Attachment 51701 Details for Bug 16527 – [SIGNED-OFF] Bug 16527: Restore sticky due date behavior

[patch] [SIGNED-OFF] Bug 16527: Restore sticky due date behavior

SIGNED-OFF-Bug-16527-Restore-sticky-due-date-behav.patch (text/plain), 1.66 KB, created by Eivin Giske Skaaren on 2016-05-22 19:47:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Eivin Giske Skaaren

Created: 2016-05-22 19:47:42 UTC

Size: 1.66 KB

patch

obsolete

>From 2b8342d351c98349ca150dc35e269bd60ca02cd2 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Sat, 21 May 2016 09:10:46 +0100
>Subject: [PATCH] [SIGNED-OFF] Bug 16527: Restore sticky due date behavior
>
>Bug 10067 has introduced a regression on the sticky due date behavior:
>If "remember for session" is checked and the due date is changed from 1
>checkout to another, the old due date will be used.
>
>The code added by bug 10067 is really confusing and it's hard to say
>this patch won't introduce another regression.
>The manipulation of $restoreduedatespec, $duedatespec and $stickyduedate
>is not easy to follow.
>
>Test plan:
>Confirm that the regression is fixed and that this patch will not
>introduced a regression on bug 10067 (good luck!)
>
>Signed-off-by: Eivin Giske Skaaren <eivin@sysmystic.com>
>---
> circ/circulation.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/circ/circulation.pl b/circ/circulation.pl
>index 66b3dcf..d7fcfbb 100755
>--- a/circ/circulation.pl
>+++ b/circ/circulation.pl
>@@ -166,7 +166,7 @@ my $stickyduedate  = $query->param('stickyduedate') || $session->param('stickydu
> my $duedatespec    = $query->param('duedatespec')   || $session->param('stickyduedate');
> $duedatespec = eval { output_pref( { dt => dt_from_string( $duedatespec ), dateformat => 'iso' }); }
>     if ( $duedatespec );
>-my $restoreduedatespec  = $query->param('restoreduedatespec') || $session->param('stickyduedate') || $duedatespec;
>+my $restoreduedatespec  = $query->param('restoreduedatespec') || $duedatespec || $session->param('stickyduedate');
> if ($restoreduedatespec eq "highholds_empty") {
>     undef $restoreduedatespec;
> }
>-- 
>2.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 16527: 51673 | 51701 | 52508 | 52550