Bugzilla – Attachment 32572 Details for
Bug 13107
Members are restricted even if the debarment is ended - opac-user
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 13107 - Members are restricted even if the debarment is ended - opac-user
PASSED-QA-Bug-13107---Members-are-restricted-even-.patch (text/plain), 1.67 KB, created by
Katrin Fischer
on 2014-10-21 22:39:21 UTC
(
hide
)
Description:
[PASSED QA] Bug 13107 - Members are restricted even if the debarment is ended - opac-user
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2014-10-21 22:39:21 UTC
Size:
1.67 KB
patch
obsolete
>From 04544528c50bd19fb146b3608152c4e1a0ee3840 Mon Sep 17 00:00:00 2001 >From: Fridolin Somers <fridolin.somers@biblibre.com> >Date: Fri, 17 Oct 2014 18:30:34 +0200 >Subject: [PATCH] [PASSED QA] Bug 13107 - Members are restricted even if the > debarment is ended - opac-user > >This is the same issue as bug 12134. > >Test Plan: >1) Add a manual restriction to a patron with expiration date in the past. >2) Go on the OPAC and connect (opac-user.pl) >3) Note the warning message > "Your account has been frozen until until XX/XX/XXXX ..." >4) Apply this patch >5) Repeat step 2 >6) Note the warning message does not appear anymore > >Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Works as described. > >NOTE: If you set a debarment with date in the past in the GUI, >there will be no entry in borrowers.debarred and you won't be >able to see the problem. Set one with a date in the future and >then alter the date in borrower_debarments and borrowers.debarred. >--- > opac/opac-user.pl | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > >diff --git a/opac/opac-user.pl b/opac/opac-user.pl >index aae78ec..ad4f339 100755 >--- a/opac/opac-user.pl >+++ b/opac/opac-user.pl >@@ -35,6 +35,7 @@ use C4::Items; > use C4::Letters; > use C4::Branch; # GetBranches > use Koha::DateUtils; >+use Koha::Borrower::Debarments qw(IsDebarred); > > use constant ATTRIBUTE_SHOW_BARCODE => 'SHOW_BCODE'; > >@@ -84,7 +85,7 @@ my ($warning_year, $warning_month, $warning_day) = split /-/, $borr->{'dateexpir > > $borr->{'ethnicity'} = fixEthnicity( $borr->{'ethnicity'} ); > >-my $debar = $borr->{'debarred'}; >+my $debar = IsDebarred($borrowernumber); > my $userdebarred; > > if ($debar) { >-- >1.9.1
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 13107
:
32517
|
32522
| 32572