]> git.koha-community.org Git - koha.git/commit
Bug 17872: Fix small error in GetBudgetHierarchy and one of its calls
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Tue, 10 Jan 2017 09:03:36 +0000 (10:03 +0100)
committerKatrin Fischer <katrin.fischer.83@web.de>
Sun, 16 Apr 2017 12:14:08 +0000 (14:14 +0200)
commit3079f7a72515a22062bd9a65ec546cfcd6d9bb66
tree728321dc06e9fd0dd4f242014129511b4eb26c96
parent733d77520f92f9367fa0a100cdcda79d2a98232d
Bug 17872: Fix small error in GetBudgetHierarchy and one of its calls

In aqbudgetperiods.pl a commented line is removed that contains a
wrong userenv hash key. Should be branch instead of branchcode.

In aqbudgets GetBudgetHierarchy is called with the same wrong userenv
hash key. Should be userenv->{branch}.
This made another bug visible: if you call GetBudgetHierarchy with a
branch and without owner, the where clause should take into account
that the branchcode can be empty (not null).

Test plan:
[1] Run Budgets.t
[2] Run aqbudgets.pl from Administration
[3] Add the show_mine=1 parameter in the URL
[4] Change owner of one of the funds and repeat step 2 and 3.

t/Budgets.t returns green. Followed test plan, works as expected.
Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 476179a12311b32643087cc8ec10c31409bb1b99)
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
C4/Budgets.pm
admin/aqbudgetperiods.pl
admin/aqbudgets.pl