@angular/router 12.1.1 → 12.1.2

package/bundles/router-testing.umd.js

@@ -1,5 +1,5 @@
 /**
- * @license Angular v12.1.1
+ * @license Angular v12.1.2
  * (c) 2010-2021 Google LLC. https://angular.io/
  * License: MIT
  */

package/bundles/router-upgrade.umd.js

@@ -1,5 +1,5 @@
 /**
- * @license Angular v12.1.1
+ * @license Angular v12.1.2
  * (c) 2010-2021 Google LLC. https://angular.io/
  * License: MIT
  */

package/bundles/router.umd.js

@@ -1,5 +1,5 @@
 /**
- * @license Angular v12.1.1
+ * @license Angular v12.1.2
  * (c) 2010-2021 Google LLC. https://angular.io/
  * License: MIT
  */
@@ -4472,6 +4472,10 @@
             /**
              * The id of the currently active page in the router.
              * Updated to the transition's target id on a successful navigation.
+             *
+             * This is used to track what page the router last activated. When an attempted navigation fails,
+             * the router can then use this to compute how to restore the state back to the previously active
+             * page.
              */
             this.currentPageId = 0;
             this.isNgZoneEnabled = false;
@@ -4604,6 +4608,19 @@
             this.navigations = this.setupNavigations(this.transitions);
             this.processNavigations();
         }
+        Object.defineProperty(Router.prototype, "browserPageId", {
+            /**
+             * The ɵrouterPageId of whatever page is currently active in the browser history. This is
+             * important for computing the target page id for new navigations because we need to ensure each
+             * page id in the browser history is 1 more than the previous entry.
+             */
+            get: function () {
+                var _a;
+                return (_a = this.location.getState()) === null || _a === void 0 ? void 0 : _a.ɵrouterPageId;
+            },
+            enumerable: false,
+            configurable: true
+        });
         Router.prototype.setupNavigations = function (transitions) {
             var _this = this;
             var eventsSubject = this.events;
@@ -4693,7 +4710,7 @@
                 }), 
                 // Before Preactivation
                 switchTap(function (t) {
-                    var targetSnapshot = t.targetSnapshot, navigationId = t.id, appliedUrlTree = t.extractedUrl, rawUrlTree = t.rawUrl, _b = t.extras, skipLocationChange = _b.skipLocationChange, replaceUrl = _b.replaceUrl;
+                    var targetSnapshot = t.targetSnapshot, navigationId = t.id, appliedUrlTree = t.extractedUrl, rawUrlTree = t.rawUrl, _c = t.extras, skipLocationChange = _c.skipLocationChange, replaceUrl = _c.replaceUrl;
                     return _this.hooks.beforePreactivation(targetSnapshot, {
                         navigationId: navigationId,
                         appliedUrlTree: appliedUrlTree,
@@ -4716,7 +4733,7 @@
                     _this.triggerEvent(guardsEnd);
                 }), operators.filter(function (t) {
                     if (!t.guardsResult) {
-                        _this.cancelNavigationTransition(t, '');
+                        _this.cancelNavigationTransitionRestoreHistory(t, '');
                         return false;
                     }
                     return true;
@@ -4733,7 +4750,7 @@
                                 next: function () { return dataResolved = true; },
                                 complete: function () {
                                     if (!dataResolved) {
-                                        _this.cancelNavigationTransition(t, "At least one route resolver didn't emit any value.");
+                                        _this.cancelNavigationTransitionRestoreHistory(t, "At least one route resolver didn't emit any value.");
                                     }
                                 }
                             }));
@@ -4746,7 +4763,7 @@
                 }), 
                 // --- AFTER PREACTIVATION ---
                 switchTap(function (t) {
-                    var targetSnapshot = t.targetSnapshot, navigationId = t.id, appliedUrlTree = t.extractedUrl, rawUrlTree = t.rawUrl, _b = t.extras, skipLocationChange = _b.skipLocationChange, replaceUrl = _b.replaceUrl;
+                    var targetSnapshot = t.targetSnapshot, navigationId = t.id, appliedUrlTree = t.extractedUrl, rawUrlTree = t.rawUrl, _c = t.extras, skipLocationChange = _c.skipLocationChange, replaceUrl = _c.replaceUrl;
                     return _this.hooks.afterPreactivation(targetSnapshot, {
                         navigationId: navigationId,
                         appliedUrlTree: appliedUrlTree,
@@ -4790,14 +4807,28 @@
                      * event is fired when a navigation gets cancelled but not caught by other
                      * means. */
                     if (!completed && !errored) {
-                        // Must reset to current URL tree here to ensure history.state is set. On a
-                        // fresh page load, if a new navigation comes in before a successful
-                        // navigation completes, there will be nothing in
-                        // history.state.navigationId. This can cause sync problems with AngularJS
-                        // sync code which looks for a value here in order to determine whether or
-                        // not to handle a given popstate event or to leave it to the Angular
-                        // router.
-                        _this.cancelNavigationTransition(t, "Navigation ID " + t.id + " is not equal to the current navigation id " + _this.navigationId);
+                        var cancelationReason = "Navigation ID " + t.id + " is not equal to the current navigation id " + _this.navigationId;
+                        if (_this.canceledNavigationResolution === 'replace') {
+                            // Must reset to current URL tree here to ensure history.state is set. On
+                            // a fresh page load, if a new navigation comes in before a successful
+                            // navigation completes, there will be nothing in
+                            // history.state.navigationId. This can cause sync problems with
+                            // AngularJS sync code which looks for a value here in order to determine
+                            // whether or not to handle a given popstate event or to leave it to the
+                            // Angular router.
+                            _this.cancelNavigationTransitionRestoreHistory(t, cancelationReason);
+                        }
+                        else {
+                            // We cannot trigger a `location.historyGo` if the
+                            // cancellation was due to a new navigation before the previous could
+                            // complete. This is because `location.historyGo` triggers a `popstate`
+                            // which would also trigger another navigation. Instead, treat this as a
+                            // redirect and do not reset the state.
+                            _this.cancelNavigationTransition(t, cancelationReason);
+                            // TODO(atscott): The same problem happens here with a fresh page load
+                            // and a new navigation before that completes where we won't set a page
+                            // id.
+                        }
                     }
                     // currentNavigation should always be reset to null here. If navigation was
                     // successful, lastSuccessfulTransition will have already been set. Therefore
@@ -4901,9 +4932,9 @@
             if (!this.locationSubscription) {
                 this.locationSubscription = this.location.subscribe(function (event) {
                     var currentChange = _this.extractLocationChangeInfoFromEvent(event);
+                    // The `setTimeout` was added in #12160 and is likely to support Angular/AngularJS
+                    // hybrid apps.
                     if (_this.shouldScheduleNavigation(_this.lastLocationChangeInfo, currentChange)) {
-                        // The `setTimeout` was added in #12160 and is likely to support Angular/AngularJS
-                        // hybrid apps.
                         setTimeout(function () {
                             var source = currentChange.source, state = currentChange.state, urlTree = currentChange.urlTree;
                             var extras = { replaceUrl: true };
@@ -5111,14 +5142,7 @@
             }
             var urlTree = isUrlTree(url) ? url : this.parseUrl(url);
             var mergedTree = this.urlHandlingStrategy.merge(urlTree, this.rawUrlTree);
-            var restoredState = null;
-            if (this.canceledNavigationResolution === 'computed') {
-                var isInitialPage = this.currentPageId === 0;
-                if (isInitialPage || extras.skipLocationChange || extras.replaceUrl) {
-                    restoredState = this.location.getState();
-                }
-            }
-            return this.scheduleNavigation(mergedTree, 'imperative', restoredState, extras);
+            return this.scheduleNavigation(mergedTree, 'imperative', null, extras);
         };
         /**
          * Navigate based on the provided array of commands and a starting point.
@@ -5211,6 +5235,7 @@
             });
         };
         Router.prototype.scheduleNavigation = function (rawUrl, source, restoredState, extras, priorPromise) {
+            var _a, _b;
             if (this.disposed) {
                 return Promise.resolve(false);
             }
@@ -5254,13 +5279,25 @@
             var id = ++this.navigationId;
             var targetPageId;
             if (this.canceledNavigationResolution === 'computed') {
+                var isInitialPage = this.currentPageId === 0;
+                if (isInitialPage) {
+                    restoredState = this.location.getState();
+                }
                 // If the `ɵrouterPageId` exist in the state then `targetpageId` should have the value of
-                // `ɵrouterPageId`
+                // `ɵrouterPageId`. This is the case for something like a page refresh where we assign the
+                // target id to the previously set value for that page.
                 if (restoredState && restoredState.ɵrouterPageId) {
                     targetPageId = restoredState.ɵrouterPageId;
                 }
                 else {
-                    targetPageId = this.currentPageId + 1;
+                    // If we're replacing the URL or doing a silent navigation, we do not want to increment the
+                    // page id because we aren't pushing a new entry to history.
+                    if (extras.replaceUrl || extras.skipLocationChange) {
+                        targetPageId = (_a = this.browserPageId) !== null && _a !== void 0 ? _a : 0;
+                    }
+                    else {
+                        targetPageId = ((_b = this.browserPageId) !== null && _b !== void 0 ? _b : 0) + 1;
+                    }
                 }
             }
             else {
@@ -5314,7 +5351,7 @@
          * - triggers the `NavigationCancel` event
          * - resolves the transition promise with `false`
          */
-        Router.prototype.cancelNavigationTransition = function (t, reason) {
+        Router.prototype.cancelNavigationTransitionRestoreHistory = function (t, reason) {
             if (this.canceledNavigationResolution === 'computed') {
                 // The navigator change the location before triggered the browser event,
                 // so we need to go back to the current url if the navigation is canceled.
@@ -5333,6 +5370,9 @@
             else {
                 this.resetUrlToCurrentUrlTree();
             }
+            this.cancelNavigationTransition(t, reason);
+        };
+        Router.prototype.cancelNavigationTransition = function (t, reason) {
             var navCancel = new NavigationCancel(t.id, this.serializeUrl(t.extractedUrl), reason);
             this.triggerEvent(navCancel);
             t.resolve(false);
@@ -6655,7 +6695,7 @@
     /**
      * @publicApi
      */
-    var VERSION = new core.Version('12.1.1');
+    var VERSION = new core.Version('12.1.2');
 
     /**
      * @license