From 5bdd4c02af25873aed550bd1e312d07fa9b229f0 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Mon, 26 Mar 2018 16:55:51 -0300
Subject: [PATCH] Bug 18789: (follow-up) Fix place hold page

This patch fixes very weird behaviours introduced by
  commit 0ab22e1c7c6b6e8d14a4fb7478ab2a3d42da347d
  Bug 18789: Send Koha::Patron object to the templates

The patron variable was only set when no action was defined.
This patch restores the feature the easiest way possible (less changes)
even if it is not the best one.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 reserve/request.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/reserve/request.pl b/reserve/request.pl
index f92b2ab86d..5ff0cb7d41 100755
--- a/reserve/request.pl
+++ b/reserve/request.pl
@@ -181,6 +181,7 @@ if ($borrowernumber_hold && !$action) {
 
     my $amount_outstanding = $patron->account->balance;
     $template->param(
+                patron              => $patron,
                 expiry              => $expiry,
                 diffbranch          => $diffbranch,
                 messages            => $messages,
@@ -635,7 +636,6 @@ if ( C4::Context->preference( 'AllowHoldDateInFuture' ) ) {
 }
 
 $template->param(
-    patron => $patron,
     SuspendHoldsIntranet => C4::Context->preference('SuspendHoldsIntranet'),
     AutoResumeSuspendedHolds => C4::Context->preference('AutoResumeSuspendedHolds'),
 );
-- 
2.20.1