diff options
| author | Jonathan Biemond <jonbiemond@gmail.com> | 2025-12-18 14:43:59 +0100 |
|---|---|---|
| committer | Jacob Walls <jacobtylerwalls@gmail.com> | 2025-12-19 09:48:15 -0500 |
| commit | b172cbdf33c3eaea6a5c4a53da2ed0e46306a625 (patch) | |
| tree | 9e8300d0e134f9cd7f7a38c5ae1871b475799af7 | |
| parent | 73c987eb3b6a96300f7238dff32caf53cafe2098 (diff) | |
Fixed #36808 -- Required name argument in UniqueConstraint signature.
By trading ValueError for TypeError for omitted name arguments,
we gain a little clarity.
| -rw-r--r-- | AUTHORS | 1 | ||||
| -rw-r--r-- | django/db/models/constraints.py | 2 | ||||
| -rw-r--r-- | docs/ref/models/constraints.txt | 2 | ||||
| -rw-r--r-- | tests/constraints/tests.py | 2 |
4 files changed, 4 insertions, 3 deletions
@@ -551,6 +551,7 @@ answer newbie questions, and generally made Django that much better: Jonas Lundberg <jonas.lundberg@gmail.com> Jonathan Davis <jonathandavis47780@gmail.com> Jonatas C. D. <jonatas.cd@gmail.com> + Jonathan Biemond <jonathan.biemond@gmail.com> Jonathan Buchanan <jonathan.buchanan@gmail.com> Jonathan Daugherty (cygnus) <http://www.cprogrammer.org/> Jonathan Feignberg <jdf@pobox.com> diff --git a/django/db/models/constraints.py b/django/db/models/constraints.py index 4286319978..7dfeb3b649 100644 --- a/django/db/models/constraints.py +++ b/django/db/models/constraints.py @@ -265,7 +265,7 @@ class UniqueConstraint(BaseConstraint): self, *expressions, fields=(), - name=None, + name, condition=None, deferrable=None, include=None, diff --git a/docs/ref/models/constraints.txt b/docs/ref/models/constraints.txt index 9a51c5c7a2..36a79e1c0b 100644 --- a/docs/ref/models/constraints.txt +++ b/docs/ref/models/constraints.txt @@ -130,7 +130,7 @@ ensures the age field is never less than 18. ``UniqueConstraint`` ==================== -.. class:: UniqueConstraint(*expressions, fields=(), name=None, condition=None, deferrable=None, include=None, opclasses=(), nulls_distinct=None, violation_error_code=None, violation_error_message=None) +.. class:: UniqueConstraint(*expressions, fields=(), name, condition=None, deferrable=None, include=None, opclasses=(), nulls_distinct=None, violation_error_code=None, violation_error_message=None) Creates a unique constraint in the database. diff --git a/tests/constraints/tests.py b/tests/constraints/tests.py index f988121528..39b170125f 100644 --- a/tests/constraints/tests.py +++ b/tests/constraints/tests.py @@ -1462,7 +1462,7 @@ class UniqueConstraintTests(TestCase): def test_requires_name(self): msg = "A unique constraint must be named." with self.assertRaisesMessage(ValueError, msg): - models.UniqueConstraint(fields=["field"]) + models.UniqueConstraint(fields=["field"], name="") def test_database_default(self): models.UniqueConstraint( |