@ngxs/store 3.8.1-dev.master-c0944d0 → 3.8.1-dev.master-84a340f

package/bundles/ngxs-store.umd.js

@@ -1,8 +1,8 @@
 (function (global, factory) {
-    typeof exports === 'object' && typeof module !== 'undefined' ? factory(exports, require('@angular/core'), require('@ngxs/store/internals'), require('@angular/common'), require('rxjs'), require('rxjs/operators')) :
-    typeof define === 'function' && define.amd ? define('@ngxs/store', ['exports', '@angular/core', '@ngxs/store/internals', '@angular/common', 'rxjs', 'rxjs/operators'], factory) :
-    (global = typeof globalThis !== 'undefined' ? globalThis : global || self, factory((global.ngxs = global.ngxs || {}, global.ngxs.store = {}), global.ng.core, global.ngxs.store.internals, global.ng.common, global.rxjs, global.rxjs.operators));
-})(this, (function (exports, i0, i5, common, rxjs, operators) { 'use strict';
+    typeof exports === 'object' && typeof module !== 'undefined' ? factory(exports, require('@angular/core'), require('@ngxs/store/internals'), require('@angular/common'), require('rxjs'), require('rxjs/operators'), require('@ngxs/store/operators')) :
+    typeof define === 'function' && define.amd ? define('@ngxs/store', ['exports', '@angular/core', '@ngxs/store/internals', '@angular/common', 'rxjs', 'rxjs/operators', '@ngxs/store/operators'], factory) :
+    (global = typeof globalThis !== 'undefined' ? globalThis : global || self, factory((global.ngxs = global.ngxs || {}, global.ngxs.store = {}), global.ng.core, global.ngxs.store.internals, global.ng.common, global.rxjs, global.rxjs.operators, global.ngxs.store.operators));
+})(this, (function (exports, i0, i5, common, rxjs, operators, operators$1) { 'use strict';
 
     function _interopNamespace(e) {
         if (e && e.__esModule) return e;
@@ -1723,43 +1723,21 @@
          */
         StateContextFactory.prototype.createStateContext = function (mappedStore) {
             var root = this._internalStateOperations.getRootStateOperations();
-            function getState(currentAppState) {
-                return getValue(currentAppState, mappedStore.path);
-            }
-            function setStateValue(currentAppState, newValue) {
-                var newAppState = setValue(currentAppState, mappedStore.path, newValue);
-                root.setState(newAppState);
-                return newAppState;
-                // In doing this refactoring I noticed that there is a 'bug' where the
-                // application state is returned instead of this state slice.
-                // This has worked this way since the beginning see:
-                // https://github.com/ngxs/store/blame/324c667b4b7debd8eb979006c67ca0ae347d88cd/src/state-factory.ts
-                // This needs to be fixed, but is a 'breaking' change.
-                // I will do this fix in a subsequent PR and we can decide how to handle it.
-            }
-            function setStateFromOperator(currentAppState, stateOperator) {
-                var local = getState(currentAppState);
-                var newValue = stateOperator(local);
-                return setStateValue(currentAppState, newValue);
-            }
-            function isStateOperator(value) {
-                return typeof value === 'function';
-            }
             return {
                 getState: function () {
                     var currentAppState = root.getState();
-                    return getState(currentAppState);
+                    return getState(currentAppState, mappedStore.path);
                 },
                 patchState: function (val) {
                     var currentAppState = root.getState();
                     var patchOperator = simplePatch(val);
-                    return setStateFromOperator(currentAppState, patchOperator);
+                    return setStateFromOperator(root, currentAppState, patchOperator, mappedStore.path);
                 },
                 setState: function (val) {
                     var currentAppState = root.getState();
-                    return isStateOperator(val)
-                        ? setStateFromOperator(currentAppState, val)
-                        : setStateValue(currentAppState, val);
+                    return operators$1.isStateOperator(val)
+                        ? setStateFromOperator(root, currentAppState, val, mappedStore.path)
+                        : setStateValue(root, currentAppState, val, mappedStore.path);
                 },
                 dispatch: function (actions) {
                     return root.dispatch(actions);
@@ -1774,6 +1752,25 @@
                 type: i0.Injectable,
                 args: [{ providedIn: 'root' }]
             }], ctorParameters: function () { return [{ type: InternalStateOperations }]; } });
+    function setStateValue(root, currentAppState, newValue, path) {
+        var newAppState = setValue(currentAppState, path, newValue);
+        root.setState(newAppState);
+        return newAppState;
+        // In doing this refactoring I noticed that there is a 'bug' where the
+        // application state is returned instead of this state slice.
+        // This has worked this way since the beginning see:
+        // https://github.com/ngxs/store/blame/324c667b4b7debd8eb979006c67ca0ae347d88cd/src/state-factory.ts
+        // This needs to be fixed, but is a 'breaking' change.
+        // I will do this fix in a subsequent PR and we can decide how to handle it.
+    }
+    function setStateFromOperator(root, currentAppState, stateOperator, path) {
+        var local = getState(currentAppState, path);
+        var newValue = stateOperator(local);
+        return setStateValue(root, currentAppState, newValue, path);
+    }
+    function getState(currentAppState, path) {
+        return getValue(currentAppState, path);
+    }
 
     var NG_DEV_MODE = typeof ngDevMode === 'undefined' || ngDevMode;
     /**
@@ -2838,12 +2835,14 @@
     function createModelSelector(selectorMap) {
         var selectorKeys = Object.keys(selectorMap);
         var selectors = Object.values(selectorMap);
-        ensureValidSelectorMap({
-            prefix: '[createModelSelector]',
-            selectorMap: selectorMap,
-            selectorKeys: selectorKeys,
-            selectors: selectors,
-        });
+        if (typeof ngDevMode === 'undefined' || ngDevMode) {
+            ensureValidSelectorMap({
+                prefix: '[createModelSelector]',
+                selectorMap: selectorMap,
+                selectorKeys: selectorKeys,
+                selectors: selectors
+            });
+        }
         return createSelector(selectors, function () {
             var args = [];
             for (var _i = 0; _i < arguments.length; _i++) {
@@ -2862,12 +2861,14 @@
         ensureValueProvided(selectorKeys.length, { prefix: prefix, noun: 'non-empty selector map' });
         selectors.forEach(function (selector, index) { return ensureValidSelector(selector, {
             prefix: prefix,
-            noun: "selector for the '" + selectorKeys[index] + "' property",
+            noun: "selector for the '" + selectorKeys[index] + "' property"
         }); });
     }
 
     function createPickSelector(selector, keys) {
-        ensureValidSelector(selector, { prefix: '[createPickSelector]' });
+        if (typeof ngDevMode === 'undefined' || ngDevMode) {
+            ensureValidSelector(selector, { prefix: '[createPickSelector]' });
+        }
         var validKeys = keys.filter(Boolean);
         var selectors = validKeys.map(function (key) { return createSelector([selector], function (s) { return s[key]; }); });
         return createSelector(__spreadArray([], __read(selectors)), function () {
@@ -2883,10 +2884,12 @@
     }
 
     function createPropertySelectors(parentSelector) {
-        ensureValidSelector(parentSelector, {
-            prefix: '[createPropertySelectors]',
-            noun: 'parent selector',
-        });
+        if (typeof ngDevMode === 'undefined' || ngDevMode) {
+            ensureValidSelector(parentSelector, {
+                prefix: '[createPropertySelectors]',
+                noun: 'parent selector'
+            });
+        }
         var cache = {};
         return new Proxy({}, {
             get: function (_target, prop) {
@@ -2894,7 +2897,7 @@
                     createSelector([parentSelector], function (s) { return s === null || s === void 0 ? void 0 : s[prop]; });
                 cache[prop] = selector;
                 return selector;
-            },
+            }
         });
     }