Attachment 43569 Details for Bug 11298 – [PASSED QA] Bug 11298: Don't force the renew time to 23:59

[patch] [PASSED QA] Bug 11298: Don't force the renew time to 23:59

PASSED-QA-Bug-11298-Dont-force-the-renew-time-to-2.patch (text/plain), 1.23 KB, created by Katrin Fischer on 2015-10-18 12:08:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2015-10-18 12:08:27 UTC

Size: 1.23 KB

patch

obsolete

>From 10001859ff5c90889b2927d9c426980aa1b81250 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Tue, 4 Aug 2015 16:33:00 +0100
>Subject: [PATCH] [PASSED QA] Bug 11298: Don't force the renew time to 23:59
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>If you try to renew an item using the "Renewal due date" at the bottom
>of the checkouts table, the time will be hardcoded to 23:59 no matter
>what the user has chosen.
>
>Test plan:
>Check an item out to a patron
>renew the item and specify a due date using the datetime picker plugin
>at the bottom of the table
>Confirm the time you have chosen is correctly used.
>
>Followed test plan. It works as advertised.
>Signed-off-by: Marc VÃ©ron <veron@veron.ch>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> svc/renew | 2 --
> 1 file changed, 2 deletions(-)
>
>diff --git a/svc/renew b/svc/renew
>index fa8475a..33ac4fa 100755
>--- a/svc/renew
>+++ b/svc/renew
>@@ -49,8 +49,6 @@ my $branchcode     = $input->param('branchcode')
> my $date_due;
> if ( $input->param('date_due') ) {
>     $date_due = dt_from_string( $input->param('date_due') );
>-    $date_due->set_hour(23);
>-    $date_due->set_minute(59);
> }
> 
> my $data;
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 11298: 41356 | 43254 | 43341 | 43569