Attachment 11877 Details for Bug 8700 – Bug 8700 - RESERVESLIP fields not being replaced correctly

[patch] Bug 8700 - RESERVESLIP fields not being replaced correctly

Bug-8700---RESERVESLIP-fields-not-being-replaced-c.patch (text/plain), 1.36 KB, created by Kyle M Hall (khall) on 2012-08-29 12:45:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2012-08-29 12:45:24 UTC

Size: 1.36 KB

patch

obsolete

>From 15f01875e4214317168ea01a8259a62e5e94c892 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 29 Aug 2012 08:44:26 -0400
>Subject: [PATCH] Bug 8700 - RESERVESLIP fields not being replaced correctly
>Content-Type: text/plain; charset="utf-8"
>
>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.
>---
> C4/Reserves.pm |    6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/C4/Reserves.pm b/C4/Reserves.pm
>index 64cfc4e..aeccb12 100644
>--- a/C4/Reserves.pm
>+++ b/C4/Reserves.pm
>@@ -2079,9 +2079,9 @@ sub ReserveSlip {
>         tables => {
>             'reserves'    => $reserve,
>             'branches'    => $reserve->{branchcode},
>-            'borrowers'   => $reserve,
>-            'biblio'      => $reserve,
>-            'items'       => $reserve,
>+            'borrowers'   => $reserve->{borrowernumber},
>+            'biblio'      => $reserve->{biblionumber},
>+            'items'       => $reserve->{itemnumber},
>         },
>     );
> }
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 8700: 11877 | 12406