| Summary: | Calculation differs on aqui-home/spent and ordered.pl | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
| Component: | Acquisitions | Assignee: | Nick Clemens (kidclamp) <nick> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | bdaeuber, lucas, m.de.rooy, martin.renvoize |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 |
| Patch complexity: | Trivial patch | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: |
19.05.00, 18.11.06
|
Circulation function: | |
| Bug Depends on: | 18736 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 22791: Unify calculation between aqui-home and ordered.pl
Bug 22791: Unify calculation between aqui-home and ordered.pl Bug 22791: Unify calculation between aqui-home and ordered.pl |
||
|
Description
Nick Clemens (kidclamp)
2019-04-26 19:57:44 UTC
Created attachment 88968 [details] [review] Bug 22791: Unify calculation between aqui-home and ordered.pl To test: 1 - Enable order price rounding 2 - Order an example with 8 * 16,99 and 42% discount, 19.45% tax. 3 - Check ordered page - 94.17 is total 4 - Check aqui-home - 94.16 is total 5 - Apply patch 6 - Verify numbers now match Thought we resolved these numbers :) (In reply to Marcel de Rooy from comment #2) > Thought we resolved these numbers :) Baby steps Created attachment 89327 [details] [review] Bug 22791: Unify calculation between aqui-home and ordered.pl To test: 1 - Enable order price rounding 2 - Order an example with 8 * 16,99 and 42% discount, 19.45% tax. 3 - Check ordered page - 94.17 is total 4 - Check aqui-home - 94.16 is total 5 - Apply patch 6 - Verify numbers now match Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Created attachment 89422 [details] [review] Bug 22791: Unify calculation between aqui-home and ordered.pl To test: 1 - Enable order price rounding 2 - Order an example with 8 * 16,99 and 42% discount, 19.45% tax. 3 - Check ordered page - 94.17 is total 4 - Check aqui-home - 94.16 is total 5 - Apply patch 6 - Verify numbers now match Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Awesome work all! Pushed to master for 19.05 Pushed to 18.11.x for 18.11.06 missing dependencies for 18.05.x, no backport |