mobx 6.6.1 → 6.7.0

package/dist/mobx.esm.js

@@ -424,7 +424,7 @@ function assertNotDecorated(prototype, annotation, key) {
     var fieldName = prototype.constructor.name + ".prototype." + key.toString();
     var currentAnnotationType = prototype[storedAnnotationsSymbol][key].annotationType_;
     var requestedAnnotationType = annotation.annotationType_;
-    die("Cannot apply '@" + requestedAnnotationType + "' to '" + fieldName + "':" + ("\nThe field is already decorated with '@" + currentAnnotationType + "'.") + "\nRe-decorating fields is not allowed." + "\nUse '@override' decorator for methods overriden by subclass.");
+    die("Cannot apply '@" + requestedAnnotationType + "' to '" + fieldName + "':" + ("\nThe field is already decorated with '@" + currentAnnotationType + "'.") + "\nRe-decorating fields is not allowed." + "\nUse '@override' decorator for methods overridden by subclass.");
   }
 }
 /**
@@ -2325,7 +2325,7 @@ function reportObserved(observable) {
       }
     }
 
-    return true;
+    return observable.isBeingObserved_;
   } else if (observable.observers_.size === 0 && globalState.inBatch > 0) {
     queueForUnobservation(observable);
   }
@@ -3707,12 +3707,25 @@ function _when(predicate, effect, opts) {
 }
 
 function whenPromise(predicate, opts) {
+  var _opts$signal;
+
   if (process.env.NODE_ENV !== "production" && opts && opts.onError) {
     return die("the options 'onError' and 'promise' cannot be combined");
   }
 
+  if (opts != null && (_opts$signal = opts.signal) != null && _opts$signal.aborted) {
+    return Object.assign(Promise.reject(new Error("WHEN_ABORTED")), {
+      cancel: function cancel() {
+        return null;
+      }
+    });
+  }
+
   var cancel;
+  var abort;
   var res = new Promise(function (resolve, reject) {
+    var _opts$signal2;
+
     var disposer = _when(predicate, resolve, _extends({}, opts, {
       onError: reject
     }));
@@ -3721,6 +3734,17 @@ function whenPromise(predicate, opts) {
       disposer();
       reject(new Error("WHEN_CANCELLED"));
     };
+
+    abort = function abort() {
+      disposer();
+      reject(new Error("WHEN_ABORTED"));
+    };
+
+    opts == null ? void 0 : (_opts$signal2 = opts.signal) == null ? void 0 : _opts$signal2.addEventListener("abort", abort);
+  })["finally"](function () {
+    var _opts$signal3;
+
+    return opts == null ? void 0 : (_opts$signal3 = opts.signal) == null ? void 0 : _opts$signal3.removeEventListener("abort", abort);
   });
   res.cancel = cancel;
   return res;
@@ -4239,17 +4263,23 @@ var ObservableArrayAdministration = /*#__PURE__*/function () {
   };
 
   _proto.get_ = function get_(index) {
-    if (index < this.values_.length) {
-      this.atom_.reportObserved();
-      return this.dehanceValue_(this.values_[index]);
+    if (this.legacyMode_ && index >= this.values_.length) {
+      console.warn(process.env.NODE_ENV !== "production" ? "[mobx.array] Attempt to read an array index (" + index + ") that is out of bounds (" + this.values_.length + "). Please check length first. Out of bound indices will not be tracked by MobX" : "[mobx] Out of bounds read: " + index);
+      return undefined;
     }
 
-    console.warn(process.env.NODE_ENV !== "production" ? "[mobx] Out of bounds read: " + index : "[mobx.array] Attempt to read an array index (" + index + ") that is out of bounds (" + this.values_.length + "). Please check length first. Out of bound indices will not be tracked by MobX");
+    this.atom_.reportObserved();
+    return this.dehanceValue_(this.values_[index]);
   };
 
   _proto.set_ = function set_(index, newValue) {
     var values = this.values_;
 
+    if (this.legacyMode_ && index > values.length) {
+      // out of bounds
+      die(17, index, values.length);
+    }
+
     if (index < values.length) {
       // update at index in range
       checkIfStateModificationsAreAllowed(this.atom_);
@@ -4277,12 +4307,19 @@ var ObservableArrayAdministration = /*#__PURE__*/function () {
         values[index] = newValue;
         this.notifyArrayChildUpdate_(index, newValue, oldValue);
       }
-    } else if (index === values.length) {
-      // add a new item
-      this.spliceWithArray_(index, 0, [newValue]);
     } else {
-      // out of bounds
-      die(17, index, values.length);
+      // For out of bound index, we don't create an actual sparse array,
+      // but rather fill the holes with undefined (same as setArrayLength_).
+      // This could be considered a bug.
+      var newItems = new Array(index + 1 - values.length);
+
+      for (var i = 0; i < newItems.length - 1; i++) {
+        newItems[i] = undefined;
+      } // No Array.fill everywhere...
+
+
+      newItems[newItems.length - 1] = newValue;
+      this.spliceWithArray_(values.length, 0, newItems);
     }
   };
 
@@ -6009,16 +6046,18 @@ function assertAnnotable(adm, annotation, key) {
     var fieldName = adm.name_ + "." + key.toString();
     var currentAnnotationType = adm.appliedAnnotations_[key].annotationType_;
     var requestedAnnotationType = annotation.annotationType_;
-    die("Cannot apply '" + requestedAnnotationType + "' to '" + fieldName + "':" + ("\nThe field is already annotated with '" + currentAnnotationType + "'.") + "\nRe-annotating fields is not allowed." + "\nUse 'override' annotation for methods overriden by subclass.");
+    die("Cannot apply '" + requestedAnnotationType + "' to '" + fieldName + "':" + ("\nThe field is already annotated with '" + currentAnnotationType + "'.") + "\nRe-annotating fields is not allowed." + "\nUse 'override' annotation for methods overridden by subclass.");
   }
 }
 
+var ENTRY_0 = /*#__PURE__*/createArrayEntryDescriptor(0);
 /**
  * This array buffer contains two lists of properties, so that all arrays
  * can recycle their property definitions, which significantly improves performance of creating
  * properties on the fly.
  */
 
+
 var OBSERVABLE_ARRAY_BUFFER_SIZE = 0; // Typescript workaround to make sure ObservableArray extends Array
 
 var StubArray = function StubArray() {};
@@ -6064,6 +6103,12 @@ var LegacyObservableArray = /*#__PURE__*/function (_StubArray, _Symbol$toStringT
       allowStateChangesEnd(prev);
     }
 
+    {
+      // Seems that Safari won't use numeric prototype setter untill any * numeric property is
+      // defined on the instance. After that it works fine, even if this property is deleted.
+      Object.defineProperty(_assertThisInitialized(_this), "0", ENTRY_0);
+    }
+
     return _this;
   }