Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0050509
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajorhave not tried2022-10-13 08:492024-12-18 10:24
Reporternicola_uvaView Statuspublic 
Assigned Toaferraz 
PriorityurgentResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Versionpi
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Merge Request Statusapproved
Review Assigned Tomarvintm
OBNetwork customerGold
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0050509: WEBPOS - taxes/round/prices with 4decimal not properly calculated on the WEBPOS

DescriptionTaxes/Round/Prices not properly calculated. There is a discrepancy of 1cents beetween backoffice and POS. (see images attached)
Steps To Reproduce1)Configure currency 2 decimal,price to 4 and mask in the pos in the format: #,##0.00
2)login on webpos
3) add several lines to the ticket
4) finalize the ticket
5) compare backoffice and webpos you can see 1cents of difference (see image attached)

Below items added to the ticket with prices and taxes for each of them

product A
qty:1
net unit price: 5.9700,
gross unit price: 6.3000
tax rate: 5.5%
Taxable Amount : 5.97
Tax Amount: 0.33

product B
qty:1
net unit price: 0.0200,
gross unit price: 0.0233
tax rate: 5.5%
Taxable Amount : 0.02
Tax Amount: 0

product C
qty:1
net unit price: 0,
gross unit price: 0
tax rate: 20%
Taxable Amount : 0
Tax Amount: 0

product D
qty:1
net unit price: 5.9700,
gross unit price: 6.3000
tax rate: 5.5%
Taxable Amount : 5.97
Tax Amount: 0.33

product E
qty:1
net unit price: 8.9100,
gross unit price: 9.4000
tax rate: 5.5%
Taxable Amount : 8.91
Tax Amount: 0.49

Taxes calculation:
Actual Result -> 20.88 + 1.15 = 22.03
Expected Result -> 20.87 + 1.15 = 22.02
TagsNo tags attached.
Attached Filespng file icon image002.png [^] (62,348 bytes) 2022-10-13 08:49


png file icon image (3).png [^] (78,595 bytes) 2022-10-13 08:50


png file icon Ticket1.png [^] (305,697 bytes) 2022-10-19 17:06
png file icon Ticket2.png [^] (304,292 bytes) 2022-10-19 17:06
png file icon Ticket1-BO.png [^] (182,662 bytes) 2022-10-19 17:15

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0142294)
aferraz (viewer)
2022-10-19 16:51
edited on: 2022-10-19 17:15

 Tax engine is working as expected.

If you define a POS precision of 4, the ticket gross amount, net amount and taxes will be calculated using this precision: http://wiki.openbravo.com/wiki/Retail:Configuration_Guide#Numerical_Precision_Configuration [^]

In the issue example, the ticket amounts are correctly calculated as follows (see Ticket1.png):
grandtotal: 22.0233
totallines: 20.8752
taxbaseamt: 20.8752
taxamt: 1.1481
You can see same amounts in Sales Order window defining a proper mask in Format.xml (see Ticket1-BO.png).

By defining a POS precision of 2 in the issue example, the ticket amounts are correctly calculated as follows (see Ticket2.png):
grandtotal: 22.02
totallines: 20.87
taxbaseamt: 20.87
taxamt: 1.15
(0144853)
hgbot (developer)
2022-12-21 05:01

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/447 [^]
(0144854)
hgbot (developer)
2022-12-21 05:02

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1018 [^]
(0145173)
hgbot (developer)
2023-01-04 17:44

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.discounts/-/merge_requests/83 [^]
(0145193)
hgbot (developer)
2023-01-06 21:28

 Merge Request created: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/256 [^]
(0145985)
hgbot (developer)
2023-01-30 05:52

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1488 [^]
(0146289)
hgbot (developer)
2023-02-08 16:10

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2.restaurant/-/merge_requests/204 [^]
(0147205)
hgbot (developer)
2023-03-02 07:27

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: de6c20bc21871e6fd1a22724c3f4ca349d90b3fb
Author: Antonio Moreno <antonio.moreno@openbravo.com>
Date: 02-03-2023 07:27:27
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/de6c20bc21871e6fd1a22724c3f4ca349d90b3fb [^]

Fixed ISSUE-50509: Merge branch 'fix/50509'

---
M web/org.openbravo.mobile.core/source/utils/ob-arithmetic.js
---
(0147206)
hgbot (developer)
2023-03-02 07:27

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/447 [^]
(0147207)
hgbot (developer)
2023-03-02 07:29

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: bf4961489313b800d11143064ca75393cb596c1b
Author: Antonio Moreno <antonio.moreno@openbravo.com>
Date: 02-03-2023 07:29:21
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/bf4961489313b800d11143064ca75393cb596c1b [^]

Fixed ISSUE-50509: Merged branch.

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
M src/org/openbravo/retail/posterminal/PaidReceipts.java
M src/org/openbravo/retail/posterminal/term/Payments.java
M web/org.openbravo.retail.posterminal/app/model/business-logic/taxes-engine/engine/tax-rules.js
M web/org.openbravo.retail.posterminal/app/model/business-object/ticket/LoadTicketUtils.js
M web/org.openbravo.retail.posterminal/app/model/business-object/ticket/TicketUtils.js
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
---
(0147208)
hgbot (developer)
2023-03-02 07:29

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1018 [^]
(0147209)
hgbot (developer)
2023-03-02 07:31

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.discounts [^]
Changeset: b6dbd18d0afa4805c362912254c32a19e8620455
Author: Antonio Moreno <antonio.moreno@openbravo.com>
Date: 02-03-2023 07:31:22
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.discounts/-/commit/b6dbd18d0afa4805c362912254c32a19e8620455 [^]

Fixed ISSUE-50509: Merge branch 'fix/50509'

---
M web-test/model/business-logic/discounts-engine/engine/discount-engine.test.js
M web/org.openbravo.retail.discounts/app/model/business-logic/discounts-engine/engine/discount-engine.js
---
(0147210)
hgbot (developer)
2023-03-02 07:31

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.discounts/-/merge_requests/83 [^]
(0147211)
hgbot (developer)
2023-03-02 07:32

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 811ad79f229c1be3b5fba8563d54b6615bb64719
Author: Antonio Moreno <antonio.moreno@openbravo.com>
Date: 02-03-2023 07:32:26
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/811ad79f229c1be3b5fba8563d54b6615bb64719 [^]

Fixed ISSUE-50509: Merge branch 'fix/50509'

---
M web-jspack/org.openbravo.pos2/src/authentication/loginInitActions.js
M web-jspack/org.openbravo.pos2/src/components/Ticket/TicketLinePrice/TicketLinePrice.jsx
---
(0147212)
hgbot (developer)
2023-03-02 07:32

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1488 [^]
(0147213)
hgbot (developer)
2023-03-02 07:33

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 23427339b156bcda9545d66d2613761edc49c3ba
Author: Gabriel Lopez <gabriel.lopez@doceleguas.com>
Date: 02-03-2023 06:33:11
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/23427339b156bcda9545d66d2613761edc49c3ba [^]

Fixed ISSUE-50509: Adjusted tests

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I32263_VerifyOrderAmtBasedOnPrecision.java
---
(0147214)
hgbot (developer)
2023-03-02 07:33

 Merge request merged: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/256 [^]
(0173415)
hgbot (developer)
2024-12-18 10:24

 Merge request closed: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2.restaurant/-/merge_requests/204 [^]

- Issue History
Date Modified Username Field Change
2022-10-13 08:49 nicola_uva New Issue
2022-10-13 08:49 nicola_uva Assigned To => Retail
2022-10-13 08:49 nicola_uva File Added: image002.png
2022-10-13 08:49 nicola_uva OBNetwork customer => Gold
2022-10-13 08:49 nicola_uva Triggers an Emergency Pack => No
2022-10-13 08:50 nicola_uva File Added: image (3).png
2022-10-13 09:45 nicola_uva Steps to Reproduce Updated View Revisions
2022-10-13 18:34 nicola_uva Summary WEBPOS - taxes/round/prices with 4decimal not properly calculated => WEBPOS - taxes/round/prices with 4decimal not properly calculated on the WEBPOS
2022-10-14 08:21 radhakrishnan Assigned To Retail => radhakrishnan
2022-10-14 08:21 radhakrishnan Status new => scheduled
2022-10-14 12:31 nicola_uva Steps to Reproduce Updated View Revisions
2022-10-17 07:31 marvintm Assigned To radhakrishnan => Triage Omni OMS
2022-10-17 10:40 aferraz Status scheduled => acknowledged
2022-10-19 14:41 aferraz Assigned To Triage Omni OMS => aferraz
2022-10-19 16:43 aferraz Status acknowledged => scheduled
2022-10-19 16:44 aferraz Status scheduled => resolved
2022-10-19 16:44 aferraz Resolution open => fixed
2022-10-19 16:44 aferraz Status resolved => new
2022-10-19 16:44 aferraz Resolution fixed => open
2022-10-19 16:51 aferraz Review Assigned To => marvintm
2022-10-19 16:51 aferraz Note Added: 0142294
2022-10-19 16:51 aferraz Status new => closed
2022-10-19 16:51 aferraz Resolution open => no change required
2022-10-19 16:56 aferraz Note Edited: 0142294 View Revisions
2022-10-19 17:05 aferraz Note Edited: 0142294 View Revisions
2022-10-19 17:06 aferraz File Added: Ticket1.png
2022-10-19 17:06 aferraz File Added: Ticket2.png
2022-10-19 17:15 aferraz Note Edited: 0142294 View Revisions
2022-10-19 17:15 aferraz File Added: Ticket1-BO.png
2022-12-21 05:01 hgbot Merge Request Status => open
2022-12-21 05:01 hgbot Note Added: 0144853
2022-12-21 05:02 hgbot Note Added: 0144854
2023-01-04 17:44 hgbot Note Added: 0145173
2023-01-06 21:28 hgbot Note Added: 0145193
2023-01-12 07:42 hgbot Merge Request Status open => approved
2023-01-30 05:52 hgbot Note Added: 0145985
2023-02-08 16:10 hgbot Note Added: 0146289
2023-03-02 07:27 hgbot Resolution no change required => fixed
2023-03-02 07:27 hgbot Fixed in Version => RR23Q2
2023-03-02 07:27 hgbot Note Added: 0147205
2023-03-02 07:27 hgbot Note Added: 0147206
2023-03-02 07:29 hgbot Note Added: 0147207
2023-03-02 07:29 hgbot Note Added: 0147208
2023-03-02 07:31 hgbot Note Added: 0147209
2023-03-02 07:31 hgbot Note Added: 0147210
2023-03-02 07:32 hgbot Fixed in Version RR23Q2 => 23Q2
2023-03-02 07:32 hgbot Note Added: 0147211
2023-03-02 07:32 hgbot Note Added: 0147212
2023-03-02 07:33 hgbot Note Added: 0147213
2023-03-02 07:33 hgbot Note Added: 0147214
2024-12-18 10:24 hgbot Note Added: 0173415

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker