Bug 18025 - Expired password recovery links cause sql crash
authorLiz Rea <liz@catalyst.net.nz>
Tue, 31 Jan 2017 21:59:01 +0000 (21:59 +0000)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 17 Feb 2017 11:24:39 +0000 (11:24 +0000)
commite87dab6411a40ae0eba3d56032760d705ef62eaf
tree1588239d316296e557c612e3c957e66308dcefc5
parent745c4c3da662ac71874f9be8e88d111bc6fe8b1c
Bug 18025 - Expired password recovery links cause sql crash

When a user gets an email, but doesn't act or visit it within two days,
     attempting to create a new one causes a collision. We should just
     delete the old one, assuming they still want to reset their
     password.

To test:
create yourself a borrower with a userid and password.
Attempt a password recovery on the OPAC
update the entry in the database for that user to have an expired token
e.g. update borrower_password_recovery set valid_until = '2017-01-25
03:25:26' where borrowernumber = 12;
Attempt another password recovery operation - should error
apply the patch
Try it again - no error, new token is generated and additional email
with new link is sent.

Issue reproduced - is resolved by patch
Signed-off-by: Marc VĂ©ron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Koha/Patron/Password/Recovery.pm
opac/opac-password-recovery.pl
t/db_dependent/Passwordrecovery.t