Bug 8224 - Make authorized values appear in planning list

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
I can confirm the bugs on current master and this patch fixes both problems.

- Pull down list for "Planning" on the budget page was not showing
  categories of authorized values
- Planning page showed the categories, but budget_period_id was
  missing from the URL

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-admin-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-admin-toolbar.inc
index 3d9c1ec..ca517f9 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-admin-toolbar.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/budgets-admin-toolbar.inc
@@ -56,14 +56,12 @@
                             url: "/cgi-bin/koha/admin/aqplan.pl?budget_period_id=[% budget_period_id %]&authcat=ITEMTYPES"
                             [% UNLESS ( CAN_user_acquisition_planning_manage ) %],  id:'disabled' [% END %]
                         },
-           [% IF ( authcat ) %]
-                [% FOREACH auth_cats_loo IN auth_cats_loop %]
-                        { text: _("Plan by [% auth_cats_loo.authcat %]    "),
-                            url: "/cgi-bin/koha/admin/aqplan.pl?budget_period_id=[% auth_cats_loo.budget_period_id %]&authcat=[% auth_cats_loo.authcat %]"
-                            [% UNLESS ( CAN_user_acquisition_planning_manage ) %],  id:'disabled' [% END %]
-                        },
-                [% END %]
-           [% END %]
+        [% FOREACH auth_cats_loo IN auth_cats_loop %]
+            { text: _("Plan by [% auth_cats_loo.authcat %]    "),
+            url: "/cgi-bin/koha/admin/aqplan.pl?budget_period_id=[% budget_period_id %]&authcat=[% auth_cats_loo.authcat %]"
+            [% UNLESS ( CAN_user_acquisition_planning_manage ) %],  id:'disabled' [% END %]
+            },
+        [% END %]
                 ]
 
                    new YAHOO.widget.Button({