Openbravo Issue Tracking System - Retail Modules
View Issue Details
0047564Retail ModulesWeb POSpublic2021-06-23 20:562021-08-26 08:27
lbressan 
prakashmurugesan88 
normalmajoralways
closedfixed 
5
 
RR21Q2.2RR21Q2.2 
No
0047564: 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 => RR21Q1.4
2021-08-18 07:48marvintmTarget VersionRR21Q1.4 => RR21Q2.1
2021-08-18 14:58hgbotNote Added: 0131185
2021-08-18 14:58hgbotNote Added: 0131186
2021-08-19 07:50hgbotNote Added: 0131201
2021-08-19 07:50hgbotResolutionopen => fixed
2021-08-19 07:50hgbotStatusscheduled => closed
2021-08-19 07:50hgbotFixed in Version => RR21Q2.2
2021-08-19 07:50hgbotNote Added: 0131202
2021-08-19 07:51hgbotNote Added: 0131203
2021-08-19 07:51hgbotNote Added: 0131204
2021-08-26 08:27marvintmTarget VersionRR21Q2.1 => RR21Q2.2

Notes
(0131185)
hgbot   
2021-08-18 14:58   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/547 [^]
(0131186)
hgbot   
2021-08-18 14:58   
Merge Request created: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/119 [^]
(0131201)
hgbot   
2021-08-19 07:50   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/547 [^]
(0131202)
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: ec0c1c084cf8ae43a3ad660e22d1744bfaee7bdb
Author: Prakash M <prakash@qualiantech.com>
Date: 2021-08-18T18:26:19+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/ec0c1c084cf8ae43a3ad660e22d1744bfaee7bdb [^]

Fixed BUG-47564: 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
---
(0131203)
hgbot   
2021-08-19 07:51   
Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 97188a40d4d6696c38a95b465ab3dd3bbf3e8017
Author: Prakash M <prakash@qualiantech.com>
Date: 2021-08-18T18:16:26+05:30
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/97188a40d4d6696c38a95b465ab3dd3bbf3e8017 [^]

Related to BUG-47564 : 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
---
(0131204)
hgbot   
2021-08-19 07:51   
Merge request merged: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/119 [^]