Attachment 59233 Details for Bug 17924 – Bug 17924 - Fix error in password recovery

[patch] Bug 17924 - Fix error in password recovery

Bug-17924---Fix-error-in-password-recovery.patch (text/plain), 1.53 KB, created by Nick Clemens (kidclamp) on 2017-01-19 12:03:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2017-01-19 12:03:33 UTC

Size: 1.53 KB

patch

obsolete

>From 64d1dfd925f13b373597e087ee4c95e73635744b Mon Sep 17 00:00:00 2001
>From: Magnus Enger <magnus@libriotech.no>
>Date: Tue, 17 Jan 2017 14:50:49 +0000
>Subject: [PATCH] Bug 17924 - Fix error in password recovery
>
>To test:
>
>- Set OpacResetPassword = allowed
>- Make sure you have a user with a username and email
>- Go to the OPAC and make sure you are not logged in
>- Click on the "Forgot your password?" link
>- Fill in "Login" and "Email" and click on "Submit"
>- You will get an error that says:
>
>Undefined subroutine &C4::Letters::GetPreparedLetter called at
>/home/vagrant/kohaclone/Koha/Patron/Password/Recovery.pm line 142.
>
>- Apply the patch
>- The steps above will have added your user to the borrower_password_recovery
>  table in the database. Delete it from there to make sure you start a
>  fresh reset request.
>- Repeat the steps above and notice that the error is gone. You should
>  get a nice message instead that says "An email has been sent to [your
>  email]".
>- Sign off
>
>Signed-off-by: Zoe Schoeler <crazy.mental.onion@gmail.com>
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> Koha/Patron/Password/Recovery.pm | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/Koha/Patron/Password/Recovery.pm b/Koha/Patron/Password/Recovery.pm
>index cc59a6c..04491e6 100644
>--- a/Koha/Patron/Password/Recovery.pm
>+++ b/Koha/Patron/Password/Recovery.pm
>@@ -19,6 +19,7 @@ package Koha::Patron::Password::Recovery;
> 
> use Modern::Perl;
> use C4::Context;
>+use C4::Letters;
> use Crypt::Eksblowfish::Bcrypt qw(en_base64);
> 
> use vars qw(@ISA @EXPORT);
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 17924: 59094 | 59132 | 59233