[Bug 11743] New: Remove unused DataTables plugin from OPAC theme directories
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Bug ID: 11743 Summary: Remove unused DataTables plugin from OPAC theme directories Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org Two theme directories contain a copy of the main DataTables plugin which is unused and can be removed: opac-tmpl/ccsr/en/lib/jquery/plugins/jquery.dataTables.min.js opac-tmpl/bootstrap/lib/jquery/plugins/jquery.dataTables.min.js All OPAC templates now use the copy in opac-tmpl/lib/jquery/plugins/ -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 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.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 25207 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25207&action=edit Bug 11743 - Remove unused DataTables plugin from OPAC theme directories The Bootstrap and CCSR theme directories contain older copies of the DataTables plugin which is no longer in use. This patch removes them both. To test, apply the patch and confirm that DataTables still work on the following pages in both the Bootstrap and CCSR OPAC themes: - The Cart. Add items to the Cart, open it and confirm that the table is sorted correctly. - The detail page: View the detail page for a title (opac-detail.pl) with multiple items and confirm that the table of items is sorted correctly. - A subscription's "full history" page: Navigate to the detail page of a subscription in your catalog which has received multiple issues. From the subscriptions tab click the "More details" link, then the "Full history" link. Confirm that the table of issues is sorted correctly. - Search history: Enable the EnableOpacSearchHistory system preference. Perform a few searches so that your current session will record them. View the search history page and confirm that the tables of current and past searches are sorted correctly. - Suggestions (enable 'suggestion' system pref if necessary) - Tags (enable TagsEnabled if necessary) - Most popular (enable OpacTopissue if necessary) - User summary (opac-user.pl): Checkouts, overdues, and holds tables should all sort correctly. - Self checkout: Enable WebBasedSelfCheck if necessary. Check out as a patron who has multiple existing checkouts. Confirm that the table is sorted correctly. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25207|0 |1 is obsolete| | --- Comment #2 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 25312 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25312&action=edit Bug 11743 - Remove unused DataTables plugin from OPAC theme directories The Bootstrap and CCSR theme directories contain older copies of the DataTables plugin which is no longer in use. This patch removes them both. To test, apply the patch and confirm that DataTables still work on the following pages in both the Bootstrap and CCSR OPAC themes: - The Cart. Add items to the Cart, open it and confirm that the table is sorted correctly. - The detail page: View the detail page for a title (opac-detail.pl) with multiple items and confirm that the table of items is sorted correctly. - A subscription's "full history" page: Navigate to the detail page of a subscription in your catalog which has received multiple issues. From the subscriptions tab click the "More details" link, then the "Full history" link. Confirm that the table of issues is sorted correctly. - Search history: Enable the EnableOpacSearchHistory system preference. Perform a few searches so that your current session will record them. View the search history page and confirm that the tables of current and past searches are sorted correctly. - Suggestions (enable 'suggestion' system pref if necessary) - Tags (enable TagsEnabled if necessary) - Most popular (enable OpacTopissue if necessary) - User summary (opac-user.pl): Checkouts, overdues, and holds tables should all sort correctly. - Self checkout: Enable WebBasedSelfCheck if necessary. Check out as a patron who has multiple existing checkouts. Confirm that the table is sorted correctly. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25312|0 |1 is obsolete| | --- Comment #3 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 25341 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25341&action=edit Bug 11743 - Remove unused DataTables plugin from OPAC theme directories The Bootstrap and CCSR theme directories contain older copies of the DataTables plugin which is no longer in use. This patch removes them both. To test, apply the patch and confirm that DataTables still work on the following pages in both the Bootstrap and CCSR OPAC themes: - The Cart. Add items to the Cart, open it and confirm that the table is sorted correctly. - The detail page: View the detail page for a title (opac-detail.pl) with multiple items and confirm that the table of items is sorted correctly. - A subscription's "full history" page: Navigate to the detail page of a subscription in your catalog which has received multiple issues. From the subscriptions tab click the "More details" link, then the "Full history" link. Confirm that the table of issues is sorted correctly. - Search history: Enable the EnableOpacSearchHistory system preference. Perform a few searches so that your current session will record them. View the search history page and confirm that the tables of current and past searches are sorted correctly. - Suggestions (enable 'suggestion' system pref if necessary) - Tags (enable TagsEnabled if necessary) - Most popular (enable OpacTopissue if necessary) - User summary (opac-user.pl): Checkouts, overdues, and holds tables should all sort correctly. - Self checkout: Enable WebBasedSelfCheck if necessary. Check out as a patron who has multiple existing checkouts. Confirm that the table is sorted correctly. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Existing uses of datatables.min.js don't point to the theme directory. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |gmcharlt@gmail.com --- Comment #4 from Galen Charlton <gmcharlt@gmail.com> --- Pushed to master. Thanks, Owen! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11743 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |fridolyn.somers@biblibre.co | |m --- Comment #5 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Pushed to 3.14.x, will be in 3.14.07 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org