Bugzilla – Attachment 64608 Details for
Bug 18616
The "Add forgot password link to OPAC" should allow patrons to use their library card number in addition to username
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18616 - The "Add forgot password link to OPAC" should allow patrons to use their library card number in addition to username
Bug-18616---The-Add-forgot-password-link-to-OPAC-s.patch (text/plain), 2.04 KB, created by
Nick Clemens (kidclamp)
on 2017-06-23 23:20:50 UTC
(
hide
)
Description:
Bug 18616 - The "Add forgot password link to OPAC" should allow patrons to use their library card number in addition to username
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2017-06-23 23:20:50 UTC
Size:
2.04 KB
patch
obsolete
>From 8b34214ee40a258d30cb4cd3efed69414cdaa18f Mon Sep 17 00:00:00 2001 >From: David Kuhn <techservspec@gmail.com> >Date: Tue, 6 Jun 2017 09:38:07 -0700 >Subject: [PATCH] Bug 18616 - The "Add forgot password link to OPAC" should > allow patrons to use their library card number in addition to username >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit > >Allow patrons to enter either their library card number or user name in the "Log in" box for password recovery. > >Most patrons at our library use their card number to log in and are unaware of what their userid is. However there are some who have set a > >customized userid and would prefer to use that. This patch would allow either to be entered for password recovery. > >To test: >1. Enable the password recovery feature. >2. In the OPAC, click on "Forgot you password?" link and enter a valid library card number. >3. The error message "No account found with the provided information" appears. >4. Apply the patch. >5. Repeat step 2. The recovery email is now sent. > >Note: Moved patch from 16711 back here and re-tested. >Signed-off-by: Marc Véron <veron@veron.ch> > >Signed-off-by: Nick Clemens <nick@bywatersolutions.com> >--- > opac/opac-password-recovery.pl | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/opac/opac-password-recovery.pl b/opac/opac-password-recovery.pl >index f15a93e..f4c5e2b 100755 >--- a/opac/opac-password-recovery.pl >+++ b/opac/opac-password-recovery.pl >@@ -57,9 +57,9 @@ if ( $query->param('sendEmail') || $query->param('resendEmail') ) { > my $borrower; > my $search_results; > >- # Find the borrower by his userid or email >+ # Find the borrower by userid, card number, or email > if ($username) { >- $search_results = Koha::Patrons->search( { userid => $username } ); >+ $search_results = Koha::Patrons->search( { -or => { userid => $username, cardnumber => $username } } ); > } > elsif ($email) { > $search_results = Koha::Patrons->search( { -or => { email => $email, emailpro => $email, B_email => $email } } ); >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 18616
:
63855
|
63881
|
63882
|
63883
|
64066
| 64608