Skip to content

Conversation

@hregis
Copy link
Contributor

@hregis hregis commented Nov 14, 2025

need the constant "WEBPORTAL_LOGIN_BY_MEMBER_ACCOUNT" = 1
and the constant "ADHERENT_LOGIN_NOT_REQUIRED" = 0

@hregis
Copy link
Contributor Author

hregis commented Nov 14, 2025

@eldy

i replace :

// Check crypted password
$cryptType = '';
if (getDolGlobalString('DATABASE_PWD_ENCRYPTED')) {
	$cryptType = getDolGlobalString('DATABASE_PWD_ENCRYPTED');
}

// By default, we use default setup for encryption rule
if (!in_array($cryptType, array('auto'))) {
	$cryptType = 'auto';
}

// Check crypted password according to crypt algorithm
if ($cryptType == 'auto') {
	if ($passcrypted && dol_verifyHash($pass, $passcrypted, '0')) {
		$passok = true;
	}
}

by just :

// Check crypted password according to crypt algorithm
if ($passcrypted && dol_verifyHash($pass, $passcrypted, '0')) {
	$passok = true;
}

it's the same way !

@hregis
Copy link
Contributor Author

hregis commented Nov 14, 2025

@eldy @frederic34 the next phan error it's not me ! 😄

@hregis hregis requested a review from eldy November 14, 2025 18:50
@hregis
Copy link
Contributor Author

hregis commented Nov 17, 2025

@frederic34 @mdeweerd What's going on?

@hregis
Copy link
Contributor Author

hregis commented Nov 18, 2025

@eldy it's ok now

@hregis
Copy link
Contributor Author

hregis commented Nov 19, 2025

@eldy @kkhelifa-opendsi it's ok for you ?

@mdeweerd
Copy link
Contributor

@hregis The change to windows-ci.yml must be reverted because the tag v2025.12.2 does not exist (yet?) on mdeweerd/logToCheckStyle.

@hregis
Copy link
Contributor Author

hregis commented Nov 19, 2025

@mdeweerd I received a request on my personal repository and I accepted it... it's been withdrawn... it's ok now ?

@mdeweerd
Copy link
Contributor

@mdeweerd I received a request on my personal repository and I accepted it... it's been withdrawn... it's ok now ?
Yes, 👌🏻

@hregis
Copy link
Contributor Author

hregis commented Nov 19, 2025

@mdeweerd The edit request to modify "windows-ci.yml " had your username, but maybe it came from a bot, I didn't pay attention...

@mdeweerd
Copy link
Contributor

@mdeweerd The edit request to modify "windows-ci.yml " had your username, but maybe it came from a bot, I didn't pay attention...

I did not make a request against your repository - but you may have dependency bot doing this.
It's strange however that it proposed a wrong tag - especially since it was already changed to the correct value.

@hregis
Copy link
Contributor Author

hregis commented Nov 19, 2025

@mdeweerd maybe an error by me ! no problems ! thanks

@hregis
Copy link
Contributor Author

hregis commented Nov 21, 2025

@eldy travis error is not for my PR

@hregis
Copy link
Contributor Author

hregis commented Nov 24, 2025

@eldy can merge this PR please, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants