[ofbiz-framework] branch release18.12 updated: Fixed: Bypass Java GString to String conversion in ObjectType::simpleTypeOrObjectConvert (OFBIZ-12182)

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[ofbiz-framework] branch release18.12 updated: Fixed: Bypass Java GString to String conversion in ObjectType::simpleTypeOrObjectConvert (OFBIZ-12182)

jleroux@apache.org
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/release18.12 by this push:
     new 16fb51c  Fixed: Bypass Java GString to String conversion in ObjectType::simpleTypeOrObjectConvert (OFBIZ-12182)
16fb51c is described below

commit 16fb51c3704979e06a792c6dd425e723f587d8e8
Author: Jacques Le Roux <[hidden email]>
AuthorDate: Fri Feb 19 09:28:53 2021 +0100

    Fixed: Bypass Java GString to String conversion in ObjectType::simpleTypeOrObjectConvert (OFBIZ-12182)
   
    Follows https://markmail.org/message/azn7e4g2rbpo7tvj
   
    Conflicts handled by hand
     framework/base/src/main/java/org/apache/ofbiz/base/util/ObjectType.java
---
 .../base/src/main/java/org/apache/ofbiz/base/util/ObjectType.java    | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/framework/base/src/main/java/org/apache/ofbiz/base/util/ObjectType.java b/framework/base/src/main/java/org/apache/ofbiz/base/util/ObjectType.java
index c2b7da7..6d27f5c 100644
--- a/framework/base/src/main/java/org/apache/ofbiz/base/util/ObjectType.java
+++ b/framework/base/src/main/java/org/apache/ofbiz/base/util/ObjectType.java
@@ -510,13 +510,12 @@ public class ObjectType {
         if (obj == null || UtilValidate.isEmpty(type) || "Object".equals(type) || "java.lang.Object".equals(type)) {
             return obj;
         }
-        if ("PlainString".equals(type)) {
+        if ("PlainString".equals(type)
+                || ("org.codehaus.groovy.runtime.GStringImpl".equals(obj.getClass().getName()) && "String".equals(type))) {
             return obj.toString();
         }
         if (obj instanceof Node) {
             Node node = (Node) obj;
-
-
             String nodeValue =  node.getTextContent();
 
             if (nodeValue == null) {