It would be nice to have a way to prevent patrons from placing holds at OPAC when they have overdues.
Created attachment 147494 [details] [review] Bug 33086: Allow to prevent holds at OPAC for a patron with overdues outstanding Test plan: 1) Add overdues to a patron 2) Set the OverduesBlockHolds system preference to "Don't block" 3) Check that you can place a hold for this patron, both at OPAC and Intranet 4) Set the OverduesBlockHolds system preference to "Block" 5) Check that you cannot place a hold for this patron at OPAC, with the following message: "This title cannot be requested because you have overdues" 6) Check that you can place a hold for this patron at Intranet, but with the following warning: "Patron has overdues"
Created attachment 147495 [details] [review] Bug 33086: Allow to prevent holds at OPAC for a patron with overdues outstanding Test plan: 1) Add overdues to a patron 2) Set the OverduesBlockHolds system preference to "Don't block" 3) Check that you can place a hold for this patron, both at OPAC and Intranet 4) Set the OverduesBlockHolds system preference to "Block" 5) Check that you cannot place a hold for this patron at OPAC, with the following message: "This title cannot be requested because you have overdues" 6) Check that you can place a hold for this patron at Intranet, but with the following warning: "Patron has overdues"
Created attachment 147571 [details] [review] Bug 33086: Allow to prevent holds at OPAC for a patron with overdues outstanding Test plan: 1) Add overdues to a patron 2) Set the OverduesBlockHolds system preference to "Don't block" 3) Check that you can place a hold for this patron, both at OPAC and Intranet 4) Set the OverduesBlockHolds system preference to "Block" 5) Check that you cannot place a hold for this patron at OPAC, with the following message: "This title cannot be requested because you have overdues" 6) Check that you can place a hold for this patron at Intranet, but with the following warning: "Patron has overdues" Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
I think the syspref name needs to be more explicit: OverduesBlockPlacingHolds - Holds are not blocked for the patron - i.e. existing holds can be filled, this should be in the description - I feel we could block earlier too, disabling the hold button on the OPAC, but that's just a suggestion - I think we need the check added to 'CanItemBeReserved' as well - ILSDI would allow the hold without, and there may be more
Maybe we could think of something depending on patron category, close to BlockExpiredPatronOpacActions' fro Bug 6739
In my opinion preference should be a boolean Yes/No