[Bug 19654] New: Move tools templates JavaScript to the footer: Batch MARC tools
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Bug ID: 19654 Summary: Move tools templates JavaScript to the footer: Batch MARC tools Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org CC: dominic.pichette@inlibro.com, jonathan.druart@bugs.koha-community.org, nick@bywatersolutions.com, simon.pouchol@biblibre.com, testopia@bugs.koha-community.org Depends on: 19647 Blocks: 17858 In order to conform to best practices for performance staff client template javascript should be in the footer instead of the header. This patch will do so for the batch MARC tools templates. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17858 [Bug 17858] Omnibus: Move staff client JavaScript to the footer https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19647 [Bug 19647] Move patron lists templates JS to the footer -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 69236 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69236&action=edit Bug 19654: Move tools templates JavaScript to the footer: Batch MARC tools This patch modifies the staff client patron lists templates so that JavaScript is included in the footer instead of the header. To test, apply the patch and test the JavaScript-driven features of each modified template: All button controls, DataTables functionality, form validation, etc. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Claire Gravely <claire_gravely@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |claire_gravely@hotmail.com --- Comment #2 from Claire Gravely <claire_gravely@hotmail.com> --- When testing the Batch item modification, after entering barcodes and continuing I get: ReferenceError: $ is not defined. Batchmod.pl:861:1 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #69236|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=19654 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 69740 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69740&action=edit Bug 19654: Move tools templates JavaScript to the footer: Batch MARC tools This patch modifies the staff client patron lists templates so that JavaScript is included in the footer instead of the header. To test, apply the patch and test the JavaScript-driven features of each modified template: All button controls, DataTables functionality, form validation, etc. Revised: My original patch included changes to batchMod-edit.tt but didn't take into account the MARC plugins which are loaded in the item edit form. I have discarded those changes for now. Tools -> Batch record modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit records for deletion - Page highlighting in the sidebar - Checkbox selection links - Warning when no records selected - Patron autocomplete in the "Check out" tab Tools -> Batch item modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab Tools -> Batch item deletion - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit items for deletion - Patron autocomplete in the "Check out" tab - Checkbox selection links - Column-hiding checkboxes - Datatables Tools -> Batch record modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit records for modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Checkbox selection links - Datatables - MARC preview - Warning when no records selected - Progress bar after submitting items for modification -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Owen Leonard <oleonard@myacpl.org> 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=19654 Claire Gravely <claire_gravely@hotmail.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=19654 Claire Gravely <claire_gravely@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #69740|0 |1 is obsolete| | --- Comment #4 from Claire Gravely <claire_gravely@hotmail.com> --- Created attachment 69770 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69770&action=edit Bug 19654: Move tools templates JavaScript to the footer: Batch MARC tools This patch modifies the staff client patron lists templates so that JavaScript is included in the footer instead of the header. To test, apply the patch and test the JavaScript-driven features of each modified template: All button controls, DataTables functionality, form validation, etc. Revised: My original patch included changes to batchMod-edit.tt but didn't take into account the MARC plugins which are loaded in the item edit form. I have discarded those changes for now. Tools -> Batch record modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit records for deletion - Page highlighting in the sidebar - Checkbox selection links - Warning when no records selected - Patron autocomplete in the "Check out" tab Tools -> Batch item modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab Tools -> Batch item deletion - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit items for deletion - Patron autocomplete in the "Check out" tab - Checkbox selection links - Column-hiding checkboxes - Datatables Tools -> Batch record modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit records for modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Checkbox selection links - Datatables - MARC preview - Warning when no records selected - Progress bar after submitting items for modification Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Josef Moravec <josef.moravec@gmail.com> 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=19654 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #69770|0 |1 is obsolete| | --- Comment #5 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 69774 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69774&action=edit Bug 19654: Move tools templates JavaScript to the footer: Batch MARC tools This patch modifies the staff client patron lists templates so that JavaScript is included in the footer instead of the header. To test, apply the patch and test the JavaScript-driven features of each modified template: All button controls, DataTables functionality, form validation, etc. Revised: My original patch included changes to batchMod-edit.tt but didn't take into account the MARC plugins which are loaded in the item edit form. I have discarded those changes for now. Tools -> Batch record modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit records for deletion - Page highlighting in the sidebar - Checkbox selection links - Warning when no records selected - Patron autocomplete in the "Check out" tab Tools -> Batch item modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab Tools -> Batch item deletion - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit items for deletion - Patron autocomplete in the "Check out" tab - Checkbox selection links - Column-hiding checkboxes - Datatables Tools -> Batch record modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Submit records for modification - Page highlighting in the sidebar - Patron autocomplete in the "Check out" tab - Checkbox selection links - Datatables - MARC preview - Warning when no records selected - Progress bar after submitting items for modification Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> 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=19654 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |josef.moravec@gmail.com |y.org | CC| |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=19654 Bug 19654 depends on bug 19647, which changed state. Bug 19647 Summary: Move patron lists templates JS to the footer https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19647 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #6 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 18.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19654 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to Master |RESOLVED --- Comment #7 from Nick Clemens <nick@bywatersolutions.com> --- Awesome work all! Enhancement, not backporting for 17.11 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org