fix: add model parents to create unique_constraint_by_fields

s-aleshin · s-aleshin · commit 58e856527cde · 2025-08-20T13:52:33.000+04:00
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py
@@ -1607,7 +1607,9 @@ def get_unique_together_validators(self):
             source_map[source].append(name)
 
         unique_constraint_by_fields = {
-            constraint.fields: constraint for constraint in self.Meta.model._meta.constraints
+            constraint.fields: constraint
+            for model_cls in (self.Meta.model, *self.Meta.model._meta.parents)
+            for constraint in model_cls._meta.constraints
             if isinstance(constraint, models.UniqueConstraint)
         }
 
diff --git a/tests/test_validators.py b/tests/test_validators.py
@@ -627,13 +627,7 @@ class Meta:
                 fields=("username", "company_id"),
                 name="unique_username_company_custom_msg",
                 violation_error_message="Username must be unique within a company.",
-                violation_error_code="duplicate_username",
-            )
-            if django_version[0] >= 5
-            else models.UniqueConstraint(
-                fields=("username", "company_id"),
-                name="unique_username_company_custom_msg",
-                violation_error_message="Username must be unique within a company.",
+                **(dict(violation_error_code="duplicate_username") if django_version[0] >= 5 else {}),
             ),
             models.UniqueConstraint(
                 fields=("company_id", "role"),

Original file line number	Diff line number	Diff line change
`@@ -1607,7 +1607,9 @@ def get_unique_together_validators(self):`
`1607`	`1607`	`source_map[source].append(name)`
`1608`	`1608`
`1609`	`1609`	`unique_constraint_by_fields = {`
`1610`		`- constraint.fields: constraint for constraint in self.Meta.model._meta.constraints`
	`1610`	`+ constraint.fields: constraint`
	`1611`	`+ for model_cls in (self.Meta.model, *self.Meta.model._meta.parents)`
	`1612`	`+ for constraint in model_cls._meta.constraints`
`1611`	`1613`	`if isinstance(constraint, models.UniqueConstraint)`
`1612`	`1614`	`}`
`1613`	`1615`