Bugzilla – Attachment 73098 Details for
Bug 20340
Ability to use authentication plugin
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 20340: Followup to allow OPAC login to complete
Bug-20340-Followup-to-allow-OPAC-login-to-complete.patch (text/plain), 1.46 KB, created by
Mark Tompsett
on 2018-03-20 04:05:58 UTC
(
hide
)
Description:
Bug 20340: Followup to allow OPAC login to complete
Filename:
MIME Type:
Creator:
Mark Tompsett
Created:
2018-03-20 04:05:58 UTC
Size:
1.46 KB
patch
obsolete
>From b252385dca72a46122c463a9abd908d7b4abbd79 Mon Sep 17 00:00:00 2001 >From: Mark Tompsett <mtompset@hotmail.com> >Date: Tue, 20 Mar 2018 04:03:44 +0000 >Subject: [PATCH] Bug 20340: Followup to allow OPAC login to complete > >The Plugin authenticates correctly, the problem is there >is no patron to match with, and so there is nothing to check >for authorization and user-details once a login is attempted. >This will at least let it fail gracefully. >--- > opac/opac-user.pl | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/opac/opac-user.pl b/opac/opac-user.pl >index ac91d4f..022aa3d 100755 >--- a/opac/opac-user.pl >+++ b/opac/opac-user.pl >@@ -90,12 +90,12 @@ if (!$borrowernumber) { > > # get borrower information .... > my $patron = Koha::Patrons->find( $borrowernumber ); >-my $borr = $patron->unblessed; >+my $borr = $patron ? $patron->unblessed : {}; > > my ( $today_year, $today_month, $today_day) = Today(); > my ($warning_year, $warning_month, $warning_day) = split /-/, $borr->{'dateexpiry'}; > >-my $debar = Koha::Patrons->find( $borrowernumber )->is_debarred; >+my $debar = $patron ? $patron->is_debarred : undef; > my $userdebarred; > > if ($debar) { >@@ -118,7 +118,7 @@ if ( $userdebarred || $borr->{'gonenoaddress'} || $borr->{'lost'} ) { > $canrenew = 0; > } > >-my $amountoutstanding = $patron->account->balance; >+my $amountoutstanding = $patron ? $patron->account->balance : 0; > if ( $amountoutstanding > 5 ) { > $borr->{'amountoverfive'} = 1; > } >-- >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 20340
:
72469
|
73089
|
73098
|
73106
|
88470
|
88472
|
88473
|
88474
|
88475
|
88476
|
88477
|
88478
|
88487
|
88488
|
88511
|
88512
|
88518
|
88519
|
88520
|
88521
|
88522
|
88523
|
88526
|
88527
|
91304
|
91305
|
91306
|
91307