A comment in line 299 of ldapprov.module says

 // The user name in $form['_account']['#value'] is already changed.
// Has to look for the old name in the authmap.

which is not correct in my case. The following if() fails and a user update is not done.

Changing the if to

if (isset($values['name']) && ($row = db_fetch_object($result)) && $values['name'] != $row->authname)
solves the problem.

CommentFileSizeAuthor
#2 ldapprov.module.patch615 bytesarnd

Comments

arnd’s picture

Sorry, forgot the patch:

arnd’s picture

StatusFileSize
new615 bytes

File upload doesn't work in Konqueror on this site :(

miglius’s picture

Status: Active » Fixed

Committed to the cvs. Thanks!

Status: Fixed » Closed (fixed)

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

  • Commit caa9382 on master, 6.x-2.x, 6.x-1.x by miglius:
    ldap_provisioning: fixed old username lookup bug, #413924 by arnd
    
    

  • Commit caa9382 on master, 6.x-2.x, 6.x-1.x by miglius:
    ldap_provisioning: fixed old username lookup bug, #413924 by arnd