From 2785183a6b3c1bc231f78a0e506617379ea3998e Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Fri, 29 Jul 2016 13:16:40 +0200 Subject: [PATCH] Bug 16992: [QA Follow-up] Member-password should pass an userid If we do not fill a new userid, we should keep the old one. Script member-password should pass that to Koha::Patron. Otherwise things go wrong. Signed-off-by: Marcel de Rooy Without this patch, you could effectively disable a login. Signed-off-by: Katrin Fischer --- members/member-password.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/members/member-password.pl b/members/member-password.pl index 64d6347d8a..c24a26213b 100755 --- a/members/member-password.pl +++ b/members/member-password.pl @@ -64,7 +64,7 @@ push( @errors, 'SHORTPASSWORD' ) if ( $newpassword && $minpw && ( length($newpas if ( $newpassword && !scalar(@errors) ) { my $digest = Koha::AuthUtils::hash_password( $input->param('newpassword') ); - my $uid = $input->param('newuserid'); + my $uid = $input->param('newuserid') || $bor->{userid}; my $dbh = C4::Context->dbh; if ( Koha::Patrons->find( $member )->update_password($uid, $digest) ) { $template->param( newpassword => $newpassword ); -- 2.39.5