Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add a failing test which demonstrate the following problem: #400

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

sly7-7 commented Sep 28, 2012

Using the RESTAdapter, when creating a child record, then commit, the parent is not commited (which is ok), but its state remains in updated.inFlight.
As a result no other modifications on this parent record is possible.

NB: This issue was tracked with @tomdale, I hope he has not wasted his time tracking an imaginary bug...

sly7-7 added some commits Sep 28, 2012

Add a failing test which demonstrate the following problem:
Using the RESTAdapter, when creating a child record, then commit, the parent is not commited (which is ok), but its state remains in updated.inFlight.
As a result no other modifications on this parent record is possible.
Contributor

sly7-7 commented Oct 2, 2012

I'm quite sure the fix is not the good one, but it makes the test pass, and does'nt not break others. (BTW it works in my app)

Contributor

sly7-7 commented Oct 3, 2012

Seems like a duplicate of #381

mehulkar commented Oct 3, 2012

has it already been discussed somewhere if the parent should be committed when children are committed?

Contributor

sly7-7 commented Oct 3, 2012

Hmm, I don't know, do you encounter an issue within this situation ?

Contributor

sly7-7 commented Oct 15, 2012

Closing this, updated against mater

@sly7-7 sly7-7 closed this Oct 15, 2012

@sly7-7 So why this was closed? The issue still exists.

EDIT: Got it

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