Attachment 62456 Details for Bug 18314 – Bug 18314: Resetting the password removes the lock

[patch] Bug 18314: Resetting the password removes the lock

Bug-18314-Resetting-the-password-removes-the-lock.patch (text/plain), 1.14 KB, created by Biblibre Sandboxes on 2017-04-20 11:25:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Biblibre Sandboxes

Created: 2017-04-20 11:25:16 UTC

Size: 1.14 KB

patch

obsolete

>From 5807fffbd3847c2ac59c12b2e90fd8e0d3279cb7 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Tue, 21 Mar 2017 21:18:10 -0300
>Subject: [PATCH] Bug 18314: Resetting the password removes the lock
>
>When a password is changed (updated by a staff member or using the
>"recover password via email" feature, the counter/flag needs to be
>reset.
>
>Signed-off-by: Jonathan Field <jonathan.field@ptfs-europe.com>
>---
> Koha/Patron.pm | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
>diff --git a/Koha/Patron.pm b/Koha/Patron.pm
>index 3c1c0d4..a766478 100644
>--- a/Koha/Patron.pm
>+++ b/Koha/Patron.pm
>@@ -333,7 +333,12 @@ sub update_password {
>     my ( $self, $userid, $password ) = @_;
>     eval { $self->userid($userid)->store; };
>     return if $@; # Make sure the userid is not already in used by another patron
>-    $self->password($password)->store;
>+    $self->update(
>+        {
>+            password       => $password,
>+            login_attempts => 0,
>+        }
>+    );
>     logaction( "MEMBERS", "CHANGE PASS", $self->borrowernumber, "" ) if C4::Context->preference("BorrowersLog");
>     return 1;
> }
>-- 
>2.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 18314: 61630 | 61631 | 61632 | 61633 | 61634 | 62453 | 62454 | 62455 | 62456 | 62457 | 62872 | 62873 | 62874 | 62875 | 62876 | 62877 | 62878 | 62884 | 62885 | 62886 | 62887 | 62888 | 62889 | 62890 | 62891