From 38398f359912db4c0761175afbd71caebd2a9331 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 --- svc/holds | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/svc/holds b/svc/holds index 3ff704f..43bcf2a 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(); } } -- 1.7.2.5