[Bug 18438] New: Check in: Modal about holds hides important check in messages
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Bug ID: 18438 Summary: Check in: Modal about holds hides important check in messages Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: veron@veron.ch QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com To reproduce: - Have a patron X who has an item checked out where an other patron has a hold on it - Make sure patron X triggers a message while checking in (e.g. add a manual fine, add a manual restriction) - Check the item in on Home > Circulation > Check in (/koha/circ/circulation.pl) Result: A modal window appears that hides the messages about fines, restriction. 'Confirm hold' triggers the check-in; librarians have noch chance to see the other messages. Introduced by Bug 11360. See also comment #20 on Bug 11360 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11360#c20 "If a checkin triggers the modal, any messages or alerts which were also triggered by the checkin will be hidden by the modal and will disappear after clearing the modal. For instance: A warning that the item was not checked out, or a checkin message which has been configured for that particular item type." -- 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=18438 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |11360 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11360 [Bug 11360] Disable barcode field and submit button when a hold is found -- 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=18438 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=14224 -- 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=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |alexbuckley@catalyst.net.nz |ity.org | CC| |alexbuckley@catalyst.net.nz -- 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=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #1 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 62349 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62349&action=edit Bug 18438 - Implemented data-dismiss="modal" attribute on 'Confirm hold' and 'confirm' buttons so when they are cliked the modal box dissapears and any warning messages that were hidden by the box are now visible. The form is submitted via the js onclick function when the 'Comfirm holds' button is clicked because the checkin needs to be triggered by clicking this button. However when the 'Confirm' button is displayed this means that the item was not checked out when the user is trying to check it back in. In which case the check in does not need to be triggered and so in this patch the 'Confirm' button does not submt the form it simply closes the modal box to show any warning messages behind. Test plan: 1. Check out an item to one patron whilst having that item also being on hold to another patron 2. Check the item back in 3. Notice a modal box appears greying out the background with three buttons 'Confirm hold', 'Print and confirm' and 'Ignore'. Click confirm and notice that the page refreshes and no yellow warning messages are able to be displayed 4. Now that you have checked the item in. Try checking it in a second time by clicking on the Checkin tab and writing in the barcode. 5. The modal box will appear again, this time with three buttons 'Confirm', 'Print and confirm' and 'Cancel hold' 6. Click the 'Confirm' button and the page refreshes again and the yellow warning message hidden by the modal box is not properly displayed to the user 7. Apply patch 8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out" 9. Drop the hold on the item and make sure it is not checked out. 10. Repeat steps 1 and 2 and notice after clicking the 'Confirm hold' button the page refreshes and the item is successfully checked back in -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #2 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 62350 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62350&action=edit Bug 18438 - Removed unused js function call and if statement check for running js function -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- This appears to be working really well, but I don't like that it introduces a new instance of "onclick." https://wiki.koha-community.org/wiki/Coding_Guidelines#JS9:_Avoid_the_use_of... I realize it's a pretty trivial use of it, but I would prefer that it be done according to coding guidelines. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #4 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 62422 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62422&action=edit Bug 18438 - Removed onclick event on 'Confirm hold' button to adhere to coding guidelines following feedback -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #5 from Alex Buckley <alexbuckley@catalyst.net.nz> --- (In response to Owen Leonard in comment 3) Hi Owen As that onclick event isn't necessary I have removed it in the patch I just attached. Thanks -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62349|0 |1 is obsolete| | --- Comment #6 from Marc Véron <veron@veron.ch> --- Created attachment 62444 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62444&action=edit Bug 18438 - Implemented data-dismiss="modal" attribute on 'Confirm hold' and 'confirm' buttons so when they are cliked the modal box dissapears and any warning messages that were hidden by the box are now visible. The form is submitted via the js onclick function when the 'Comfirm holds' button is clicked because the checkin needs to be triggered by clicking this button. However when the 'Confirm' button is displayed this means that the item was not checked out when the user is trying to check it back in. In which case the check in does not need to be triggered and so in this patch the 'Confirm' button does not submt the form it simply closes the modal box to show any warning messages behind. Test plan: 1. Check out an item to one patron whilst having that item also being on hold to another patron 2. Check the item back in 3. Notice a modal box appears greying out the background with three buttons 'Confirm hold', 'Print and confirm' and 'Ignore'. Click confirm and notice that the page refreshes and no yellow warning messages are able to be displayed 4. Now that you have checked the item in. Try checking it in a second time by clicking on the Checkin tab and writing in the barcode. 5. The modal box will appear again, this time with three buttons 'Confirm', 'Print and confirm' and 'Cancel hold' 6. Click the 'Confirm' button and the page refreshes again and the yellow warning message hidden by the modal box is not properly displayed to the user 7. Apply patch 8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out" 9. Drop the hold on the item and make sure it is not checked out. 10. Repeat steps 1 and 2 and notice after clicking the 'Confirm hold' button the page refreshes and the item is successfully checked back in Tested 3 patches together, followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62350|0 |1 is obsolete| | --- Comment #7 from Marc Véron <veron@veron.ch> --- Created attachment 62445 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62445&action=edit Bug 18438 - Removed unused js function call and if statement check for running js function Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62422|0 |1 is obsolete| | --- Comment #8 from Marc Véron <veron@veron.ch> --- Created attachment 62446 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62446&action=edit Bug 18438 - Removed onclick event on 'Confirm hold' button to adhere to coding guidelines following feedback Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Marc Véron <veron@veron.ch> 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=18438 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org Status|Signed Off |Failed QA --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Marc Véron from comment #6)
8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out"
Yes, but the hold is not confirmed! The form submission (and so the page reload) does something: it confirms the hold, which is why we expect here :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #10 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 62708 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62708&action=edit Bug 18438 - Removed the data-dismiss="modal" from all confirm buttons except where an item is waiting. This means the hold is confirmed for a item by the form submitting and page reloading in all cases except where the item is being attempted to be checked in again where it is actually waiting in the library for pickuop by user with a hold on it. The confirm button in this case does not require the page reloading because the hidden input 'cancel_reserve' value of it is 0 Same test plan as comment 1 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |josef.moravec@gmail.com --- Comment #11 from Josef Moravec <josef.moravec@gmail.com> --- Works good, just small issue - I think that after dismissing modal, the focus should be on the input field for checking in... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #12 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 62955 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62955&action=edit Bug 18438 - Added focus on the checkin barcode input box after modal box is dismissed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #13 from Josef Moravec <josef.moravec@gmail.com> --- (In reply to Alex Buckley from comment #12)
Created attachment 62955 [details] [review] Bug 18438 - Added focus on the checkin barcode input box after modal box is dismissed
Alex, you should avoid to use onclick parameter on submit button. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62955|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #14 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63002 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63002&action=edit Bug 18438 - Amended implementation of focus on barcode input box when the modal box is closed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Josef Moravec <josef.moravec@gmail.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=18438 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62444|0 |1 is obsolete| | Attachment #62445|0 |1 is obsolete| | Attachment #62446|0 |1 is obsolete| | Attachment #62708|0 |1 is obsolete| | Attachment #63002|0 |1 is obsolete| | --- Comment #15 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63003 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63003&action=edit [SIGNED-OFF] Bug 18438 - Implemented data-dismiss="modal" attribute on 'Confirm hold' and 'confirm' buttons so when they are cliked the modal box dissapears and any warning messages that were hidden by the box are now visible. The form is submitted via the js onclick function when the 'Comfirm holds' button is clicked because the checkin needs to be triggered by clicking this button. However when the 'Confirm' button is displayed this means that the item was not checked out when the user is trying to check it back in. In which case the check in does not need to be triggered and so in this patch the 'Confirm' button does not submt the form it simply closes the modal box to show any warning messages behind. Test plan: 1. Check out an item to one patron whilst having that item also being on hold to another patron 2. Check the item back in 3. Notice a modal box appears greying out the background with three buttons 'Confirm hold', 'Print and confirm' and 'Ignore'. Click confirm and notice that the page refreshes and no yellow warning messages are able to be displayed 4. Now that you have checked the item in. Try checking it in a second time by clicking on the Checkin tab and writing in the barcode. 5. The modal box will appear again, this time with three buttons 'Confirm', 'Print and confirm' and 'Cancel hold' 6. Click the 'Confirm' button and the page refreshes again and the yellow warning message hidden by the modal box is not properly displayed to the user 7. Apply patch 8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out" 9. Drop the hold on the item and make sure it is not checked out. 10. Repeat steps 1 and 2 and notice after clicking the 'Confirm hold' button the page refreshes and the item is successfully checked back in Tested 3 patches together, followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #16 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63004 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63004&action=edit [SIGNED-OFF] Bug 18438 - Removed unused js function call and if statement check for running js function Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #17 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63005 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63005&action=edit [SIGNED-OFF] Bug 18438 - Removed onclick event on 'Confirm hold' button to adhere to coding guidelines following feedback Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #18 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63006 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63006&action=edit [SIGNED-OFF] Bug 18438 - Removed the data-dismiss="modal" from all confirm buttons except where an item is waiting. This means the hold is confirmed for a item by the form submitting and page reloading in all cases except where the item is being attempted to be checked in again where it is actually waiting in the library for pickuop by user with a hold on it. The confirm button in this case does not require the page reloading because the hidden input 'cancel_reserve' value of it is 0 Same test plan as comment 1 Signed-off-by: Josef Moravec <josef.moravec@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #19 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63007 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63007&action=edit [SIGNED-OFF] Bug 18438 - Amended implementation of focus on barcode input box when the modal box is closed. Signed-off-by: Josef Moravec <josef.moravec@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #20 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I do not understand where we are with these patches. What are the expected behaviour changes? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #21 from Josef Moravec <josef.moravec@gmail.com> --- (In reply to Jonathan Druart from comment #20)
I do not understand where we are with these patches. What are the expected behaviour changes?
As I understand, the change is, that the page doesn't reload after dismissing hold modal window, so the message under the window does not disappear and user haven't wait for another reload... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |ASSIGNED --- Comment #22 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Josef Moravec from comment #21)
(In reply to Jonathan Druart from comment #20)
I do not understand where we are with these patches. What are the expected behaviour changes?
As I understand, the change is, that the page doesn't reload after dismissing hold modal window, so the message under the window does not disappear and user haven't wait for another reload...
Yes, it is what I understood too, but it is the behaviour I already get against master. Alex, can you squash the patches and provide a simple test plan please? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #23 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Hi Jonathan Sure thing, I'll do that when I have a bit of time this evening -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63003|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63004|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63005|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63006|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63007|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #24 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63371 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63371&action=edit Bug 18438 - Implemented data-dismiss="modal" in returns.tt so that any warning messages hidden by a hold modal are displayed after it is dismissed Test plan: 1. Check out an item to one patron whilst having that item also being on hold to another patron 2. Check the item back in 3. Notice a modal box appears greying out the background with three buttons 'Confirm hold', 'Print and confirm' and 'Ignore'. Click confirm and notice that the page refreshes and no yellow warning messages are able to be displayed 4. Now that you have checked the item in. Try checking it in a second time by clicking on the Checkin tab and writing in the barcode. 5. The modal box will appear again, this time with three buttons 'Confirm', 'Print and confirm' and 'Cancel hold' 6. Click the 'Confirm' button and the page refreshes again and the yellow warning message hidden by the modal box is not properly displayed to the user. Notice that the focus is on the barcode input box. 7. Apply patch 8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out". Also notice the focus is in the barcode inputbox. 9. Drop the hold on the item and make sure it is not checked out. 10. Repeat steps 1 and 2 and notice after clicking the 'Confirm hold' button the page refreshes and the item is successfully checked back in. With the focus on the barcode input. 11. View koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt and notice that the button on line 345 does not use an onclick parameter Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63371|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 --- Comment #25 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 63372 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63372&action=edit Bug 18438 - Implemented data-dismiss="modal" in returns.tt so that any warning messages hidden by a hold modal are displayed after it is dismissed Test plan: 1. Check out an item to one patron whilst having that item also being on hold to another patron 2. Check the item back in 3. Notice a modal box appears greying out the background with three buttons 'Confirm hold', 'Print and confirm' and 'Ignore'. Click confirm and notice that the page refreshes and no yellow warning messages are able to be displayed 4. Now that you have checked the item in. Try checking it in a second time by clicking on the Checkin tab and writing in the barcode. 5. The modal box will appear again, this time with three buttons 'Confirm', 'Print and confirm' and 'Cancel hold' 6. Click the 'Confirm' button and the page refreshes again and the yellow warning message hidden by the modal box is not properly displayed to the user. Notice that the focus is on the barcode input box. 7. Apply patch 8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out". Also notice the focus is in the barcode inputbox. 9. Drop the hold on the item and make sure it is not checked out. 10. Repeat steps 1 and 2 and notice after clicking the 'Confirm hold' button the page refreshes and the item is successfully checked back in. With the focus on the barcode input. 11. View koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt and notice that the button on line 345 does not use an onclick parameter -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff --- Comment #26 from Alex Buckley <alexbuckley@catalyst.net.nz> --- This patch is just the previous 5 patches squashed down to one with an up to date test plan, the patch functionality itself has not been changed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63372|0 |1 is obsolete| | --- Comment #27 from Marc Véron <veron@veron.ch> --- Created attachment 63458 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63458&action=edit Bug 18438 - Implemented data-dismiss="modal" in returns.tt so that any warning messages hidden by a hold modal are displayed after it is dismissed Test plan: 1. Check out an item to one patron whilst having that item also being on hold to another patron 2. Check the item back in 3. Notice a modal box appears greying out the background with three buttons 'Confirm hold', 'Print and confirm' and 'Ignore'. Click confirm and notice that the page refreshes and no yellow warning messages are able to be displayed 4. Now that you have checked the item in. Try checking it in a second time by clicking on the Checkin tab and writing in the barcode. 5. The modal box will appear again, this time with three buttons 'Confirm', 'Print and confirm' and 'Cancel hold' 6. Click the 'Confirm' button and the page refreshes again and the yellow warning message hidden by the modal box is not properly displayed to the user. Notice that the focus is on the barcode input box. 7. Apply patch 8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out". Also notice the focus is in the barcode inputbox. 9. Drop the hold on the item and make sure it is not checked out. 10. Repeat steps 1 and 2 and notice after clicking the 'Confirm hold' button the page refreshes and the item is successfully checked back in. With the focus on the barcode input. 11. View koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt and notice that the button on line 345 does not use an onclick parameter Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Marc Véron <veron@veron.ch> 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=18438 Lisette <lisetteslatah@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lisetteslatah@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 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.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63458|0 |1 is obsolete| | --- Comment #28 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 63549 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63549&action=edit Bug 18438 - Implemented data-dismiss="modal" in returns.tt so that any warning messages hidden by a hold modal are displayed after it is dismissed Test plan: 1. Check out an item to one patron whilst having that item also being on hold to another patron 2. Check the item back in 3. Notice a modal box appears greying out the background with three buttons 'Confirm hold', 'Print and confirm' and 'Ignore'. Click confirm and notice that the page refreshes and no yellow warning messages are able to be displayed 4. Now that you have checked the item in. Try checking it in a second time by clicking on the Checkin tab and writing in the barcode. 5. The modal box will appear again, this time with three buttons 'Confirm', 'Print and confirm' and 'Cancel hold' 6. Click the 'Confirm' button and the page refreshes again and the yellow warning message hidden by the modal box is not properly displayed to the user. Notice that the focus is on the barcode input box. 7. Apply patch 8. Try checking in the item again, and this time after clicking the 'confirm' button on the modal box notice that the yellow warning message is displayed telling the user the item was "Not checked out". Also notice the focus is in the barcode inputbox. 9. Drop the hold on the item and make sure it is not checked out. 10. Repeat steps 1 and 2 and notice after clicking the 'Confirm hold' button the page refreshes and the item is successfully checked back in. With the focus on the barcode input. 11. View koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt and notice that the button on line 345 does not use an onclick parameter Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Amended-patch: remove spaces -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #29 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 17.05, thanks Alex! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |katrin.fischer@bsz-bw.de --- Comment #30 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- This patch has been pushed to 16.11.x and will be in 16.11.08. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtj@kohaaloha.com --- Comment #31 from Mason James <mtj@kohaaloha.com> --- (In reply to Katrin Fischer from comment #30)
This patch has been pushed to 16.11.x and will be in 16.11.08.
Blocked by Enhancement (bz11360), skipping for 16.05.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |19116 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19116 [Bug 19116] Holds not set to waiting when "Confirm" is used -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|19116 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19116 [Bug 19116] Holds not set to waiting when "Confirm" is used -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |19116 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19116 [Bug 19116] Holds not set to waiting when "Confirm" is used -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18438 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|19116 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19116 [Bug 19116] Holds not set to waiting when "Confirm" is used -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org