Bugzilla – Attachment 57888 Details for
Bug 17583
Use Koha::Patron->is_expired from circulation.pl
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 17583: Use Koha::Patron->is_expired from the circulation page
Bug-17583-Use-KohaPatron-isexpired-from-the-circul.patch (text/plain), 1.66 KB, created by
Jonathan Druart
on 2016-12-02 08:30:56 UTC
(
hide
)
Description:
Bug 17583: Use Koha::Patron->is_expired from the circulation page
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2016-12-02 08:30:56 UTC
Size:
1.66 KB
patch
obsolete
>From d5ceb53fd7feb5b28d5a8c9639d3f8ee8cfb3c36 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Tue, 8 Nov 2016 12:26:00 +0100 >Subject: [PATCH] Bug 17583: Use Koha::Patron->is_expired from the circulation > page > >Bug 17579 introduces a Koha::Patron->is_expired method. >Let's use it from this script. > >Test plan: >Confirm that you see the message on the interface from the circulation >page for an expired patron > >Signed-off-by: Josef Moravec <josef.moravec@gmail.com> >--- > circ/circulation.pl | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) > >diff --git a/circ/circulation.pl b/circ/circulation.pl >index 9a376a1..6efcbe6 100755 >--- a/circ/circulation.pl >+++ b/circ/circulation.pl >@@ -268,15 +268,8 @@ if ($borrowernumber) { > $borrower = GetMemberDetails( $borrowernumber, 0 ); > my ( $od, $issue, $fines ) = GetMemberIssuesAndFines( $borrowernumber ); > >- # Warningdate is the date that the warning starts appearing >- my ( $today_year, $today_month, $today_day) = Today(); >- my ($warning_year, $warning_month, $warning_day) = split /-/, $borrower->{'dateexpiry'}; >- my ( $enrol_year, $enrol_month, $enrol_day) = split /-/, $borrower->{'dateenrolled'}; > # if the expiry date is before today ie they have expired >- if ( !$borrower->{'dateexpiry'} || $warning_year*$warning_month*$warning_day==0 >- || Date_to_Days($today_year, $today_month, $today_day ) >- > Date_to_Days($warning_year, $warning_month, $warning_day) ) >- { >+ if ( $patron->is_expired ) { > #borrowercard expired, no issues > $template->param( > noissues => ($force_allow_issue) ? 0 : "1", >-- >2.9.3
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 17583
:
57329
|
57330
|
57334
|
57364
|
57380
|
57384
|
57387
|
57488
|
57523
|
57524
|
57525
|
57526
|
57527
|
57528
|
57888
|
57889
|
57890
|
57891
|
57892
|
57893
|
57914
|
57915
|
57916
|
57917
|
57918
|
57919
|
57920