username replaced with auto generated password when account is edited

jaymallison - April 11, 2008 - 02:18
Project:Auto username
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

If you disable the "Update on user edit" feature at "/admin/user/auto-username" and then go to "/user/[uid]/edit" and resave the user account, it will overwrite the username with an auto generated password, and this will continue to happen each time you edit the account.

I'd try to fix it, but I have a presentation with a client for this site in the morning and don't have time to fix it right now. The module seems to work great if you leave the "Update on user edit" feature enabled. I ultimately don't want that to happen on this site, but for now it'll work.

Thanks for this module, its exactly what my client wanted.

#1

Crell - April 11, 2008 - 15:55
Status:active» fixed

Oopsies. The use edit form *must* have a value for that field or it won't validate, so I'm making it a "value" field with a random string so that it's unique that just gets replaced anyway. I forgot an if statement for it. Fixed now. Thanks.

#2

jaymallison - April 11, 2008 - 18:39

sweet, can you post the code to patch it up? That way I can get this issue cleared up in my copy. Or I guess you could post an updated dev version, whatever works for you. Thanks for taking care of it so quickly.

#3

Crell - April 11, 2008 - 19:09

It's already committed to 5.x-1.x-dev. You can grab it straight from CVS if you can't wait the 8 hours or so until the tarball is rebuilt. :-) (There were a few other things fixed as well, so I don't want to try and roll just that limited patch.)

#4

jaymallison - April 11, 2008 - 22:31

no worries, I didn't know that it was rebuilt automatically. My bad, I haven't used the "Issues" system on here before.

#5

Anonymous (not verified) - April 25, 2008 - 22:31
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.