Prevent context matching if the oldContext is null

All existing tests still pass. It's not possible to test this particular issue without migrating the entire test suite off of Backburner.
tildeio · Feb 24, 2018 · b8571bd · b8571bd
1 parent 5b08deb
commit b8571bd
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/router/transition-intent/named-transition-intent.js b/lib/router/transition-intent/named-transition-intent.js
@@ -114,7 +114,11 @@ export default class NamedTransitionIntent extends TransitionIntent {
           newHandlerInfo.context
         );
         var oldContext = oldHandlerInfo && oldHandlerInfo.context;
-        if (result.names.length > 0 && newHandlerInfo.context === oldContext) {
+        if (
+          result.names.length > 0
+          && oldContext !== null
+          && newHandlerInfo.context === oldContext
+        ) {
           // If contexts match in isActive test, assume params also match.
           // This allows for flexibility in not requiring that every last
           // handler provide a `serialize` method