{"action":"created","issue":{"url":"https://api.github.com/repos/emberjs/ember.js/issues/9527","labels_url":"https://api.github.com/repos/emberjs/ember.js/issues/9527/labels{/name}","comments_url":"https://api.github.com/repos/emberjs/ember.js/issues/9527/comments","events_url":"https://api.github.com/repos/emberjs/ember.js/issues/9527/events","html_url":"https://github.com/emberjs/ember.js/pull/9527","id":48219959,"number":9527,"title":"RFC: Improved CP Syntax","user":{"login":"cibernox","id":265339,"avatar_url":"https://avatars.githubusercontent.com/u/265339?v=3","gravatar_id":"","url":"https://api.github.com/users/cibernox","html_url":"https://github.com/cibernox","followers_url":"https://api.github.com/users/cibernox/followers","following_url":"https://api.github.com/users/cibernox/following{/other_user}","gists_url":"https://api.github.com/users/cibernox/gists{/gist_id}","starred_url":"https://api.github.com/users/cibernox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cibernox/subscriptions","organizations_url":"https://api.github.com/users/cibernox/orgs","repos_url":"https://api.github.com/users/cibernox/repos","events_url":"https://api.github.com/users/cibernox/events{/privacy}","received_events_url":"https://api.github.com/users/cibernox/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"krisselden","id":61024,"avatar_url":"https://avatars.githubusercontent.com/u/61024?v=3","gravatar_id":"","url":"https://api.github.com/users/krisselden","html_url":"https://github.com/krisselden","followers_url":"https://api.github.com/users/krisselden/followers","following_url":"https://api.github.com/users/krisselden/following{/other_user}","gists_url":"https://api.github.com/users/krisselden/gists{/gist_id}","starred_url":"https://api.github.com/users/krisselden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krisselden/subscriptions","organizations_url":"https://api.github.com/users/krisselden/orgs","repos_url":"https://api.github.com/users/krisselden/repos","events_url":"https://api.github.com/users/krisselden/events{/privacy}","received_events_url":"https://api.github.com/users/krisselden/received_events","type":"User","site_admin":false},"milestone":null,"comments":55,"created_at":"2014-11-10T00:31:07Z","updated_at":"2015-05-29T15:14:57Z","closed_at":"2015-01-02T20:25:57Z","pull_request":{"url":"https://api.github.com/repos/emberjs/ember.js/pulls/9527","html_url":"https://github.com/emberjs/ember.js/pull/9527","diff_url":"https://github.com/emberjs/ember.js/pull/9527.diff","patch_url":"https://github.com/emberjs/ember.js/pull/9527.patch"},"body":"This PR implements https://github.com/emberjs/rfcs/pull/11\r\n\r\nIt is based on #9489, which will be merge soon. Also supersedes #9478 (I'll close it in a minute).\r\n\r\nThere is no feature flags for this because it mostly a refactor that does not change current semantics. It could be done though.\r\n\r\n### Tasks\r\n\r\nDone in #9489 \r\n\r\n- [x] Raise deprecation warnings when passing `cacheable` or `readOnly` to the constructor. No extrictly necessary, but is not used anywhere in the code since 2012 and simplifies things.\r\n- [x] Deprecate usage of `cacheable()` and `readOnly()`. Are the default.\r\n\r\nDone in this PR:\r\n\r\n- [x] Refactor CPs to work internally with setter and getters. Passing a function still works: If the function has arity < 2 its considered a getter. If arity is >=2 is considered both setter and getter. \r\n\r\nSteps in futures PRs:\r\n- [x] Use new cp syntax all iver the place in ember internals.\r\n- [x] Although passing functions with arity >= 2 is still allowed, is deprecated. Recommend user {get, set} instead.\r\n- [ ] Implement logic in https://github.com/emberjs/rfcs/pull/12\r\n- [ ] Deprecate all the things!\r\n- [ ] Ember 2.0. Delete all the things!\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/emberjs/ember.js/issues/comments/106842682","html_url":"https://github.com/emberjs/ember.js/pull/9527#issuecomment-106842682","issue_url":"https://api.github.com/repos/emberjs/ember.js/issues/9527","id":106842682,"user":{"login":"samselikoff","id":2922250,"avatar_url":"https://avatars.githubusercontent.com/u/2922250?v=3","gravatar_id":"","url":"https://api.github.com/users/samselikoff","html_url":"https://github.com/samselikoff","followers_url":"https://api.github.com/users/samselikoff/followers","following_url":"https://api.github.com/users/samselikoff/following{/other_user}","gists_url":"https://api.github.com/users/samselikoff/gists{/gist_id}","starred_url":"https://api.github.com/users/samselikoff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samselikoff/subscriptions","organizations_url":"https://api.github.com/users/samselikoff/orgs","repos_url":"https://api.github.com/users/samselikoff/repos","events_url":"https://api.github.com/users/samselikoff/events{/privacy}","received_events_url":"https://api.github.com/users/samselikoff/received_events","type":"User","site_admin":false},"created_at":"2015-05-29T15:14:57Z","updated_at":"2015-05-29T15:14:57Z","body":"yah I agree that guide needs to be updated, I keep looking at it for reference and forgetting about the return."}}