Skip to content

Commit 2621d8e

Browse files
iroquetaBeta Bot
authored andcommitted
Cherry pick branch 'genexuslabs:fix/strcmp-null-vs-empty' into beta
1 parent 90d5d99 commit 2621d8e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

common/src/main/java/com/genexus/CommonUtil.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2809,7 +2809,7 @@ else if (className.equals("long") || className.equals("java.lang.Long") || class
28092809
}
28102810
else if (className.equals("string") || className.indexOf("java.lang.String") != -1)
28112811
{
2812-
return objStr.equals("null") ? null : objStr;
2812+
return (obj == JSONObject.NULL)? null : objStr;
28132813
}
28142814
else if (className.equals("double") || className.equals("java.lang.Double") || className.equals("[D"))
28152815
{

java/src/test/java/com/genexus/TestCommonUtil.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ public void testConvertObjectTo() {
295295
try{
296296
Class stringClass = Class.forName("java.lang.String");
297297
result = CommonUtil.convertObjectTo(obj, stringClass, true);
298-
Assert.assertEquals(null, result);
298+
Assert.assertEquals("null", result);
299299
} catch (Exception e){
300300
Assert.fail("Test failed " + e);
301301
}

0 commit comments

Comments
 (0)