Merge pull request #14087 from trentmwillis/lazy-handler

[BUGFIX beta] Check that handler exists before triggering event

packages/ember-routing/lib/system/router.js

@@ -1059,7 +1059,7 @@ export function triggerEvent(handlerInfos, ignoreFailure, args) {
     handlerInfo = handlerInfos[i];
     handler = handlerInfo.handler;
 
-    if (handler.actions && handler.actions[name]) {
+    if (handler && handler.actions && handler.actions[name]) {
       if (handler.actions[name].apply(handler, args) === true) {
         eventWasHandled = true;
       } else {

packages/ember-routing/tests/system/router_test.js

@@ -2,7 +2,7 @@ import HashLocation from 'ember-routing/location/hash_location';
 import HistoryLocation from 'ember-routing/location/history_location';
 import AutoLocation from 'ember-routing/location/auto_location';
 import NoneLocation from 'ember-routing/location/none_location';
-import Router from 'ember-routing/system/router';
+import Router, { triggerEvent } from 'ember-routing/system/router';
 import { runDestroy } from 'ember-runtime/tests/utils';
 import buildOwner from 'container/tests/test-helpers/build-owner';
 import { setOwner } from 'container/owner';
@@ -192,3 +192,67 @@ QUnit.test('Router#handleURL should remove any #hashes before doing URL transiti
 
   router.handleURL('/foo/bar?time=morphin#pink-power-ranger');
 });
+
+QUnit.test('Router#triggerEvent allows actions to bubble when returning true', function(assert) {
+  assert.expect(2);
+
+  let handlerInfos = [
+    {
+      name: 'application',
+      handler: {
+        actions: {
+          loading() {
+            assert.ok(false, 'loading not handled by application route');
+          }
+        }
+      }
+    },
+    {
+      name: 'about',
+      handler: {
+        actions: {
+          loading() {
+            assert.ok(true, 'loading handled by about route');
+            return false;
+          }
+        }
+      }
+    },
+    {
+      name: 'about.me',
+      handler: {
+        actions: {
+          loading() {
+            assert.ok(true, 'loading handled by about.me route');
+            return true;
+          }
+        }
+      }
+    }
+  ];
+
+  triggerEvent(handlerInfos, false, ['loading']);
+});
+
+QUnit.test('Router#triggerEvent ignores handlers that have not loaded yet', function(assert) {
+  assert.expect(1);
+
+  let handlerInfos = [
+    {
+      name: 'about',
+      handler: {
+        actions: {
+          loading() {
+            assert.ok(true, 'loading handled by about route');
+          }
+        }
+      }
+    },
+    {
+      name: 'about.me',
+      handler: undefined
+    }
+  ];
+
+  triggerEvent(handlerInfos, false, ['loading']);
+});