altf4arnold/datatracker
1
0
Fork 0
Code Issues Pull requests Actions 5 Packages Projects Releases Wiki Activity

fix: remove order from DocumentInfo as apparently unneeded (fixes #5245) (#5417)

Browse source 
* fix: remove order from DocumentInfo as apparently unneeded (fixes #5245)

Fixes #5245. Includes migration, however I did not test the reverse.

* Merge branch 'main' into pr/Spectre17/5417

* chore: Renumber migrations

* Revert "Merge branch 'main' into pr/Spectre17/5417"

This reverts commit 77b4a46f84e9ece6778dec6e8127aedad71c2625.

---------

Co-authored-by: Jennifer Richards <jennifer@staff.ietf.org>
Co-authored-by: Robert Sparks <rjsparks@nostrum.com>
This commit is contained in:
Peter Yee 2023-05-15 08:35:54 -07:00 committed by GitHub
parent 6688cceb04
commit 369fe0dfe5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 34 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
ietf/doc/migrations/0003_remove_document_info_order.py Normal file
View file

@ -0,0 +1,34 @@
# Copyright The IETF Trust 2023, All Rights Reserved
from django.db import migrations, models
def forward(apps, schema_editor):
migrations.RemoveField(
model_name='dochistory',
name='order',
)
migrations.RemoveField(
model_name='document',
name='order',
)
def reverse(apps, schema_editor):
migrations.AddField(
model_name='dochistory',
name='order',
field = models.IntegerField(default=1, blank=True),
)
migrations.AddField(
model_name='document',
name='order',
)
class Migration(migrations.Migration):
dependencies = [
("doc", "0002_auto_20230320_1222"),
]
operations = [migrations.RunPython(forward, reverse)]

1
ietf/doc/models.py
View file

@ -113,7 +113,6 @@ class DocumentInfo(models.Model):
pages = models.IntegerField(blank=True, null=True)
words = models.IntegerField(blank=True, null=True)
formal_languages = models.ManyToManyField(FormalLanguageName, blank=True, help_text="Formal languages used in document")
order = models.IntegerField(default=1, blank=True) # This is probably obviated by SessionPresentaion.order
intended_std_level = ForeignKey(IntendedStdLevelName, verbose_name="Intended standardization level", blank=True, null=True)
std_level = ForeignKey(StdLevelName, verbose_name="Standardization level", blank=True, null=True)
ad = ForeignKey(Person, verbose_name="area director", related_name='ad_%(class)s_set', blank=True, null=True)