When we upgraded Bootstrap in the staff client we missed updating some references to Bootstrap's "hidden" event. It should now be "hidden.bs.modal." $("#marcPreview").on("hidden.bs.modal", function(){ // Action triggered when modal is closed });
Created attachment 77630 [details] [review] Bug 21186: Incorrect Bootstrap modal event name in multiple templates Modals in the staff client which load data via AJAX require an "on close" action in order to clear the modal of the loaded data. The problem presents itself the second time the modal is opened: The "loading" message doesn't appear, and the data from the previous link is shown until the new data loads. To test, apply the patch and test the following pages. In each case, click multiple instances of the modal trigger link/button to confirm that the "loading" message always appears between re-displays of the modal. - Acquisitions -> Vendor -> Add to basket -> From a staged file -> Add orders - Test the "MARC" and "Card" links - Acquisitions -> EDIFACT messages - Test the "View message" button - Acquisitions -> Vendor -> Add to basket -> From an existing record - Test "View MARC" link - Acquisitions -> Vendor -> Invoices -> Invoice -> Receipt page - Test the "Order," "MARC," and "Card" links. - Catalog -> Bibliographic record details - Test the MARC Preview link. - Cataloging -> Cataloging search results - Test the Actions -> MARC preview and Actions -> Card preview menu items - Cataloging -> Cataloging search results -> Merge - Test the "View MARC" linkscataloguing/merge.tt - Tools -> Patron lists -> Actions -> Print patron cards - Tools -> Batch record modification -> Results -> Show MARC - Tools -> Staged MARC record management -> View batch - Test MARC preview shown when you click a staged title
Created attachment 78412 [details] [review] Bug 21186: Incorrect Bootstrap modal event name in multiple templates Modals in the staff client which load data via AJAX require an "on close" action in order to clear the modal of the loaded data. The problem presents itself the second time the modal is opened: The "loading" message doesn't appear, and the data from the previous link is shown until the new data loads. To test, apply the patch and test the following pages. In each case, click multiple instances of the modal trigger link/button to confirm that the "loading" message always appears between re-displays of the modal. - Acquisitions -> Vendor -> Add to basket -> From a staged file -> Add orders - Test the "MARC" and "Card" links - Acquisitions -> EDIFACT messages - Test the "View message" button - Acquisitions -> Vendor -> Add to basket -> From an existing record - Test "View MARC" link - Acquisitions -> Vendor -> Invoices -> Invoice -> Receipt page - Test the "Order," "MARC," and "Card" links. - Catalog -> Bibliographic record details - Test the MARC Preview link. - Cataloging -> Cataloging search results - Test the Actions -> MARC preview and Actions -> Card preview menu items - Cataloging -> Cataloging search results -> Merge - Test the "View MARC" linkscataloguing/merge.tt - Tools -> Patron lists -> Actions -> Print patron cards - Tools -> Batch record modification -> Results -> Show MARC - Tools -> Staged MARC record management -> View batch - Test MARC preview shown when you click a staged title
Just to confirm Owen's bug report: https://getbootstrap.com/2.3.2/javascript.html#modals (just "hidden") vs. https://getbootstrap.com/docs/3.3/javascript/#modals-events and https://getbootstrap.com/docs/4.0/components/modal/#events ("hidden.bs.modal") Eye balls fine, but have not tested it.
*** Bug 21664 has been marked as a duplicate of this bug. ***
Created attachment 81182 [details] [review] Bug 21186: (follow-up) Incorrect Bootstrap modal event name in multiple templates This patch addresses a few more instances of this problem which I didn't catch in my first pass. To test, apply the patch and clear your browser cache if necessary. - Go to Circulation -> Check in and check in an item which has a hold. - When the 'Hold found' modal appears the checkin barcode field behind it should be disabled. - When the modal is dismissed the checkin field should be enabled again and focus should be on that field. - Go to Administration -> Library groups. - Click the "Add group" button. The cursor focus should be on the title field. - Go to Tools -> MARC modification templates. - Click the "New template" button. The cursor focus should be on the name field.
Trying to sign off, but I don't know how to trigger some of these tests: Acquisitions -> Vendor -> Add to basket -> From a staged file -> Add orders - Test the "MARC" and "Card" links DONE - Acquisitions -> EDIFACT messages - Test the "View message" button DON'T KNOW HOW TO GET EDIFACT MESSAGES - Acquisitions -> Vendor -> Add to basket -> From an existing record - Test "View MARC" link DONE - Acquisitions -> Vendor -> Invoices -> Invoice -> Receipt page - Test the "Order," "MARC," and "Card" links. DON'T KNOW HOW TO GET INVOICES - Catalog -> Bibliographic record details - Test the MARC Preview link. DONE - Cataloging -> Cataloging search results - Test the Actions -> MARC preview and Actions -> Card preview menu items DONE - Cataloging -> Cataloging search results -> Merge - Test the "View MARC" linkscataloguing/merge.tt DON'T KNOW HOW TO MAKE THIS HAPPEN - Tools -> Patron lists -> Actions -> Print patron cards DONE - Tools -> Batch record modification -> Results -> Show MARC DONE - Tools -> Staged MARC record management -> View batch - Test MARC preview shown when you click a staged title DONE - Go to Circulation -> Check in and check in an item which has a hold. - When the 'Hold found' modal appears the checkin barcode field behind it should be disabled. - When the modal is dismissed the checkin field should be enabled again and focus should be on that field. DONE - Go to Administration -> Library groups. - Click the "Add group" button. The cursor focus should be on the title field. DONE - Go to Tools -> MARC modification templates. - Click the "New template" button. The cursor focus should be on the name field. DONE
Created attachment 81189 [details] [review] Bug 21186: Incorrect Bootstrap modal event name in multiple templates Modals in the staff client which load data via AJAX require an "on close" action in order to clear the modal of the loaded data. The problem presents itself the second time the modal is opened: The "loading" message doesn't appear, and the data from the previous link is shown until the new data loads. To test, apply the patch and test the following pages. In each case, click multiple instances of the modal trigger link/button to confirm that the "loading" message always appears between re-displays of the modal. - Acquisitions -> Vendor -> Add to basket -> From a staged file -> Add orders - Test the "MARC" and "Card" links - Acquisitions -> EDIFACT messages - Test the "View message" button - Acquisitions -> Vendor -> Add to basket -> From an existing record - Test "View MARC" link - Acquisitions -> Vendor -> Invoices -> Invoice -> Receipt page - Test the "Order," "MARC," and "Card" links. - Catalog -> Bibliographic record details - Test the MARC Preview link. - Cataloging -> Cataloging search results - Test the Actions -> MARC preview and Actions -> Card preview menu items - Cataloging -> Cataloging search results -> Merge - Test the "View MARC" linkscataloguing/merge.tt - Tools -> Patron lists -> Actions -> Print patron cards - Tools -> Batch record modification -> Results -> Show MARC - Tools -> Staged MARC record management -> View batch - Test MARC preview shown when you click a staged title Signed-off-by: Cori Lynn Arnold <carnold@dgiinc.com>
Created attachment 81190 [details] [review] Bug 21186: (follow-up) Incorrect Bootstrap modal event name in multiple templates This patch addresses a few more instances of this problem which I didn't catch in my first pass. To test, apply the patch and clear your browser cache if necessary. - Go to Circulation -> Check in and check in an item which has a hold. - When the 'Hold found' modal appears the checkin barcode field behind it should be disabled. - When the modal is dismissed the checkin field should be enabled again and focus should be on that field. - Go to Administration -> Library groups. - Click the "Add group" button. The cursor focus should be on the title field. - Go to Tools -> MARC modification templates. - Click the "New template" button. The cursor focus should be on the name field. Signed-off-by: Cori Lynn Arnold <carnold@dgiinc.com>
signed off
Created attachment 81369 [details] [review] Bug 21186: Incorrect Bootstrap modal event name in multiple templates Modals in the staff client which load data via AJAX require an "on close" action in order to clear the modal of the loaded data. The problem presents itself the second time the modal is opened: The "loading" message doesn't appear, and the data from the previous link is shown until the new data loads. To test, apply the patch and test the following pages. In each case, click multiple instances of the modal trigger link/button to confirm that the "loading" message always appears between re-displays of the modal. - Acquisitions -> Vendor -> Add to basket -> From a staged file -> Add orders - Test the "MARC" and "Card" links - Acquisitions -> EDIFACT messages - Test the "View message" button - Acquisitions -> Vendor -> Add to basket -> From an existing record - Test "View MARC" link - Acquisitions -> Vendor -> Invoices -> Invoice -> Receipt page - Test the "Order," "MARC," and "Card" links. - Catalog -> Bibliographic record details - Test the MARC Preview link. - Cataloging -> Cataloging search results - Test the Actions -> MARC preview and Actions -> Card preview menu items - Cataloging -> Cataloging search results -> Merge - Test the "View MARC" linkscataloguing/merge.tt - Tools -> Patron lists -> Actions -> Print patron cards - Tools -> Batch record modification -> Results -> Show MARC - Tools -> Staged MARC record management -> View batch - Test MARC preview shown when you click a staged title Signed-off-by: Cori Lynn Arnold <carnold@dgiinc.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 81370 [details] [review] Bug 21186: (follow-up) Incorrect Bootstrap modal event name in multiple templates This patch addresses a few more instances of this problem which I didn't catch in my first pass. To test, apply the patch and clear your browser cache if necessary. - Go to Circulation -> Check in and check in an item which has a hold. - When the 'Hold found' modal appears the checkin barcode field behind it should be disabled. - When the modal is dismissed the checkin field should be enabled again and focus should be on that field. - Go to Administration -> Library groups. - Click the "Add group" button. The cursor focus should be on the title field. - Go to Tools -> MARC modification templates. - Click the "New template" button. The cursor focus should be on the name field. Signed-off-by: Cori Lynn Arnold <carnold@dgiinc.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Awesome work all! Pushed to master for 18.11
Pushed to 18.05.x for 18.05.06
Is this also for 17.11.x ?
(In reply to Fridolin SOMERS from comment #14) > Is this also for 17.11.x ? Yes.
Pushed to 17.11.x for 17.11.13