mobx-state-tree 5.3.0 → 5.3.1-alpha.1

package/dist/mobx-state-tree.module.js

@@ -5842,62 +5842,47 @@ var defaultObjectOptions = {
 };
 function toPropertiesObject(declaredProps) {
     var keysList = Object.keys(declaredProps);
-    var alreadySeenKeys = {};
+    var alreadySeenKeys = new Set();
     keysList.forEach(function (key) {
-        if (alreadySeenKeys[key]) {
-            throw fail$1(key + " is declared twice in the model. Model should not contain same keys");
-        }
-        else {
-            alreadySeenKeys[key] = true;
+        if (alreadySeenKeys.has(key)) {
+            throw fail$1(key + " is declared twice in the model. Model should not contain the same keys");
         }
+        alreadySeenKeys.add(key);
     });
     // loop through properties and ensures that all items are types
     return keysList.reduce(function (props, key) {
-        var _a, _b, _c;
         // warn if user intended a HOOK
-        if (key in Hook)
+        if (key in Hook) {
             throw fail$1("Hook '" + key + "' was defined as property. Hooks should be defined as part of the actions");
-        // the user intended to use a view
-        var descriptor = Object.getOwnPropertyDescriptor(props, key);
+        }
+        var descriptor = Object.getOwnPropertyDescriptor(declaredProps, key);
         if ("get" in descriptor) {
             throw fail$1("Getters are not supported as properties. Please use views instead");
         }
-        // undefined and null are not valid
         var value = descriptor.value;
         if (value === null || value === undefined) {
             throw fail$1("The default value of an attribute cannot be null or undefined as the type cannot be inferred. Did you mean `types.maybe(someType)`?");
-            // its a primitive, convert to its type
         }
         else if (isPrimitive(value)) {
-            return Object.assign({}, props, (_a = {},
-                _a[key] = optional(getPrimitiveFactoryFromValue(value), value),
-                _a));
-            // map defaults to empty object automatically for models
+            props[key] = optional(getPrimitiveFactoryFromValue(value), value);
         }
         else if (value instanceof MapType) {
-            return Object.assign({}, props, (_b = {},
-                _b[key] = optional(value, {}),
-                _b));
+            props[key] = optional(value, {});
         }
         else if (value instanceof ArrayType) {
-            return Object.assign({}, props, (_c = {}, _c[key] = optional(value, []), _c));
-            // its already a type
-        }
-        else if (isType(value)) {
-            return props;
-            // its a function, maybe the user wanted a view?
+            props[key] = optional(value, []);
         }
+        else if (isType(value)) ;
         else if (devMode() && typeof value === "function") {
             throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed a function. Did you forget to invoke it, or did you intend to declare a view / action?");
-            // no other complex values
         }
         else if (devMode() && typeof value === "object") {
             throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed an object. Try passing another model type or a types.frozen.");
-            // WTF did you pass in mate?
         }
         else {
             throw fail$1("Invalid type definition for property '" + key + "', cannot infer a type from a value like '" + value + "' (" + typeof value + ")");
         }
+        return props;
     }, declaredProps);
 }
 /**

package/dist/mobx-state-tree.umd.js

@@ -5824,62 +5824,47 @@
     };
     function toPropertiesObject(declaredProps) {
         var keysList = Object.keys(declaredProps);
-        var alreadySeenKeys = {};
+        var alreadySeenKeys = new Set();
         keysList.forEach(function (key) {
-            if (alreadySeenKeys[key]) {
-                throw fail$1(key + " is declared twice in the model. Model should not contain same keys");
-            }
-            else {
-                alreadySeenKeys[key] = true;
+            if (alreadySeenKeys.has(key)) {
+                throw fail$1(key + " is declared twice in the model. Model should not contain the same keys");
             }
+            alreadySeenKeys.add(key);
         });
         // loop through properties and ensures that all items are types
         return keysList.reduce(function (props, key) {
-            var _a, _b, _c;
             // warn if user intended a HOOK
-            if (key in Hook)
+            if (key in Hook) {
                 throw fail$1("Hook '" + key + "' was defined as property. Hooks should be defined as part of the actions");
-            // the user intended to use a view
-            var descriptor = Object.getOwnPropertyDescriptor(props, key);
+            }
+            var descriptor = Object.getOwnPropertyDescriptor(declaredProps, key);
             if ("get" in descriptor) {
                 throw fail$1("Getters are not supported as properties. Please use views instead");
             }
-            // undefined and null are not valid
             var value = descriptor.value;
             if (value === null || value === undefined) {
                 throw fail$1("The default value of an attribute cannot be null or undefined as the type cannot be inferred. Did you mean `types.maybe(someType)`?");
-                // its a primitive, convert to its type
             }
             else if (isPrimitive(value)) {
-                return Object.assign({}, props, (_a = {},
-                    _a[key] = optional(getPrimitiveFactoryFromValue(value), value),
-                    _a));
-                // map defaults to empty object automatically for models
+                props[key] = optional(getPrimitiveFactoryFromValue(value), value);
             }
             else if (value instanceof MapType) {
-                return Object.assign({}, props, (_b = {},
-                    _b[key] = optional(value, {}),
-                    _b));
+                props[key] = optional(value, {});
             }
             else if (value instanceof ArrayType) {
-                return Object.assign({}, props, (_c = {}, _c[key] = optional(value, []), _c));
-                // its already a type
-            }
-            else if (isType(value)) {
-                return props;
-                // its a function, maybe the user wanted a view?
+                props[key] = optional(value, []);
             }
+            else if (isType(value)) ;
             else if (typeof value === "function") {
                 throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed a function. Did you forget to invoke it, or did you intend to declare a view / action?");
-                // no other complex values
             }
             else if (typeof value === "object") {
                 throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed an object. Try passing another model type or a types.frozen.");
-                // WTF did you pass in mate?
             }
             else {
                 throw fail$1("Invalid type definition for property '" + key + "', cannot infer a type from a value like '" + value + "' (" + typeof value + ")");
             }
+            return props;
         }, declaredProps);
     }
     /**