[ofbiz-framework] branch release17.12 updated: Fixed: getEntityRefData service has a weird issue (OFBIZ-11401)

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

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


The following commit(s) were added to refs/heads/release17.12 by this push:
     new 4079b54  Fixed: getEntityRefData service has a weird issue (OFBIZ-11401)
4079b54 is described below

commit 4079b54ece0644467f2a3f945256fa81279539a9
Author: Jacques Le Roux <[hidden email]>
AuthorDate: Thu Jun 18 17:51:36 2020 +0200

    Fixed: getEntityRefData service has a weird issue (OFBIZ-11401)
   
    It's reproductible on trunk demo though only visible in log, UI shows no issue
   
    R17 and R18 are also affected, R16 is OK.
   
    Ankush: This stacktrace should be coming for all entities where
    default-resource-name is defined on entity definition but no property exist in
    defined resource for key EntityDescription.entityname and FieldDescription.field.
   
    As we want to have these entity labels as much as possible, we can simply print
    warning about the missing entity descriptions and field descriptions for
    readability purpose instead of printing huge exceptions stacktrace.
   
    Thanks: Ankush Upadhyay and Pierre Smits for the help
   
    Handled by hand (swallowing exceptions is lazy and bad :p)
---
 .../java/org/apache/ofbiz/webtools/WebToolsServices.java    | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/framework/webtools/src/main/java/org/apache/ofbiz/webtools/WebToolsServices.java b/framework/webtools/src/main/java/org/apache/ofbiz/webtools/WebToolsServices.java
index 4e660a3..68a2865 100644
--- a/framework/webtools/src/main/java/org/apache/ofbiz/webtools/WebToolsServices.java
+++ b/framework/webtools/src/main/java/org/apache/ofbiz/webtools/WebToolsServices.java
@@ -663,7 +663,9 @@ public class WebToolsServices {
                         if (bundle != null) {
                             try {
                                 entityDescription = bundle.getString("EntityDescription." + entity.getEntityName());
-                            } catch (Exception exception) {}
+                            } catch (Exception exception) {
+                                Debug.logWarning("EntityDescription for entity " + entity.getEntityName() + " is missing", module);
+                            }
                         }
                         if (UtilValidate.isEmpty(entityDescription)) {
                             entityDescription = entity.getDescription();
@@ -682,7 +684,10 @@ public class WebToolsServices {
                             if (bundle != null) {
                                 try {
                                     fieldDescription = bundle.getString("FieldDescription." + entity.getEntityName() + "." + field.getName());
-                                } catch (Exception exception) {}
+                                } catch (Exception exception) {
+                                    Debug.logWarning("FieldDescription for entity.field " + entity.getEntityName() + "."
+                                            + field.getName() + " is missing", module);
+                                }
                             }
                             if (UtilValidate.isEmpty(fieldDescription)) {
                                 fieldDescription = field.getDescription();
@@ -690,7 +695,9 @@ public class WebToolsServices {
                             if (UtilValidate.isEmpty(fieldDescription) && bundle != null) {
                                 try {
                                 fieldDescription = bundle.getString("FieldDescription." + field.getName());
-                                } catch (Exception exception) {}
+                                } catch (Exception exception) {
+                                    Debug.logWarning("FieldDescription for field " + field.getName() + " is missing", module);
+                                }
                             }
                             if (UtilValidate.isEmpty(fieldDescription)) {
                                 fieldDescription = ModelUtil.javaNameToDbName(field.getName()).toLowerCase();