[Koha-bugs] [Bug 23258] Batch holding

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23258

--- Comment #9 from Owen Leonard <oleonard at myacpl.org> ---
Created attachment 106598
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106598&action=edit
Bug 23258: (follow-up) Batch holds template followup

This patch revises the batch holds template to bring it up to date with
current guidelines including corrected indentation, Bootstrap grid,
terminology fixes ("hold" instead of "reserve") and JavaScript in the
footer.

Also changed:
- Add specificity to page title and breadcrumbs
- Add "Select all" and "Clear all" links to confirmation page
- Add jQuery validation of the form, enforcing barcode and patron entry
- Swith from "Batch holding" to "Batch place and cancel holds."
- Made suggested change to avoid errors in the log:
  my $op  = $input->param('op') // '';
- Fixed some other QA tool errors

To test, apply the patch and test the process of placing and cancelling
holds in batches:

 - By uploading a file of barcodes
 - By submitting barcodes in the textarea
 - Confirm that holds are placed and cancelled

 At each step of the process confirm that everything looks good and
 works properly.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.