Bug 27885: Populate biblionumbers parameter using biblionumbers array
authorNick Clemens <nick@bywatersolutions.com>
Fri, 5 Mar 2021 17:32:28 +0000 (17:32 +0000)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 30 Jul 2021 14:25:32 +0000 (10:25 -0400)
commit7cca187cc4bbe5651615a05400fc65bd46b5fe46
tree7bbefa315f24faf262aa7f05bf728dac02d205bf
parent3b552ac937b8c2b1402c42d46088b6562c54e345
Bug 27885: Populate biblionumbers parameter using biblionumbers array

Currently we send $biblionumbers as the parameter, but this is just apassthrough form when a list of biblios is selected for placing a hold

If passed a single biblionumber we push it into @biblionumbers and use that for building the biblio loop

This patch uses @biblionumbers to avoid sending a blank variable in the URL

To test:
1 - On the staff client click 'place hold' for an individual record
2 - Use the form to find a patron
3 - Note the url is:
    http://localhost:8081/cgi-bin/koha/reserve/request.pl?biblionumbers=
4 - Apply patch
5 - Repeat
6 - The url is now like:
    http://localhost:8081/cgi-bin/koha/reserve/request.pl?biblionumbers=248
    (but with whatever biblionumber you chose)
7 - Perform a search and select multiple biblios and confirm you can place holds as before

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
JD Amended patch: Add missing space

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
reserve/request.pl