Hello,

the method LdapUserConf::deleteProvisionedLdapEntries() checks the users language and then tries to access his saved provisionEntries via $account->ldap_user_prov_entries[$language].

This fails here, as the users language is 'de' while all provisioned entries are under $account->ldap_user_prov_entries[LANGUAGE_NONE].

Furthermore, I guess after the entries were deleted, the $account object should be cleaned up, shouldn't it?

Comments

grahl’s picture

grahl
he/him
Location Zürich
commented
Status: Active » Fixed

Thanks for the useful bug report! All other instance of that field use LANGUAGE_NONE so it's clearly safe.

The account object does not need to be cleaned up since the function is called from within hook_user_delete().

  • grahl committed 3cbac99 on 7.x-2.x 
    Issue #2456263 by haggins, grahl: Bug in deleteProvisionedLdapEntries()

Status: Fixed » Closed (fixed)

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