Skip to content

Commit

Permalink
BUG Fix the password reset message to be shown consistently.
Browse files Browse the repository at this point in the history
If we detect any of the password reset GET params, it's safe to assume
that someone intended a password reset, regardless of other conditions.
  • Loading branch information
mateusz committed Oct 24, 2013
1 parent 7cf8e65 commit 1a39f61
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions security/Security.php
Original file line number Diff line number Diff line change
Expand Up @@ -658,9 +658,8 @@ public function changepassword() {
'Form' => $this->ChangePasswordForm()));

} else {
// show an error message if the auto login token is invalid and the
// user is not logged in
if(!isset($_REQUEST['t']) || !$member) {
// Show friendly message if it seems like the user arrived here via password reset feature.
if(isset($_REQUEST['m']) || isset($_REQUEST['t'])) {
$customisedController = $controller->customise(
array('Content' =>
_t(
Expand Down

0 comments on commit 1a39f61

Please sign in to comment.