]> git.koha-community.org Git - koha.git/commit
Bug 15009 - Planning dropdown button in aqbudget can have empty line
authorBlou <philippe.blouin@inlibro.com>
Tue, 13 Oct 2015 20:12:49 +0000 (16:12 -0400)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 29 Apr 2016 12:59:16 +0000 (12:59 +0000)
commitb8076d8ec7236360274e1a4e861969c86bb679ea
tree671a456946bcb99e7a5344c76dd73b80805735af
parentb4ce86df28ce937ea06b8f31010bf4cd85755c52
Bug 15009 - Planning dropdown button in aqbudget can have empty line

When displaying a budget, the Planning button in the admin toolbar displays

Plan by months
Plan by libraries
Plan by item types
Plan by

The last one is empty, due to C4::Budgets::GetBudgetAuthCats returning an empty field if the budget has no sort defined.
This prevents returning an array with empty element(s)

TEST:
1) Admin -> Budgets
2) Select a budget
   a) you must have '' (empty) in your aqbudgets.sort1_authcat field.
   b) edit the budget (direct DB or interface) to get that.
3) Click on Planning dropdown, see the "Plan by <nothing> " entry.
4) apply the patch, revalidate.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
No more empty option
No errors

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
C4/Budgets.pm