Bug 26799 - Remove the use of jquery.checkboxes plugin from patron payment page
Summary: Remove the use of jquery.checkboxes plugin from patron payment page
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: Katrin Fischer
URL:
Keywords:
Depends on:
Blocks: 26006 26806
  Show dependency tree / graph
 
Reported: 2020-10-23 13:51 UTC by Owen Leonard
Modified: 2021-06-14 21:33 UTC (History)
1 user (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 26799: Remove the use of jquery.checkboxes plugin from patron payment page (2.30 KB, patch)
2020-10-23 14:08 UTC, Owen Leonard 		Details | Diff | Splinter Review
Bug 26799: Remove the use of jquery.checkboxes plugin from patron payment page (2.36 KB, patch)
2020-10-24 20:15 UTC, Lisette Scheer 		Details | Diff | Splinter Review
Bug 26799: Remove the use of jquery.checkboxes plugin from patron payment page (2.42 KB, patch)
2020-10-25 01:23 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 13:51:34 UTC 
The patron payment page in the staff interface (pay.tt) includes the jquery.checkboxes plugin which is old and unmaintained. It should be replaced.
Comment 1 Owen Leonard 2020-10-23 14:08:15 UTC 
Created attachment 112276 [details] [review]
Bug 26799: Remove the use of jquery.checkboxes plugin from patron payment page

This patch removes the use of the jquery.checkboxes plugin from the
patron payment screen (members/pay.tt).

To test, apply the patch and locate a patron with unpaid fines.

- View the "Accounting" page for that patron.
- In the table of outstanding fines, test the "Select all" and "Clear
  all" links work correctly to check the checkboxes corresponding to
  each fine.
- Confirm that the "Select all" action enables the "Pay selected" and
  "Write off selected" buttons.
- Confirm that the "Clear all" action disables the "Pay selected" and
  "Write off selected" buttons.
Comment 2 Lisette Scheer 2020-10-24 20:15:47 UTC 
Created attachment 112397 [details] [review]
Bug 26799: Remove the use of jquery.checkboxes plugin from patron payment page

This patch removes the use of the jquery.checkboxes plugin from the
patron payment screen (members/pay.tt).

To test, apply the patch and locate a patron with unpaid fines.

- View the "Accounting" page for that patron.
- In the table of outstanding fines, test the "Select all" and "Clear
  all" links work correctly to check the checkboxes corresponding to
  each fine.
- Confirm that the "Select all" action enables the "Pay selected" and
  "Write off selected" buttons.
- Confirm that the "Clear all" action disables the "Pay selected" and
  "Write off selected" buttons.

Signed-off-by: Lisette Scheer <lisettes@latahlibrary.org>
Comment 3 Katrin Fischer 2020-10-25 01:23:03 UTC 
Created attachment 112442 [details] [review]
Bug 26799: Remove the use of jquery.checkboxes plugin from patron payment page

This patch removes the use of the jquery.checkboxes plugin from the
patron payment screen (members/pay.tt).

To test, apply the patch and locate a patron with unpaid fines.

- View the "Accounting" page for that patron.
- In the table of outstanding fines, test the "Select all" and "Clear
  all" links work correctly to check the checkboxes corresponding to
  each fine.
- Confirm that the "Select all" action enables the "Pay selected" and
  "Write off selected" buttons.
- Confirm that the "Clear all" action disables the "Pay selected" and
  "Write off selected" buttons.

Signed-off-by: Lisette Scheer <lisettes@latahlibrary.org>

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