Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033836Retail ModulesWeb POSpublic2016-08-03 12:562016-10-05 12:39
Reportermalsasua 
Assigned Tomigueldejuana 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q2.3Fixed in VersionRR16Q2.3 
Merge Request Status
Review Assigned Toguilleaer
OBNetwork customer
Support ticket
Regression levelProduction - Confirmed Stable
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033836: wrong "net unit price" with tax include taxes and price precision 6
Descriptionwhen the pricelist is configured with price include taxes, the tax is set as "Document based amount by rate" in "Document based amount by rate" field, the price precision is 6, POS precision 2, then the "net unit price" calculated in sales order line is wrong.

For this reason, the invoice created by cashup is set as not paid, with outstanding amount 0.01$


Steps To Reproducein backoffice:
. price list version: "The White Valley Group Sale Price List Version"
  . product Alpine Poles: unit price and list price: 117
  . product Alpine Ski Boot: unit price and list price: 117
. as system administrator, go to Currency window:
 . currency EUR: price precision: 6
                 POS precision: 2

in POS
. new receipt:
 . Alpine Poles: 1 unit
 . Alpine Ski Poles: 3 units
. pay it
. do cashup

in backoffice:
. go to sales order window, receipt created in above step
line1: "net unit price": 96.69
line2: "net unit price": 96.70
>>>Error: the "net unit price" should be 96.69 in the two lines, as it is explained in http://wiki.openbravo.com/wiki/How_Price_Including_Taxes_are_Calculated [^]

. go to sales invoice window, invoice created by cashup
>>>Error: invoice is set as payment complete: No, outstanding amt 0.01, total gross amount is wrong.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0033611 closed migueldejuana wrong "net unit price" with tax include taxes and price precision 6 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-08-29 10:30migueldejuanaTypedefect => backport
2016-08-29 10:30migueldejuanaTarget Version => RR16Q2.3
2016-09-22 12:21hgbotCheckin
2016-09-22 12:21hgbotNote Added: 0090158
2016-09-22 12:21hgbotCheckin
2016-09-22 12:21hgbotNote Added: 0090159
2016-09-22 12:21hgbotCheckin
2016-09-22 12:21hgbotNote Added: 0090160
2016-09-22 13:37migueldejuanaStatusscheduled => resolved
2016-09-22 13:37migueldejuanaResolutionopen => fixed
2016-09-27 09:27guilleaerReview Assigned Toadrianromero => guilleaer
2016-09-27 09:27guilleaerStatusresolved => closed
2016-09-27 09:27guilleaerFixed in Version => RR16Q2.3
2016-10-05 12:39hgbotCheckin
2016-10-05 12:39hgbotNote Added: 0090493
2016-10-05 12:39hgbotCheckin
2016-10-05 12:39hgbotNote Added: 0090494

Notes
(0090158)
hgbot   
2016-09-22 12:21   
Repository: retail/backports/3.0RR16Q2.3/pi-mobile
Changeset: ddf5d9469812411523fe4381bcba681613a29ebe
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Sep 22 12:14:58 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.3/pi-mobile/rev/ddf5d9469812411523fe4381bcba681613a29ebe [^]

Related to issue 0033836: wrong "net unit price" with tax include taxes and price precision 6

- Checking invoice, check also: totalNet + totalTaxAmount = totalGross

---
M src-test/org/openbravo/test/mobile/common/selenium/database/InvoiceLinesOfDocumentnoRow.java
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
---
(0090159)
hgbot   
2016-09-22 12:21   
Repository: retail/backports/3.0RR16Q2.3/pi-mobile
Changeset: ac890f93aaedc398fee9ba01e938958ff65b97f6
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Sep 22 12:15:32 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.3/pi-mobile/rev/ac890f93aaedc398fee9ba01e938958ff65b97f6 [^]

Related to issue 0033836: wrong "net unit price" with tax include taxes and price precision 6

- Allow to check invoice with more than one line
- Add the test

---
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
---
(0090160)
hgbot   
2016-09-22 12:21   
Repository: retail/backports/3.0RR16Q2.3/org.openbravo.retail.posterminal
Changeset: c93525e5e52c32f724aa9721a9275c3aede8250d
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Sep 22 12:01:25 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.3/org.openbravo.retail.posterminal/rev/c93525e5e52c32f724aa9721a9275c3aede8250d [^]

Fixed to issue 0033836: wrong "net unit price" with tax include taxes and price precision 6

- If totalines + totalTaxAmount != grandTotalAmount we have to adjust taxes so we choose the tax with more amount to adjust the invoice

---
M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java
---
(0090493)
hgbot   
2016-10-05 12:39   
Repository: tools/automation/pi-mobile
Changeset: ddf5d9469812411523fe4381bcba681613a29ebe
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Sep 22 12:14:58 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/ddf5d9469812411523fe4381bcba681613a29ebe [^]

Related to issue 0033836: wrong "net unit price" with tax include taxes and price precision 6

- Checking invoice, check also: totalNet + totalTaxAmount = totalGross

---
M src-test/org/openbravo/test/mobile/common/selenium/database/InvoiceLinesOfDocumentnoRow.java
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
---
(0090494)
hgbot   
2016-10-05 12:39   
Repository: tools/automation/pi-mobile
Changeset: ac890f93aaedc398fee9ba01e938958ff65b97f6
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Sep 22 12:15:32 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/ac890f93aaedc398fee9ba01e938958ff65b97f6 [^]

Related to issue 0033836: wrong "net unit price" with tax include taxes and price precision 6

- Allow to check invoice with more than one line
- Add the test

---
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
---