[Koha-bugs] [Bug 10067] decreaseLoanHighHolds messes with specify due date

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10067

Viktor Sarge <viktor.sarge at regionhalland.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |viktor.sarge at regionhalland.
                   |                            |se

--- Comment #4 from Viktor Sarge <viktor.sarge at regionhalland.se> ---
I've been giving this some thought and I don't mind the current workflow where
you get a chance to tell a patron that the book will have a shorter loan period
than usual. But the problem is that decreaseLoanHighHolds hooks into
duedatespec and overwrites it. So even if you auto clear after using highholds
you will lose any date you had entered manually. 

My two cents is that this could be solved with this workflow: 
1) When highholds is active pick upp anything stored in "duedatespec" and stick
it into the new parameter "specifiedDueDateToRestore" 
2) Let highholds overwrite duedatespec as it currently do. 
3) When circulation.pl finds a non empty specifiedDueDateToRestore parameter
stick this back into duedatespec. 

This way the due date from highholds will be the one off value it's supposed to
be and manually defined due dates will be restored. 

I've been looking at 10067 from a few different angles and this is what I can
come up with that does'nt change peoples current workflow. I have yet to
discover any fatal flaws with this approach althoguh there might be several for
all I know.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.