Just like the title.

When I just enable the "LDAP Authentication", "LDAP Servers" and "LDAP User Module" modules. There have been shown this warning.

Comments

jerenus’s picture

Status: Active » Needs review
StatusFileSize
new917 bytes

Quick patch.

frob’s picture

I have not fully tested the patch yet, but it made the notice go away.

erik frèrejean’s picture

Since ldap_authentication_get_valid_conf() explicitly returns FALSE when the LdapAuthenticationConf isn't configured it is probably neater to check for that boolean rather than the isset check in #1

johnbarclay’s picture

Assigned: Unassigned » johnbarclay
benstallings’s picture

Status: Needs review » Reviewed & tested by the community

I can confirm that this patch makes the error message go away.

gold’s picture

I can also confirm that the patch at #3 gets rid of the Notice.

jyee’s picture

Another confirmation that #3 works and is rtbc

johnbarclay’s picture

That 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.

LavenPillay’s picture

Hi Folks,
Just wanted to say that the update in post #8 does the trick.
(note : applied manually)

Thanks a lot !

star-szr’s picture

Status: Reviewed & tested by the community » Fixed

This was pushed and is now in the 7.x-2.x-dev version: http://drupalcode.org/project/ldap.git/commit/fe3a3d5

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

jackbravo’s picture

Version: 7.x-2.x-dev » 7.x-1.x-dev
Status: Closed (fixed) » Needs work
Issue tags: +needs backport to 7.x-1.x

This also happens on 7.x-1.x

jackbravo’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Needs work » Closed (fixed)

Never mind... I don't know what code I was seeing. Sorry for the noise.

daniel418’s picture

Hi 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 !

rfdparker’s picture

Status: Closed (fixed) » Active

Still having this issue with the 7.x-2.0-beta8 release of ldap with Drupal 7.38.

mechanoid’s picture

Confirmed that solution #8 (and #14) works.

Error message has disappeared after adding '$auth_conf && ' into statement.

Leo Jiménez’s picture

Same issue in ldap authentication-7.x-2.0-beta8, solution #8 fix the problem.

/**
 * Implements hook_init().
 */
function ldap_authentication_init() {
  $auth_conf = ldap_authentication_get_valid_conf();
  if ($auth_conf && $auth_conf->templateUsagePromptUser) {
    ldap_authentication_check_for_email_template();
  }
}
darvanen’s picture

Status: Active » Closed (fixed)
StatusFileSize
new584 bytes

This 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.

osagiesammy’s picture

Fixed thanks.