https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24680 --- Comment #18 from Ere Maijala <ere.maijala@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.