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
Fixed #31051 -- Allowed dumpdata to handle circular references in natural keys. #12458
Conversation
0de0cc7
to
68d98d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@matthijskooijman Thanks 👍 Sorry for a delay but I'm still trying to split these changes into related/unrelated bits.
Rebased. |
I tried to move all tests that work without this patch to a separate PR #12680. |
Rebased on top of #12680. |
…ural keys. Since #26291 forward references in natural keys are properly handled by loaddata, so sorting depenencies in dumpdata doesn't need to break on cycles. This patch allows circular references in natural keys by breaking sort_depenencies() on loops.
…ign keys are not used. There is no need to sort dependencies when natural foreign keys are not used.
@matthijskooijman Thanks again 👍 I pushed minor edits and reorganized commits. |
This is a slightly cleaned up version of #12166, fixing just ticket-31051.