Merge pull request #18 from prakanth97/array_fix

Fix bug in open array conversion
ballerina-platform · May 2, 2024 · 5ff04c0 · 5ff04c0
2 parents 1f53bb0 + 444e7c4
commit 5ff04c0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/ballerina/tests/from_json_with_disable_projection_option.bal b/ballerina/tests/from_json_with_disable_projection_option.bal
@@ -297,3 +297,10 @@ isolated function testDisableDataProjectionInRecordTypeWithParseAsType() {
     test:assertTrue(val4 is error);
     test:assertEquals((<error>val4).message(), "undefined field 'company1'");
 }
+
+@test:Config
+isolated function testDisableProjectionForOpenArray() returns error? {
+    string jsonStr = string `[1, 2, 3, 4]`;
+    int[] val = check parseString(jsonStr, options);
+    test:assertEquals(val, [1, 2, 3, 4]);
+}
diff --git a/native/src/main/java/io/ballerina/lib/data/jsondata/json/JsonTraverse.java b/native/src/main/java/io/ballerina/lib/data/jsondata/json/JsonTraverse.java
@@ -229,7 +229,8 @@ private Object traverseArrayValue(BArray array, Object currentJsonNode) {
                     ArrayType arrayType = (ArrayType) rootArray;
                     int expectedArraySize = arrayType.getSize();
                     long sourceArraySize = array.getLength();
-                    if (!allowDataProjection && expectedArraySize < sourceArraySize) {
+                    if (!allowDataProjection && arrayType.getState() == ArrayType.ArrayState.CLOSED
+                            && expectedArraySize < sourceArraySize) {
                         throw DiagnosticLog.error(DiagnosticErrorCode.ARRAY_SIZE_MISMATCH);
                     }