Openbravo Issue Tracking System - Retail Modules
View Issue Details
0047563Retail ModulesWeb POSpublic2021-06-23 20:562021-08-19 07:50
lbressan 
prakashmurugesan88 
normalmajoralways
closedfixed 
5
 
RR21Q3.1RR21Q3.1 
No
0047563: 25651 - Incorrect tax id getting set when tax have multi level taxes
Incorrect tax id getting set when tax have multi level taxes after OB upgrade 20Q3. In SO lines instead of setting summary level tax id, it sets the child tax id. This was not the case in 18Q3.

Expected behavior: Incase of multi-tax summary level tax id should be set in POS order lines.
1. A compound tax corresponding to the Third Party is created.
2. An order is created from the pos, for the third party with the tax category for which the tax was created.
3. When checking from the backoffice it is detected that the tax line was incorrectly reflected in the order line (appears with **)

A video is attached with the step by step and the test carried out in livebuild
https://drive.google.com/file/d/1UqkHF4a7cGT_JNIwB0Ttc5mph-tOtt8g/view?usp=sharing [^]
No tags attached.
blocks defect 0047235 closed prakashmurugesan88 25651 - Incorrect tax id getting set when tax have multi level taxes 
Issue History
2021-08-18 07:38marvintmTypedefect => backport
2021-08-18 07:38marvintmTarget Version => RR21Q3.1
2021-08-18 13:21hgbotNote Added: 0131182
2021-08-18 13:23hgbotNote Added: 0131183
2021-08-19 07:50hgbotResolutionopen => fixed
2021-08-19 07:50hgbotStatusscheduled => closed
2021-08-19 07:50hgbotFixed in Version => RR21Q3.1
2021-08-19 07:50hgbotNote Added: 0131198
2021-08-19 07:50hgbotNote Added: 0131197
2021-08-19 07:50hgbotNote Added: 0131199
2021-08-19 07:50hgbotNote Added: 0131200

Notes
(0131182)
hgbot   
2021-08-18 13:21   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/546 [^]
(0131183)
hgbot   
2021-08-18 13:23   
Merge Request created: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/118 [^]
(0131197)
hgbot   
2021-08-19 07:50   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/546 [^]
(0131198)
hgbot   
2021-08-19 07:50   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 14b6bff9f69d2c58910c2adf6ac35ddbe093d2ff
Author: Prakash M <prakash@qualiantech.com>
Date: 2021-08-18T16:29:01+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/14b6bff9f69d2c58910c2adf6ac35ddbe093d2ff [^]

Fixed BUG-47563: Fixes issue where summary level tax id is not set to line when tax have multi level taxes

---
M web/org.openbravo.retail.posterminal/app/model/business-logic/taxes-engine/engine/tax-rules.js
---
(0131199)
hgbot   
2021-08-19 07:50   
Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 06bd5b6b4805740459886d4ee0135fe94ceb245b
Author: Prakash M <prakash@qualiantech.com>
Date: 2021-08-18T16:31:24+05:30
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/06bd5b6b4805740459886d4ee0135fe94ceb245b [^]

Related to BUG-47563 : Added test to Verify Summary Level Tax is Set to Line in case of multi level taxes

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/taxes/I47235_VerifySummaryLevelTaxSetLine.java
---
(0131200)
hgbot   
2021-08-19 07:50   
Merge request merged: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/118 [^]