]> git.koha-community.org Git - koha.git/commit
Bug 37197: Reports option to send to batch patron modification needs to use POST
authorPhil Ringnalda <phil@chetcolibrary.org>
Thu, 27 Jun 2024 03:19:21 +0000 (20:19 -0700)
committerLucas Gass <lucas@bywatersolutions.com>
Wed, 9 Oct 2024 22:04:22 +0000 (22:04 +0000)
commit2d038cfda56a01f453fe7fb7397cf6e44b1ff7f8
tree1d6d1d996efc162678320e5a6986cb9ae4798a04
parent602938a82b25015de981e0e9255e08ff62fce95c
Bug 37197: Reports option to send to batch patron modification needs to use POST

When you have an SQL report that selects cardnumber from borrowers, you
get a menuitem to send the results to Batch patron modification. Currently
that fails with a message about no card numbers or borrowernumbers given,
because it tries to do a GET with the op show, when modborrowers.pl is
expecting a POST with the op cud-show.

Test plan:
1. Without the patch, Reports - Create from SQL - name it and paste

select cardnumber from borrowers

   in the SQL textarea and save, then Run report.
2. Above the results is a "Batch operations with 20 visible records" menu,
   choose Batch patron modification
3. Note that instead of modifying the records, you're stuck with a message
   saying "No patron card numbers or borrowernumbers given." even though
   there's a whole string of them in the URL.
4. Apply patch, refresh the page with the results (oddly, no apparent need
   to even restart_all), and choose the menuitem again, but this time with
   a successful result.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit a85086d8278b3c0b5ad578c3af7b0e70637a47be)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt