check false key and ignore the field

Thomas · Thomas · commit 492e1a2f9af7 · 2024-06-21T03:33:06.000-04:00
diff --git a/server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/authentication/util/AdvancedMapUtils.java b/server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/authentication/util/AdvancedMapUtils.java
@@ -13,6 +13,7 @@ public class AdvancedMapUtils {
      */
     public static String getString(Map<String, Object> map, String key) {
         if(key == null) return null;
+        if(key.equals("false")) return null;
         String[] parts = key.split("\\.");
         Object current = map;
 
diff --git a/server/api-service/lowcoder-server/src/test/java/org/lowcoder/api/authentication/AdvancedMapUtilsTest.java b/server/api-service/lowcoder-server/src/test/java/org/lowcoder/api/authentication/AdvancedMapUtilsTest.java
@@ -17,10 +17,13 @@ public void testGetStringFromMap() throws Exception {
                 Collections.singletonMap("def", Collections.singletonMap("hi", "hello world")),
                 Collections.singletonMap("def", Collections.singletonMap("hi", "another value"))
         ));
+        nestedMap.put("false", "123");
 
         String value0 = AdvancedMapUtils.getString(nestedMap, "abc[0].def.hi");
         String value1 = AdvancedMapUtils.getString(nestedMap, "abc[1].def.hi");
+        String value2 = AdvancedMapUtils.getString(nestedMap, "false");
         Assertions.assertSame("hello world", value0);
         Assertions.assertSame("another value", value1);
+        Assertions.assertSame(null, value2);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -17,10 +17,13 @@ public void testGetStringFromMap() throws Exception {`
`17`	`17`	`Collections.singletonMap("def", Collections.singletonMap("hi", "hello world")),`
`18`	`18`	`Collections.singletonMap("def", Collections.singletonMap("hi", "another value"))`
`19`	`19`	`));`
	`20`	`+ nestedMap.put("false", "123");`
`20`	`21`
`21`	`22`	`String value0 = AdvancedMapUtils.getString(nestedMap, "abc[0].def.hi");`
`22`	`23`	`String value1 = AdvancedMapUtils.getString(nestedMap, "abc[1].def.hi");`
	`24`	`+ String value2 = AdvancedMapUtils.getString(nestedMap, "false");`
`23`	`25`	`Assertions.assertSame("hello world", value0);`
`24`	`26`	`Assertions.assertSame("another value", value1);`
	`27`	`+ Assertions.assertSame(null, value2);`
`25`	`28`	`}`
`26`	`29`	`}`