From 0bfd512759b54bef56f279cf195c9186691933e5 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Marc=20V=C3=A9ron?= <veron@veron.ch>
Date: Fri, 18 Dec 2015 16:01:12 +0100
Subject: [PATCH]  Bug 15049: (followup) Add warning about "No active currency"

Fix translatibiity issues and use include in admin/aqplan.tt as well.
See comment #24

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
---
 .../prog/en/includes/budgets-active-currency.inc   |  2 +-
 .../prog/en/modules/acqui/acqui-home.tt            |  2 +-
 .../intranet-tmpl/prog/en/modules/admin/aqplan.tt  | 14 ++++++++------
 3 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-active-currency.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-active-currency.inc
index fbaa7e8e9d..18620599e2 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-active-currency.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-active-currency.inc
@@ -6,7 +6,7 @@
     <div class="dialog alert">
         <h3>No active currency is defined</h3>
         [% IF CAN_user_parameters_parameters_remaining_permissions %]
-            <p>Please <a href="/cgi-bin/koha/admin/currency.pl">specify an active currency</a>.</p>
+            <p><a href="/cgi-bin/koha/admin/currency.pl">Please specify an active currency.</a></p>
         [% ELSE %]
             <p>Your administrator must specify an active currency.</p>
         [% END %]
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt
index ab062679d8..82f201f057 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt
@@ -101,7 +101,7 @@ $(document).ready(function() {
     [% INCLUDE 'budgets-active-currency.inc' hide = 'yes' %]
     [% UNLESS ( loop_budget ) %]
         [% IF ( CAN_user_acquisition_period_manage ) %]
-        <div class="dialog alert">You must <a href="/cgi-bin/koha/admin/aqbudgetperiods.pl">define a budget</a> in Administration</div>
+        <div class="dialog alert"><a href="/cgi-bin/koha/admin/aqbudgetperiods.pl">You must define a budget in Administration</a></div>
         [% ELSE %]
         <div class="dialog alert">Your administrator must define a budget in Administration</div>
         [% END %]
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt
index 9b2f2affe9..86a196251b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt
@@ -85,13 +85,15 @@
 
     [% IF ( budget_lines ) %]
 
-    [% IF ( currency ) %]<p><b>Currency = [% currency %]</b>.
-    [% IF ( show_actual ) %]
-        <b>Each cell contain both actual and estimated values.</b>
+    [% IF ( currency ) %]<p><b>Currency = [% currency %].</b>
+        [% IF ( show_actual ) %]
+            <b>Each cell contain both actual and estimated values.</b>
+        [% ELSE %]
+            <b>Cells contain estimated values only.</b>
+        [% END %]
+        </p>
     [% ELSE %]
-        <b>Cells contain estimated values only.</b>
-    [% END %]
-    </p>[% ELSE %]<div class="dialog alert"><h3>No active currency is defined</h3><p>Please <a href="/cgi-bin/koha/admin/currency.pl">specify an active currency</a>.</p></div>
+        [% INCLUDE 'budgets-active-currency.inc' hide = 'yes' %]
 [% IF ( show_actual ) %]
         <p><b>Each cell contain both actual and estimated values.</b></p>
     [% ELSE %]
-- 
2.39.5