Merge pull request #4 from apierzch/master

support adding multiple values to array

Author: imlucas

index.js

@@ -4,8 +4,16 @@ module.exports = function(patches){
   var update = {};
   patches.map(function(p){
     if(p.op === 'add'){
+      var path = toDot(p.path);
       if(!update.$push) update.$push = {};
-      update.$push[toDot(p.path)] = p.value;
+      if(!update.$push[path]) {
+        update.$push[path] = p.value;
+      } else if (update.$push[path]) {
+        if(!update.$push[path].$each) {
+          update.$push[path] = {$each : [update.$push[path]]};
+        }
+        update.$push[path].$each.push(p.value);
+      }
     }
     else if(p.op === 'remove'){
       if(!update.$unset) update.$unset = {};

test/index.test.js

@@ -4,7 +4,7 @@ var chai = require('chai');
 
 describe('jsonpatch to mongodb', function() {
 
-  it('should work with add', function() {
+  it('should work with single add', function() {
     var patches = [{
       op: 'add',
       path: '/name',
@@ -20,6 +20,30 @@ describe('jsonpatch to mongodb', function() {
     assert.deepEqual(toMongodb(patches), expected);
   });
 
+  it('should work with multiple adds', function() {
+    var patches = [{
+      op: 'add',
+      path: '/name',
+      value: 'dave'
+    },{
+      op: 'add',
+      path: '/name',
+      value: 'bob'
+    },{
+      op: 'add',
+      path: '/name',
+      value: 'john'
+    }];
+
+    var expected = {
+      $push: {
+        name: {$each: ['dave', 'bob', 'john']}
+      }
+    };
+
+    assert.deepEqual(toMongodb(patches), expected);
+  });
+
   it('should work with remove', function() {
     var patches = [{
       op: 'remove',