[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Martin Renvoize changed: What|Removed |Added Resolution|--- |FIXED Status|Pushed to Stable|RESOLVED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Fridolin SOMERS changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable CC||fridolin.som...@biblibre.co ||m --- Comment #40 from Fridolin SOMERS --- Pushed to 17.11.x for 17.11.08 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Jonathan Druartchanged: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #39 from Jonathan Druart --- Pushed to master for 18.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #38 from Marcel de Rooy--- is_waiting_or_transit ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #37 from Marcel de Rooy--- (In reply to Katrin Fischer from comment #17) > is_found is also a bit confusing, because this also includes transit items. Yes, confirming it again. is_found_or_transit ? New report btw -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #36 from Marcel de Rooy--- This is a quick fix, the other point mentioned in earlier comments should go on a new enhancement. Why not add it to 18.05 still, Joubu ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Marcel de Rooychanged: What|Removed |Added Status|In Discussion |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #35 from Marcel de Rooy--- Created attachment 75395 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75395=edit Bug 18856: (QA follow-up) Rename method to avoid confusion Signed-off-by: Kyle M Hall Signed-off-by: Marcel de Rooy Hmm. I would have liked the opac parameter even better ;) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Marcel de Rooychanged: What|Removed |Added Attachment #70424|0 |1 is obsolete|| Attachment #70425|0 |1 is obsolete|| --- Comment #34 from Marcel de Rooy --- Created attachment 75394 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75394=edit Bug 18856: Don't show cancel option for waiting holds in OPAC This is the alternative patch of Kyle's """ If a hold is 'waiting' for the patron to collect then the patron should be prevented from cancelling the hold via their account in the opac. If a patron tries to cancel the hold, Koha will give an 'are you sure' alert and when you click Yes the page just refreshes and the hold remains. Staff can cancel the hold from the staff interface but they can then action the waiting hold. I think therefore that it is correct behaviour that a patron cannot cancel a hold when it reaches waiting state via the opac but it would be useful to either have a warning to prevent the cancellation or a useful message when they attempt to do so. The template was using a method that tells Koha if *staff* can cancel a hold, instead of patron. Test Plan: 1) Set up a waiting hold 2) Try to cancel it from the opac 3) Note you cannot 4) Apply this patch 5) Reload the page 6) Note the cancel button has disappeared for found holds """ It sounds better to keep the ->is_cancelable method, for readability Signed-off-by: Kyle M Hall Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #33 from Hugh Rundle--- If I am understanding correctly what the patch does (removes the cancel button from allocated holds), then yes, it qualifies as a quick fix. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #32 from Jonathan Druart--- What's wrong with the patches, could they be considered ok as a quick fix? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #31 from Hugh Rundle--- +1 on a quick fix, current behaviour is baffling to patrons. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #30 from Katrin Fischer--- We have another library complaining about this :( I think at the moment, we just need a quick fix that doesn't appear broken to the patron. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #29 from Katrin Fischer--- I agree with Hugh and Liz that we might want to rethink the policy of not allowing patrons to cancel waiting holds, but think the fix might need a bit more thought. Would we like to know that the user cancelled it (for statistics)? My suggestion would be to fix the current behaviour (not allow cancelling but give a good message/not show the button) for the stable releases and do a better fix for the next release that allows to change the behaviour. Given we have a feature in Koha to 'punish' people who don't pick up their waiting items (ExpireReservesMaxPickUpDelayCharge), libraries might have different ideas about this? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #28 from Hugh Rundle--- I like Liz's suggested way of dealing with it. On the topic of alerts, there is a related problem in that the holds cancellation alert is quite confusing anyway, because "ok" cancels the hold and "cancel" doesn't. I've listed this as a separate bug since it affects unallocated holds as well: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20278 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Liz Reachanged: What|Removed |Added CC||l...@catalyst.net.nz --- Comment #27 from Liz Rea --- I'm with Hugh. What should probably happen here, is that if a user cancels the waiting hold from the OPAC, it should be immediately expired, and put on the "expired holds" ("holds waiting over x days) list in the staff interface. Items on this list are necessarily actioned by librarians, i.e. pulled off the hold shelf and returned to circulation or moved on to the next borrower. If we need a message for the borrower, it should be something like "This hold is waiting for you at the library, canceling this hold will remove you from the list and immediately make the item available to other library users. Cancel this hold anyway? / Do not cancel this hold" We could even do a separate tab for these on that waitingreserves page, if that makes the decision easier. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Hugh Rundlechanged: What|Removed |Added CC||hu...@brimbank.vic.gov.au --- Comment #26 from Hugh Rundle --- I don't want to derail a quick bugfix given that the current behaviour is clearly misleading, but is there a reason this initial assumption has gone unchallenged? >If a hold is 'waiting' for the patron to collect then the patron should be >prevented from cancelling the hold via their account in the opac. Why? Surely we should be making it easy for patrons to self-manage their loans and holds. As a library patron I would expect to be able to cancel a hold at any time. If the problem is that staff need to be notified somehow about cancelled allocated holds then it would be better to tackle that. Would that be a new bug or have i missed something? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Juan Romay Sieirachanged: What|Removed |Added CC||juan.sie...@xercode.es --- Comment #25 from Juan Romay Sieira --- I think de better option is to add a parameter in is_cancelable, what Marcel explains -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Margaretchanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=20164 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Jonathan Druartchanged: What|Removed |Added Status|Passed QA |In Discussion --- Comment #24 from Jonathan Druart --- Looking at bug 4319: We also have C4::Reserves::IsItemOnHoldAndFound that does exactly the same thing. So maybe "is_cancelable" is too specific? What could we do to keep this code simple, understandable and consistent? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #23 from Barton Chittenden--- (In reply to Marcel de Rooy from comment #21) > (In reply to Jonathan Druart from comment #16) > > Yes Barton you are right, that's why I added a comment to the POD of the > > method to tell it only applies for the OPAC. > > Maybe rename the method to something more verbose (is_cancelable_from_opac) > > to remove the ambiguity? > > Maybe include a parameter for opac/intranet/api ? > $hold->is_cancelable{( from => 'opac' or opac=>1 }) ? This strikes the right balance for me. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #22 from Katrin Fischer--- You are absolutely right, Marcel. Tired brain. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #21 from Marcel de Rooy--- (In reply to Jonathan Druart from comment #16) > Yes Barton you are right, that's why I added a comment to the POD of the > method to tell it only applies for the OPAC. > Maybe rename the method to something more verbose (is_cancelable_from_opac) > to remove the ambiguity? Maybe include a parameter for opac/intranet/api ? $hold->is_cancelable{( from => 'opac' or opac=>1 }) ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #20 from Marcel de Rooy--- (In reply to Katrin Fischer from comment #17) > is_found is also a bit confusing, because this also includes transit items. > found = F in the reserves table is for waiting. So you might not think about > the transit items as well immediately. It confused me on testing this. found=W is waiting found=F is finished (only in deleted reserves) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #19 from Kyle M Hall--- Created attachment 70425 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70425=edit Bug 18856 [QA Followup] - Rename method to avoid confusion Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Kyle M Hallchanged: What|Removed |Added Attachment #70398|0 |1 is obsolete|| Attachment #70399|0 |1 is obsolete|| Attachment #70419|0 |1 is obsolete|| --- Comment #18 from Kyle M Hall --- Created attachment 70424 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70424=edit Bug 18856: Don't show cancel option for waiting holds in OPAC This is the alternative patch of Kyle's """ If a hold is 'waiting' for the patron to collect then the patron should be prevented from cancelling the hold via their account in the opac. If a patron tries to cancel the hold, Koha will give an 'are you sure' alert and when you click Yes the page just refreshes and the hold remains. Staff can cancel the hold from the staff interface but they can then action the waiting hold. I think therefore that it is correct behaviour that a patron cannot cancel a hold when it reaches waiting state via the opac but it would be useful to either have a warning to prevent the cancellation or a useful message when they attempt to do so. The template was using a method that tells Koha if *staff* can cancel a hold, instead of patron. Test Plan: 1) Set up a waiting hold 2) Try to cancel it from the opac 3) Note you cannot 4) Apply this patch 5) Reload the page 6) Note the cancel button has disappeared for found holds """ It sounds better to keep the ->is_cancelable method, for readability Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Kyle M Hallchanged: What|Removed |Added Status|In Discussion |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Axelle Aix-Marseille Universitéchanged: What|Removed |Added CC||axelle.clari...@univ-amu.fr -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #17 from Katrin Fischer--- is_found is also a bit confusing, because this also includes transit items. found = F in the reserves table is for waiting. So you might not think about the transit items as well immediately. It confused me on testing this. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #16 from Jonathan Druart--- Yes Barton you are right, that's why I added a comment to the POD of the method to tell it only applies for the OPAC. Maybe rename the method to something more verbose (is_cancelable_from_opac) to remove the ambiguity? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Jonathan Druartchanged: What|Removed |Added Attachment #70398|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #15 from Barton Chittenden--- (In reply to Jonathan Druart from comment #14) > Kyle, Katrin and Marcel, could you take a look at this l > ast patch? > It sounds better to me to keep the method and fix its behaviour instead of > testing is_found that can be confusing. Feel free to obsolete it if you > disagree and I will pushed Kyle's patches with a follow-up to remove the > method and the tests. The problem with 'is_cancelable' is that staff and opac have different views of whether the item is cancelable or not, whereas is_found is un-ambiguous. is_cancelable *sounds* like it's clearer because it signals its intent, but under the circumstances, I think 'is_found' is better. Just my humble opinion; I'm also probably missing context because I haven't taken the time to read the code, so take my words with a large grain of salt. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Jonathan Druartchanged: What|Removed |Added Status|Passed QA |In Discussion --- Comment #14 from Jonathan Druart --- Kyle, Katrin and Marcel, could you take a look at this l ast patch? It sounds better to me to keep the method and fix its behaviour instead of testing is_found that can be confusing. Feel free to obsolete it if you disagree and I will pushed Kyle's patches with a follow-up to remove the method and the tests. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Jonathan Druartchanged: What|Removed |Added Attachment #70398|0 |1 is obsolete|| --- Comment #13 from Jonathan Druart --- Created attachment 70419 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70419=edit [ALTERNATIVE-PATCH] Bug 18856: Don't show cancel option for waiting holds in OPAC This is the alternative patch of Kyle's """ If a hold is 'waiting' for the patron to collect then the patron should be prevented from cancelling the hold via their account in the opac. If a patron tries to cancel the hold, Koha will give an 'are you sure' alert and when you click Yes the page just refreshes and the hold remains. Staff can cancel the hold from the staff interface but they can then action the waiting hold. I think therefore that it is correct behaviour that a patron cannot cancel a hold when it reaches waiting state via the opac but it would be useful to either have a warning to prevent the cancellation or a useful message when they attempt to do so. The template was using a method that tells Koha if *staff* can cancel a hold, instead of patron. Test Plan: 1) Set up a waiting hold 2) Try to cancel it from the opac 3) Note you cannot 4) Apply this patch 5) Reload the page 6) Note the cancel button has disappeared for found holds """ It sounds better to keep the ->is_cancelable method, for readability -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Katrin Fischerchanged: What|Removed |Added Attachment #69077|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Katrin Fischerchanged: What|Removed |Added Attachment #70294|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #12 from Katrin Fischer--- Created attachment 70399 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70399=edit Bug 18856: (follow-up) Catch missed instance of is_cancelable Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #11 from Katrin Fischer--- Created attachment 70398 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70398=edit Bug 18856: Don't show cancel option for waiting holds in OPAC If a hold is 'waiting' for the patron to collect then the patron should be prevented from cancelling the hold via their account in the opac. If a patron tries to cancel the hold, Koha will give an 'are you sure' alert and when you click Yes the page just refreshes and the hold remains. Staff can cancel the hold from the staff interface but they can then action the waiting hold. I think therefore that it is correct behaviour that a patron cannot cancel a hold when it reaches waiting state via the opac but it would be useful to either have a warning to prevent the cancellation or a useful message when they attempt to do so. The template was using a method that tells Koha if *staff* can cancel a hold, instead of patron. Test Plan: 1) Set up a waiting hold 2) Try to cancel it from the opac 3) Note you cannot 4) Apply this patch 5) Reload the page 6) Note the cancel button has disappeared for found holds Signed-off-by: Kyle M Hall Signed-off-by: Caroline Cyr La Rose Signed-off-by: Marcel de Rooy Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Katrin Fischerchanged: What|Removed |Added Status|Signed Off |Passed QA --- Comment #10 from Katrin Fischer --- Fixed the patch subjects according to guidelines (: (follow-up) state what was fixed) Should is_cancelable be removed after this change? It seems the only remaining occurrences are in tests: Koha/Hold.pm:=head3 is_cancelable Koha/Hold.pm:sub is_cancelable { t/db_dependent/Hold.t:# Test method is_cancelable t/db_dependent/Hold.t:ok( $hold->is_cancelable(), "Unfound hold is cancelable" ); t/db_dependent/Hold.t:ok( $hold->is_cancelable, "Waiting hold is cancelable" ); t/db_dependent/Hold.t:ok( !$hold->is_cancelable, "In transit hold is not cancelable" ); Sorry Marcel, forgot to check the QA contact first! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #9 from Kyle M Hall--- Created attachment 70294 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70294=edit Bug 18856 [QA Followup] - Catch missed instance of is_cancelable -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Kyle M Hallchanged: What|Removed |Added Status|In Discussion |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Jessie Zairochanged: What|Removed |Added CC||jza...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Jonathan Druartchanged: What|Removed |Added Status|Passed QA |In Discussion CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #8 from Jonathan Druart --- What about the other occurrence of is_cancelable in the same template file? 695 [% IF ( RESERVE.is_cancelable ) %] -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Marcel de Rooychanged: What|Removed |Added Version|16.11 |master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Marcel de Rooychanged: What|Removed |Added Status|Signed Off |Passed QA Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Marcel de Rooychanged: What|Removed |Added QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Marcel de Rooychanged: What|Removed |Added Attachment #67520|0 |1 is obsolete|| --- Comment #7 from Marcel de Rooy --- Created attachment 69077 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69077=edit Bug 18856 - Cancel Waiting Hold in OPAC does not give useful message If a hold is 'waiting' for the patron to collect then the patron should be prevented from cancelling the hold via their account in the opac. If a patron tries to cancel the hold, Koha will give an 'are you sure' alert and when you click Yes the page just refreshes and the hold remains. Staff can cancel the hold from the staff interface but they can then action the waiting hold. I think therefore that it is correct behaviour that a patron cannot cancel a hold when it reaches waiting state via the opac but it would be useful to either have a warning to prevent the cancellation or a useful message when they attempt to do so. The template was using a method that tells Koha if *staff* can cancel a hold, instead of patron. Test Plan: 1) Set up a waiting hold 2) Try to cancel it from the opac 3) Note you cannot 4) Apply this patch 5) Reload the page 6) Note the cancel button has disappeared for found holds Signed-off-by: Kyle M Hall Signed-off-by: Caroline Cyr La Rose Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Marcel de Rooychanged: What|Removed |Added CC||m.de.r...@rijksmuseum.nl --- Comment #6 from Marcel de Rooy --- Just confirming the above by doublechecking the code: Cancel button calls opac-modrequest.pl (once called only in opac-user) opac-modrequest.pl checks with CanReserveBeCanceledFromOpac. This sub returns false for waiting or transit. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Kyle M Hallchanged: What|Removed |Added Attachment #67480|0 |1 is obsolete|| --- Comment #5 from Kyle M Hall --- Created attachment 67520 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67520=edit Bug 18856 - Cancel Waiting Hold in OPAC does not give useful message If a hold is 'waiting' for the patron to collect then the patron should be prevented from cancelling the hold via their account in the opac. If a patron tries to cancel the hold, Koha will give an 'are you sure' alert and when you click Yes the page just refreshes and the hold remains. Staff can cancel the hold from the staff interface but they can then action the waiting hold. I think therefore that it is correct behaviour that a patron cannot cancel a hold when it reaches waiting state via the opac but it would be useful to either have a warning to prevent the cancellation or a useful message when they attempt to do so. The template was using a method that tells Koha if *staff* can cancel a hold, instead of patron. Test Plan: 1) Set up a waiting hold 2) Try to cancel it from the opac 3) Note you cannot 4) Apply this patch 5) Reload the page 6) Note the cancel button has disappeared for found holds Signed-off-by: Kyle M Hall Signed-off-by: Caroline Cyr La Rose -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Caroline Cyr La Rosechanged: What|Removed |Added Status|Needs Signoff |Signed Off CC||caroline.cyr-la-rose@inlibr ||o.com --- Comment #4 from Caroline Cyr La Rose --- Works as described! :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Kyle M Hallchanged: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Kyle M Hallchanged: What|Removed |Added CC||k...@bywatersolutions.com Severity|enhancement |normal Assignee|oleon...@myacpl.org |k...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #3 from Kyle M Hall--- Created attachment 67480 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67480=edit Bug 18856 - Cancel Waiting Hold in OPAC does not give useful message If a hold is 'waiting' for the patron to collect then the patron should be prevented from cancelling the hold via their account in the opac. If a patron tries to cancel the hold, Koha will give an 'are you sure' alert and when you click Yes the page just refreshes and the hold remains. Staff can cancel the hold from the staff interface but they can then action the waiting hold. I think therefore that it is correct behaviour that a patron cannot cancel a hold when it reaches waiting state via the opac but it would be useful to either have a warning to prevent the cancellation or a useful message when they attempt to do so. The template was using a method that tells Koha if *staff* can cancel a hold, instead of patron. Test Plan: 1) Set up a waiting hold 2) Try to cancel it from the opac 3) Note you cannot 4) Apply this patch 5) Reload the page 6) Note the cancel button has disappeared for found holds -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 --- Comment #2 from Katrin Fischer--- I did some digging and it looks like there should be no button when the hold is waiting or in transfer already. (see bug 12876) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Katrin Fischerchanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=12876 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=2812 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 18856] Cancel Waiting Hold in OPAC does not give useful message
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18856 Barton Chittendenchanged: What|Removed |Added CC||bar...@bywatersolutions.com --- Comment #1 from Barton Chittenden --- I think that this is an actual bug -- it's confusing to patrons who think that their hold was actually cancelled, and it doesn't give them any indication that they need to talk to the library who would actually have to cancel the hold. --Barton -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/