From: [email protected] <[email protected]>

If you have a list of holds on a biblio, and one or more are in-transit, then 
the
array that is fed to modrequest.pl is not fully-populated, lacking the branch 
on the
in-transit rows.  If you then attempt to edit one of the remaining holds' pickup
location, it doesn't modify the one you expect, but ones *above* that.  Also, 
holds
at the bottom of the list get the first pickup library in the list, since they 
are
getting undef passed in.

Signed-off-by: Nicole C. Engard <[email protected]>
---
 .../prog/en/modules/reserve/request.tmpl           |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tmpl 
b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tmpl
index 58a5ad9..53ff7b7 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tmpl
@@ -639,7 +639,7 @@ function checkMultiHold() {
                 <!-- TMPL_IF name="found" -->
                 Item waiting at <b> <!-- TMPL_VAR NAME="wbrname" --></b> 
<input type="hidden" name="pickup" value="<!-- TMPL_VAR NAME="wbrcode" -->" />
             <!-- TMPL_ELSE -->
-                Waiting to be pulled
+                Waiting to be pulled <input type="hidden" name="pickup" 
value="<!-- TMPL_VAR NAME="wbrcode" -->" />
             <!-- /TMPL_IF -->
           <!-- TMPL_ELSE -->
             Item being transferred to <b> <!-- TMPL_VAR NAME="wbrname" --></b> 
<input type="hidden" name="pickup" value="<!-- TMPL_VAR NAME="wbrcode" -->" />
-- 
1.7.2.3

_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-patches
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to