@angular/router 12.2.0-rc.0 → 12.2.3

package/bundles/router-testing.umd.js

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

package/bundles/router-upgrade.umd.js

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

package/bundles/router.umd.js

@@ -1,5 +1,5 @@
 /**
- * @license Angular v12.2.0-rc.0
+ * @license Angular v12.2.3
  * (c) 2010-2021 Google LLC. https://angular.io/
  * License: MIT
  */
@@ -4650,6 +4650,11 @@
                         t.extractedUrl.toString() !== _this.browserUrlTree.toString();
                     var processCurrentUrl = (_this.onSameUrlNavigation === 'reload' ? true : urlTransition) &&
                         _this.urlHandlingStrategy.shouldProcessUrl(t.rawUrl);
+                    // If the source of the navigation is from a browser event, the URL is
+                    // already updated. We already need to sync the internal state.
+                    if (isBrowserTriggeredNavigation(t.source)) {
+                        _this.browserUrlTree = t.rawUrl;
+                    }
                     if (processCurrentUrl) {
                         return rxjs.of(t).pipe(
                         // Fire NavigationStart event
@@ -4881,7 +4886,12 @@
                                 var mergedTree = _this.urlHandlingStrategy.merge(e.url, _this.rawUrlTree);
                                 var extras = {
                                     skipLocationChange: t.extras.skipLocationChange,
-                                    replaceUrl: _this.urlUpdateStrategy === 'eager'
+                                    // The URL is already updated at this point if we have 'eager' URL
+                                    // updates or if the navigation was triggered by the browser (back
+                                    // button, URL bar, etc). We want to replace that item in history if
+                                    // the navigation is rejected.
+                                    replaceUrl: _this.urlUpdateStrategy === 'eager' ||
+                                        isBrowserTriggeredNavigation(t.source)
                                 };
                                 _this.scheduleNavigation(mergedTree, 'imperative', null, extras, { resolve: t.resolve, reject: t.reject, promise: t.promise });
                             }, 0);
@@ -5267,7 +5277,8 @@
             var lastNavigation = this.getTransition();
             // We don't want to skip duplicate successful navs if they're imperative because
             // onSameUrlNavigation could be 'reload' (so the duplicate is intended).
-            var browserNavPrecededByRouterNav = source !== 'imperative' && (lastNavigation === null || lastNavigation === void 0 ? void 0 : lastNavigation.source) === 'imperative';
+            var browserNavPrecededByRouterNav = isBrowserTriggeredNavigation(source) && lastNavigation &&
+                !isBrowserTriggeredNavigation(lastNavigation.source);
             var lastNavigationSucceeded = this.lastSuccessfulId === lastNavigation.id;
             // If the last navigation succeeded or is in flight, we can use the rawUrl as the comparison.
             // However, if it failed, we should compare to the final result (urlAfterRedirects).
@@ -5438,6 +5449,9 @@
             }
         }
     }
+    function isBrowserTriggeredNavigation(source) {
+        return source !== 'imperative';
+    }
 
     /**
      * @license
@@ -5889,7 +5903,7 @@
             });
         };
         RouterLinkActive.prototype.isLinkActive = function (router) {
-            var options = 'paths' in this.routerLinkActiveOptions ?
+            var options = isActiveMatchOptions(this.routerLinkActiveOptions) ?
                 this.routerLinkActiveOptions :
                 // While the types should disallow `undefined` here, it's possible without strict inputs
                 (this.routerLinkActiveOptions.exact || false);
@@ -5923,6 +5937,12 @@
         routerLinkActiveOptions: [{ type: core.Input }],
         routerLinkActive: [{ type: core.Input }]
     };
+    /**
+     * Use instead of `'paths' in options` to be compatible with property renaming
+     */
+    function isActiveMatchOptions(options) {
+        return !!options.paths;
+    }
 
     /**
      * @license
@@ -6727,7 +6747,7 @@
     /**
      * @publicApi
      */
-    var VERSION = new core.Version('12.2.0-rc.0');
+    var VERSION = new core.Version('12.2.3');
 
     /**
      * @license