[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Baptistechanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=18885 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Julian Mauricechanged: What|Removed |Added Status|Pushed to Master|Pushed to Stable CC||julian.maur...@biblibre.com --- Comment #12 from Julian Maurice --- Patch pushed to 3.22.x, will be in 3.22.10 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Frédéric Demianschanged: What|Removed |Added CC||frede...@tamil.fr --- Comment #11 from Frédéric Demians --- Pushed in 16.05. Will be in 16.05.02. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 J Schmidtchanged: What|Removed |Added CC||jschm...@switchinc.org -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Kirstenchanged: What|Removed |Added CC||direc...@sandownlibrary.us --- Comment #10 from Kirsten --- Can we have this fix backported to 3.22? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Kyle M Hallchanged: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #9 from Kyle M Hall --- Pushed to master for 16.11, thanks Jonathan! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Katrin Fischerchanged: What|Removed |Added Attachment #52508|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 --- Comment #8 from Katrin Fischer--- Created attachment 52550 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52550=edit [PASSED QA] 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 Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Jonathan Druartchanged: What|Removed |Added Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Jonathan Druartchanged: What|Removed |Added Attachment #51701|0 |1 is obsolete|| --- Comment #7 from Jonathan Druart --- Created attachment 52508 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52508=edit 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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Kyle M Hallchanged: What|Removed |Added CC||k...@bywatersolutions.com Status|Signed Off |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 --- Comment #6 from Eivin Giske Skaaren--- This change seems to fix the issues raised by Nick but I also noticed when testing and trying to understand 10067 again that there has been other changes to the parts of Koha we are dealing with here.. (And the mix of TT, jQuery and Perl to handle this together with the highholds is pretty confusing.) Another regression might be that the decreaseLoanHighHolds does not work anymore. There was several changes to that since last I looked into this and when testing on a new kohadevbox the checkout length was not decreased, even when the new checkbox "Don't decrease checkout length based on holds" was not selected. That has nothing to do with this patch however so I signed off. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Eivin Giske Skaarenchanged: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Eivin Giske Skaarenchanged: What|Removed |Added Attachment #51673|0 |1 is obsolete|| --- Comment #5 from Eivin Giske Skaaren --- Created attachment 51701 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51701=edit [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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 --- Comment #4 from Eivin Giske Skaaren--- I am without a koha dev env because my laptop died.. Installing another computer today. The stickyduedate behaviour and 10067 is confusing because there is some jquery code that intercepts the request and I had to send that data to serverside somehow to be able to manipulate. (from the top of my head..) If I am able to get a dev env up and running I will look into it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Jonathan Druartchanged: What|Removed |Added Assignee|koha-b...@lists.koha-commun |jonathan.dru...@bugs.koha-c |ity.org |ommunity.org -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 --- Comment #3 from Jonathan Druart--- Created attachment 51673 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51673=edit 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!) -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Jonathan Druartchanged: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Jonathan Druartchanged: What|Removed |Added Severity|normal |major --- Comment #2 from Jonathan Druart --- This should be fixed before the next release (soon!) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Jonathan Druartchanged: What|Removed |Added Depends on||10067 CC||ei...@sysmystic.com, ||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #1 from Jonathan Druart --- It seems that it is a regression introduced by bug 10067. I'd say that the following diff will fix it: 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; } But I have no idea if it won't break bug 10067. Eivin, could you have a look please? Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10067 [Bug 10067] decreaseLoanHighHolds messes with specify due date -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16527] Sticky due date calendar unexpected behaviour
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16527 Nick Clemenschanged: What|Removed |Added Summary|Stick due date calendar |Sticky due date calendar |unexpected behaviour|unexpected behaviour -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/