Openbravo Issue Tracking System - Retail Modules
View Issue Details
0033835Retail ModulesWeb POSpublic2016-08-03 12:562016-09-01 08:46
malsasua 
migueldejuana 
immediatemajoralways
closedfixed 
5
 
RR16Q3.1RR16Q3.1 
guilleaer
Production - Confirmed Stable
No
0033835: wrong "net unit price" with tax include taxes and price precision 6
when 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$


in 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 invoice window, invoice created by cashup
>>>Error: invoice is set as payment complete: No, outstanding amt 0.01, total gross amount is wrong.
No tags attached.
blocks defect 0033611 closed migueldejuana wrong "net unit price" with tax include taxes and price precision 6 
Issue History
2016-08-29 10:30migueldejuanaTypedefect => backport
2016-08-29 10:30migueldejuanaTarget Version => RR16Q3.1
2016-08-30 08:50hgbotCheckin
2016-08-30 08:50hgbotNote Added: 0089526
2016-08-30 08:50hgbotStatusscheduled => resolved
2016-08-30 08:50hgbotResolutionopen => fixed
2016-08-30 08:50hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q3.1/org.openbravo.retail.posterminal/rev/e9968782596ac900ed7fc1e4838d0affa075b050 [^]
2016-08-30 08:53hgbotCheckin
2016-08-30 08:53hgbotNote Added: 0089527
2016-08-30 09:06guilleaerReview Assigned Toadrianromero => guilleaer
2016-08-30 09:06guilleaerStatusresolved => closed
2016-08-30 09:06guilleaerFixed in Version => RR16Q3.1
2016-08-31 12:22lorenzofidalgoSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=12996#r12996
2016-09-01 08:46hgbotCheckin
2016-09-01 08:46hgbotNote Added: 0089632

Notes
(0089526)
hgbot   
2016-08-30 08:50   
Repository: retail/backports/3.0RR16Q3.1/org.openbravo.retail.posterminal
Changeset: e9968782596ac900ed7fc1e4838d0affa075b050
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Tue Aug 30 08:49:48 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.1/org.openbravo.retail.posterminal/rev/e9968782596ac900ed7fc1e4838d0affa075b050 [^]

Fixed issue 33835: 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
---
(0089527)
hgbot   
2016-08-30 08:53   
Repository: retail/backports/3.0RR16Q3.1/pi-mobile
Changeset: 2e84ade816b0b69a66c0c206620afbd3f1ae222c
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Tue Aug 30 08:53:01 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.1/pi-mobile/rev/2e84ade816b0b69a66c0c206620afbd3f1ae222c [^]

Related to issue 33835: Add tests

---
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
M src-test/org/openbravo/test/mobile/common/selenium/database/rows/InvoiceLinesOfDocumentnoRow.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/I33695_CashupInvoiceTaxPrecision.java
---
(0089632)
hgbot   
2016-09-01 08:46   
Repository: tools/automation/pi-mobile
Changeset: 2e84ade816b0b69a66c0c206620afbd3f1ae222c
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Tue Aug 30 08:53:01 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/2e84ade816b0b69a66c0c206620afbd3f1ae222c [^]

Related to issue 33835: Add tests

---
M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java
M src-test/org/openbravo/test/mobile/common/selenium/database/rows/InvoiceLinesOfDocumentnoRow.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/I33695_CashupInvoiceTaxPrecision.java
---