Attachment 37907 Details for Bug 10423 – Bug 10423: Show date due with time if it's a hourly loan

[patch] Bug 10423: Show date due with time if it's a hourly loan

Bug-10423-Show-date-due-with-time-if-its-a-hourly-.patch (text/plain), 2.68 KB, created by Jonathan Druart on 2015-04-15 13:37:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2015-04-15 13:37:46 UTC

Size: 2.68 KB

patch

obsolete

>From 102e995ece0594d2572c84ea8a956ae139bbcefe Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 15 Apr 2015 15:33:54 +0200
>Subject: [PATCH] Bug 10423: Show date due with time if it's a hourly loan
>
>On printing the summary from the patron account, the hours are not
>displayed if needed.
>The as_date_due flag should be set to display it correctly.
>Problem, GetPendingIssues modify the value retrieved from the database.
>In order to not add regression and check all calls to GetPendingIssues,
>this patch backup the value before the change.
>
>Test plan:
>Check some items out, specify a hourly loan for some.
>Click on print > print summary and confirm the date due are correctly
>formatted.
>---
> C4/Members.pm                                                       | 2 ++
> koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt | 2 +-
> members/summary-print.pl                                            | 2 ++
> 3 files changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/C4/Members.pm b/C4/Members.pm
>index 4ca83b9..2978926 100644
>--- a/C4/Members.pm
>+++ b/C4/Members.pm
>@@ -1200,6 +1200,8 @@ sub GetPendingIssues {
>             $_->{issuedate} = dt_from_string($_->{issuedate}, 'sql');
>         }
>         $_->{date_due} or next;
>+        $_->{date_due_iso} = $_->{date_due};
>+        # FIXME no need to have this value
>         $_->{date_due} = dt_from_string($_->{date_due}, 'sql');
>         if ( DateTime->compare($_->{date_due}, $today) == -1 ) {
>             $_->{overdue} = 1;
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
>index 1d62a23..92526e8 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
>@@ -50,7 +50,7 @@
>                         <td>[% issue.author %]</td>
>                         <td>[% issue.itemcallnumber %]</td>
>                         <td>[% issue.itemtype_description %]</td>
>-                        <td>[% issue.date_due | $KohaDates %]</td>
>+                        <td>[% issue.date_due | $KohaDates as_due_date => 1 %]</td>
>                         <td>[% issue.barcode %]</td>
>                         <td>[% issue.charge %]</td>
>                         <td>[% issue.replacementprice %]</td>
>diff --git a/members/summary-print.pl b/members/summary-print.pl
>index 39897db..a072c05 100755
>--- a/members/summary-print.pl
>+++ b/members/summary-print.pl
>@@ -95,6 +95,8 @@ sub build_issue_data {
> 
>         $row{'charge'} = sprintf( "%.2f", $charge );
> 
>+        $row{date_due} = $row{date_due_iso};
>+
>         push( @{$return}, \%row );
>     }
> 
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 10423: 18690 | 37907 | 37947 | 37955 | 38005 | 38109