[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates

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

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239

--- Comment #3 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
I've rebased the patches and done a first code review. I like the concept of
this as it's small and flexible. 

I wonder, if you are still around for this Magnus? I can spend some time on ILL
developments right now as we are looking into a backend for our ILL workflow
and this would help a great deal.

Looking at the code it seems like providing no due date in the ill request
would be problematic. I'd suggest to check if the date is set and then have a
fallback to normal due_date calculation if it isn't. 

There is also a leftover warn.

I can also sign-off, but I think for QA we'd need tests...

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