Just like the title.
When I just enable the "LDAP Authentication", "LDAP Servers" and "LDAP User Module" modules. There have been shown this warning.
| Comment | File | Size | Author |
|---|---|---|---|
| #18 | ldap-authentication_notice-2182413-18.patch | 584 bytes | darvanen |
| #3 | ldap-property_of_non_object_in_ldap_authentication_init-2182413-3.patch | 545 bytes | erik frèrejean |
| #1 | ldap-variable_verification-2182413-1.patch | 917 bytes | jerenus |
| 20140127102445.jpg | 20.56 KB | jerenus |
Comments
Comment #1
jerenus commentedQuick patch.
Comment #2
frobI have not fully tested the patch yet, but it made the notice go away.
Comment #3
erik frèrejeanSince
ldap_authentication_get_valid_conf()explicitly returnsFALSEwhen theLdapAuthenticationConfisn't configured it is probably neater to check for that boolean rather than the isset check in #1Comment #4
johnbarclay commentedComment #5
benstallings commentedI can confirm that this patch makes the error message go away.
Comment #6
goldI can also confirm that the patch at #3 gets rid of the Notice.
Comment #7
jyee commentedAnother confirmation that #3 works and is rtbc
Comment #8
johnbarclay commentedThat fix solves the warning, but removes functionality. Here's the correct fix:
if ($auth_conf && $auth_conf->templateUsagePromptUser) {
ldap_authentication_check_for_email_template();
}
This will be pushed at some point, but am having GIT issues at the moment.
Comment #9
LavenPillay commentedHi Folks,
Just wanted to say that the update in post #8 does the trick.
(note : applied manually)
Thanks a lot !
Comment #10
star-szrThis was pushed and is now in the 7.x-2.x-dev version: http://drupalcode.org/project/ldap.git/commit/fe3a3d5
Comment #12
jackbravo commentedThis also happens on 7.x-1.x
Comment #13
jackbravo commentedNever mind... I don't know what code I was seeing. Sorry for the noise.
Comment #14
daniel418 commentedHi Folks,
Just wanted to say that the update in post #8 does the trick.
(note : applied manually)
steps:
#cd sites/all/modules/ldap/ldap_authentication
#sudo vi ldap_authentication.module
if ($auth_conf && $auth_conf->templateUsagePromptUser) {
ldap_authentication_check_for_email_template();
}
Thanks a lot !
Comment #15
rfdparker commentedStill having this issue with the
7.x-2.0-beta8release ofldapwith Drupal 7.38.Comment #16
mechanoid commentedConfirmed that solution #8 (and #14) works.
Error message has disappeared after adding '
$auth_conf &&' into statement.Comment #17
Leo Jiménez commentedSame issue in ldap authentication-7.x-2.0-beta8, solution #8 fix the problem.
Comment #18
darvanenThis has already been commited to the dev branch. If you need the beta to work before a new one comes out, use this patch, or just change the code as per #8 because that's what the patch does.
Comment #19
osagiesammy commentedFixed thanks.