[Bug 33086] New: Allow to prevent holds at OPAC for a patron with overdues outstanding
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Bug ID: 33086 Summary: Allow to prevent holds at OPAC for a patron with overdues outstanding Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs@lists.koha-community.org Reporter: matthias.meusburger@biblibre.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com It would be nice to have a way to prevent patrons from placing holds at OPAC when they have overdues. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 --- Comment #1 from Matthias Meusburger <matthias.meusburger@biblibre.com> --- Created attachment 147494 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=147494&action=edit 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" -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Matthias Meusburger <matthias.meusburger@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Matthias Meusburger <matthias.meusburger@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |matthias.meusburger@biblibr |ity.org |e.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Matthias Meusburger <matthias.meusburger@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #147494|0 |1 is obsolete| | --- Comment #2 from Matthias Meusburger <matthias.meusburger@biblibre.com> --- Created attachment 147495 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=147495&action=edit 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" -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #147495|0 |1 is obsolete| | --- Comment #3 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 147571 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=147571&action=edit 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> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |nick@bywatersolutions.com --- Comment #4 from Nick Clemens <nick@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 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #5 from Fridolin Somers <fridolin.somers@biblibre.com> --- Maybe we could think of something depending on patron category, close to BlockExpiredPatronOpacActions' fro Bug 6739 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- In my opinion preference should be a boolean Yes/No -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com --- Comment #7 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Please, make it dependent on bug 40101. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=40101 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|matthias.meusburger@biblibr |tomascohen@gmail.com |e.com | Status|Failed QA |ASSIGNED --- Comment #8 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- Hey folks, I have a working implementation for this one, using circulation rules instead. Making up my mind on how to organize things in the UI before submission. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 --- Comment #9 from Fridolin Somers <fridolin.somers@biblibre.com> --- (In reply to Tomás Cohen Arazi (tcohen) from comment #8)
Hey folks, I have a working implementation for this one, using circulation rules instead. Making up my mind on how to organize things in the UI before submission.
Looks better to have circulation rules so it can depend on patron category + library + itemtype . Good luck ;) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33086 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |40101 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40101 [Bug 40101] Add `Koha::Patron->can_place_holds` -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org