Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0047538Retail ModulesWeb POSpublic2021-08-12 09:562021-09-01 06:51
Reporterjoniturralde93 
Assigned Toprakashmurugesan88 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR21Q4 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0047538: Returning a line in another terminal with different taxes makes the return tax inconsistent
DescriptionWhen returning a line done in a different organization, Web POS might apply different taxes.
Steps To ReproduceTo configure in live builds:
- New tax category
- Two new tax rates for this tax category with different rates (select for Spain and the configuration needed so it's loaded in the Web POS)
- Set that tax category in a product (for example 'Ceiling Lamp')
- Go to Organization Vall Blanca store and select cross store as 'White Valley Spain S.A'
- Do the same for Category Tree Store (and activate it if necessary)
- Add the modified product to both the price lists and assortments

- Log into Web POS (VBS-1) and buy the product (make sure the applied rate is correct)
- Log into CTST1 and go to Verified returns and search the completed order
- Return the line
- Check that the total is the same, but the net unit price and the tax amount has changed (it applied a different tax)
Proposed SolutionPrice and taxes should remain the same for returns.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0047862 closed aferraz Openbravo Localizations SAF-T: Cross store return with different taxes 
Attached Filespatch I47538_19Q3_PosTerminal.patch (23,251) 2021-09-01 06:51
https://issues.openbravo.com/file_download.php?file_id=16162&type=bug
patch I47538_19Q3_RetailReturns.patch (1,798) 2021-09-01 06:51
https://issues.openbravo.com/file_download.php?file_id=16163&type=bug
Issue History
Date ModifiedUsernameFieldChange
2021-08-12 09:56joniturralde93New Issue
2021-08-12 09:56joniturralde93Assigned To => Retail
2021-08-12 09:56joniturralde93Resolution time => 1630533600
2021-08-12 09:56joniturralde93Triggers an Emergency Pack => No
2021-08-17 13:32prakashmurugesan88Assigned ToRetail => prakashmurugesan88
2021-08-17 13:32prakashmurugesan88Statusnew => scheduled
2021-08-30 07:54hgbotNote Added: 0131414
2021-08-30 07:55hgbotNote Added: 0131415
2021-08-31 08:05hgbotResolutionopen => fixed
2021-08-31 08:05hgbotStatusscheduled => closed
2021-08-31 08:05hgbotFixed in Version => RR21Q4
2021-08-31 08:05hgbotNote Added: 0131450
2021-08-31 08:05hgbotNote Added: 0131451
2021-08-31 08:05hgbotNote Added: 0131452
2021-08-31 08:05hgbotNote Added: 0131453
2021-09-01 06:51prakashmurugesan88File Added: I47538_19Q3_PosTerminal.patch
2021-09-01 06:51prakashmurugesan88File Added: I47538_19Q3_RetailReturns.patch
2021-10-18 17:22aferrazRelationship addedrelated to 0047862
2021-10-19 12:54aferrazRelationship replacedblocks 0047862

Notes
(0131414)
hgbot   
2021-08-30 07:54   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/556 [^]
(0131415)
hgbot   
2021-08-30 07:55   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/25 [^]
(0131450)
hgbot   
2021-08-31 08:05   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 231d73fb0fc9c20c7202056aff1b858d8dc64174
Author: Prakash M <prakash@qualiantech.com>
Date: 2021-08-31T11:32:28+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/231d73fb0fc9c20c7202056aff1b858d8dc64174 [^]

Fixed BUG-47538: Retained original receipt tax info in case of verified return
* Tax rules are generated from line taxes of original receipt in case of verified return
* Included needed tax info in paid receipt

---
M src/org/openbravo/retail/posterminal/PaidReceipts.java
M web/org.openbravo.retail.posterminal/app/model/business-logic/taxes-engine/engine/tax-rules.js
M web/org.openbravo.retail.posterminal/app/model/business-object/ticket/LoadTicketUtils.js
---
(0131451)
hgbot   
2021-08-31 08:05   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/556 [^]
(0131452)
hgbot   
2021-08-31 08:05   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/25 [^]
(0131453)
hgbot   
2021-08-31 08:05   
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns [^]
Changeset: 49f310858cbd642068e1c18dd3e8293c96457826
Author: Prakash M <prakash@qualiantech.com>
Date: 2021-08-27T19:06:23+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/commit/49f310858cbd642068e1c18dd3e8293c96457826 [^]

Related to BUG-47538: Storing line tax details in separate property
* This new property is used for forming Tax Rules in case of verified return
* New property is required as line tax details are getting updated in the middle of process

---
M web/org.openbravo.retail.returns/app/model/business-object/ticket/actions/ReturnVerifiedTicket.js
M web/org.openbravo.retail.returns/js/modalReturnLines.js
---