From e731c50a9d75fb52e1112c188846bd845fcdbbb9 Mon Sep 17 00:00:00 2001 From: Mark Tompsett Date: Mon, 6 Apr 2015 19:50:30 -0400 Subject: [PATCH] Bug 4502 - An attempt to make things more sensible. The reason the budget_period_id was not defined was because in two cases it was not passed! This patch adds those missing parameters. And as a result, cuts out the attempt to default the authcat to '' unless the budget_period_id is defined. Additionally, the start and end months don't seem to be passed, so rather than have it blow up, checking them forces the else case logic. budget_period_id is the budget id. If you have two budgets, you can craft a URL to work with budget_period_id matching those two ids. Anything else should trigger the new error which was modified to reflect more of what the problem is. Follow the test plan in comment #6. Feel free to also to attempt crafting URLs and triggering errors. --- admin/aqplan.pl | 4 ---- koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt | 4 +++- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/admin/aqplan.pl b/admin/aqplan.pl index 879bb0e..bdd5dda 100755 --- a/admin/aqplan.pl +++ b/admin/aqplan.pl @@ -106,10 +106,6 @@ if ( $budget_period_locked == 1 && not defined $show_actual ) { $authcat = 'Asort1' if not defined $authcat; # defaults to Asort if no authcat given -# If no active budget is defined, budget_period_id is undef -# This is the behavior of GetBudgetPeriod -$authcat = '' unless $budget_period_id; - my $budget_id = $input->param('budget_id'); my $op = $input->param("op"); 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 fdf84b6..5ff6f4d 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt @@ -226,7 +226,7 @@ [% ELSIF not budget_period_id %] -
No active budget defined. You should define one before continuing.
+
That budget is not defined. You should selected a defined budget to continue.
[% ELSE %]
No funds to display for this search criteria
[% END %] @@ -237,6 +237,7 @@
+

Filter

[% BLOCK planning_types %] @@ -311,6 +312,7 @@
+
-- 1.9.1