Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.4
-
Component/s: Authentication
-
Labels:
-
Affected Branches:MOODLE_24_STABLE
-
Fixed Branches:MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
-
Pull from Repository:
-
Pull Master Branch:wip_master_mdl-37088_bad_operator_ldap_sync_user
Description
Hello,
There is a little mistake here :
http://git.moodle.org/gw?p=moodle.git;a=blob;f=auth/ldap/auth.php;h=fed0bae7aadf466717deba63f921425e3f63097b;hb=HEAD#l714
Operator should be != or variables must be casted (because one is int and other is string).
Currently this condition always return true. Fortunately there is no bad effect, just a resource/time waste.
Thanks.
Attachments
Issue Links
- is duplicated by
-
MDL-36118 LDAP sync_users performs redundant actions
-
- Closed
-