fix JSONPath#set, for issue #842

alibaba · Oct 29, 2022 · 7ed8a94 · 7ed8a94
1 parent d120ce9
commit 7ed8a94
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/core/src/main/java/com/alibaba/fastjson2/JSONPath.java b/core/src/main/java/com/alibaba/fastjson2/JSONPath.java
@@ -4814,6 +4814,11 @@ public void set(Context context, Object value) {
             if (object instanceof java.util.List) {
                 List list = (List) object;
                 if (index >= 0) {
+                    if (index > list.size()) {
+                        for (int i = list.size(); i < index; ++i) {
+                            list.add(null);
+                        }
+                    }
                     if (index < list.size()) {
                         list.set(index, value);
                     } else if (index <= list.size()) {

diff --git a/core/src/test/java/com/alibaba/fastjson2/issues/Issue842.java b/core/src/test/java/com/alibaba/fastjson2/issues/Issue842.java
@@ -1,6 +1,7 @@
 package com.alibaba.fastjson2.issues;
 
 import com.alibaba.fastjson2.JSON;
+import com.alibaba.fastjson2.JSONObject;
 import com.alibaba.fastjson2.JSONPath;
 import org.junit.jupiter.api.Test;
 
@@ -25,4 +26,18 @@ public void test1() {
         JSONPath.set(data, jsonPath, "设置成功");
         assertEquals("{\"items\":[{\"data\":[\"设置成功\"]}]}", JSON.toJSONString(data));
     }
+
+    @Test
+    public void test2() {
+        Map data = new LinkedHashMap<>();
+        JSONPath.set(data, "$.obj.5", "设置成功");
+        assertEquals("{\"obj\":[null,null,null,null,null,\"设置成功\"]}", JSONObject.toJSONString(data));
+    }
+
+    @Test
+    public void test3() {
+        Map data = new LinkedHashMap<>();
+        JSONPath.set(data, "$.obj.2.5", "设置成功");
+        assertEquals("{\"obj\":[null,null,[null,null,null,null,null,\"设置成功\"]]}", JSONObject.toJSONString(data));
+    }
 }