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

Errors with certain LDAP responses #25

Closed ericmerrill closed 7 months ago

ericmerrill commented 3 years ago

I saw a bunch of these messages from cron:

PHP Warning:  count(): Parameter must be an array or an object that implements Countable in /var/www/learning.up.edu/moodle/local/ldap/locallib.php on line 369

Warning: count(): Parameter must be an array or an object that implements Countable in /var/www/learning.up.edu/moodle/local/ldap/locallib.php on line 369

It seems that in locallib.php there is no additional check if the attribute is set before accessing it.

mackensen commented 7 months ago

This was probably fixed in #34.