Skip to content
Snippets Groups Projects
Verified Commit 95b4ee28 authored by Nik | Klampfradler's avatar Nik | Klampfradler
Browse files

Fix unique constraint on LessonDocumentation

parent c3b5d5d5
No related branches found
No related tags found
1 merge request!225Resolve "Lesson documentations are not unique [Ticket#75177]"
Pipeline #28179 failed
......@@ -14,6 +14,7 @@ Fixed
* The _Delete personal note_ action didn't work due to wrong usage of ``bulk_update``.
* Groups and persons were shown multiple times in some forms due to filtering by permissions.
* Unique constraint on lesson documentations did not work and caused racey duplicates.
`2.0rc4`_ - 2021-08-01
----------------------
......
# Generated by Django 3.2.3 on 2021-08-20 12:38
from django.db import migrations, models
class Migration(migrations.Migration):
dependencies = [
('alsijil', '0013_fix_uniqueness_per_site'),
]
operations = [
migrations.RemoveConstraint(
model_name='lessondocumentation',
name='unique_documentation_per_object',
),
migrations.AddConstraint(
model_name='lessondocumentation',
constraint=models.UniqueConstraint(fields=('week', 'year', 'lesson_period'), name='unique_documentation_per_lp'),
),
migrations.AddConstraint(
model_name='lessondocumentation',
constraint=models.UniqueConstraint(fields=('week', 'year', 'event'), name='unique_documentation_per_ev'),
),
migrations.AddConstraint(
model_name='lessondocumentation',
constraint=models.UniqueConstraint(fields=('week', 'year', 'extra_lesson'), name='unique_documentation_per_el'),
),
]
......@@ -373,8 +373,16 @@ class LessonDocumentation(RegisterObjectRelatedMixin, ExtensibleModel):
check=lesson_related_constraint_q, name="one_relation_only_lesson_documentation",
),
models.UniqueConstraint(
fields=("lesson_period", "week", "year", "event", "extra_lesson"),
name="unique_documentation_per_object",
fields=("week", "year", "lesson_period"),
name="unique_documentation_per_lp",
),
models.UniqueConstraint(
fields=("week", "year", "event"),
name="unique_documentation_per_ev",
),
models.UniqueConstraint(
fields=("week", "year", "extra_lesson"),
name="unique_documentation_per_el",
),
]
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment