[ofbiz-framework] branch release22.01 updated: Fixed: Tax calculation not considering productStoreGroup (OFBIZ-12686)

[jleroux]

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch release22.01
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/release22.01 by this push:
     new 2c754d1b29 Fixed: Tax calculation not considering productStoreGroup 
(OFBIZ-12686)
2c754d1b29 is described below

commit 2c754d1b2907ea9e7361c3798d0eaae5ba5ee0ef
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Sat Aug 27 12:14:48 2022 +0200

    Fixed: Tax calculation not considering productStoreGroup (OFBIZ-12686)
    
    As reported by Ingo the productStoreId field does not exist in the 
ProductPrice
    entity, it's productStoreGroupId. This fixes it
    
    Thanks: Ingo for issue report
---
 .../ofbiz/accounting/tax/TaxAuthorityServices.java | 23 +++++++---------------
 1 file changed, 7 insertions(+), 16 deletions(-)

diff --git 
a/applications/accounting/src/main/java/org/apache/ofbiz/accounting/tax/TaxAuthorityServices.java
 
b/applications/accounting/src/main/java/org/apache/ofbiz/accounting/tax/TaxAuthorityServices.java
index 3e791f4f02..0915d79ea9 100644
--- 
a/applications/accounting/src/main/java/org/apache/ofbiz/accounting/tax/TaxAuthorityServices.java
+++ 
b/applications/accounting/src/main/java/org/apache/ofbiz/accounting/tax/TaxAuthorityServices.java
@@ -656,22 +656,13 @@ public class TaxAuthorityServices {
      */
     private static GenericValue getProductPrice(Delegator delegator, 
GenericValue product, GenericValue productStore, String taxAuthGeoId,
             String taxAuthPartyId) throws GenericEntityException {
-        if 
(UtilValidate.isNotEmpty(productStore.getString("primaryStoreGroupId"))) {
-            return EntityQuery.use(delegator).from("ProductPrice")
-                    .where("productId", product.get("productId"),
-                            "taxAuthPartyId", taxAuthPartyId,
-                            "taxAuthGeoId", taxAuthGeoId,
-                            "productPricePurposeId", "PURCHASE",
-                            "productStoreId", 
productStore.get("primaryStoreGroupId"))
-                    .orderBy("-fromDate").filterByDate().queryFirst();
-        } else {
-            return EntityQuery.use(delegator).from("ProductPrice")
-                    .where("productId", product.get("productId"),
-                            "taxAuthPartyId", taxAuthPartyId,
-                            "taxAuthGeoId", taxAuthGeoId,
-                            "productPricePurposeId", "PURCHASE")
-                            .orderBy("-fromDate").filterByDate().queryFirst();
-        }
+        return EntityQuery.use(delegator).from("ProductPrice")
+                .where("productId", product.get("productId"),
+                        "taxAuthPartyId", taxAuthPartyId,
+                        "taxAuthGeoId", taxAuthGeoId,
+                        "productPricePurposeId", "PURCHASE",
+                        "productStoreGroupId", 
productStore.get("primaryStoreGroupId"))
+                .orderBy("-fromDate").filterByDate().queryFirst();
     }
 
     /**