Bug 26795 - Remove the use of jquery.checkboxes plugin from ILL pages
Summary: Remove the use of jquery.checkboxes plugin from ILL pages
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 26006 26806
  Show dependency tree / graph
 
Reported: 2020-10-23 12:19 UTC by Owen Leonard
Modified: 2021-06-14 21:33 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00
Circulation function:

Attachments
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages (2.14 KB, patch)
2020-10-23 12:51 UTC, Owen Leonard 		Details | Diff | Splinter Review
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages (2.19 KB, patch)
2020-10-31 21:55 UTC, David Nind 		Details | Diff | Splinter Review
Bug 26795: Remove the use of jquery.checkboxes plugin from ILL pages (2.25 KB, patch)
2020-10-31 23:12 UTC, Katrin Fischer 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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!