Fixed bug with scoped name functions because webpack remove them when uglify

Author: Josema

package.json

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

src/api/computed.js

@@ -1,7 +1,9 @@
 
 dop.computed = function(callback) {
     dop.util.invariant(isFunction(callback), 'dop.computed needs a function as first parameter');
-    return function $DOP_COMPUTED_FUNCTION(object, property, shallWeSet, oldValue) {
+    var f = function (object, property, shallWeSet, oldValue) {
         return dop.core.createComputed(object, property, callback, shallWeSet, oldValue);
-    };
+    }
+    f._name = dop.cons.COMPUTED_FUNCTION;
+    return f;
 };

src/api/isBroadcastFunction.js

@@ -1,4 +1,4 @@
 
 dop.isBroadcastFunction = function(fun) {
-    return (isFunction(fun) && fun.name===dop.cons.BROADCAST_FUNCTION);
+    return (isFunction(fun) && fun._name===dop.cons.BROADCAST_FUNCTION);
 };

src/api/isRemoteFunction.js

@@ -1,4 +1,4 @@
 
 dop.isRemoteFunction = function(fun) {
-    return (isFunction(fun) && fun.name===dop.cons.REMOTE_FUNCTION);
+    return (isFunction(fun) && fun._name===dop.cons.REMOTE_FUNCTION);
 };

src/api/setBroadcastFunction.js

@@ -4,7 +4,8 @@ dop.setBroadcastFunction = function (object, namefunction) {
     var path = dop.getObjectPath(object),
         object_id = path.shift();
     path.push(namefunction);
-    dop.getObjectTarget(object)[namefunction] = function $DOP_BROADCAST_FUNCTION() {
+    dop.getObjectTarget(object)[namefunction] = function() {
         return dop.protocol.broadcast(object_id, path, arguments);
     }
+    dop.getObjectTarget(object)[namefunction]._name = dop.cons.BROADCAST_FUNCTION;
 };

src/core/mutators/set.js

@@ -22,7 +22,7 @@ dop.core.set = function(object, property, value) {
             if (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)
+            else if (isFunction(value) && value._name==dop.cons.COMPUTED_FUNCTION)
                 objectTarget[property] = value(objectTarget, property, false, oldValue);
             // other
             else

src/core/objects/configureObject.js

@@ -66,12 +66,12 @@ dop.core.configureObject = function(object, propertyParent, parent) {
         value = object_target[property];
         is_function = isFunction(value);
         // remote function
-        if (is_function && value.name==dop.core.createRemoteFunction.name) {
+        if (is_function && value._name==dop.cons.REMOTE_FUNCTION_UNSETUP) {
             path = dop.getObjectPath(object);
             object_target[property] = value(path[0], path.slice(1).concat(property));
         }
         // storing computed value function
-        else if (is_function && value.name==dop.cons.COMPUTED_FUNCTION)
+        else if (is_function && value._name==dop.cons.COMPUTED_FUNCTION)
             object_target[property] = value(object_proxy, property, false, undefined);
         // object or array
         else if (dop.isObjectRegistrable(value))

src/core/objects/setPatch.js

@@ -11,7 +11,7 @@ dop.core.setPatch = function(object, patch, mutator) {
 
 
 dop.core.setPatchFunctionMutator = function(destiny, prop, value, typeofValue, path){
-    if (isFunction(value) && value.name==dop.core.createRemoteFunction.name)
+    if (isFunction(value) && value._name==dop.cons.REMOTE_FUNCTION_UNSETUP)
         dop.set(destiny, prop, value(dop.getObjectId(destiny), path.slice(0)));
     else
         return dop.core.setPatchMutator(destiny, prop, value, typeofValue, path);

src/core/protocol/createRemoteFunction.js

@@ -1,12 +1,16 @@
 
-dop.core.createRemoteFunction = function $DOP_REMOTE_FUNCTION_UNSETUP(node) {
-    return function $DOP_REMOTE_FUNCTION_UNSETUP(object_id, path) {
+dop.core.createRemoteFunction = function(node) {
+    var f = function (object_id, path) {
         // // http://jsperf.com/dynamic-name-of-functions
         // return new Function(
         //     "var a=arguments;return function " + path[path.length-1] + "(){return a[0](a[1], a[2], a[3], arguments)}"
         // )(dop.protocol.call, node, object_id, path)
-        return function $DOP_REMOTE_FUNCTION() {
+        var f2 = function $DOP_REMOTE_FUNCTION() {
             return dop.protocol.call(node, object_id, path, arguments);
         }
+        f2._name = dop.cons.REMOTE_FUNCTION
+        return f2;
     }
+    f._name = dop.cons.REMOTE_FUNCTION_UNSETUP;
+    return f;
 };

src/dop.js

@@ -49,8 +49,9 @@ var dop = {
         SEND: '~SEND',
         DISCONNECT: '~DISCONNECT',
         REMOTE_FUNCTION: '$DOP_REMOTE_FUNCTION',
+        REMOTE_FUNCTION_UNSETUP: '$DOP_REMOTE_FUNCTION_UNSETUP',
         BROADCAST_FUNCTION: '$DOP_BROADCAST_FUNCTION',
-        COMPUTED_FUNCTION: '$DOP_COMPUTED_FUNCTION'
+        COMPUTED_FUNCTION: '$DOP_COMPUTED_FUNCTION',
     }
 
 };