embedded associations are now automatically added to the parent's transa...

[ghempton]

...ction

Should fix #686

[thingista]

Hi Gordon,

Sorry I might be confused... but how does it work to fix #686? Does it mean I have to explicitly create a transaction? I mean for instance, if I have a very simple model+controller like this:

App.Solution = DS.Model.extend({
sequences: DS.hasMany('App.Sequence', { embedded:'always' }),
});

App.SolutionsNewController = Ember.ObjectController.extend({
create: function(){
this.store.commit()
}
});

"create" still fires two POSTS to the server, even if the first one already sends all the necessary data (both solution and sequences data).

Thanks a lot
PJ

[ghempton]

@thingista, yeah good point, I don't handle the default transaction here. This just solves for explicitly adding the parent to a transaction in which case all embedded records will be added as well.

[bobbus]

@thingista , I think you are talking about issue #437 and #440 , for the #686 it should be fixed with this patch.
Thanks @ghempton !

[darthdeus]

@ghempton this doesn't seem to merge without conflicts anymore :\

[ghempton]

@darthdeus I just rebased, should work now