]> 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)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 26 Jul 2024 12:56:32 +0000 (13:56 +0100)
commita85086d8278b3c0b5ad578c3af7b0e70637a47be
tree55d1798ff34585a1128ca61352c2ae3bc5352808
parent43985e90db4bbf7a936c4ce96306461249ded73c
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>
koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt