Attachment 31286 Details for Bug 12847 – Bug 12847 [Followup] - Remove code that calculates "overdue" key as C4::Members::GetPendingIssues already creates it

[patch] Bug 12847 [Followup] - Remove code that calculates "overdue" key as C4::Members::GetPendingIssues already creates it

-Bug-12847-Follwup---Remove-code-that-calculates-o.patch (text/plain), 1.51 KB, created by Kyle M Hall (khall) on 2014-08-29 14:10:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2014-08-29 14:10:02 UTC

Size: 1.51 KB

patch

obsolete

>From 45ab7956d2b5caf98d103b304455e0ca75732511 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Fri, 29 Aug 2014 09:46:16 -0400
>Subject: [PATCH]  Bug 12847 [Follwup] - Remove code that calculates "overdue" key as C4::Members::GetPendingIssues already creates it
>
>---
> C4/Members.pm |    7 +------
> 1 files changed, 1 insertions(+), 6 deletions(-)
>
>diff --git a/C4/Members.pm b/C4/Members.pm
>index 2c388fe..7cf2cde 100644
>--- a/C4/Members.pm
>+++ b/C4/Members.pm
>@@ -2356,15 +2356,10 @@ sub IssueSlip {
>         $dt_issuedate->truncate( to => 'day' );
>         my $dt_lastreneweddate = dt_from_string( $it->{'lastreneweddate'} );
>         $dt_lastreneweddate->truncate( to => 'day' );
>-        my $dt_date_due = dt_from_string( $it->{'date_due'} );
>-        $dt_date_due->truncate( to => 'day' );
> 
>         if ( $dt_issuedate == $now || $dt_lastreneweddate == $now ) {
>             $it->{'now'} = 1;
>         }
>-        elsif ( $dt_date_due < $now ) {
>-            $it->{'overdue'} = 1;
>-        }
> 
>         # Recreate date_due as DateTime because we truncated the hours
>         $it->{'date_due'} = output_pref( dt_from_string( $it->{'date_due'} ) );
>@@ -2389,7 +2384,7 @@ sub IssueSlip {
>                 'biblio' => $_,
>                 'items'  => $_,
>                 'issues' => $_,
>-            }, grep { !$_->{'overdue'} } @issues ],
>+            }, grep { !$_->{'overdue'} } @issues ], # overdue key created by GetPendingIssues
> 
>             'overdue' => [ map {
>                 'biblio' => $_,
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 12847: 31280 | 31281 | 31286 | 35969 | 36048 | 36049 | 37908 | 37946 | 37992 | 38020 | 38302