mobx 6.13.6 → 6.15.0

package/dist/mobx.umd.development.js

@@ -1012,7 +1012,8 @@
       // lone setter -> action setter
       if (descriptor.set) {
         // TODO make action applicable to setter and delegate to action.make_
-        var set = createAction(key.toString(), descriptor.set);
+        var set = isAction(descriptor.set) ? descriptor.set // See #4553
+        : createAction(key.toString(), descriptor.set);
         // own
         if (source === adm.target_) {
           return adm.defineProperty_(key, {
@@ -1377,13 +1378,14 @@
         _this.equals = equals;
         _this.value_ = enhancer(value, undefined, name_);
         if ( notifySpy && isSpyEnabled()) {
+          var _this$value_;
           // only notify spy if this is a stand-alone observable
           spyReport({
             type: CREATE,
             object: _this,
             observableKind: "value",
             debugObjectName: _this.name_,
-            newValue: "" + _this.value_
+            newValue: "" + ((_this$value_ = _this.value_) == null ? void 0 : _this$value_.toString())
           });
         }
         return _this;
@@ -2551,6 +2553,9 @@
         };
         abortSignal == null || abortSignal.addEventListener == null || abortSignal.addEventListener("abort", dispose);
         dispose[$mobx] = this;
+        if ("dispose" in Symbol && typeof Symbol.dispose === "symbol") {
+          dispose[Symbol.dispose] = dispose;
+        }
         return dispose;
       };
       _proto.toString = function toString() {
@@ -4684,8 +4689,8 @@
           if (!change) {
             return this;
           }
-          // ideally, value = change.value would be done here, so that values can be
-          // changed by interceptor. Same applies for other Set and Map api's.
+          // implemented reassignment same as it's done for ObservableMap
+          value = change.newValue;
         }
         if (!this.has(value)) {
           transaction(function () {
@@ -5740,6 +5745,8 @@
       return eq(a, b, depth);
     }
     // Copied from https://github.com/jashkenas/underscore/blob/5c237a7c682fb68fd5378203f0bf22dce1624854/underscore.js#L1186-L1289
+    // Modified: "Deep compare objects" part to iterate over keys in forward order instead of reverse order.
+    //
     // Internal recursive comparison function for `isEqual`.
     function eq(a, b, depth, aStack, bStack) {
       // Identical objects are equal. `0 === -0`, but they aren't identical.
@@ -5852,15 +5859,14 @@
       } else {
         // Deep compare objects.
         var keys = Object.keys(a);
-        var key;
-        length = keys.length;
+        var _length = keys.length;
         // Ensure that both objects contain the same number of properties before comparing deep equality.
-        if (Object.keys(b).length !== length) {
+        if (Object.keys(b).length !== _length) {
           return false;
         }
-        while (length--) {
+        for (var i = 0; i < _length; i++) {
           // Deep compare each member
-          key = keys[length];
+          var key = keys[i];
           if (!(hasProp(b, key) && eq(a[key], b[key], depth - 1, aStack, bStack))) {
             return false;
           }