[Koha-patches] [PATCH] [SIGNED-OFF] Fix for Bug 4913 - Budget pages should show currency name instead of symbol
Galen Charlton
gmcharlt at gmail.com
Sat Nov 13 04:53:23 CET 2010
From: Owen Leonard <oleonard at myacpl.org>
This patch adds a new include file to handle the display of the active
currency name and a warning if no currency is active.
Signed-off-by: Galen Charlton <gmcharlt at gmail.com>
---
admin/aqbudgetperiods.pl | 4 +++-
admin/aqbudgets.pl | 4 +++-
admin/aqplan.pl | 5 +++--
.../prog/en/includes/budgets-active-currency.inc | 1 +
.../prog/en/modules/admin/aqbudgetperiods.tmpl | 4 +++-
.../prog/en/modules/admin/aqbudgets.tmpl | 6 +++---
.../prog/en/modules/admin/aqplan.tmpl | 2 +-
7 files changed, 17 insertions(+), 9 deletions(-)
create mode 100644 koha-tmpl/intranet-tmpl/prog/en/includes/budgets-active-currency.inc
diff --git a/admin/aqbudgetperiods.pl b/admin/aqbudgetperiods.pl
index 62383f1..39f3cd0 100755
--- a/admin/aqbudgetperiods.pl
+++ b/admin/aqbudgetperiods.pl
@@ -82,7 +82,9 @@ my ($template, $borrowernumber, $cookie, $staff_flags ) = get_template_and_user(
my $cur = GetCurrency();
-$template->param( cur => $cur->{symbol} );
+$template->param( symbol => $cur->{symbol},
+ currency => $cur->{currency}
+ );
my $cur_format = C4::Context->preference("CurrencyFormat");
my $num;
diff --git a/admin/aqbudgets.pl b/admin/aqbudgets.pl
index 34a4021..d266727 100755
--- a/admin/aqbudgets.pl
+++ b/admin/aqbudgets.pl
@@ -52,7 +52,9 @@ my ($template, $borrowernumber, $cookie, $staffflags ) = get_template_and_user(
);
my $cur = GetCurrency();
-$template->param( cur => $cur->{symbol} );
+$template->param( symbol => $cur->{symbol},
+ currency => $cur->{currency}
+ );
my $op = $input->param('op');
diff --git a/admin/aqplan.pl b/admin/aqplan.pl
index 8efa427..8b68d51 100755
--- a/admin/aqplan.pl
+++ b/admin/aqplan.pl
@@ -60,7 +60,9 @@ my $budget_period_id = $input->param('budget_period_id');
my $period = GetBudgetPeriod($budget_period_id);
my $count = GetPeriodsCount();
my $cur = GetCurrency;
-
+$template->param( symbol => $cur->{symbol},
+ currency => $cur->{currency}
+ );
$template->param( period_button_only => 1 ) if $count == 0;
@@ -450,7 +452,6 @@ $template->param(
show_actual => $show_actual,
show_percent => $show_percent,
show_mine => $show_mine,
- cur => $cur->{symbol},
cur_format => $cur_format,
CGIextChoice => $CGIextChoice,
CGIsepChoice => $CGIsepChoice,
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
new file mode 100644
index 0000000..a4571f9
--- /dev/null
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-active-currency.inc
@@ -0,0 +1 @@
+<!-- TMPL_IF name="currency" --><p><b>Currency = <!-- TMPL_VAR NAME="currency" --></b></p><!-- TMPL_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><!-- /TMPL_IF -->
\ No newline at end of file
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tmpl
index 78c51b8..523fd13 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tmpl
@@ -254,7 +254,9 @@
<!-- DEFAULT display budget periods list -->
<!-- TMPL_IF name="else" -->
<h2>Budgets administration</h2>
- <!-- TMPL_IF name="cur" --><p><b>Currency = <!-- TMPL_VAR name="cur" --></b></p><!-- TMPL_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><!-- /TMPL_IF -->
+
+ <!-- TMPL_INCLUDE NAME="budgets-active-currency.inc" -->
+
<table id="periodsh">
<thead>
<tr>
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tmpl
index 5e0db00..80d9156 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tmpl
@@ -157,9 +157,10 @@
<!-- TMPL_UNLESS name="delete_confirm" --><!-- TMPL_INCLUDE NAME="budgets-admin-toolbar.inc" --><!-- /TMPL_UNLESS -->
<!-- TMPL_IF name="else" -->
-<!-- TMPL_UNLESS NAME="cur" --><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><!-- /TMPL_UNLESS -->
-
<h1>Funds for '<!-- TMPL_VAR name="budget_period_description" -->'</h1>
+
+<!-- TMPL_INCLUDE NAME="budgets-active-currency.inc" -->
+
<table id="budgeth">
<thead>
<tr>
@@ -240,7 +241,6 @@
</tbody>
</table>
-<!-- TMPL_IF name="cur" --><p><b>Currency = <!-- TMPL_VAR NAME="cur" --> <!-- TMPL_VAR NAME="cur_format" --></b></p><!-- /TMPL_IF -->
<!-- TMPL_IF NAME="pagination_bar" --><div class="paginationBar"><!-- TMPL_VAR NAME="pagination_bar" --></div><!-- /TMPL_IF -->
<!-- /TMPL_IF --> <!-- else -->
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tmpl
index 170c7a0..2192178 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tmpl
@@ -139,7 +139,7 @@ YAHOO.util.Event.onAvailable("popmenu", function () {
<!-- TMPL_IF NAME="budget_lines" -->
- <!-- TMPL_IF name="cur" --><p><b>Currency = <!-- TMPL_VAR name="cur" --></b></p><!-- TMPL_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><!-- /TMPL_IF -->
+ <!-- TMPL_INCLUDE NAME="budgets-active-currency.inc" -->
<table id="plan" width="100%">
<thead>
--
1.7.0
More information about the Koha-patches
mailing list