Various fixes for loading embedded records. #541

Closed
wants to merge 7 commits into from

5 participants

@chunkerchunker

This fixes the following issues:

(I realize there's a lot of work going on embedded records on another branch, but this enables us to use the current functionality with the current master..)

ebryn and others added some commits Dec 8, 2012
@ebryn ebryn Update Ember and update a test to use createWithMixins. 351cb9c
@ebryn ebryn Fix typo 486c40a
@lukemelia lukemelia Define Ember's new `toStringExtension` on `DS.Model` to include the r…
…ecord's `id` property in `toString` output
3123f1e
@ebryn ebryn Update Ember again, replace a couple usages of create with createWith…
…Mixins.
c2fbee1
Alex Kwiatkowski & Ian Lesperance Add hasOne association
Models can now define a hasOne association as the inverse of a belongsTo
association:

  App.Person = DS.Model.extend({
    heart: DS.hasOne(App.Heart)
  });

  App.Heart = DS.Model.extend({
    person: DS.belongsTo(App.Person)
  });

Includes updates for DS.RESTSerializer to parse hasOne information out
of incoming record data but not include it with outgoing data.
d81bde6
@chunkerchunker chunkerchunker Fixes for loading embedded records.
This fixes the following issues:
- emberjs#540
- emberjs#523
- emberjs#504
- emberjs#516

(I realize there's a lot of work going on embedded records on another branch, but this enables us to use the current functionality with the current master..)
b3dfd3c
@ivanvanderbyl

I just tried this patch out and confirmed it resolves #504 and thus #523

@chunkerchunker chunkerchunker Merge commit 'd81bde6' from mhelabs
Conflicts:
	packages/ember-data/tests/integration/associations_test.js
15c472c
@wagenet
Ember.js member

@chunkerchunker This doesn't merge cleanly and it also looks like you caught some other commits. Can you rebase on master?

@chunkerchunker

Sorry, i should have closed this earlier. This is no longer relevant, given the merge of embedded-records into master (which i haven't had a chance yet to kick the tires on).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment