From 023c228706c560797b7e9f98741ed1a890170ec2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-S=C3=A9bastien=20Caux?= <git@jscaux.org> Date: Tue, 12 Mar 2024 18:23:02 +0100 Subject: [PATCH] Use `SET_NULL` rather than `CASCADE` for SubsidyPayment attachments --- .../migrations/0030_auto_20240312_1822.py | 24 +++++++++++++++++++ scipost_django/finances/models.py | 4 ++-- 2 files changed, 26 insertions(+), 2 deletions(-) create mode 100644 scipost_django/finances/migrations/0030_auto_20240312_1822.py diff --git a/scipost_django/finances/migrations/0030_auto_20240312_1822.py b/scipost_django/finances/migrations/0030_auto_20240312_1822.py new file mode 100644 index 000000000..c8e6dec70 --- /dev/null +++ b/scipost_django/finances/migrations/0030_auto_20240312_1822.py @@ -0,0 +1,24 @@ +# Generated by Django 3.2.18 on 2024-03-12 17:22 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('finances', '0029_subsidyattachment_git_url_and_more'), + ] + + operations = [ + migrations.AlterField( + model_name='subsidypayment', + name='invoice', + field=models.OneToOneField(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='invoice_for', to='finances.subsidyattachment'), + ), + migrations.AlterField( + model_name='subsidypayment', + name='proof_of_payment', + field=models.OneToOneField(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='proof_of_payment_for', to='finances.subsidyattachment'), + ), + ] diff --git a/scipost_django/finances/models.py b/scipost_django/finances/models.py index 931f24a4f..e80b06160 100644 --- a/scipost_django/finances/models.py +++ b/scipost_django/finances/models.py @@ -150,14 +150,14 @@ class SubsidyPayment(models.Model): date_scheduled = models.DateField() invoice = models.OneToOneField( "finances.SubsidyAttachment", - on_delete=models.CASCADE, + on_delete=models.SET_NULL, blank=True, null=True, related_name="invoice_for", ) proof_of_payment = models.OneToOneField( "finances.SubsidyAttachment", - on_delete=models.CASCADE, + on_delete=models.SET_NULL, blank=True, null=True, related_name="proof_of_payment_for", -- GitLab