[kmymoney] [Bug 402708] Division by zero errors in investement reports

[Thomas Baumgart]

https://bugs.kde.org/show_bug.cgi?id=402708

Thomas Baumgart  changed:

   What|Removed |Added

   Version Fixed In||5.0.3
 Resolution|--- |FIXED
 Status|REPORTED|RESOLVED
  Latest Commit||https://commits.kde.org/kmy
   ||money/741e848593162aa670c73
   ||5b812b0574dbb483907

--- Comment #3 from Thomas Baumgart  ---
Git commit 741e848593162aa670c735b812b0574dbb483907 by Thomas Baumgart.
Committed on 17/01/2019 at 07:28.
Pushed by tbaumgart into branch '5.0'.

Fix division by zero errors in QueryTable

Summary:
Code used to try to divide by zero. Fixed it to always check for the
denominator being 0 beforehand.

Original patch by Zoltan Ivanfi
FIXED-IN: 5.0.3

Reviewers: #kmymoney, tbaumgart

Reviewed By: #kmymoney, tbaumgart

Subscribers: aacid, tbaumgart, ostroffjh

Differential Revision: https://phabricator.kde.org/D17871

M  +8-5kmymoney/plugins/views/reports/core/querytable.cpp
M  +28   -0kmymoney/plugins/views/reports/core/tests/querytable-test.cpp
M  +1-0kmymoney/plugins/views/reports/core/tests/querytable-test.h

https://commits.kde.org/kmymoney/741e848593162aa670c735b812b0574dbb483907

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney] [Bug 402708] Division by zero errors in investement reports

[Jack]

https://bugs.kde.org/show_bug.cgi?id=402708

--- Comment #2 from Jack  ---
I just added KMyMoney as reviewer in Phabricator.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney] [Bug 402708] Division by zero errors in investement reports

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=402708

--- Comment #1 from zoltan1...@gmail.com ---
I uploaded a fix for review. The instructions said the it will automatically
get assigned, but it did not and complains that "This revision needs review,
but there are no reviewers specified."

https://phabricator.kde.org/D17871

-- 
You are receiving this mail because:
You are watching all bug changes.