Skip to content

Commit

Permalink
Merge pull request #16433 from emberjs/rwjblue-remove-content-alias
Browse files Browse the repository at this point in the history
Remove content alias
  • Loading branch information
rwjblue authored Mar 28, 2018
2 parents c6616bf + 8089eb5 commit 0fed741
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 21 deletions.
10 changes: 0 additions & 10 deletions packages/ember-runtime/lib/mixins/controller.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { Mixin } from 'ember-metal';
import { deprecatingAlias } from '../computed/computed_macros';
import ActionHandler from './action_handler';

/**
Expand Down Expand Up @@ -40,13 +39,4 @@ export default Mixin.create(ActionHandler, {
@public
*/
model: null,

/**
@private
*/
content: deprecatingAlias('model', {
id: 'ember-runtime.controller.content-alias',
until: '2.17.0',
url: 'https://emberjs.com/deprecations/v2.x/#toc_controller-content-alias'
})
});
11 changes: 0 additions & 11 deletions packages/ember-runtime/tests/controllers/controller_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -106,17 +106,6 @@ QUnit.module('Controller deprecations');

QUnit.module('Controller Content -> Model Alias');

QUnit.test('`content` is a deprecated alias of `model`', function(assert) {
assert.expect(2);
let controller = Controller.extend({
model: 'foo-bar'
}).create();

expectDeprecation(function () {
assert.equal(controller.get('content'), 'foo-bar', 'content is an alias of model');
});
});

QUnit.test('`content` is not moved to `model` when `model` is unset', function(assert) {
assert.expect(2);
let controller;
Expand Down

0 comments on commit 0fed741

Please sign in to comment.