[Koha-bugs] [Bug 22806] CanBookBeReserve and CanItemBeReserve do not check AllowHoldsOnPatronsPossessions

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

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

Nick Clemens <nick at bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #40 from Nick Clemens <nick at 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.