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

fago drupal-devel at drupal.org
Mon Sep 12 15:49:13 UTC 2005


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

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

right, i fixed that and rerolled 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




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

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

Attachment: http://drupal.org/files/issues/php5.head.patch (6.4 KB)

and the patch for head




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

Tue, 30 Aug 2005 23:23:16 +0000 : fago

Attachment: http://drupal.org/files/issues/php5.head_0.patch (6.78 KB)

sry, i had attached the wrong patch for head.


this is the right one.
please review.




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

Sun, 11 Sep 2005 22:45:17 +0000 : m3avrck

I've changed the status to CVS since this patch affects CVS as well and
should go in there first before 4.6.3.


One problem, if I enter a valid email address, I'm getting the error
'Sorry the email address is not recognized'.


Otherwise patch looks great! However, it is slightly out of date
against HEAD so should be rerolled to be clean, thanks!







More information about the drupal-devel mailing list