Fixed #29854 -- Made _all_related_fields() return deterministically ordered fields.

Thanks to Rick Yang and Baptiste Mispelon for the investigation.
This commit is contained in:
Collin Anderson 2022-04-26 14:15:26 -04:00 committed by Mariusz Felisiak
parent c5fd5e3cc3
commit 3b898ea61e
1 changed files with 10 additions and 5 deletions

View File

@ -1,4 +1,5 @@
import logging import logging
import operator
from datetime import datetime from datetime import datetime
from django.db.backends.ddl_references import ( from django.db.backends.ddl_references import (
@ -35,11 +36,15 @@ def _is_relevant_relation(relation, altered_field):
def _all_related_fields(model): def _all_related_fields(model):
return model._meta._get_fields( # Related fields must be returned in a deterministic order.
forward=False, return sorted(
reverse=True, model._meta._get_fields(
include_hidden=True, forward=False,
include_parents=False, reverse=True,
include_hidden=True,
include_parents=False,
),
key=operator.attrgetter("name"),
) )