From 463d52ce1562903aff1097da3dfbb4db4d2de31c Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 16 Oct 2014 10:29:11 -0400 Subject: [PATCH] Bug 13099 - "Item hasn't been transferred yet" shows pickup branch rather than holding branch The "Item hasn't been transferred yet" message on circulation.pl and moremember.pl in the holds table shows the pickup branch rather than holding branch. Test Plan: 1) Place an item on hold for a patron 2) Check the item in to set the hold to waiting 3) Place an item level hold for another patron, make the pickup location be a different library 4) Load the holds tab on the patron details page for the second patron 5) Note it says "Item hasn't been transferred yet from " 6) Apply this patch 7) Repeat step 4 8) Note the library listed is now the holding library Signed-off-by: Paola Rossi Signed-off-by: Katrin Fischer Works as described and passes tests and QA script. Signed-off-by: Tomas Cohen Arazi --- svc/holds | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/svc/holds b/svc/holds index 3ff704f3d2..43bcf2a54a 100755 --- a/svc/holds +++ b/svc/holds @@ -128,7 +128,7 @@ while ( my $h = $holds_rs->next() ) { $h->branchcode()->branchcode() ) { $hold->{not_transferred} = 1; - $hold->{not_transferred_by} = $h->branchcode()->branchname(); + $hold->{not_transferred_by} = $h->item()->holdingbranch()->branchname(); } } -- 2.39.5