(ofbiz-framework) branch release24.09 updated: Fixed: Error viewing entities with fromDate key in entity maintenance (OFBIZ-13222)

[nmalin]

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

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


The following commit(s) were added to refs/heads/release24.09 by this push:
     new ecb95e194b Fixed: Error viewing entities with fromDate key in entity 
maintenance (OFBIZ-13222)
ecb95e194b is described below

commit ecb95e194b7cbf6f0e064d0392ec820bd80e163f
Author: Nicolas Malin <nicolas.ma...@nereide.fr>
AuthorDate: Tue Apr 29 15:41:08 2025 +0200

    Fixed: Error viewing entities with fromDate key in entity maintenance 
(OFBIZ-13222)
    
    Fix NPE with previous commit when we use 'create' button from webtools
---
 .../src/main/java/org/apache/ofbiz/entity/util/EntityUtil.java     | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git 
a/framework/entity/src/main/java/org/apache/ofbiz/entity/util/EntityUtil.java 
b/framework/entity/src/main/java/org/apache/ofbiz/entity/util/EntityUtil.java
index bba17e9b06..64977cd4ed 100644
--- 
a/framework/entity/src/main/java/org/apache/ofbiz/entity/util/EntityUtil.java
+++ 
b/framework/entity/src/main/java/org/apache/ofbiz/entity/util/EntityUtil.java
@@ -592,8 +592,11 @@ public final class EntityUtil {
      */
     public static String entityToPath(GenericValue gv) {
         StringBuilder path = new StringBuilder(gv.getEntityName());
-        for (String pkName : gv.getModelEntity().getPkFieldNames()) {
-            path.append("/").append(UtilCodec.encodeUrl(gv.getString(pkName)));
+        List<String> pkFieldNames = gv.getModelEntity().getPkFieldNames();
+        if (pkFieldNames.stream().noneMatch(pkName -> gv.get(pkName) == null)) 
{
+            for (String pkName : pkFieldNames) {
+                
path.append("/").append(UtilCodec.encodeUrl(gv.getString(pkName)));
+            }
         }
         return path.toString();
     }