Merge pull request #12188 from rwjblue/prevent-issue-with-_actions

[BUGFIX release] Ensure `_actions` specified to extend works.
emberjs · Aug 24, 2015 · c2c1079 · c2c1079
2 parents fe9d4eb + dcc0dc3
commit c2c1079
Show file tree

Hide file tree

Showing 2 changed files with 66 additions and 3 deletions.
diff --git a/packages/ember-routing/tests/system/route_test.js b/packages/ember-routing/tests/system/route_test.js
@@ -213,6 +213,39 @@ QUnit.test('can access `actions` hash via `_actions` [DEPRECATED]', function() {
   }, 'Usage of `_actions` is deprecated, use `actions` instead.');
 });
 
+QUnit.test('actions in both `_actions` and `actions` results in an assertion', function() {
+  expectAssertion(function() {
+    EmberRoute.extend({
+      _actions: { },
+      actions: { }
+    }).create();
+  }, 'Specifying `_actions` and `actions` in the same mixin is not supported.');
+});
+
+QUnit.test('actions added via `_actions` can be used [DEPRECATED]', function() {
+  expect(3);
+
+  let route;
+  expectDeprecation(function() {
+    route = EmberRoute.extend({
+      _actions: {
+        bar: function() {
+          ok(true, 'called bar action');
+        }
+      }
+    }, {
+      actions: {
+        foo: function() {
+          ok(true, 'called foo action');
+        }
+      }
+    }).create();
+  }, 'Specifying actions in `_actions` is deprecated, please use `actions` instead.');
+
+  route.send('foo');
+  route.send('bar');
+});
+
 QUnit.module('Ember.Route serialize', {
   setup: setup,
   teardown: teardown

diff --git a/packages/ember-runtime/lib/mixins/action_handler.js b/packages/ember-runtime/lib/mixins/action_handler.js
@@ -5,7 +5,6 @@
 import Ember from 'ember-metal/core';
 import { Mixin } from 'ember-metal/mixin';
 import { get } from 'ember-metal/property_get';
-import { deprecateProperty } from 'ember-metal/deprecate_property';
 
 /**
   `Ember.ActionHandler` is available on some familiar classes including
@@ -185,13 +184,44 @@ var ActionHandler = Mixin.create({
                    ') does not have a `send` method', typeof target.send === 'function');
       target.send(...arguments);
     }
+  },
+
+  willMergeMixin(props) {
+    Ember.assert('Specifying `_actions` and `actions` in the same mixin is not supported.', !props.actions || !props._actions);
+
+    if (props._actions) {
+      Ember.deprecate(
+        'Specifying actions in `_actions` is deprecated, please use `actions` instead.',
+        false,
+        { id: 'ember-runtime.action-handler-_actions', until: '3.0.0' }
+      );
+
+      props.actions = props._actions;
+      delete props._actions;
+    }
   }
 });
 
 export default ActionHandler;
 
 export function deprecateUnderscoreActions(factory) {
-  deprecateProperty(factory.prototype, '_actions', 'actions', {
-    id: 'ember-runtime.action-handler-_actions', until: '3.0.0'
+  function deprecate() {
+    Ember.deprecate(
+      `Usage of \`_actions\` is deprecated, use \`actions\` instead.`,
+      false,
+      { id: 'ember-runtime.action-handler-_actions', until: '3.0.0' }
+    );
+  }
+
+  Object.defineProperty(factory.prototype, '_actions', {
+    configurable: true,
+    enumerable: false,
+    set(value) {
+      Ember.assert(`You cannot set \`_actions\` on ${this}, please use \`actions\` instead.`);
+    },
+    get() {
+      deprecate();
+      return get(this, 'actions');
+    }
   });
 }