search

LafColITS / moodle-local_ldap

Various synchronization scripts between Moodle and LDAP directories (see https://tracker.moodle.org/browse/MDL-25011 )
9 stars 7 forks source link

MNET-enhance user lookup #12

Closed OdyX closed 6 years ago

OdyX commented 6 years ago

Since the username is not unique the script should only look for local users. It's possible that a user from a remote system connected via MNET has the same username which could lead to unintended consequences.

mackensen commented 6 years ago

Please fix the calling of the $CFG variable.

OdyX commented 6 years ago

Oh, indeed. Good catch; thanks!

mackensen commented 6 years ago

Looks good, thanks for the report and patch.