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

package/dist/mobx-state-tree.js

@@ -5846,62 +5846,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);
 }
 /**