[Koha-bugs] [Bug 24680] Hold modification endpoints don't always work properly

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

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

--- Comment #18 from Ere Maijala <ere.maijala at helsinki.fi> ---
(In reply to Jonathan Druart from comment #15)
> (In reply to Ere Maijala from comment #14)
> > Jonathan, unfortunately ModReserve wants them all, and I think that changing
> > it to make the call easier is far more risky.
> 
> Indeed!
> 
> However, should not we test for "exists" instead of "defined" (//)?
> As it, it will be impossible to set suspend_until to NULL, which I think is
> something we want to.

Right, that was indeed broken (in a couple of ways). I don't think this applies
to the other parameters, so I didn't change them. I also added a test to verify
setting suspended_until to null.

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