Bug 10067: decreaseLoanHighHolds messes with specify due date
authorEivin Giske Skaaren <eskaaren@yahoo.no>
Mon, 25 May 2015 17:10:54 +0000 (17:10 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 17 Nov 2015 13:02:35 +0000 (10:02 -0300)
commit16213187d91695babb08da4301aeffb2517bf753
treefc97977c1aba46257fbf4da8d72bf2ff1d7f8acd
parent0fe08231f427bdc65b1ff1219ad7507c284f1dbd
Bug 10067: decreaseLoanHighHolds messes with specify due date

To test:

1. On a branch up to date with master: apply the patch.

2. Try to check out items both with and without holds.
(The decreaseLoanHighHolds function should be configured and kick in when testing an item.

3. Expected results are:
The due date date field should be cleared after checkout if "Remember for session"
is not selected. If it is selected then the chosen date should stay in the field even
if the warning about reduced loan time for an item with too many holds is displayed.
Also the correct due dates are shown in the table after a checkout.

Known limitation: Does not work too well when jumping between different parts
of the system as it relies on the values sent to the circulation.pl script.

Sponsored-by: Halland County Library
Refactored to both fix specific issue where date is changed
and still have the old behaviour with date kept intact.

Rebase
Remove code whitespace cleanup

Remove one more code whitespace cleanup

Removed duplicate tag

Refactored stickyduedate session parameter

Fix datefield set by highholds if blanc

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
circ/circulation.pl
koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt