Hotfix: Production server is failing to migrate cards with missing data

nfoert · Oct 5, 2024 · 3e7a7c3 · 3e7a7c3
1 parent f075d42
commit 3e7a7c3
Show file tree

Hide file tree

Showing 4 changed files with 45 additions and 41 deletions.
diff --git a/cardie/main/migrations/0010_migrate_card_v0_to_v1.py b/cardie/main/migrations/0010_migrate_card_v0_to_v1.py
@@ -9,30 +9,31 @@ def migrate_json_data(apps, schema_editor):
 
         obj_json['version'] = 1
 
-        text_items = obj_json["information"]["items"]["text"]
-        link_items = obj_json["information"]["items"]["links"]
+        if obj_json:
+            text_items = obj_json["information"]["items"]["text"]
+            link_items = obj_json["information"]["items"]["links"]
 
-        del obj_json["information"]["items"]
-        obj_json["information"]["items"] = []
+            del obj_json["information"]["items"]
+            obj_json["information"]["items"] = []
 
-        if text_items:
-            for item in range(len(text_items)):
-                text_items[item]["url"] = ""
-                text_items[item]["url_enabled"] = False
+            if text_items:
+                for item in range(len(text_items)):
+                    text_items[item]["url"] = ""
+                    text_items[item]["url_enabled"] = False
 
-                obj_json["information"]["items"].append(text_items[item])
+                    obj_json["information"]["items"].append(text_items[item])
 
-        if link_items:
-            for item in range(len(link_items)):
-                link_items[item]["url_enabled"] = True
+            if link_items:
+                for item in range(len(link_items)):
+                    link_items[item]["url_enabled"] = True
 
-                obj_json["information"]["items"].append(link_items[item])
+                    obj_json["information"]["items"].append(link_items[item])
 
-        for item in range(len(obj_json["information"]["items"])):
-            obj_json["information"]["items"][item]["uuid"] = item
-
-        obj.data = obj_json
-        obj.save()
+            for item in range(len(obj_json["information"]["items"])):
+                obj_json["information"]["items"][item]["uuid"] = item
+            
+            obj.data = obj_json
+            obj.save()
 
 class Migration(migrations.Migration):
 

diff --git a/cardie/main/migrations/0011_migrate_card_v1_to_v2.py b/cardie/main/migrations/0011_migrate_card_v1_to_v2.py
@@ -9,14 +9,15 @@ def migrate_json_data(apps, schema_editor):
 
         obj_json["version"] = 2
 
-        items = obj_json["information"]["items"]
+        if obj_json:
+            items = obj_json["information"]["items"]
 
-        if items:
-            for item in range(len(items)):
-                items[item]["position"] = item
+            if items:
+                for item in range(len(items)):
+                    items[item]["position"] = item
 
-        obj.data = obj_json
-        obj.save()
+            obj.data = obj_json
+            obj.save()
 
 class Migration(migrations.Migration):
 

diff --git a/cardie/main/migrations/0012_migrate_card_v2_to_v3.py b/cardie/main/migrations/0012_migrate_card_v2_to_v3.py
@@ -13,25 +13,26 @@ def migrate_json_data(apps, schema_editor):
 
         obj_json["version"] = 3
 
-        color_data = {
-            "background": "#ffffff",
-            "accent": "#000000",
-            "text": "#000000"
-        }
+        if obj_json:
+            color_data = {
+                "background": "#ffffff",
+                "accent": "#000000",
+                "text": "#000000"
+            }
 
-        obj_json["colors"] = color_data
+            obj_json["colors"] = color_data
 
-        username = obj_json["details"]["username"]
-        pronouns = obj_json["details"]["pronouns"]
+            username = obj_json["details"]["username"]
+            pronouns = obj_json["details"]["pronouns"]
 
-        obj_json["details"]["primary"] = username
-        obj_json["details"]["secondary"] = pronouns
+            obj_json["details"]["primary"] = username
+            obj_json["details"]["secondary"] = pronouns
 
-        del obj_json["details"]["username"]
-        del obj_json["details"]["pronouns"]
+            del obj_json["details"]["username"]
+            del obj_json["details"]["pronouns"]
 
-        obj.data = obj_json
-        obj.save()
+            obj.data = obj_json
+            obj.save()
 
 class Migration(migrations.Migration):
 

diff --git a/cardie/main/migrations/0013_migrate_card_v3_to_v4.py b/cardie/main/migrations/0013_migrate_card_v3_to_v4.py
@@ -9,10 +9,11 @@ def migrate_json_data(apps, schema_editor):
 
         obj_json["version"] = 4
 
-        obj_json["font_style"] = "Simple"
+        if obj_json:
+            obj_json["font_style"] = "Simple"
 
-        obj.data = obj_json
-        obj.save()
+            obj.data = obj_json
+            obj.save()
 
 class Migration(migrations.Migration):