Fixes for db atomic save errors

Author: geekygirlsarah

web/migrations/0004_missinglookup.py

@@ -0,0 +1,25 @@
+# Generated by Django 4.2.27 on 2025-12-28 00:33
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('web', '0003_lookupdata_date_time'),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name='MissingLookup',
+            fields=[
+                ('id', models.BigAutoField(primary_key=True, serialize=False)),
+                ('date_time', models.DateTimeField(auto_now_add=True)),
+                ('item_type', models.CharField(max_length=20)),
+                ('item_value', models.CharField(max_length=100)),
+                ('language_context', models.CharField(blank=True, max_length=50, null=True)),
+                ('site_visit', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='web.sitevisit')),
+            ],
+        ),
+    ]

web/views.py

@@ -10,6 +10,7 @@
     HttpResponseNotFound,
     HttpResponseServerError
 )
+from django.db import transaction
 from django.db.models import Count, Q
 from django.shortcuts import HttpResponse, render
 from django.utils.html import escape, strip_tags
@@ -49,7 +50,8 @@ def store_url_info(request):
             user_agent=user_agent,
             referer=referer,
         )
-        visit.save()
+        with transaction.atomic():
+            visit.save()
         return visit
     except Exception as e:
         logging.error(f"Failed to store URL info: {e}")
@@ -68,7 +70,8 @@ def store_lookup_info(request, visit, language1, version1, language2, version2,
             structure=structure,
             site_visit=visit
         )
-        info.save()
+        with transaction.atomic():
+            info.save()
     except Exception as e:
         logging.error(f"Failed to store lookup info: {e}")
 
@@ -83,7 +86,8 @@ def store_missing_info(visit, item_type, item_value, language_context=None):
             language_context=language_context,
             site_visit=visit
         )
-        info.save()
+        with transaction.atomic():
+            info.save()
     except Exception as e:
         logging.error(f"Failed to store missing info: {e}")