From 837fa140e7c6621b3cfa9b0ce825844cac4543f5 Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Thu, 6 Aug 2015 10:49:03 +0200 Subject: [PATCH] Bug 14653: Acq planning filter always use default active period In Acq planning /cgi-bin/koha/admin/aqplan.pl, left side filter and export forms have no parameter budget_period_id nor authcat, so the use the default active period. As said in code : - IF PERIOD_ID IS DEFINED, GET THE PERIOD - ELSE GET THE ACTIVE PERIOD BY DEFAULT my $period = GetBudgetPeriod($budget_period_id); This patch corrects the filter form. Test plan : - Create an active budget "B_ACTIVE" with 1 fund - Create an inactive budget "B_OLD" with 3 funds - Go to "B_OLD" planning by libraries - You see 3 funds - In filter form, select "by months" in "Select planning type" => Without patch, you see planning of "B_ACTIVE" with 1 fund => Without patch, you see planning of "B_OLD" with 3 funds Signed-off-by: Hector Castro Signed-off-by: Jonathan Druart Signed-off-by: Tomas Cohen Arazi --- koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) 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 2510d043d4..2b16b64e4d 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt @@ -285,8 +285,10 @@ -
-
+
+ + +
[% IF ( budget_lines ) %] -- 2.39.5