Bug 12992 - Fund planning does not display actual values
Summary: Fund planning does not display actual values
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-25 07:39 UTC by Jonathan Druart
Modified: 2015-06-04 23:33 UTC (History)
1 user (show)

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


Attachments
Bug 12992: Fund planning should display actual values (1.58 KB, patch)
2014-09-25 07:42 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 12992: Fund planning should display actual values (1.63 KB, patch)
2014-09-28 06:27 UTC, Chris Cormack
Details | Diff | Splinter Review
[PASSED QA] Bug 12992: Fund planning should display actual values (1.70 KB, patch)
2014-10-03 14:11 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2014-09-25 07:39:28 UTC
This is broken since the migration to TT.
Comment 1 Jonathan Druart 2014-09-25 07:42:51 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2014-09-28 06:27:58 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall 2014-10-03 14:11:25 UTC
Created attachment 31986 [details] [review]
[PASSED QA] Bug 12992: Fund planning should display actual values

The 'show_actual' variable is not correctly manage in the template.
This has certainly been introduced by the migration to TT.

Test plan:
1/ Go on a bugdet planning view (admin/aqplan.pl)
2/ Check the "Show actual/estimated values" checkbox on the left
(Filters box)
3/ Go
4/ Verify the actual values are now displayed (i.e. the columns contain
2 values).

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 4 Tomás Cohen Arazi 2014-11-17 00:38:43 UTC
Patch pushed to master.

Thanks Jonathan!