-
Notifications
You must be signed in to change notification settings - Fork 164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for cyclic relationships in Manage #889
Comments
Those don't look like cyclic relationships to me. If they were cyclic the |
They are not cyclic in terms of object references, but they are in terms of data. We already support the object reference scenario: var person = new Person { Name = "Peter", Id = 1 };
person.Dog = new Dog();
person.Dog.Owner = person; That works just fine, because when we get to managing the owner, we can see that it's already been managed. Also, this is more of a "name":"peter",
"id":1,
"dog": {
"name":"Rex",
"owner": {
"id":1
}
} This will work just fine with Javascript because the second time we try to |
➤ nirinchev commented: Closing since it doesn't appear to have been a real-world issue. |
We don't support cyclic relationships when the same object is not represented by the same reference e.g.
Ideally,
Manage
should figure out that we've already persistedPerson(1)
and merge the properties instead of overwriting them each time it sees a new instance. Obviously, this raises the question what is the correct behavior when dealing with the following object:The text was updated successfully, but these errors were encountered: