Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Canary - generated controller using id not model.id #10144

Closed
zyllorion opened this issue Jan 5, 2015 · 7 comments
Closed

Canary - generated controller using id not model.id #10144

zyllorion opened this issue Jan 5, 2015 · 7 comments

Comments

@zyllorion
Copy link

DEPRECATION: You attempted to access `id` from `(generated container.detail controller)`, but object proxying is deprecated. Please use `model.id` instead.
        at __exports__.default.Mixin.create.unknownProperty (http://localhost:4200/assets/vendor.js:43071:17)
        at get (http://localhost:4200/assets/vendor.js:30251:22)
        at Object.merge.valueFn (http://localhost:4200/assets/vendor.js:53367:18)

Obviously there's no controller.js, but there is a view.js with computed model.

@stefanpenner
Copy link
Member

can you provide a JSBIN

@zyllorion
Copy link
Author

Not easily, would have to try and recreate from scratch, too much company logic and complex models :( Will see if I can recreate when I get a mo. It's only a warning at the moment so not a show stopper. Suspect it's just a case of adding model. in the generator code somewhere.
Happy to inspect and debug if you point me at what to look for :)

@stefanpenner
Copy link
Member

Without steps to reproduce and clarity of which versions you are using this issue isn't very actionable.

@zyllorion
Copy link
Author

Sorry, thought canary would be enough :)

DEBUG: Ember : 1.11.0-beta.1+canary.111861f0
DEBUG: Ember Data : 1.0.0-beta.14.1
DEBUG: jQuery : 1.11.2

Will take a quick look in the source to try and find the generator code then try and mock something up to reproduce.

@rwjblue
Copy link
Member

rwjblue commented Jan 5, 2015

Duplicate of #10143.

@rwjblue rwjblue closed this as completed Jan 5, 2015
@rwjblue
Copy link
Member

rwjblue commented Jan 5, 2015

Your container.detail template is using id instead of model.id (as the message indicates).

@zyllorion
Copy link
Author

Aha, it was a nested component. Could the error message be enhanced to name the view/component with error? Currently it says it's the controller which is a bit misleading.
Thanks for the help!

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

No branches or pull requests

3 participants