rewrite array flattening to avoid RangeError

Signed-off-by: andrew-coleman <[email protected]>

Author: andrew-coleman

src/functions.js

@@ -1671,7 +1671,7 @@ const functions = (() => {
                 var res =  lookup(input[ii], key);
                 if (typeof res !== 'undefined') {
                     if (Array.isArray(res)) {
-                        result.push(...res);
+                        res.forEach(val => result.push(val));
                     } else {
                         result.push(res);
                     }

src/jsonata.js

@@ -283,7 +283,7 @@ var jsonata = (function() {
                     resultSequence.push(res);
                 } else {
                     // res is a sequence - flatten it into the parent sequence
-                    Array.prototype.push.apply(resultSequence, res);
+                    res.forEach(val => resultSequence.push(val));
                 }
             });
         }

test/test-suite/groups/flattening/large.json

@@ -0,0 +1,16 @@
+[
+    {
+        "expr": "( $data := { 'number': [1..2e5].$string() }; $values := $data.number; $count($values) )",
+        "data": null,
+        "result": 200000,
+        "timelimit": 10000,
+        "depth": 10
+    },
+    {
+        "expr": "( $data := { 'number': [1..2e5].$string() }; $values := $lookup($data, 'number'); $count($values) )",
+        "data": null,
+        "result": 200000,
+        "timelimit": 10000,
+        "depth": 10
+    }
+]