https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22806 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #40 from Nick Clemens <nick@bywatersolutions.com> --- (In reply to Arthur Suzuki from comment #39)
(In reply to Nick Clemens from comment #37)
This works as described, I do see that we have checks for this in reserve.pl and opac-reserve.pl - can they be removed now that they are being added to the modules?
Indeed! Done in the QA patch :)
Thanks Arthur! Two things I note: 1 - With this patchset the message to the patron changes from" This title cannot be requested because it's already in your possession. To: There are no items that can be placed on hold. We should pass the exact reason if possible 2 - On the staff side before these patches I am warned that patron has this in possession, but I can still place the hold After the patches I cannot place a hold unless AllowHoldPolicyOverride is set This is probably appropriate, but it is a behaviour change and should be noted in release notes -- You are receiving this mail because: You are watching all bug changes.