[Bug 16948] New: 'Return to batch item deletion' button should only show when there are items that could not be deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16948 Bug ID: 16948 Summary: 'Return to batch item deletion' button should only show when there are items that could not be deleted Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Tools Assignee: gmcharlt@gmail.com Reporter: aleishaamohia@hotmail.com QA Contact: testopia@bugs.koha-community.org Referring to Owen Leonard's comment from Bug 15824: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15824#c18 Having both the link and the button makes sense with batch item modifications when the list of modified items is displayed. On the batch item deletion confirmation screen I think we should move the button into the [% IF ( not_deleted_loop ) %] block so that it is only shown if the page is listing items which could not be deleted. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16948 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16948 --- Comment #1 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 53539 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53539&action=edit Bug 16948: 'Return to batch item deletion' button should only show when there are items that could not be deleted To test: 1) Go to Tools --> Batch item deletion 2) Put in an item barcode and click Continue 3) Deselect the item (so that you don't actually delete it) and click Delete selected items 4) Notice there are two links that say 'Return to batch item deletion' - one as a link and one as a button. 5) Apply patch and refresh page. You may have to go back and resend the form 6) Button should be gone, only one link shows. 7) Check out the same item that you are testing with to a patron 8) Put the barcode into batch item deletion again and follow the prompts. Do not deselect it after clicking Continue this time. (It will not be deleted as it has been checked out). 9) After clicking Delete selected items, you should be redirected and see a table with the item in it and the reason it can't be deleted. The button and the link saying 'Return to batch item deletion' should both be present. Sponsored-by: Catalyst IT -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16948 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored Assignee|gmcharlt@gmail.com |aleishaamohia@hotmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16948 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=16949 CC| |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=16948 Claire Gravely <c.gravely@arts.ac.uk> 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=16948 Claire Gravely <c.gravely@arts.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #53539|0 |1 is obsolete| | --- Comment #2 from Claire Gravely <c.gravely@arts.ac.uk> --- Created attachment 53558 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53558&action=edit Bug 16948: 'Return to batch item deletion' button should only show when there are items that could not be deleted To test: 1) Go to Tools --> Batch item deletion 2) Put in an item barcode and click Continue 3) Deselect the item (so that you don't actually delete it) and click Delete selected items 4) Notice there are two links that say 'Return to batch item deletion' - one as a link and one as a button. 5) Apply patch and refresh page. You may have to go back and resend the form 6) Button should be gone, only one link shows. 7) Check out the same item that you are testing with to a patron 8) Put the barcode into batch item deletion again and follow the prompts. Do not deselect it after clicking Continue this time. (It will not be deleted as it has been checked out). 9) After clicking Delete selected items, you should be redirected and see a table with the item in it and the reason it can't be deleted. The button and the link saying 'Return to batch item deletion' should both be present. Sponsored-by: Catalyst IT Signed-off-by: Claire Gravely <c.gravely@arts.ac.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16948 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=16948 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #53558|0 |1 is obsolete| | --- Comment #3 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 53595 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53595&action=edit Bug 16948: 'Return to batch item deletion' button should only show when there are items that could not be deleted To test: 1) Go to Tools --> Batch item deletion 2) Put in an item barcode and click Continue 3) Deselect the item (so that you don't actually delete it) and click Delete selected items 4) Notice there are two links that say 'Return to batch item deletion' - one as a link and one as a button. 5) Apply patch and refresh page. You may have to go back and resend the form 6) Button should be gone, only one link shows. 7) Check out the same item that you are testing with to a patron 8) Put the barcode into batch item deletion again and follow the prompts. Do not deselect it after clicking Continue this time. (It will not be deleted as it has been checked out). 9) After clicking Delete selected items, you should be redirected and see a table with the item in it and the reason it can't be deleted. The button and the link saying 'Return to batch item deletion' should both be present. Sponsored-by: Catalyst IT Signed-off-by: Claire Gravely <c.gravely@arts.ac.uk> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16948 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 16.11, thanks Aleisha! -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org