Added deep and shadow options on method dop.set

Author: Josema

package.json

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

src/api/set.js

@@ -1,8 +1,8 @@
 
-dop.set = function(object, property, value) {
+dop.set = function(object, property, value, options) {
     // dop.util.invariant(dop.isRegistered(object), 'Object passed to dop.set must be a registered object');
     (dop.isRegistered(object)) ?
-        dop.core.set(object, property, value)
+        dop.core.set(object, property, value, options)
     :
         object[property] = value;
     return value;

src/core/mutators/set.js

@@ -1,5 +1,11 @@
 
-dop.core.set = function(object, property, value) {
+dop.core.set = function(object, property, value, options) {
+
+    if (!isObject(options))
+        options = {}
+
+    options.deep = typeof options.deep == 'boolean' ? options.deep : true
+    options.shadow = typeof options.shadow == 'boolean' ? options.shadow : false
 
     // If is a different value
     if (object[property] !== value) {
@@ -19,7 +25,7 @@ dop.core.set = function(object, property, value) {
             //     property = Number(property);
 
             // object or array
-            if (dop.isObjectRegistrable(value) && !(dop.isRegistered(value) && dop.getObjectParent(value) === objectProxy))
+            if (options.deep && dop.isObjectRegistrable(value) && !(dop.isRegistered(value) && dop.getObjectParent(value) === objectProxy))
                 objectTarget[property] = dop.core.configureObject(value, property, objectProxy);
             // computed value
             else if (isFunction(value) && value._name==dop.cons.COMPUTED_FUNCTION)
@@ -29,6 +35,7 @@ dop.core.set = function(object, property, value) {
                 objectTarget[property] = value;
 
             if (
+                !options.shadow &&
                 (objectTarget===objectProxy || object===objectProxy) &&
                 !(isFunction(oldValue) && isFunction(value)) &&
                 (path = dop.getObjectPath(object))

test/collectors.js

@@ -122,3 +122,16 @@ test('Emit', function(t) {
     t.equal(collector.mutations.length, 0);
     t.end();
 });
+
+
+
+
+test('Testing shadow option on set', function(t) {
+    var collector = dop.collect();
+    dop.set(object, 'noshadow', [2,3,4])
+    dop.set(object, 'shadow', [2,3,4], {shadow:true})
+    t.equal(collector.mutations.length, 1);
+    t.deepEqual(object.noshadow, object.shadow);
+    collector.destroy();
+    t.end();
+});

test/sets_dels.js

@@ -186,3 +186,18 @@ test('Update a property freeze return same value and dont change property', func
     t.equal(object.new, 'newsealvalue');
     t.end();
 });
+
+
+
+
+
+
+test('set() deep option', function(t) {
+    var object = dop.register({})
+    set(object, 'deep', {a:1,b:{e:true}})
+    set(object, 'nodeep', {a:1,b:{e:true}}, {deep:false})
+    t.equal(dop.isRegistered(object.deep), true)
+    t.equal(dop.isRegistered(object.nodeep), false)
+    t.deepEqual(object.deep, object.nodeep)
+    t.end();
+});