Bug 26239

Summary: Number::Format issues with large negative numbers
Product: Koha Reporter: Nick Clemens <nick>
Component: Architecture, internals, and plumbingAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: amit.gupta, amitddng135
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00
Bug Depends on: 15770    
Bug Blocks:    
Attachments: Bug 26239: Do not format large negative numbers
Bug 26239: Do not format large negative numbers
Error screenshot
Bug 26239: Do not format large negative numbers

Description Nick Clemens 2020-08-18 16:43:19 UTC
Follow-up to bug 15770 for negative numbers
Comment 1 Nick Clemens 2020-08-18 16:44:22 UTC
Created attachment 108528 [details] [review]
Bug 26239: Do not format large negative numbers

At several places we got the following error if we use numbers too big
for Number::Format
Template process failed: undef error - round() overflow. Try smaller
precision or use Math::BigFloat at /home/koha/src/Koha/Number/Price.pm line 44

It make the app explodes.
The goal here is to handle these errors gracefully and easily.

We fixed it for positive numbers in bug 15770, but we neglected the case of negative numbers

Test plan:
- Add a manual credit to a patron of 100000000000000
- ISE!
- Apply patch
- Restart all the things
-
Comment 2 Amit Gupta 2020-08-18 17:16:31 UTC
Created attachment 108530 [details] [review]
Bug 26239: Do not format large negative numbers

Test plan:
- Add a manual credit to a patron of 100000000000000
- ISE!
- Apply patch
- Restart all the things
-------------------------------
Checked and worked OK.
Comment 3 Amit Gupta 2020-08-18 17:17:19 UTC
Created attachment 108531 [details]
Error screenshot
Comment 4 Katrin Fischer 2020-08-18 21:31:10 UTC
Created attachment 108560 [details] [review]
Bug 26239: Do not format large negative numbers

At several places we got the following error if we use numbers too big
for Number::Format
Template process failed: undef error - round() overflow. Try smaller
precision or use Math::BigFloat at /home/koha/src/Koha/Number/Price.pm line 44

It make the app explodes.
The goal here is to handle these errors gracefully and easily.

We fixed it for positive numbers in bug 15770, but we neglected the case of negative numbers

Test plan:
- Add a manual credit to a patron of 100000000000000
- ISE!
- Apply patch
- Restart all the things

Works perfectly.

Signed-off-by: Amit Gupta <amit.gupta@informaticsglobal.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2020-08-19 07:36:40 UTC
Pushed to master for 20.11, thanks to everybody involved!