Bug 26795

Summary: Remove the use of jquery.checkboxes plugin from ILL pages
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: aleisha, amit.gupta, amitddng135, david, katrin.fischer, lucas, testopia
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00
Bug Depends on:    
Bug Blocks: 26006, 26806    
Attachments: Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages

Description Owen Leonard 2020-10-23 12:19:18 UTC
Two ill request pages include the jquery.checkboxes plugin but do not use them. The include should be removed.

- intranet-tmpl/prog/en/modules/ill/ill-requests.tt
- intranet-tmpl/prog/en/modules/members/ill-requests.tt
Comment 1 Owen Leonard 2020-10-23 12:51:01 UTC
Created attachment 112261 [details] [review]
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages

This patch removes the inclusion of the jquery.checkboxes plugin from
two staff interface ILL pages where the plugin is unused.

To test you need at least one patron with some ILL requests.

- Apply the patch and locate a patron with ILL requests.
- Click the "ILL requests history" tab.
- Confirm that the table of requests has no checkboxes.
- Choose "Ill requests" from the "More" menu in the header.
- Confirm that the table of requests has no checkboxes.
Comment 2 David Nind 2020-10-31 21:55:56 UTC
Created attachment 112769 [details] [review]
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages

This patch removes the inclusion of the jquery.checkboxes plugin from
two staff interface ILL pages where the plugin is unused.

To test you need at least one patron with some ILL requests.

- Apply the patch and locate a patron with ILL requests.
- Click the "ILL requests history" tab.
- Confirm that the table of requests has no checkboxes.
- Choose "Ill requests" from the "More" menu in the header.
- Confirm that the table of requests has no checkboxes.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2020-10-31 22:08:19 UTC
Testing notes: See https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26758#c3 for how to setup ILL.
Comment 4 Katrin Fischer 2020-10-31 23:12:10 UTC
Created attachment 112770 [details] [review]
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages

This patch removes the inclusion of the jquery.checkboxes plugin from
two staff interface ILL pages where the plugin is unused.

To test you need at least one patron with some ILL requests.

- Apply the patch and locate a patron with ILL requests.
- Click the "ILL requests history" tab.
- Confirm that the table of requests has no checkboxes.
- Choose "Ill requests" from the "More" menu in the header.
- Confirm that the table of requests has no checkboxes.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2020-11-03 09:59:24 UTC
Pushed to master for 20.11, thanks to everybody involved!