Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 37887: OPAC password recovery needs to use a cud- op while POSTing new password

Browse source 
To pass xt/find-missing-op-in-forms.t after bug 37728 updates it to notice
that there can be more than one form in a template, opac-password-recovery.tt
needs to have a param named 'op' which starts with 'cud-' for the form that
POSTs the new password.

Luckily, testing this doesn't require that you set Koha up to actually send
email (though you can), because you can get the link to reset the password
by looking at the list of notices sent to the patron in the staff interface:
the failure to send notice works just fine.

Test plan:
 1. There's no behavior change to test, you just need to see that resetting
    the password still works, so start with the patch applied.
 2. Administration - System preferences - change OpacResetPassword to allowed
 3. Set a patron so you can use them, which requires that you know the
    username, and they have an email address. I give Acosta, Edna one of
    my email addresses, and copy her cardnumber.
 4. Open the OPAC, and below the login form click the "Forgot your password?"
    link
 5. Enter the cardnumber for Login and the email you used for Email and
    click Submit. If you didn't set up sending email, you'll get an error
    message, but ignore it, not a problem
 6. Back in the staff interface, check out to the patron you are using, and
    on the left sidebar choose Notices and find the Koha password recovery
    notice and click that linked phrase
 7. In the popup with the notice text, open the recovery link in a new tab
 8. Following the instructions for the content of a new password, enter one
    in both fields and click Submit
 9  In the success message, click the link to Log in to your account and
    log in with the new password

Sponsored-by: Chetco Community Public Library
Signed-off-by: Olivier V <olivier.vezina@inLibro.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Phil Ringnalda 2024-09-10 20:17:20 -07:00 committed by Katrin Fischer
parent 81bc750e00
commit 9e43658e6f
Signed by: kfischer
GPG key ID: 0EF6E2C03357A834
2 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-password-recovery.tt
View file

@ -151,6 +151,7 @@
[% UNLESS ( errLinkNotValid ) %] [% UNLESS ( errLinkNotValid ) %]
<form id="mainform" name="mainform" action="/cgi-bin/koha/opac-password-recovery.pl" method="post" autocomplete="off"> <form id="mainform" name="mainform" action="/cgi-bin/koha/opac-password-recovery.pl" method="post" autocomplete="off">
[% INCLUDE 'csrf-token.inc' %] [% INCLUDE 'csrf-token.inc' %]
<input type="hidden" name="op" value="cud-reset_password" />
<legend class="sr-only">Password not valid</legend> <legend class="sr-only">Password not valid</legend>
<input type="hidden" name="koha_login_context" value="opac" /> <input type="hidden" name="koha_login_context" value="opac" />
<fieldset class="brief"> <fieldset class="brief">

2
opac/opac-password-recovery.pl
View file

@ -148,7 +148,7 @@ if ( $op eq 'cud-sendEmail' || $op eq 'cud-resendEmail' ) {
); );
} }
} }
elsif ( $query->param('passwordReset') ) { elsif ( $op eq 'cud-reset_password' ) {
( $borrower_number, $username ) = GetValidLinkInfo($uniqueKey); ( $borrower_number, $username ) = GetValidLinkInfo($uniqueKey);
my $error; my $error;