[ofbiz-framework] branch trunk updated: Improved: Try to reduce the SARIF file used by CodeQL for Java classes (OFBIZ-12361)

[jleroux]

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

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


The following commit(s) were added to refs/heads/trunk by this push:
     new b62cfbd  Improved: Try to reduce the SARIF file used by CodeQL for 
Java classes (OFBIZ-12361)
b62cfbd is described below

commit b62cfbdc3a85dc8f7352d668af4e6c9cda6c87b6
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Wed Nov 17 08:06:39 2021 +0100

    Improved: Try to reduce the SARIF file used by CodeQL for Java classes 
(OFBIZ-12361)
    
    Try a syntax change from Andrew Eisenberg
    It also needs a not functional change for codeQL to run
    
    Thanks to Andrew Eisenberg from GH CodeQL
---
 .github/workflows/codeql-analysis.yml                            | 9 +++------
 .../org/apache/ofbiz/party/party/PartyRelationshipServices.java  | 1 -
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/codeql-analysis.yml 
b/.github/workflows/codeql-analysis.yml
index e4995f9..ab77932 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -65,6 +65,7 @@ jobs:
       uses: github/codeql-action/init@v1
       with:
         languages: ${{ matrix.language }}
+        debug: true
         # If you wish to specify custom queries, you can do so here or in a 
config file.
         # By default, queries listed here will override any specified in a 
config file.
         # Prefix the list here with "+" to use these queries and those in the 
config file.
@@ -86,9 +87,5 @@ jobs:
     #   make bootstrap
     #   make release
 
-    #- name: Perform CodeQL Analysis
-    # uses: github/codeql-action/analyze@v1
-
-    - uses: github/codeql-action/analyze@v1
-      with:
-        debug: true
+    - name: Perform CodeQL Analysis
+      uses: github/codeql-action/analyze@v1
diff --git 
a/applications/party/src/main/java/org/apache/ofbiz/party/party/PartyRelationshipServices.java
 
b/applications/party/src/main/java/org/apache/ofbiz/party/party/PartyRelationshipServices.java
index 8865b08..22ebe89 100644
--- 
a/applications/party/src/main/java/org/apache/ofbiz/party/party/PartyRelationshipServices.java
+++ 
b/applications/party/src/main/java/org/apache/ofbiz/party/party/PartyRelationshipServices.java
@@ -46,7 +46,6 @@ import org.apache.ofbiz.service.ServiceUtil;
 public class PartyRelationshipServices {
 
     private static final String MODULE = 
PartyRelationshipServices.class.getName();
-
     private static final String RES_ERROR = "PartyErrorUiLabels";
 
     /** Creates and updates a PartyRelationship creating related PartyRoles if 
needed.