Merge branch 'apache:trunk' into OFBIZ-9498

Author: gilPts

.github/workflows/codeql-analysis.yml

@@ -26,15 +26,9 @@ name: "CodeQL"
 on:
   push:
     branches: [ trunk, release* ]
-    paths:
-        - '**.java'
-        - '**.js'
   pull_request:
     # The branches below must be a subset of the branches above
     branches: [ trunk ]
-    paths:
-        - '**.java'
-        - '**.js'
   schedule:
     - cron: '27 15 * * 1'
 

.gitignore

@@ -18,7 +18,6 @@ runtime/catalina/work/*
 runtime/tempfiles/*
 runtime/indexes/products/
 !runtime/tempfiles/README
-applications/content/index/
 changelog
 .classpath
 .project

NOTICE

@@ -1,5 +1,5 @@
 Apache OFBiz
-Copyright 2001-2021 The Apache Software Foundation
+Copyright 2001-2022 The Apache Software Foundation
 
 This product includes software developed at
 The Apache Software Foundation (http://www.apache.org/).

README.adoc

@@ -21,13 +21,13 @@ under the License.
 The Apache OFBiz Project
 
 image:https://img.shields.io/badge/License-Apache%202.0-blue.svg[link=http://www.apache.org/licenses/LICENSE-2.0]
-image:https://img.shields.io/badge/Version-trunk-blue.svg[link=Version]
-image:https://github.com/apache/ofbiz-framework/actions/workflows/gradle.yaml/badge.svg?branch=trunk[link=https://github.com/apache/ofbiz-framework/actions/workflows/gradle.yaml/badge.svg?branch=trunk]
+image:https://img.shields.io/badge/Version-trunk-blue.svg[link=https://github.com/apache/ofbiz-framework]
+image:https://github.com/apache/ofbiz-framework/actions/workflows/gradle.yaml/badge.svg?branch=trunk[link=https://github.com/apache/ofbiz-framework/actions/workflows/gradle.yaml]
+image:https://sonarcloud.io/api/project_badges/measure?project=apache_ofbiz-framework&metric=alert_status[link=https://sonarcloud.io/dashboard?id=apache_ofbiz-framework]
 image:https://github.com/apache/ofbiz-framework/actions/workflows/codeql-analysis.yml/badge.svg[link=https://github.com/apache/ofbiz-framework/actions/workflows/codeql-analysis.yml]
-image:https://qpkb254zxeu.montastic.io/badge[link=https://qpkb254zxeu.montastic.io]
 
 If you are reading this file in AsciiDoc format you may want to see it at
-https://ci.apache.org/projects/ofbiz/site/trunk/readme/html5/README.html[HTML] +
+https://nightlies.apache.org/ofbiz/trunk/readme/html5/[HTML]
 or https://ci.apache.org/projects/ofbiz/site/trunk/readme/pdf/README.pdf[PDF] format
 
 ---

VERSION

@@ -1 +1 @@
-Trunk
+Trunk 

applications/accounting/config/AccountingUiLabels.xml

@@ -0,0 +1,53 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+<entity-engine-xml>
+<!-- 
+# what should the default of the payment to invoice processing be?
+# paymentProcessing by invoice or paymentprocessing by invoiceitem?
+# there is still a box on the menu to change this option by the user.
+# value Y: applying payments to the invoice as a whole (invoiceItemSeqNr = null) with the option to apply per invoice item
+# value N: applying payments to every invoice item with the option to apply to an invoice as a whole
+#value YY: do not show the option box to the user and do only invoice processing
+#value NN: do not show the option box to the user and do only invoiceitem processing
+-->
+    <SystemProperty systemResourceId="accounting" systemPropertyId="invoiceProcessing" 
+    systemPropertyValue="YY"
+    description="Options are: Y, N, YY, NN"
+    />
+
+    <SystemProperty systemResourceId="accounting" systemPropertyId="accounting.fixedasset.autocreate" 
+    systemPropertyValue="Y"
+    description="Create a fixed asset when an 'asset usage' type product is created. Options: Y, N"/>
+
+    <SystemProperty systemResourceId="accounting" systemPropertyId="accounting.payment.application.autocreate" 
+    systemPropertyValue="Y"
+    description="Create a payment application when a payment is received/sent or invoice is approved (take the oldest not closed one). Options: Y, N"/>
+
+   <SystemProperty systemResourceId="accounting" systemPropertyId="accounting.payment.purchaseorder.autocreate" 
+    systemPropertyValue="Y"
+    description="Create a 'not-paid' payment record if the purchase order is approved. Options: Y, N"/>
+
+    <SystemProperty systemResourceId="accounting" systemPropertyId="accounting.payment.salesorder.autocreate" 
+    systemPropertyValue="Y"
+    description="Create a 'not-paid' payment record if the sales order is completed and no payment exist yet. Options:: Y, N"/>
+
+    <!-- <SystemProperty systemResourceId="accounting" systemPropertyId="create.invoice.per.shipment" systemPropertyValue="Y"
+    description="create invoice per shipment. Options: = Y (Invoice per shipment), N (Invoice per order)"/> -->
+</entity-engine-xml>

applications/accounting/data/AccountingSystemPropertyData.xml

@@ -27,6 +27,7 @@ under the License.
     <entity-resource type="model" reader-name="main" loader="main" location="entitydef/entitymodel_reports.xml"/>
     <entity-resource type="eca" reader-name="main" loader="main" location="entitydef/eecas.xml"/>
     <entity-resource type="data" reader-name="seed" loader="main" location="data/AccountingSecurityPermissionSeedData.xml"/>
+    <entity-resource type="data" reader-name="seed" loader="main" location="data/AccountingSystemPropertyData.xml"/>
     <entity-resource type="data" reader-name="seed" loader="main" location="data/AccountingPortletData.xml"/>
     <entity-resource type="data" reader-name="seed-initial" loader="main" location="data/AccountingScheduledServiceData.xml"/>
 

applications/accounting/ofbiz-component.xml

@@ -84,7 +84,7 @@ under the License.
     <service name="setAcctgCompany" engine="groovy"
         location="component://accounting/groovyScripts/admin/AcctgAdminServices.groovy" invoke="setAcctgCompany" auth="true">
         <description>Set Accounting Company when select</description>
-        <permission-service service-name="acctgPrefPermissionCheck" main-action="CREATE"/>
+        <permission-service service-name="acctgPrefPermissionCheck" main-action="VIEW"/>
         <attribute type="String" mode="INOUT" name="organizationPartyId" optional="true"/>
     </service>
 

applications/accounting/servicedef/services_admin.xml

@@ -37,6 +37,7 @@
 
 import org.apache.commons.collections4.CollectionUtils;
 import org.apache.commons.csv.CSVFormat;
+import org.apache.commons.csv.CSVFormat.Builder;
 import org.apache.commons.csv.CSVRecord;
 import org.apache.ofbiz.accounting.payment.PaymentGatewayServices;
 import org.apache.ofbiz.accounting.payment.PaymentWorker;
@@ -3310,7 +3311,7 @@ public static Map<String, Object> updatePaymentApplicationDefBd(DispatchContext
             errorMessageList.add(UtilProperties.getMessage(RESOURCE, "AccountingNoAmount", locale));
         } else {
             successMessage = UtilProperties.getMessage(RESOURCE,
-                    "AccountingApplicationSuccess",
+                    "AccountingPaymentApplicationSuccess",
                     UtilMisc.<String, Object>toMap("amountApplied", amountApplied,
                             "paymentId", paymentId,
                             "isoCode", currencyUomId,
@@ -3672,16 +3673,16 @@ public static Map<String, Object> importInvoice(DispatchContext dctx, Map<String
         String organizationPartyId = (String) context.get("organizationPartyId");
         String encoding = System.getProperty("file.encoding");
         String csvString = Charset.forName(encoding).decode(fileBytes).toString();
-        final BufferedReader csvReader = new BufferedReader(new StringReader(csvString));
-        CSVFormat fmt = CSVFormat.DEFAULT.withHeader();
+        Builder csvFormatBuilder = Builder.create().setHeader();
+        CSVFormat fmt = csvFormatBuilder.build();
         List<String> errMsgs = new LinkedList<>();
         List<String> newErrMsgs;
         String lastInvoiceId = null;
         String currentInvoiceId = null;
         String newInvoiceId = null;
         int invoicesCreated = 0;
 
-        try {
+        try (BufferedReader csvReader = new BufferedReader(new StringReader(csvString))) {
             for (final CSVRecord rec : fmt.parse(csvReader)) {
                 currentInvoiceId = rec.get("invoiceId");
                 if (lastInvoiceId == null || !currentInvoiceId.equals(lastInvoiceId)) {
@@ -3758,11 +3759,18 @@ public static Map<String, Object> importInvoice(DispatchContext dctx, Map<String
                         try {
                             invoiceResult = dispatcher.runSync("createInvoice", invoice);
                             if (ServiceUtil.isError(invoiceResult)) {
+                                // Eclipse reports here: Resource leak: '<unassigned Closeable value>' is not closed at this location
+                                // but it's OK. As csvReader is in a try-with-ressource it will be closed anyway
+                                // I prefer to not put @SuppressWarnings("resource") to the whole method
+                                // BTW to be consistent Eclipse should also reports the same issue in PartyService (see there)
                                 return ServiceUtil.returnError(ServiceUtil.getErrorMessage(invoiceResult));
                             }
                         } catch (GenericServiceException e) {
-                            csvReader.close();
                             Debug.logError(e, MODULE);
+                            // Eclipse reports here: Resource leak: '<unassigned Closeable value>' is not closed at this location
+                            // but it's OK. As csvReader is in a try-with-ressource it will be closed anyway
+                            // I prefer to not put @SuppressWarnings("resource") to the whole method
+                            // BTW to be consistent Eclipse should also reports the same issue in PartyService (see there)
                             return ServiceUtil.returnError(e.getMessage());
                         }
                         newInvoiceId = (String) invoiceResult.get("invoiceId");
@@ -3824,10 +3832,13 @@ public static Map<String, Object> importInvoice(DispatchContext dctx, Map<String
                         try {
                             Map<String, Object> result = dispatcher.runSync("createInvoiceItem", invoiceItem);
                             if (ServiceUtil.isError(result)) {
+                                // Eclipse reports here: Resource leak: '<unassigned Closeable value>' is not closed at this location
+                                // but it's OK. As csvReader is in a try-with-ressource it will be closed anyway
+                                // I prefer to not put @SuppressWarnings("resource") to the whole method
+                                // BTW to be consistent Eclipse should also reports the same issue in PartyService (see there)
                                 return ServiceUtil.returnError(ServiceUtil.getErrorMessage(result));
                             }
                         } catch (GenericServiceException e) {
-                            csvReader.close();
                             Debug.logError(e, MODULE);
                             return ServiceUtil.returnError(e.getMessage());
                         }