rc3

Author: Josema

.gitignore

@@ -21,3 +21,4 @@ yarn.lock
 examples/drag/package-lock.json
 examples/todomvc/package-lock.json
 .vscode/launch.json
+*.tgz

package.json

@@ -1,6 +1,6 @@
 {
     "name": "dop",
-    "version": "1.0.0-rc2",
+    "version": "1.0.0-rc3",
     "main": "./dist/dop.nodejs.js",
     "browser": "./dist/dop.js",
     "unpkg": "./dist/dop.min.js",

src/core/objects/setPatch.js

@@ -24,16 +24,6 @@ dop.core.setPatchMutator = function(destiny, prop, value, tof_value) {
         instructions_patchs = dop.protocol.instructionsPatchs,
         mutation
 
-    // Any value
-    if (tof_value != 'object' && tof_value != 'array') {
-        dop.set(destiny, prop, value)
-    }
-
-    // Not defined object or array
-    else if (!destiny.hasOwnProperty(prop)) {
-        dop.set(destiny, prop, tof_value == 'array' ? [] : {})
-    }
-
     // Array instruction
     if (tof_value == 'array') {
         type_instruction = value[0]
@@ -49,7 +39,12 @@ dop.core.setPatchMutator = function(destiny, prop, value, tof_value) {
         }
 
         // Array mutations
-        else if (isArray(destiny[prop])) {
+        else {
+            // We must create the array in case the
+            if (!destiny.hasOwnProperty(prop)) {
+                dop.set(destiny, prop, [])
+            }
+
             if (!isArray(type_instruction)) value = [value]
 
             for (var index = 0, total = value.length; index < total; ++index) {
@@ -65,6 +60,16 @@ dop.core.setPatchMutator = function(destiny, prop, value, tof_value) {
 
         return true // Skiping to dont go inside of [instructionPatch, ...]
     }
+
+    // Any value
+    if (tof_value != 'object' && tof_value != 'array') {
+        dop.set(destiny, prop, value)
+    }
+
+    // Not defined object
+    else if (!destiny.hasOwnProperty(prop)) {
+        dop.set(destiny, prop, {})
+    }
 }
 
 // // Array mutations

test/setpatchs2.js

@@ -0,0 +1,28 @@
+var test = require('tape')
+var dopClient = require('./.proxy').create()
+var dopServer = require('./.proxy').create()
+
+function getPatch(collector) {
+    var patch = dopServer.core.getPatch(collector.mutations)
+    var patchServer = dopServer.decode(dopServer.encode(patch))
+    collector.destroy()
+    for (var id in patchServer) return patchServer[id].chunks
+}
+function setPatch(destiny, chunks) {
+    dopClient.core.setPatch(destiny, chunks, dopClient.core.setPatchMutator)
+}
+
+test('Creating a new property with a new object must generate only one mutation in client', function(t) {
+    var objServer = dopServer.register({})
+    var objClient = dopClient.register({})
+    var collectorServer = dopServer.collect()
+    dopServer.set(objServer, 'prop', { new: 'object' })
+
+    var collectorClient = dopClient.collect()
+    setPatch(objClient, getPatch(collectorServer))
+
+    t.equal(collectorClient.mutations.length, 1)
+    t.deepEqual(objServer, objClient)
+    collectorClient.destroy()
+    t.end()
+})