Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0052576
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2023-05-26 12:292023-06-26 12:57
ReporterpsanjuanView Statuspublic 
Assigned Toranjith_qualiantech_com 
PriorityurgentResolutionfixedFixed in VersionRR23Q3
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSLinux 64 bitDatabasePostgreSQLJava version7.x
OS VersionOpenbravo Appliance 14.04Database version9.3.xAnt version1.9.x
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0052576: It should be possible to manage multi Price List in a channel touchpoint

DescriptionIt should be possible to manage multi Price List in a channel touchpoint, for that it will be required to enable the preference "Web POS Enable Multi Price List".
Steps To ReproduceCurrently, it is not possible to use multi-price list in a terminal because:

when a BP is selected in POS, two validations are executed:

- pricelist include tax/not include taxes should be the same that the main pricelist (the one configure by default at Org. level)

- currency should be the same that main pricelist (the one configure by default at Org. level)

If not, a popup will be displayed informing about this problem, and the main pricelist will be used to this customer
Proposed SolutionThis validations should not be executed but just in the case when a "Verified Return" is being created, as in that particular scenario nothing nor the currency, nor the price list, nor the sales taxes, etc...should be changed.

It should always be possible to change the price list assigned to a BP, but that one should only be used for new sales transactions, not for returning already created transactions for a specific price list or currency.

Business scenario:
- Vall Blanca store has configured a price list "The White Valley Group Sale Price List including taxes", however the customer Arturo Montoro has another one "The White Valley Group Sale Price List excluding taxes"
- A new sales is created in the POS for the customer Arturo Montoro, the following message is shown:
"Customer Price List('The White Valley Group Sale Price List excluding taxes') configuration is not same as Terminal Price List('The White Valley Group Sale Price List including taxes') configuration. So Terminal Price List will be used for this receipt"

That should not happen if a NEW sale is being created, Web POS should be able to manage a sale with a diferente price list than the price list of the store.

Only in the case that an existing transaction is being return, these checks must be performed. A return implies no change of any data included in the original sale, including sales taxes, prices, promotions, price list, currency, etc..

Same way if an already existing transaction created with a a given price list/currency/sales taxes, etc is being used for whatever kind of action (ie. payment or whatever other process), nothing should be recalculated on that original transaction.

In that sense if the feature "Cancel and Replace" is used, cancel transaction should be exactly the same as the original one (same price list, currency, sales taxes..etc) and new one could have different settings, for instance a new BP price list or new discounts or promotions.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0041927 closedranjith_qualiantech_com Data inconsistency is provoked when multiprice list is used in WEBPOS [ Quotations and Verified Returns ] 

-  Notes
(0151016)
hgbot (developer)
2023-06-12 09:33

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1239 [^]
(0151018)
hgbot (developer)
2023-06-12 09:34

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/56 [^]
(0151019)
hgbot (developer)
2023-06-12 09:34

 Merge Request created: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/295 [^]
(0151551)
hgbot (developer)
2023-06-23 10:56

 Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 0bb9cd186f0abe25ab21772c6b6b71ffe7ae1c1a
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 23-06-2023 08:56:15
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/0bb9cd186f0abe25ab21772c6b6b71ffe7ae1c1a [^]

Verifies ISSUE-52576: Removed Customer PriceList change validation

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/multipricelists/I52576_VerifyVRWithMultiPriceList.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/customer/I39251_VerifyPriceWhenCustomerPriceListWasModified.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I32561_VerifyPriceListBasedOnCustomerPL.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sharedpaymentmethods/SPMDiscountsWithoutPriceIncludeTaxes.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sharedpaymentmethods/SPMMulticurrencyPayInEURUSD.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sharedpaymentmethods/SPMSaleAndReloadSharedPayments.java
---
(0151552)
hgbot (developer)
2023-06-23 10:56

 Merge request merged: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/295 [^]
(0151553)
hgbot (developer)
2023-06-23 10:56

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: ca10df1b3afc33c0a3b1ea8a3fb344bfadb97378
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 23-06-2023 07:24:06
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/ca10df1b3afc33c0a3b1ea8a3fb344bfadb97378 [^]

Related to ISSUE-52576: Removed Pricelist validation if MultiPriceList is enabled
* Product price should be changed based on receipt customer's price list

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/retail/posterminal/PaidReceiptProperties.java
M web/org.openbravo.retail.posterminal/js/components/businesspartner_selector.js
M web/org.openbravo.retail.posterminal/js/model/order.js
---
(0151554)
hgbot (developer)
2023-06-23 10:56

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1239 [^]
(0151555)
hgbot (developer)
2023-06-23 10:56

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns [^]
Changeset: 53edbb91cafb2e56d53936b52861bd75cb5de228
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 23-06-2023 07:24:08
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/commit/53edbb91cafb2e56d53936b52861bd75cb5de228 [^]

Fixed ISSUE-52576: Added validation when adding multiple verfied returns ticket
* When adding verified return lines, existing receipt's and new receipt's
  pricelist configuration should be same

---
M web-test/model/business-object/ticket/actions/ReturnVerifiedTicket-actionPreparation.test.js
M web-test/model/business-object/ticket/actions/ReturnVerifiedTicket.test.js
M web/org.openbravo.retail.returns/app/model/business-object/ticket/actions/ReturnVerifiedTicket.js
M web/org.openbravo.retail.returns/js/modalReturnLines.js
M web/org.openbravo.retail.returns/js/returnUtils.js
---
(0151556)
hgbot (developer)
2023-06-23 10:56

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/56 [^]
(0151638)
hgbot (developer)
2023-06-26 12:57

 Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 3041fa6c3cc6086aae18a061d79da703e5d3948d
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 26-06-2023 16:26:30
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/3041fa6c3cc6086aae18a061d79da703e5d3948d [^]

Verifies ISSUE-52576: Updated Automated tests
* Optimized payment methods tests for oracle

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sharedpaymentmethods/SPMDiscountsWithoutPriceIncludeTaxes.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sharedpaymentmethods/SPMMulticurrencyPayInEURUSD.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/sharedpaymentmethods/SPMSaleAndReloadSharedPayments.java
---

- Issue History
Date Modified Username Field Change
2023-05-26 12:29 psanjuan New Issue
2023-05-26 12:29 psanjuan Assigned To => Retail
2023-05-26 12:29 psanjuan Triggers an Emergency Pack => No
2023-05-26 12:32 psanjuan Proposed Solution updated
2023-05-26 12:45 psanjuan Proposed Solution updated
2023-05-26 12:59 psanjuan Proposed Solution updated
2023-05-26 13:02 psanjuan Proposed Solution updated
2023-05-26 13:48 psanjuan Proposed Solution updated
2023-05-30 07:53 marvintm Relationship added related to 0041927
2023-05-31 15:39 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2023-06-01 09:19 ranjith_qualiantech_com Status new => scheduled
2023-06-12 09:33 hgbot Note Added: 0151016
2023-06-12 09:34 hgbot Note Added: 0151018
2023-06-12 09:34 hgbot Note Added: 0151019
2023-06-23 10:56 hgbot Note Added: 0151551
2023-06-23 10:56 hgbot Note Added: 0151552
2023-06-23 10:56 hgbot Note Added: 0151553
2023-06-23 10:56 hgbot Note Added: 0151554
2023-06-23 10:56 hgbot Resolution open => fixed
2023-06-23 10:56 hgbot Status scheduled => closed
2023-06-23 10:56 hgbot Fixed in Version => RR23Q3
2023-06-23 10:56 hgbot Note Added: 0151555
2023-06-23 10:56 hgbot Note Added: 0151556
2023-06-26 12:57 hgbot Note Added: 0151638

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker