From a7ef547ec95f664656d9e2fd5dab919a6d00e5ca Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 29 Aug 2012 08:44:26 -0400 Subject: [PATCH] Bug 8700 - RESERVESLIP fields not being replaced correctly The slip RESERVESLIP is not replacing fields correctly. C4::Reserves::ReserveSlip calls C4::Letters::GetPreparedLetter, and passes the $reserve hashref to it for each table except branches ( which is passed the branchcode ). The problem is, if you pass a hashref for a table, it uses that hashref for the replacing, rather than looking up the data from the database. Fixed by passing the correct keys for each of the tables requested. Signed-off-by: Marc Veron Tested following the test plan. Could reproduce the bug. After applying the patch slip printed as expected. Signed-off-by: Paul Poulain --- C4/Reserves.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/C4/Reserves.pm b/C4/Reserves.pm index 585d1e56b2..1dc6f72ab8 100644 --- a/C4/Reserves.pm +++ b/C4/Reserves.pm @@ -2081,9 +2081,9 @@ sub ReserveSlip { tables => { 'reserves' => $reserve, 'branches' => $reserve->{branchcode}, - 'borrowers' => $reserve, - 'biblio' => $reserve, - 'items' => $reserve, + 'borrowers' => $reserve->{borrowernumber}, + 'biblio' => $reserve->{biblionumber}, + 'items' => $reserve->{itemnumber}, }, ); } -- 2.20.1