Download & Extend
Generate password » Issues

'Generated password' message needed?

Posted by ramsalt on March 18, 2009 at 9:04am
Project:Generate password
Version:6.x-1.x-dev
Component:Code
Category:support request
Priority:normal
Assigned:Unassigned
Status:patch (to be ported)

Issue Summary

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

Comments

#1

Posted by japanitrat on March 18, 2009 at 12:52pm

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.

#2

Posted by yingtho on April 6, 2009 at 11:06am

I agree with the proposal.

#3

Posted by joelstein on June 16, 2010 at 9:43pm
Status:active» fixed

The UI was cleaned up a bit in this ticket #772690: Module revamp and the latest release.

#4

Posted by System Message on June 30, 2010 at 9:50pm
Status:fixed» closed (fixed)

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

#5

Posted by yogesh1110 on August 30, 2011 at 4:48am

Here is the patch to display password message based on permission 'administer users'

AttachmentSize
remove_pawword_msg-405774-5.patch 1.13 KB

#6

Posted by yogesh1110 on September 8, 2011 at 6:53am
Status:closed (fixed)» patch (to be ported)

Here is the patch to display password message based on permission 'administer users'

AttachmentSize
remove_pawword_msg-405774-5.patch 1.13 KB

#7

Posted by Lukas von Blarer on November 22, 2011 at 12:21pm

I wrote a patch for D7 that adds a option to disable these messages globally.

AttachmentSize
patch_commit_39eb94029419.patch 3.1 KB
nobody click here