[drupal-devel] [bug] Password reset page - username and e-mail checks don't work

[fago]

Issue status update for 
http://drupal.org/node/29548
Post a follow up: 
http://drupal.org/project/comments/add/29548

 Project:      Drupal
 Version:      4.6.3
 Component:    user system
 Category:     bug reports
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  kumo
 Updated by:   fago
 Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/php5.head.patch (6.4 KB)

and the patch for head




fago



Previous comments:
------------------------------------------------------------------------

Wed, 24 Aug 2005 15:51:57 +0000 : kumo

Attachment: http://drupal.org/files/issues/user_pass_reset_checks.patch (1.73 KB)

On the password reset page, if a user enters a non-existant username or
e-mail address, the 'Further instructions have been sent to your e-mail
address.' message is shown


The attached patch fixes this issue. It also removes an extraneous
message shown if no username or password is given or when the
unrecognised username or e-mail address messages are shown.




------------------------------------------------------------------------

Thu, 25 Aug 2005 17:55:51 +0000 : fago

+1
now it works.


so, it doesn't display a message if the user enters nothing any more,
but i think this is ok, because the login form works in the same way.


however, usually patches are based in drupal's root dir.
http://drupal.org/patch




------------------------------------------------------------------------

Fri, 26 Aug 2005 14:18:09 +0000 : fago

i've noticed a similar problem in user_view()
i think that's a php5 specific issue?


in php5  is something like if ($account = user_load(array('uid' =>
$uid, 'status' => 1))) 
always true. perhaps we should fix user_load() instead?




------------------------------------------------------------------------

Tue, 30 Aug 2005 23:15:14 +0000 : fago

Attachment: http://drupal.org/files/issues/php5.4.6.patch (4.88 KB)

i've fixed this everywhere in core, where i found it and created a
patch.
as this affects 4.6 too, here is the patch for 4.6