isObjectRegistrable only accepts POJO objects

Author: Josema

Gruntfile.js

@@ -72,6 +72,35 @@ module.exports = function(grunt) {
             }
         },
 
+        replace: {
+            dist: {
+                options: {
+                    patterns: [
+                        {
+                            match: /\.core\./g,
+                            replacement: '.c.'
+                        },
+                        {
+                            match: /\.util\./g,
+                            replacement: '.u.'
+                        },
+                        {
+                            match: /\.protocol\./g,
+                            replacement: '.p.'
+                        }
+                    ]
+                },
+                files: [
+                    {
+                        expand: true,
+                        flatten: true,
+                        src: 'dist/dop.min.js',
+                        dest: 'dist'
+                    }
+                ]
+            }
+        },
+
         'optimize-js': {
             options: {
                 sourceMap: false,
@@ -92,10 +121,11 @@ module.exports = function(grunt) {
     grunt.loadNpmTasks('grunt-contrib-concat');
     grunt.loadNpmTasks('grunt-contrib-uglify');
     grunt.loadNpmTasks('grunt-contrib-watch');
+    grunt.loadNpmTasks('grunt-replace');
     grunt.loadNpmTasks('grunt-optimize-js');
 
 
-    var tasks = ['copy', 'concat:nodejs', 'concat:browser', 'uglify'/*, 'optimize-js'*/];
+    var tasks = ['copy', 'concat:nodejs', 'concat:browser', 'uglify', /*'replace', 'optimize-js'*/];
     if (grunt.option('build') === undefined)
         tasks.push('watch');
     grunt.registerTask('default', tasks);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "dop",
-  "version": "0.22.1",
+  "version": "0.22.2",
   "main": "./dist/dop.nodejs.js",
   "browser": "./dist/dop.js",
   "unpkg": "./dist/dop.min.js",
@@ -48,6 +48,7 @@
     "grunt-contrib-uglify": "^0.6.0",
     "grunt-contrib-watch": "^0.6.1",
     "grunt-optimize-js": "^0.6.0",
+    "grunt-replace": "^1.0.1",
     "js-combinatorics": "^0.5.2",
     "lodash.merge": "^4.6.0",
     "tap": "^7.1.2",

src/api/isObjectRegistrable.js

@@ -1,14 +1,13 @@
 
 dop.isObjectRegistrable = function(object) {
-    var tof = dop.util.typeof(object);
-    return (tof === 'object' || tof == 'array');
+    if (!object || typeof object !== "object") return false;
+    var prototype = Object.getPrototypeOf(object);
+    return prototype === Object.prototype || prototype === Array.prototype;
 };
 
-// dop.isObjectRegistrable = function(object) {
-//     if (!object)
-//         return false;
-//     var prototype = Object.getPrototypeOf(object);
-//     return (prototype === Object.prototype || prototype === Array.prototype);
+//dop.isObjectRegistrable = function(object) {
+//     var tof = dop.util.typeof(object);
+//     return (tof === 'object' || tof == 'array');
 // };
 
 // function Test(){}

src/api/register.js

@@ -1,6 +1,6 @@
 
 dop.register = function(object) {
-    dop.util.invariant(dop.util.typeof(object) == 'object', 'dop.register needs a regular plain object as first parameter');
+    dop.util.invariant(dop.isObjectRegistrable(object) && !isArray(object), 'dop.register needs a regular plain object as first parameter');
     return (dop.isRegistered(object)) ?
         dop.getObjectProxy(object)
     :

test/zarrays.js

@@ -10,10 +10,7 @@ function gify(obj) {
     this.myClassProperty=123;
     return JSON.stringify(obj);
 }
-function isObjectRegistrable(object) {
-    var tof = dop.util.typeof(object);
-    return (tof === 'object' || tof == 'array');
-}
+var isObjectRegistrable = dop.isObjectRegistrable;