Skip to content

Deprecate Ember.ObjectController.#10062

Merged
rwjblue merged 1 commit intoemberjs:masterfrom
rwjblue:deprecate-object-controller
Jan 2, 2015
Merged

Deprecate Ember.ObjectController.#10062
rwjblue merged 1 commit intoemberjs:masterfrom
rwjblue:deprecate-object-controller

Conversation

@rwjblue
Copy link
Member

@rwjblue rwjblue commented Dec 28, 2014

  • Deprecate on init of Ember.ObjectController's when not auto-generated.
  • Deprecate on get of a proxied property.
  • Deprecate on set of a proxied property.

@rwjblue rwjblue force-pushed the deprecate-object-controller branch from 1daec3e to 833460b Compare December 28, 2014 05:24
@mixonic mixonic mentioned this pull request Dec 28, 2014
@rwjblue rwjblue force-pushed the deprecate-object-controller branch from 833460b to 0bca015 Compare December 28, 2014 18:41
@rwjblue rwjblue force-pushed the deprecate-object-controller branch from 0bca015 to 8b6c8e8 Compare January 2, 2015 02:56
@rwjblue
Copy link
Member Author

rwjblue commented Jan 2, 2015

Rebased.

@rwjblue
Copy link
Member Author

rwjblue commented Jan 2, 2015

Submitted PR to Ghost (TryGhost/Ghost#4748) showing the upgrade path. The following are the key upgrade concepts:

  • Update template usage to reference {{model.propertyName}} instead of {{propertyName}}.
  • Update CP and Observer dependent keys to use model.propertyName.
  • Update route and controller access to ensure model.propertyName is used for any get or set operations.

* Deprecate on `init` of `Ember.ObjectController`'s when not
  auto-generated.
* Deprecate on `get` of a proxied property.
* Deprecate on `set` of a proxied property.
@rwjblue rwjblue force-pushed the deprecate-object-controller branch from 093aeed to 33042d9 Compare January 2, 2015 20:11
rwjblue added a commit that referenced this pull request Jan 2, 2015
@rwjblue rwjblue merged commit 3f78acc into emberjs:master Jan 2, 2015
@rwjblue rwjblue deleted the deprecate-object-controller branch January 2, 2015 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant