From 369fe0dfe5634a94305dfb6f4d8cc6ef7e5c5836 Mon Sep 17 00:00:00 2001
From: Peter Yee <github@houseofyee.com>
Date: Mon, 15 May 2023 08:35:54 -0700
Subject: [PATCH] fix: remove order from DocumentInfo as apparently unneeded
 (fixes #5245) (#5417)

* 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>
---
 .../0003_remove_document_info_order.py        | 34 +++++++++++++++++++
 ietf/doc/models.py                            |  1 -
 2 files changed, 34 insertions(+), 1 deletion(-)
 create mode 100644 ietf/doc/migrations/0003_remove_document_info_order.py

diff --git a/ietf/doc/migrations/0003_remove_document_info_order.py b/ietf/doc/migrations/0003_remove_document_info_order.py
new file mode 100644
index 000000000..085f8df87
--- /dev/null
+++ b/ietf/doc/migrations/0003_remove_document_info_order.py
@@ -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)]
diff --git a/ietf/doc/models.py b/ietf/doc/models.py
index ee56c0dc3..2e6db02a6 100644
--- a/ietf/doc/models.py
+++ b/ietf/doc/models.py
@@ -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)