[Bug 3166] New: Fines are summing off by 1 penny due to rounding issue when summing.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 Summary: Fines are summing off by 1 penny due to rounding issue when summing. Product: Koha Version: rel_3_0 Platform: All OS/Version: All Status: NEW Severity: minor Priority: PATCH-Sent Component: Patrons AssignedTo: galen.charlton@liblime.com ReportedBy: drdrulm@yahoo.com Estimated Hours: 0.0 Fines are summing off by 1 penny due to rounding issue when summing. This only occurs when summing many many records. The round off error occurs in the summation. There is a multiplicand of 100 and then a divisor of 100. To ensure the summation will work for many fines, just make the multiplicand 1000 and the divisor 1000. If there is still a problem with this, then bump it to 10000. -D.Ulm -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 --- Comment #1 from Galen Charlton <galen.charlton@liblime.com> 2009-04-27 20:43:45 --- Darrell, do you have an example in your database that I can look at? -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 Ryan Higgins <rch@liblime.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rch@liblime.com --- Comment #2 from Ryan Higgins <rch@liblime.com> 2009-04-30 22:26:33 --- I think the proper way to do this is to do integer math. Do we really need to support more than two decimal places ? If so, I suggest a system preference for CurrencyDecimalPlaces and a fcn C4::Output::as_currency(). (Or C4::Koha::). The db will then always hold an integer value of 'cents' (or amount * 10^(CurrencyDecimalPlaces) ). -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 Cookie Wolfrom <cookie.wolfrom@liblime.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cookie.wolfrom@liblime.com -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 Darrell Ulm <drdrulm@yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |drdrulm@yahoo.com --- Comment #3 from Darrell Ulm <drdrulm@yahoo.com> 2009-07-22 14:04:28 --- Patch sent again! :) Simple one. Please accept. Occurs with patrons with long list of fines. -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 --- Comment #4 from Galen Charlton <galen.charlton@liblime.com> 2009-07-22 14:27:45 --- Pushed patch. Bug should remain open, as we should switch to purely integer math as Ryan suggests. -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 David Schuster <dschust1@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dschust1@gmail.com -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 J. David Bavousett <dbavousett@ptfs.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dbavousett@ptfs.com -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 Melanie <melanie@salpublib.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |melanie@salpublib.org -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3166 Adrea Lund <adrea@moablibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |adrea@moablibrary.org -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
participants (2)
-
bugzilla-daemon＠kohaorg.ec2.liblime.com -
bugzilla-daemon＠liblime.com