error_log php/ldap'ish reports

[warmerdam]

The /etc/httpd/logs/error_log file is full of stuff like this:

[client 204.11.125.18] PHP Notice: Use of undefined constant LDAP_DEFAULT_ORG - assumed 'LDAP_DEFAULT_ORG' in /var/www/html/modules/ldap_integration/ldapauth.module on line 13 [client 204.11.125.18] PHP Notice: Use of undefined constant LDAP_DEFAULT_BASE_DN - assumed 'LDAP_DEFAULT_BASE_DN' in /var/www/html/modules/ldap_integration/ldapauth.module on line 14 [client 204.11.125.18] PHP Notice: Use of undefined constant LDAP_DEFAULT_USER_ATTRIBUTE - assumed 'LDAP_DEFAULT_USER_ATTRIBUTE' in /var/www/html/modules/ldap_integration/ldapauth.module on line 15 [client 204.11.125.18] PHP Notice: Use of undefined constant localhost - assumed 'localhost' in /var/www/html/modules/ldap_integration/ldap_integration/LDAPInterface.php on line 8 [client 204.11.125.18] PHP Notice: Undefined index: time in /var/www/html/includes/bootstrap.inc on line 60

Is there something that should be fixed? If only to reduce error logging?

[tmitchell]

I'm not sure about this. I recall seeing it appear before, but it was actually shown when the module was enabled and was displayed on the web pages not just in logs. I seem to recall that once LDAP account information was entered correctly it went away.

Over the past couple weeks there was an update of the ldap_integration module. I applied changes, in case it might affect this bug.

I filed the bug report with developers at: ​http://drupal.org/node/121783

[warmerdam]

Tyler,

Was this ldap_integration module upgrade something you backported from a new verison of Drupal? If so, the drupal developers aren't likely to appreciate the bug report on a munged homebrew mix of stuff. And it would behoove us to deal with it ourselves.

[tmitchell]

Egads - no I haven't back ported anything! They also had a newer version of the module for drupal 4.7 than what we were using - so I upgraded. But I don't think it has all their most recent fixes that went into their version 5 release of the module. Sorry - by 'applied' I didn't mean 'patch' .. I simply updated the module.