Missing id warning

[kenjiqq]

There does not seem to be any errors or warning of you define a idAttribute that the resource you fetch from the server does not contain. Should probably be some warning that the id was not found. Right now it just doesn't inject any of the objects into the store so you just get an empty array from a filter query etc

[jmdobry]

Is this when you use DS.findAll?

DS.inject throws a RuntimeError if the idAttribute defined by the resource is not present in the attrs argument. DS.inject is used internally by find, findAll, create, save, and update. Are you not seeing any error message from the error thrown by DS.inject?

Perhaps you could provide some sample code or a Plunker/JSBin/JSFiddle that demonstrates the issue?

[kenjiqq]

You figured it out? Was just trying to make a jsbin to show it

[jmdobry]

In any case, I verified that the error is thrown, and added a $log.error if for some reason the thrown error is not printed by the console. In fact, if a response to a DS.findAll call contains items that are missing the field specified by the resource's idAttribute, then the promise will be rejected, as shown in this test here https://github.com/jmdobry/angular-data/blob/master/test/integration/datastore/async_methods/findAll.test.js#L87

[jmdobry]

Fixed as of 0.10.1.

[kenjiqq]

http://jsbin.com/rogadaho/2/edit?html,js,console here is a jsBin that demonstrates the problem if you need it. No errors here, but if you change to the other id attribute it works fine

[jmdobry]

Thanks. If you change https://github.com/jmdobry/angular-data/releases/download/0.10.0/angular-data-0.10.0.js to https://github.com/jmdobry/angular-data/releases/download/0.10.1/angular-data-0.10.1.js in your JSBin you'll see the error reported to your console.

[kenjiqq]

Great!