]> git.koha-community.org Git - koha.git/commit
Bug 7940 - Placing a hold on a single item from the staff cart causes errors
authorKyle M Hall <kyle@bywatersolutions.com>
Tue, 10 Apr 2012 15:36:36 +0000 (11:36 -0400)
committerJared Camins-Esakov <jcamins@cpbibliography.com>
Sat, 5 May 2012 12:16:06 +0000 (08:16 -0400)
commit0a476f82315e23b0853e38a3b8894cfba1914915
tree89e75a294db14ac8ce5c5778ddc2b31c218e7854
parent809af2ba094a9cd192b411775560e17ad50b3c39
Bug 7940 - Placing a hold on a single item from the staff cart causes errors

This is caused by the javascript function placeHold() in basket.pl
The cause of this error is thus: when a staff member uses the cart
to place holds on multiple items at once, the cart redirects to
reserver/request.pl with the params 'biblionumbers' ( a string of
biblionumbers separated by slashes ( e.g. '5/4/3/' ) and the param
multi_hold with a value of 1.

When multi_hold is enabled, request.pl splits the string 'biblionumbers'
on those slashes and works on that list.

In placeHold(), when only one item is checked, the system passes
the param biblionumbers with a single biblionumber ( e.g. '5/' )
and does *not* pass the multi_hold param. This causes request.pl
to not parse the biblionumbers param, and thus reserve.pl has
no biblionumber to work on ( hence our error here ).

There are two options to resolve this:
A) Add the multi_hold param even for a single hold from the cart.
B) In the event of a single hold being placed from the cart,
   switch to the standard single hold url ( i.e. request.pl?biblionumber=234 )

This commit resolves the situation using option B, as it seems more
logical than using the multi-holds system for a single hold.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
passes tests, works as advertised. Good catch!

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt