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>

diff --git a/reserve/request.pl b/reserve/request.pl
index f92b2ab..5ff0cb7 100755 (executable)
--- 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'),
 );