fix(django): resolve `RemovedInDjango60Warning: CheckConstraint.check…

… is deprecated`
zyv · Aug 20, 2024 · 0405886 · 0405886
1 parent 49159a7
commit 0405886
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 10 deletions.
diff --git a/logbook/migrations/0004_logentry_arrival_after_departure.py b/logbook/migrations/0004_logentry_arrival_after_departure.py
@@ -13,7 +13,7 @@ class Migration(migrations.Migration):
         migrations.AddConstraint(
             model_name="logentry",
             constraint=models.CheckConstraint(
-                check=models.Q(("arrival_time__gt", django.db.models.expressions.F("departure_time"))),
+                condition=models.Q(("arrival_time__gt", django.db.models.expressions.F("departure_time"))),
                 name="arrival_after_departure",
             ),
         ),

diff --git a/logbook/migrations/0005_logentry_copilot_not_pilot.py b/logbook/migrations/0005_logentry_copilot_not_pilot.py
@@ -13,7 +13,7 @@ class Migration(migrations.Migration):
         migrations.AddConstraint(
             model_name="logentry",
             constraint=models.CheckConstraint(
-                check=models.Q(("copilot", django.db.models.expressions.F("pilot")), _negated=True),
+                condition=models.Q(("copilot", django.db.models.expressions.F("pilot")), _negated=True),
                 name="copilot_not_pilot",
             ),
         ),

diff --git a/logbook/migrations/0009_logentry_no_pic_no_xc.py b/logbook/migrations/0009_logentry_no_pic_no_xc.py
@@ -12,7 +12,7 @@ class Migration(migrations.Migration):
         migrations.AddConstraint(
             model_name="logentry",
             constraint=models.CheckConstraint(
-                check=models.Q(
+                condition=models.Q(
                     ("time_function", "PIC"),
                     models.Q(models.Q(("time_function", "PIC"), _negated=True), ("cross_country", False)),
                     _connector="OR",

diff --git a/logbook/migrations/0015_certificate_validity_after_issue.py b/logbook/migrations/0015_certificate_validity_after_issue.py
@@ -12,7 +12,7 @@ class Migration(migrations.Migration):
         migrations.AddConstraint(
             model_name="certificate",
             constraint=models.CheckConstraint(
-                check=models.Q(
+                condition=models.Q(
                     ("valid_until__isnull", True),
                     ("valid_until__gt", models.F("issue_date")),
                     _connector="OR",

diff --git a/logbook/migrations/0020_certificate_supersedes_self.py b/logbook/migrations/0020_certificate_supersedes_self.py
@@ -12,7 +12,7 @@ class Migration(migrations.Migration):
         migrations.AddConstraint(
             model_name="certificate",
             constraint=models.CheckConstraint(
-                check=models.Q(("id", models.F("supersedes")), _negated=True),
+                condition=models.Q(("id", models.F("supersedes")), _negated=True),
                 name="supersedes_self",
             ),
         ),

diff --git a/logbook/models/log_entry.py b/logbook/models/log_entry.py
@@ -45,10 +45,10 @@ class LogEntry(models.Model):
 
     class Meta:
         constraints = (
-            CheckConstraint(check=Q(arrival_time__gt=F("departure_time")), name="arrival_after_departure"),
-            CheckConstraint(check=~Q(copilot=F("pilot")), name="copilot_not_pilot"),
+            CheckConstraint(condition=Q(arrival_time__gt=F("departure_time")), name="arrival_after_departure"),
+            CheckConstraint(condition=~Q(copilot=F("pilot")), name="copilot_not_pilot"),
             CheckConstraint(
-                check=(
+                condition=(
                     Q(time_function=FunctionType.PIC)  # PIC time may be XC or not XC
                     | ~Q(time_function=FunctionType.PIC) & Q(cross_country=False)  # non-PIC time must be non-XC
                 ),

diff --git a/logbook/models/pilot.py b/logbook/models/pilot.py
@@ -44,11 +44,11 @@ class Certificate(models.Model):
     class Meta:
         constraints = (
             CheckConstraint(
-                check=Q(valid_until__isnull=True) | Q(valid_until__gt=F("issue_date")),
+                condition=Q(valid_until__isnull=True) | Q(valid_until__gt=F("issue_date")),
                 name="validity_after_issue",
             ),
             CheckConstraint(
-                check=~Q(id=F("supersedes")),
+                condition=~Q(id=F("supersedes")),
                 name="supersedes_self",
             ),
             UniqueConstraint(