Merge pull request #22 from zeidoo/master

add support for array creation
jonschlinkert · Apr 28, 2021 · db9602f · db9602f
2 parents 042bb56 + 6169f1b
commit db9602f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -38,7 +38,9 @@ function set(target, path, value, options) {
     let prop = keys[i];
 
     if (!isObject(target[prop])) {
-      target[prop] = {};
+      let nextProp = i + 1 < len ? keys[i + 1] : "";
+      let shouldBeArray = !isNaN(+nextProp);
+      target[prop] = shouldBeArray ? [] : {};
     }
 
     if (i === len - 1) {

diff --git a/test.js b/test.js
@@ -26,6 +26,17 @@ describe('set', function() {
     assert.equal(o.a.b, 'c');
   });
 
+  it('should create a nested array if it does not already exist', function() {
+    const o = {};
+    set(o, 'a.0', 'c');
+    set(o, 'a.1', 'd');
+    assert(Array.isArray(o.a));
+    assert.equal(o.a[0], 'c');
+    let actual = "";
+    o.a.map(i=> actual +=i);
+    assert.equal(actual, "cd");
+  });
+
   it('should merge an existing value with the given value', function() {
     var o = {a: {b: {c: 'd'}}};
     set(o, 'a.b', {y: 'z'}, { merge: true });