Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033611Retail ModulesWeb POSpublic2016-08-03 12:562016-09-20 11:16
Reportermalsasua 
Assigned Tomigueldejuana 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Toadrianromero
OBNetwork customer
Support ticket
Regression levelProduction - Confirmed Stable
Regression date
Regression introduced in releaseRR16Q2
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033611: 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 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
related to defect 0032263RR16Q2 closed ranjith_qualiantech_com Retail Modules Sales Order's "Total Net Amount" different from sum of sales order line's "line net amount", when price precision = 6 
related to defect 0032264 closed adrianromero Retail Modules Taxes at document level are not properly calculated in some cases 
depends on backport 0033835RR16Q3.1 closed migueldejuana Retail Modules wrong "net unit price" with tax include taxes and price precision 6 
depends on backport 0033836RR16Q2.3 closed migueldejuana Retail Modules wrong "net unit price" with tax include taxes and price precision 6 
related to design defect 0032265 closed aferraz Openbravo ERP [SER QA 1434] Taxes at document level are not properly calculated in some cases 
Attached Filespng centimos.png (173,825) 2016-08-03 14:30
https://issues.openbravo.com/file_download.php?file_id=9674&type=bug
png

png invoice.png (165,473) 2016-08-04 11:12
https://issues.openbravo.com/file_download.php?file_id=9676&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2016-08-03 12:56malsasuaNew Issue
2016-08-03 12:56malsasuaAssigned To => Retail
2016-08-03 12:56malsasuaTriggers an Emergency Pack => No
2016-08-03 14:26malsasuaRegression level => Production - Confirmed Stable
2016-08-03 14:28malsasuaSummarywrong line net amount with tax include taxes and price precision 6 => wrong "net unit price" with tax include taxes and price precision 6
2016-08-03 14:30malsasuaFile Added: centimos.png
2016-08-03 18:30eugeniIssue Monitored: eugeni
2016-08-04 09:39malsasuaResolution time => 1471471200
2016-08-04 09:39malsasuaPriorityhigh => immediate
2016-08-04 11:12malsasuaDescription Updatedbug_revision_view_page.php?rev_id=12793#r12793
2016-08-04 11:12malsasuaSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=12795#r12795
2016-08-04 11:12malsasuaFile Added: invoice.png
2016-08-11 11:03migueldejuanaAssigned ToRetail => ranjith_qualiantech_com
2016-08-11 11:08migueldejuanaRelationship addedrelated to 0032263
2016-08-16 11:56ranjith_qualiantech_comStatusnew => scheduled
2016-08-18 12:15migueldejuanaRelationship addeddepends on 0032265
2016-08-18 14:20migueldejuanaRelationship addedrelated to 0032264
2016-08-22 08:55migueldejuanaAssigned Toranjith_qualiantech_com => migueldejuana
2016-08-22 13:50malsasuaRelationship addedhas duplicate 0033762
2016-08-24 15:48hgbotCheckin
2016-08-24 15:48hgbotNote Added: 0089354
2016-08-24 15:48hgbotCheckin
2016-08-24 15:48hgbotNote Added: 0089355
2016-08-24 15:48hgbotCheckin
2016-08-24 15:48hgbotNote Added: 0089356
2016-08-24 15:49migueldejuanaStatusscheduled => resolved
2016-08-24 15:49migueldejuanaResolutionopen => fixed
2016-08-25 13:38adrianromeroReview Assigned To => adrianromero
2016-08-25 13:38adrianromeroNote Added: 0089395
2016-08-25 13:38adrianromeroStatusresolved => closed
2016-08-29 10:27migueldejuanaStatusclosed => new
2016-08-29 10:27migueldejuanaResolutionfixed => open
2016-08-29 10:30migueldejuanaStatusnew => scheduled
2016-08-29 10:30migueldejuanaStatusscheduled => resolved
2016-08-29 10:30migueldejuanaResolutionopen => fixed
2016-08-29 10:31migueldejuanaStatusresolved => closed
2016-08-30 14:59guilleaerNote Added: 0089554
2016-08-31 11:30migueldejuanaRegression introduced in release => RR16Q2
2016-08-31 11:32migueldejuanaNote Added: 0089594
2016-08-31 12:22lorenzofidalgoSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=12997#r12997
2016-08-31 13:54malsasuaRelationship deletedhas duplicate 0033762
2016-09-20 11:16hgbotCheckin
2016-09-20 11:16hgbotNote Added: 0090103
2017-01-16 10:42aferrazRelationship deleteddepends on 0032265
2017-01-16 10:42aferrazRelationship addedrelated to 0032265

Notes
(0089354)
hgbot   
2016-08-24 15:48   
Repository: tools/automation/pi-mobile
Changeset: 3c179fa76f681057c01827f1e4951e5998b71e6e
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Aug 23 10:52:57 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/3c179fa76f681057c01827f1e4951e5998b71e6e [^]

Related to issue 0033611: 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/WebPOSDatabaseHelper.java
M src-test/org/openbravo/test/mobile/common/selenium/database/rows/InvoiceLinesOfDocumentnoRow.java
---
(0089355)
hgbot   
2016-08-24 15:48   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 5794a0f5facd7a269a9269b73c24b524b8bdf2cd
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Wed Aug 24 11:19:59 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/5794a0f5facd7a269a9269b73c24b524b8bdf2cd [^]

Fixed to issue 0033611: 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
---
(0089356)
hgbot   
2016-08-24 15:48   
Repository: tools/automation/pi-mobile
Changeset: 327cedc87d3677b5f1ba8b39f93480e71e2d9408
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Wed Aug 24 11:16:02 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/327cedc87d3677b5f1ba8b39f93480e71e2d9408 [^]

Related to issue 0033611: 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
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/I33695_CashupInvoiceTaxPrecision.java
---
(0089395)
adrianromero   
2016-08-25 13:38   
Verified.
(0089554)
guilleaer   
2016-08-30 14:59   
Regression fields are pending
(0089594)
migueldejuana   
2016-08-31 11:32   
Could be introduced by https://issues.openbravo.com/view.php?id=0032996 [^]
(0090103)
hgbot   
2016-09-20 11:16   
Repository: tools/automation/pi-mobile
Changeset: bbcffb7e052c235fa5e34363e44fd2c3b3b3299c
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Sep 20 11:16:26 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/bbcffb7e052c235fa5e34363e44fd2c3b3b3299c [^]

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

- Allow to check invoice with more than one line (fix)

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