Fixes to the "reset password" mode of Special:ChangeEmail
authorAlexandre Emsenhuber <ialex.wiki@gmail.com>
Mon, 12 Nov 2012 20:07:13 +0000 (21:07 +0100)
committerAlexandre Emsenhuber <ialex.wiki@gmail.com>
Mon, 4 Feb 2013 19:42:47 +0000 (20:42 +0100)
commit89995281960123f54b915a10252b509cbbc022d2
tree6bf331684eb24e2004bbc2ad74789ce144f3c81b
parent0cfa07f623d0623e83e3f162ee4e4a37fc1d7a53
Fixes to the "reset password" mode of Special:ChangeEmail

- Use the current User object when the user modifies its own password instead of different object (so that the fields are correctly updated on that object)
- Only set the cookies when changing the password of the current user and not when resetting the user's password. This was hiding the problem below.
- Make the internal call to Special:UserLogin call from Special:ChangeEmail actually work.
  The problem is that the $this->mNewPass field is cleared by attemptReset(). This was hidden because of the above and because the user is always redirected.
- Do not show the form and the message after successful submission (this was hidden for the same reason as above).
- Let Special:UserLogin handle the redirect itself when calling (because it might want to show something, such as what is injected by the UserLoginComplete hook)

Change-Id: I6cf15e23c905dad9612bab76a2dae5eb613fea9b
includes/specials/SpecialChangePassword.php