Updating records: /put doesn't send record id

[gopeter]

I'm trying to update records over my API, but Ember doesn't send the ID of the record.
I'm using Ember 1.2.0 and Ember Data 1.0.0 Beta 4
Thats my code:

this.store.find('customer', 1).then(function(data) {
      data.set('id', 1);
      data.set('name', data.name);
      data.set('initial', data.initial);
      data.set('description', data.description);
      data.save();
});

Now Ember sends a put request to my API, but sends this data only:

customer:{name:11, initial:11, description:null}}

But my API needs the ID to update the correct record. Shouldn't Ember Data always include the ID when sending put requests?

[recipher]

Shouldn't the api call be something like /customer/:id ? And you can pull
the id from the URL, rather than the body?

On 5 December 2013 14:50, Peter Göbel notifications@github.com wrote:

I'm trying to update records over my API, but Ember doesn't send the ID of
the record.
I'm using Ember 1.2.0 and Ember Data 1.0.0 Beta 4
Thats my code:

this.store.find('customer', 1).then(function(data) {
data.set('id', 1);
data.set('name', data.name);
data.set('initial', data.initial);
data.set('description', data.description);
data.save();
});

Now Ember sends a put request to my API, but sends this data only:

customer:{name:11, initial:11, description:null}}

But my API needs the ID to update the correct record. Shouldn't Ember Data
always include the ID when sending put requests?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1548
.

[gopeter]

Oh my god, sorry, I have to take a break now :)

[otac0n]

Well, some existing APIs need the ID to come in, and I'd rather not proxy the request. Is it possible to make this an extend() option on RESTAdapter?

[bmac]

@otac0n The easiest way to force Ember Data to always include the id would be to extend the RESTSerializer like this:

App.ApplicationSerializer = DS.RESTSerializer.extend({
  serialize: function(record, options) {
    options = options || {includeId: true};
    return this._super(record, options);
  }
});