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

Nick Clemens <[email protected]> changed:

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

--- Comment #40 from Nick Clemens <[email protected]> ---
(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.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to