Home » Download » Modules » Generate password » Issues

'Generated password' message needed?

ramsalt - March 18, 2009 - 09:04
Project:Generate password
Version:6.x-1.x-dev
Component:Code
Category:support request
Priority:normal
Assigned:Unassigned
Status:active
Description

Is the 'Generated password' message actually needed? Maybe for admins, or roles with administer users permissions, but I can't see why anonymous users have to see this. Why not throw a if(user_access('administer users')) around it?

» Login or register to post comments

#1

japanitrat - March 18, 2009 - 12:52

hm i guess we had this for testing purposes on both forms (registration / admin user-add). You're right, its not needed on the regular registration form. On the other hand, it makes sense if the admin selected "optional password" instead of "no password fields". So maybe the messages should be more descriptive for normal user registrations.

my proposals for regular user registration forms:

  1. "optional password": You did not enter a password, so we generated one for you. (maybe the generated one could also displayed here, since activation is needed anyway in most cases)
  2. "no password fields": either no message at all, or the password like above.

The password display option could also be configurable in the settings.

Login or register to post comments

#2

yingtho - April 6, 2009 - 11:06

I agree with the proposal.

Login or register to post comments
 
 

Drupal is a registered trademark of Dries Buytaert.