Fixes #98.

Stable Version 0.10.1.

CHANGELOG.md

@@ -1,4 +1,4 @@
-##### 0.10.1 - xx July 2014
+##### 0.10.1 - 20 July 2014
 
 ##### Backwards compatible API changes
 - #93 - Added `DS.createInstance(resourceName[, attrs][, options])`
@@ -9,6 +9,7 @@
 - #91 - dist/angular-data(.min).js doesn't end with a semicolon (upgraded Browserify)
 - #94 - Resource object name/class inconsistency (added `useClass` option to `DS.defineResource`)
 - #95 - observe-js outdated (Upgraded observe-js.js an refactored to new API)
+- #98 - Missing id warning
 
 ##### 0.10.0 - 18 July 2014
 

dist/angular-data.js

@@ -5020,7 +5020,9 @@ function _inject(definition, resource, attrs) {
       attrs[definition.idAttribute] = definition.computed[definition.idAttribute].apply(attrs, args);
     }
     if (!(definition.idAttribute in attrs)) {
-      throw new _this.errors.R(errorPrefix + 'attrs: Must contain the property specified by `idAttribute`!');
+      var error = new _this.errors.R(errorPrefix + 'attrs: Must contain the property specified by `idAttribute`!');
+      $log.error(error);
+      throw error;
     } else {
       try {
         definition.beforeInject(definition.name, attrs);

src/datastore/sync_methods/inject.js

@@ -56,7 +56,9 @@ function _inject(definition, resource, attrs) {
       attrs[definition.idAttribute] = definition.computed[definition.idAttribute].apply(attrs, args);
     }
     if (!(definition.idAttribute in attrs)) {
-      throw new _this.errors.R(errorPrefix + 'attrs: Must contain the property specified by `idAttribute`!');
+      var error = new _this.errors.R(errorPrefix + 'attrs: Must contain the property specified by `idAttribute`!');
+      $log.error(error);
+      throw error;
     } else {
       try {
         definition.beforeInject(definition.name, attrs);

test/integration/datastore/async_methods/findAll.test.js

@@ -84,6 +84,25 @@ describe('DS.findAll(resourceName, params[, options]): ', function () {
     assert.equal(lifecycle.serialize.callCount, 0, 'serialize should have been called');
     assert.equal(lifecycle.deserialize.callCount, 2, 'deserialize should have been called');
   });
+  it('should fail when no "idAttribute" is present on an item in the response', function () {
+    $httpBackend.expectGET(/http:\/\/test\.angular-cache\.com\/posts\??/).respond(200, [
+      { author: 'John', age: 30 },
+      { author: 'Sally', age: 31 }
+    ]);
+
+    DS.findAll('post', {}).then(function () {
+      fail('Should not have succeeded!');
+    }, function (err) {
+      assert(err.message, 'DS.inject(resourceName, attrs[, options]): attrs: Must contain the property specified by `idAttribute`!');
+      assert.deepEqual(DS.filter('post', {}), [], 'The posts should not be in the store');
+    });
+
+    $httpBackend.flush();
+
+    assert($log.error.logs[0][0].message, 'DS.inject(resourceName, attrs[, options]): attrs: Must contain the property specified by `idAttribute`!');
+    assert.equal(lifecycle.beforeInject.callCount, 0, 'beforeInject should not have been called');
+    assert.equal(lifecycle.afterInject.callCount, 0, 'afterInject should not have been called');
+  });
   it('should query the server for a collection but not store the data if cacheResponse is false', function () {
     $httpBackend.expectGET(/http:\/\/test\.angular-cache\.com\/posts\??/).respond(200, [p1, p2, p3, p4]);