Migration to Django 2.2 - missing on_delete - django.urls import reverse

1 files changed, 19 insertions, 6 deletions

diff --git a/archaeological_finds/models_finds.py b/archaeological_finds/models_finds.py
index 2c2fffd00..63afb9dff 100644
--- a/archaeological_finds/models_finds.py
+++ b/archaeological_finds/models_finds.py
@@ -25,11 +25,11 @@ from django.apps import apps
 from django.conf import settings
 from django.contrib.gis.db import models
 from django.contrib.postgres.indexes import GinIndex
-from django.core.urlresolvers import reverse
 from django.db import connection
 from django.db.models import Max, Q, F
 from django.db.models.signals import m2m_changed, post_save, post_delete, pre_delete
 from django.core.exceptions import ObjectDoesNotExist
+from django.urls import reverse
 from ishtar_common.utils import ugettext_lazy as _, pgettext_lazy
 
 from ishtar_common.data_importer import post_importer_action, ImporterError
@@ -407,7 +407,10 @@ class BaseFind(
         _("Special interest"), blank=True, default="", max_length=120
     )
     context_record = models.ForeignKey(
-        ContextRecord, related_name="base_finds", verbose_name=_("Context Record")
+        ContextRecord,
+        related_name="base_finds",
+        verbose_name=_("Context Record"),
+        on_delete=models.CASCADE,
     )
     discovery_date = models.DateField(
         _("Discovery date (exact or TPQ)"), blank=True, null=True
@@ -3244,12 +3247,17 @@ class FindInsideContainer(models.Model):
     SLUG = "find_inside_container"
 
     find = models.OneToOneField(
-        Find, verbose_name=_("Find"), related_name="inside_container", primary_key=True
+        Find,
+        verbose_name=_("Find"),
+        related_name="inside_container",
+        primary_key=True,
+        on_delete=models.DO_NOTHING,
     )
     container = models.ForeignKey(
         "archaeological_warehouse.Container",
         verbose_name=_("Container"),
         related_name="container_content",
+        on_delete=models.DO_NOTHING,
     )
 
     class Meta:
@@ -3262,12 +3270,17 @@ for attr in Find.HISTORICAL_M2M:
 
 
 class Property(LightHistorizedItem):
-    find = models.ForeignKey(Find, verbose_name=_("Find"))
+    find = models.ForeignKey(Find, verbose_name=_("Find"), on_delete=models.CASCADE)
     administrative_act = models.ForeignKey(
-        AdministrativeAct, verbose_name=_("Administrative act")
+        AdministrativeAct,
+        verbose_name=_("Administrative act"),
+        on_delete=models.CASCADE,
     )
     person = models.ForeignKey(
-        Person, verbose_name=_("Person"), related_name="properties"
+        Person,
+        verbose_name=_("Person"),
+        related_name="properties",
+        on_delete=models.CASCADE,
     )
     start_date = models.DateField(_("Start date"))
     end_date = models.DateField(_("End date"))