Bug 5084 - GetBudgetHierarchy doesn't respect the 'active' switch on budgets
Summary: GetBudgetHierarchy doesn't respect the 'active' switch on budgets
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal (vote)
Assignee: Robin Sheat
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-04 02:12 UTC by Robin Sheat
Modified: 2012-10-25 23:04 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
proposed patch (1.20 KB, patch)
2010-08-04 03:43 UTC, Robin Sheat
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Robin Sheat 2010-08-04 02:12:30 UTC
GetBudgetHierarchy produces the list of funds that are available when showing the dropdown in the orders page, and the list on the acquisition home page.

This ignores the 'active' state of the budget, which leads to a) inactive funds being shown, and b) them able to be selected when they probably shouldn't be.
Comment 1 Robin Sheat 2010-08-04 03:43:41 UTC
Created attachment 2448 [details] [review]
proposed patch
Comment 2 Chris Cormack 2010-12-14 02:13:18 UTC
Pushed, please test