Closed (fixed)
Project:
Lightweight Directory Access Protocol
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
27 Jan 2014 at 02:25 UTC
Updated:
1 Feb 2016 at 21:16 UTC
Jump to comment: Most recent, Most recent file
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.