[koha-commits] main Koha release repository branch master updated. v16.11.00-796-g476179a

Git repo owner gitmaster at git.koha-community.org
Thu Apr 13 14:34:42 CEST 2017


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, master has been updated
       via  476179a12311b32643087cc8ec10c31409bb1b99 (commit)
      from  17f4b05d23cb19cebeedb361005bf411a5f25c60 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 476179a12311b32643087cc8ec10c31409bb1b99
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Tue Jan 10 10:03:36 2017 +0100

    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 at veron.ch>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

-----------------------------------------------------------------------

Summary of changes:
 C4/Budgets.pm            |    2 +-
 admin/aqbudgetperiods.pl |    2 --
 admin/aqbudgets.pl       |    3 +--
 3 files changed, 2 insertions(+), 5 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list