Bug 18736: (follow-up) Fix missing rounding and bad formatting
authorNick Clemens <nick@bywatersolutions.com>
Fri, 24 Aug 2018 01:22:35 +0000 (01:22 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Thu, 21 Mar 2019 16:29:06 +0000 (16:29 +0000)
commit4ccbae8879a386a1846bb48c18b3722f936dc983
tree129f89c76bee16ef26638cd7c9732d578a2f5d87
parentb17ca2b701fab5307330b6196590e5d3048978e1
Bug 18736: (follow-up) Fix missing rounding and bad formatting

This patch:
Adds a missing use
Uses 'Koha::Number::Price->round()' instead of 'format()' to ensure numeric
returns
Ensures too big numbers don't crash round()
Uses syspref in 'GetBudgetHierarchy'

To test:
Follow previous test plan
Check values on admin/aqbudgets.pl are affected by syspref
Ensure values throughout acquisitions are correctly calculated/displayed
(even when greater than 1,000)

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/Acquisition.pm
C4/Budgets.pm
Koha/Number/Price.pm
acqui/spent.pl