From cdb32affa7851af9fb062609ebd36a00276ba34e Mon Sep 17 00:00:00 2001 From: Dirk Doesburg Date: Wed, 21 Jun 2023 21:50:27 +0200 Subject: [PATCH] Fix EventDocument migration for real now --- website/events/admin/__init__.py | 1 + website/events/migrations/0068_eventdocument.py | 10 +++++----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/website/events/admin/__init__.py b/website/events/admin/__init__.py index e347f9963..6fc287029 100644 --- a/website/events/admin/__init__.py +++ b/website/events/admin/__init__.py @@ -1,4 +1,5 @@ # ruff: noqa: F403 +from .documents import * from .event import * from .external_event import * from .registration import * diff --git a/website/events/migrations/0068_eventdocument.py b/website/events/migrations/0068_eventdocument.py index f45d3dba9..18fb3b218 100644 --- a/website/events/migrations/0068_eventdocument.py +++ b/website/events/migrations/0068_eventdocument.py @@ -11,12 +11,12 @@ def populate_eventdocument(apps, schema_editor): NewEventDocument.objects.create( document_ptr=event_document.document_ptr, owner=event_document.owner, - # For some reason, django INSERTs the existing Document row again (not only - # the new EventDocument row), but without the `created` field, as it's not - # actually creating a new Document. The DB doesn't allow this, as INSERT - # statements must specify all columns, and `created` is not nullable. - # So we can manually set created explicitly to solve this. + # For some reason we need to copy all fields, because + # Django uses an INSERT query on the Document table. created=event_document.created, + file=event_document.file, + members_only=event_document.members_only, + name=event_document.name, )