[Bug 11250] New: "too many holds" message appears inappropriately
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Bug ID: 11250 Summary: "too many holds" message appears inappropriately 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: koha@univ-lyon3.fr QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com In the staff interface, when placing a hold from the holds tab of a detailed record, the message "Cannot place hold Too Many Holds: Johnny SuchOne has too many holds" appears even if the unique reason is that there 's no holds allowed for the borrower. seems because of this line (request.pl line 193) : unless ( $canbookbereserved ) { $maxreserves = 1; } -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@univ-lyon3.fr See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=10247 Severity|enhancement |normal -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Heather Braum <hbraum@nekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hbraum@nekls.org -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Aaron Sakovich <asakovich@hmcpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |asakovich@hmcpl.org --- Comment #1 from Aaron Sakovich <asakovich@hmcpl.org> --- Additionally, placing a hold on an item that is not allowed to be held (Holds allowed count = 0), will result in the "{username} has too many holds" instead of a more meaningful or pertinent "{item} is not allowed to be placed on hold". -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #2 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 40759 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40759&action=edit Bug 11250 - "too many holds" message appears inappropriately This patch gives the various reasons precedence in this order: 1) Already in possesion 2) Already on hold 3) Age restricted 4) No items available 5) Too many holds Test Plan: 1) Attempt to place a hold where both "too many holds" and "no items available" messages display 2) Apply this patch 3) Refresh 4) Note only the "no items available" message displays -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #3 from Nick Clemens <nick@quecheelibrary.org> --- *** Bug 10912 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #4 from M. Tompsett <mtompset@hotmail.com> --- Comment on attachment 40759 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40759 Bug 11250 - "too many holds" message appears inappropriately Review of attachment 40759: --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=11250&attachment=40759) ----------------------------------------------------------------- Though I like the shift to the IF/ELSIF structure, I'm not sure it addresses the first two comments. ::: koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ +309,5 @@
+ <li><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowerfirstname %] [% borrowersurname %]</a> <strong>already has a hold</strong> on this item </li> + [% ELSIF ( ageRestricted ) %] + <li><strong>Age restricted</strong></li> + [% ELSIF ( none_available ) %] + <li> <strong>No items are available</strong> to be placed on hold</li>
There is a difference between none_available (but eventually some may) vs. comment #2's "{item} is not allowed to be placed on hold". Is there a way to check for this? @@ +311,5 @@
+ <li><strong>Age restricted</strong></li> + [% ELSIF ( none_available ) %] + <li> <strong>No items are available</strong> to be placed on hold</li> + [% ELSIF ( maxreserves ) %] + <li><strong>Too many holds: </strong> <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowerfirstname %] [% borrowersurname %] </a> has too many holds.</li>
Similarly there is a minor difference between a borrower who has too many holds (non-zero), and one who isn't allowed to place any at all, as per my understanding of comment #1. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtompset@hotmail.com --- Comment #5 from M. Tompsett <mtompset@hotmail.com> --- Sorry for the off-by-1 error in my comment. :( -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Amit Gupta <amitddng135@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |amitddng135@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Joonas Kylmälä <j.kylmala@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |j.kylmala@gmail.com --- Comment #6 from Joonas Kylmälä <j.kylmala@gmail.com> --- (In reply to M. Tompsett from comment #4)
There is a difference between none_available (but eventually some may) vs. comment #2's "{item} is not allowed to be placed on hold". Is there a way to check for this?
What do you mean with this paragraph? I didn't understand it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #7 from M. Tompsett <mtompset@hotmail.com> --- (In reply to Joonas Kylmälä from comment #6)
(In reply to M. Tompsett from comment #4)
There is a difference between none_available (but eventually some may) vs. comment #2's "{item} is not allowed to be placed on hold". Is there a way to check for this?
What do you mean with this paragraph? I didn't understand it.
"Too many holds" -- But I don't have any holds on the book at all! In this case, there is some sort of holds queue limitation. "None available" -- Because there is a holds limitation preventing a hold from being placed. "Too many holds" -- Right, because some items just can't be put on hold period. This is a specific case of the previous one. "None available" -- And there never will be, because it can't be put on hold. "Too many holds" -- Oh, but I'm insane and taking a quadruple major and need all 20 books! In this case, the patron is holds-happy putting everything they can on hold. "None available" -- Because you are clearly an holds-happy patron! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #8 from Joonas Kylmälä <j.kylmala@gmail.com> --- M. Tompsett: Still don't get what do you mean. What do you mean by "but eventually some may". And what is the thing you're trying to say with that there is a difference between none_available and "{item} is not allowed to be placed on hold"? Or if you could rephrase the thing what you suggested to be done here: ::: koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ +309,5 @@
+ <li><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowerfirstname %] [% borrowersurname %]</a> <strong>already has a hold</strong> on this item </li> + [% ELSIF ( ageRestricted ) %] + <li><strong>Age restricted</strong></li> + [% ELSIF ( none_available ) %] + <li> <strong>No items are available</strong> to be placed on hold</li>
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Joonas Kylmälä <j.kylmala@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |major -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Zeno Tajoli <z.tajoli@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch CC| |z.tajoli@cineca.it -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Zeno Tajoli <z.tajoli@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|z.tajoli@cineca.it | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40759|0 |1 is obsolete| | --- Comment #9 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 42657 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42657&action=edit Bug 11250 - "too many holds" message appears inappropriately This patch gives the various reasons precedence in this order: 1) Already in possesion 2) Already on hold 3) Age restricted 4) No items available 5) Too many holds Test Plan: 1) Attempt to place a hold where both "too many holds" and "no items available" messages display 2) Apply this patch 3) Refresh 4) Note only the "no items available" message displays Signed-off-by: Heather Braum <hbraum@nekls.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #10 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I don't understand anything from comments 4 to 8, could you please clarify? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Kyle M Hall <kyle.m.hall@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=12197 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Kyle M Hall <kyle.m.hall@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |12197 Referenced Bugs: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12197 [Bug 12197] Exceeding the maxreserves preference does not prevent librarian from placing the hold -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Same here about 4-8. New sequence looks good to me, but is there a reason we don't display all reasons why a hold can not be placed? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Kyle M Hall <kyle.m.hall@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle.m.hall@gmail.com --- Comment #12 from Kyle M Hall <kyle.m.hall@gmail.com> ---
There is a difference between none_available (but eventually some may) vs. comment #2's "{item} is not allowed to be placed on hold". Is there a way to check for this?
If I understand you correctly, the answer is no at this time. That would be outside the scope of this bug.
Similarly there is a minor difference between a borrower who has too many holds (non-zero), and one who isn't allowed to place any at all, as per my understanding of comment #1.
That is correct, and this patch deals with that situation. The patron will get different messages if they exceed the maxreserves vs they can't place a hold on any of the items. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #13 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Katrin Fischer from comment #11)
Same here about 4-8. New sequence looks good to me, but is there a reason we don't display all reasons why a hold can not be placed?
Kyle, I am not sure this patch addresses correctly the problem. I think the logic should be reworked in the script. But yes I know it will certainly be more difficult that just switching some conditional blocks in the template, so maybe it is worth to go first with this patch. Could you please answer Katrin? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #14 from Kyle M Hall <kyle.m.hall@gmail.com> --- (In reply to Katrin Fischer from comment #11)
Same here about 4-8. New sequence looks good to me, but is there a reason we don't display all reasons why a hold can not be placed?
Basically, it causes confusion for librarians. For example, iirc, a patron may not be able to place holds on any of the items, and thus the librarian receives the "too many holds" error which is not actually true. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #15 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Kyle M Hall from comment #14)
(In reply to Katrin Fischer from comment #11)
Same here about 4-8. New sequence looks good to me, but is there a reason we don't display all reasons why a hold can not be placed?
Basically, it causes confusion for librarians. For example, iirc, a patron may not be able to place holds on any of the items, and thus the librarian receives the "too many holds" error which is not actually true.
I agree that the 'too many' is often confusing, just wondering if we could be more precise as several of the reasons might apply at the same time - too many holds, restricted, too many fines... -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #16 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Huh, I realize now that we do currently display all reasons. I think this patch might just hide the real problem - that the "too_many" is not accurate enough to be helpful. With the patch it will show up less, but it will still show up sometimes if none of the other reasons applies. Maybe we could just rephrase? Too many holds: </strong> <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowerfirstname %] [% borrowersurname %] </a> has too many holds. <Patron> can't place holds on these item types or already has too many holds placed. Something like that? Can we pin down, when exactly the 'too many' is displayed and if that would cover it? Ultimately I think we need to make the feedback more meaningful, but would like the option to see if multiple things block placing holds. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #17 from Kyle M Hall <kyle.m.hall@gmail.com> --- (In reply to Katrin Fischer from comment #16)
Huh, I realize now that we do currently display all reasons.
I think this patch might just hide the real problem - that the "too_many" is not accurate enough to be helpful. With the patch it will show up less, but it will still show up sometimes if none of the other reasons applies.
Maybe we could just rephrase?
Too many holds: </strong> <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]">[% borrowerfirstname %] [% borrowersurname %] </a> has too many holds.
<Patron> can't place holds on these item types or already has too many holds placed.
Something like that? Can we pin down, when exactly the 'too many' is displayed and if that would cover it?
Ultimately I think we need to make the feedback more meaningful, but would like the option to see if multiple things block placing holds.
Rephrasing does nothing to fix the issue. The problem isn't that the too_many message is wrong, it's that it is triggered by almost all situations where a patron cannot place holds. That makes it not useful. This patch changes that so that it displays only if no other limitations exist, which means the patron really has simply placed too many holds. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 --- Comment #18 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- But it can also hide the information that you want to see - like no item available and age restriction could happen at the same time - that's what bugs me a bit about this approach. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42657|0 |1 is obsolete| | --- Comment #19 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 43748 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=43748&action=edit Bug 11250 - "too many holds" message appears inappropriately This patch gives the various reasons precedence in this order: 1) Already in possesion 2) Already on hold 3) Age restricted 4) No items available 5) Too many holds Test Plan: 1) Attempt to place a hold where both "too many holds" and "no items available" messages display 2) Apply this patch 3) Refresh 4) Note only the "no items available" message displays Signed-off-by: Heather Braum <hbraum@nekls.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11250 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |tomascohen@gmail.com --- Comment #20 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patch pushed to master. Thanks Kyle! -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org