Skip to content
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

Importing detached entities maintains their links to related entities #75

Open
bbobrian opened this issue Feb 18, 2015 · 2 comments
Open

Comments

@bbobrian
Copy link

See discussion:
http://stackoverflow.com/questions/28098520/breeze-entitymanager-not-removing-imported-detached-entites

@wardbell
Copy link
Member

Thanks for uncovering a bug. One shouldn't be able to export or import a detached entity. In future, Breeze will throw if you attempt to do either. That's different behavior from what your pull request would do. I've closed that request, grateful for your efforts but with the understanding that we cannot follow your lead on this one.

I'll keep the issue open until we actually fix the bug.

Back on your S.O. question I address the need that prompted your discovery.

@rastographics
Copy link

@wardbell Could we re-visit the idea of putting your solution for this problem in the core? For me at least, this is a common function I'm using all the time.

Would be great to have a method on entityManager like .importSaveResult(sandboxEm,saveResult)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants