From 0911826501c4fba52087a9db4c8edd6ec6434fb8 Mon Sep 17 00:00:00 2001
From: Sean Morley <zipsm15@gmail.com>
Date: Sun, 22 Sep 2024 14:12:41 -0400
Subject: [PATCH] Migrate to visits

---
 backend/server/adventures/admin.py            | 12 +---
 .../migrations/0001_adventure_image.py        | 19 ------
 .../adventures/migrations/0001_initial.py     | 62 ++++++++++++-------
 .../migrations/0002_adventureimage.py         | 27 --------
 .../0002_alter_adventureimage_adventure.py    | 19 ------
 ...ure_end_date_and_more.py => 0002_visit.py} | 16 ++---
 .../migrations/0003_adventure_end_date.py     | 18 ------
 ...date_remove_adventure_end_date_and_more.py | 26 ++++++++
 .../0004_transportation_end_date.py           | 18 ------
 .../migrations/0005_collection_shared_with.py | 20 ------
 .../migrations/0006_alter_adventure_link.py   | 18 ------
 .../adventures/migrations/migrate_images.py   |  5 --
 .../migrate_visits_and_categories.py          | 31 ++++++++++
 backend/server/adventures/models.py           | 13 ++--
 14 files changed, 110 insertions(+), 194 deletions(-)
 delete mode 100644 backend/server/adventures/migrations/0001_adventure_image.py
 delete mode 100644 backend/server/adventures/migrations/0002_adventureimage.py
 delete mode 100644 backend/server/adventures/migrations/0002_alter_adventureimage_adventure.py
 rename backend/server/adventures/migrations/{0007_remove_adventure_date_remove_adventure_end_date_and_more.py => 0002_visit.py} (64%)
 delete mode 100644 backend/server/adventures/migrations/0003_adventure_end_date.py
 create mode 100644 backend/server/adventures/migrations/0003_remove_adventure_date_remove_adventure_end_date_and_more.py
 delete mode 100644 backend/server/adventures/migrations/0004_transportation_end_date.py
 delete mode 100644 backend/server/adventures/migrations/0005_collection_shared_with.py
 delete mode 100644 backend/server/adventures/migrations/0006_alter_adventure_link.py
 create mode 100644 backend/server/adventures/migrations/migrate_visits_and_categories.py

diff --git a/backend/server/adventures/admin.py b/backend/server/adventures/admin.py
index de7227d..b0dd607 100644
--- a/backend/server/adventures/admin.py
+++ b/backend/server/adventures/admin.py
@@ -6,19 +6,9 @@ from worldtravel.models import Country, Region, VisitedRegion
 
 
 class AdventureAdmin(admin.ModelAdmin):
-    list_display = ('name', 'type', 'user_id', 'is_public', 'image_display')
+    list_display = ('name', 'type', 'user_id', 'is_public')
     list_filter = ('type', 'user_id', 'is_public')
 
-    def image_display(self, obj):
-        if obj.image:
-            public_url = os.environ.get('PUBLIC_URL', 'http://127.0.0.1:8000').rstrip('/')
-            public_url = public_url.replace("'", "")
-            return mark_safe(f'<img src="{public_url}/media/{obj.image.name}" width="100px" height="100px"')
-        else:
-            return
-
-    image_display.short_description = 'Image Preview'
-
 
 class CountryAdmin(admin.ModelAdmin):
     list_display = ('name', 'country_code', 'number_of_regions')
diff --git a/backend/server/adventures/migrations/0001_adventure_image.py b/backend/server/adventures/migrations/0001_adventure_image.py
deleted file mode 100644
index 6906db4..0000000
--- a/backend/server/adventures/migrations/0001_adventure_image.py
+++ /dev/null
@@ -1,19 +0,0 @@
-# Generated by Django 5.0.8 on 2024-08-15 23:20
-
-import django_resized.forms
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('adventures', 'migrate_images'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='adventure',
-            name='image',
-            field=django_resized.forms.ResizedImageField(blank=True, crop=None, force_format='WEBP', keep_meta=True, null=True, quality=75, scale=None, size=[1920, 1080], upload_to='images/'),
-        ),
-    ]
diff --git a/backend/server/adventures/migrations/0001_initial.py b/backend/server/adventures/migrations/0001_initial.py
index 4b0082c..3c1dfd3 100644
--- a/backend/server/adventures/migrations/0001_initial.py
+++ b/backend/server/adventures/migrations/0001_initial.py
@@ -1,4 +1,4 @@
-# Generated by Django 5.0.8 on 2024-08-11 13:37
+# Generated by Django 5.0.8 on 2024-09-22 15:12
 
 import django.contrib.postgres.fields
 import django.db.models.deletion
@@ -17,6 +17,37 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
+        migrations.CreateModel(
+            name='Adventure',
+            fields=[
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, primary_key=True, serialize=False, unique=True)),
+                ('type', models.CharField(choices=[('general', 'General 🌍'), ('Outdoor', 'Outdoor 🏞️'), ('lodging', 'Lodging 🛌'), ('dining', 'Dining 🍽️'), ('activity', 'Activity 🏄'), ('attraction', 'Attraction 🎢'), ('shopping', 'Shopping 🛍️'), ('nightlife', 'Nightlife 🌃'), ('event', 'Event 🎉'), ('transportation', 'Transportation 🚗'), ('culture', 'Culture 🎭'), ('water_sports', 'Water Sports 🚤'), ('hiking', 'Hiking 🥾'), ('wildlife', 'Wildlife 🦒'), ('historical_sites', 'Historical Sites 🏛️'), ('music_concerts', 'Music & Concerts 🎶'), ('fitness', 'Fitness 🏋️'), ('art_museums', 'Art & Museums 🎨'), ('festivals', 'Festivals 🎪'), ('spiritual_journeys', 'Spiritual Journeys 🧘\u200d♀️'), ('volunteer_work', 'Volunteer Work 🤝'), ('other', 'Other')], max_length=100)),
+                ('name', models.CharField(max_length=200)),
+                ('location', models.CharField(blank=True, max_length=200, null=True)),
+                ('activity_types', django.contrib.postgres.fields.ArrayField(base_field=models.CharField(max_length=100), blank=True, null=True, size=None)),
+                ('description', models.TextField(blank=True, null=True)),
+                ('rating', models.FloatField(blank=True, null=True)),
+                ('link', models.URLField(blank=True, max_length=2083, null=True)),
+                ('is_public', models.BooleanField(default=False)),
+                ('longitude', models.DecimalField(blank=True, decimal_places=6, max_digits=9, null=True)),
+                ('latitude', models.DecimalField(blank=True, decimal_places=6, max_digits=9, null=True)),
+                ('created_at', models.DateTimeField(auto_now_add=True)),
+                ('updated_at', models.DateTimeField(auto_now=True)),
+                ('image', django_resized.forms.ResizedImageField(blank=True, crop=None, force_format='WEBP', keep_meta=True, null=True, quality=75, scale=None, size=[1920, 1080], upload_to='images/')),
+                ('date', models.DateField(blank=True, null=True)),
+                ('end_date', models.DateField(blank=True, null=True)),
+                ('user_id', models.ForeignKey(default=1, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
+            ],
+        ),
+        migrations.CreateModel(
+            name='AdventureImage',
+            fields=[
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, primary_key=True, serialize=False, unique=True)),
+                ('image', django_resized.forms.ResizedImageField(crop=None, force_format='WEBP', keep_meta=True, quality=75, scale=None, size=[1920, 1080], upload_to='images/')),
+                ('adventure', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='images', to='adventures.adventure')),
+                ('user_id', models.ForeignKey(default=1, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
+            ],
+        ),
         migrations.CreateModel(
             name='Checklist',
             fields=[
@@ -53,6 +84,7 @@ class Migration(migrations.Migration):
                 ('end_date', models.DateField(blank=True, null=True)),
                 ('updated_at', models.DateTimeField(auto_now=True)),
                 ('is_archived', models.BooleanField(default=False)),
+                ('shared_with', models.ManyToManyField(blank=True, related_name='shared_with', to=settings.AUTH_USER_MODEL)),
                 ('user_id', models.ForeignKey(default=1, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
             ],
         ),
@@ -61,27 +93,10 @@ class Migration(migrations.Migration):
             name='collection',
             field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to='adventures.collection'),
         ),
-        migrations.CreateModel(
-            name='Adventure',
-            fields=[
-                ('id', models.UUIDField(default=uuid.uuid4, editable=False, primary_key=True, serialize=False, unique=True)),
-                ('type', models.CharField(choices=[('visited', 'Visited'), ('planned', 'Planned'), ('lodging', 'Lodging'), ('dining', 'Dining')], max_length=100)),
-                ('name', models.CharField(max_length=200)),
-                ('location', models.CharField(blank=True, max_length=200, null=True)),
-                ('activity_types', django.contrib.postgres.fields.ArrayField(base_field=models.CharField(max_length=100), blank=True, null=True, size=None)),
-                ('description', models.TextField(blank=True, null=True)),
-                ('rating', models.FloatField(blank=True, null=True)),
-                ('link', models.URLField(blank=True, null=True)),
-                ('image', django_resized.forms.ResizedImageField(blank=True, crop=None, force_format='WEBP', keep_meta=True, null=True, quality=75, scale=None, size=[1920, 1080], upload_to='images/')),
-                ('date', models.DateField(blank=True, null=True)),
-                ('is_public', models.BooleanField(default=False)),
-                ('longitude', models.DecimalField(blank=True, decimal_places=6, max_digits=9, null=True)),
-                ('latitude', models.DecimalField(blank=True, decimal_places=6, max_digits=9, null=True)),
-                ('created_at', models.DateTimeField(auto_now_add=True)),
-                ('updated_at', models.DateTimeField(auto_now=True)),
-                ('user_id', models.ForeignKey(default=1, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
-                ('collection', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to='adventures.collection')),
-            ],
+        migrations.AddField(
+            model_name='adventure',
+            name='collection',
+            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to='adventures.collection'),
         ),
         migrations.CreateModel(
             name='Note',
@@ -108,6 +123,7 @@ class Migration(migrations.Migration):
                 ('rating', models.FloatField(blank=True, null=True)),
                 ('link', models.URLField(blank=True, null=True)),
                 ('date', models.DateTimeField(blank=True, null=True)),
+                ('end_date', models.DateTimeField(blank=True, null=True)),
                 ('flight_number', models.CharField(blank=True, max_length=100, null=True)),
                 ('from_location', models.CharField(blank=True, max_length=200, null=True)),
                 ('to_location', models.CharField(blank=True, max_length=200, null=True)),
@@ -117,5 +133,5 @@ class Migration(migrations.Migration):
                 ('collection', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to='adventures.collection')),
                 ('user_id', models.ForeignKey(default=1, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
             ],
-        ),
+        )
     ]
diff --git a/backend/server/adventures/migrations/0002_adventureimage.py b/backend/server/adventures/migrations/0002_adventureimage.py
deleted file mode 100644
index bf80832..0000000
--- a/backend/server/adventures/migrations/0002_adventureimage.py
+++ /dev/null
@@ -1,27 +0,0 @@
-# Generated by Django 5.0.8 on 2024-08-15 23:17
-
-import django.db.models.deletion
-import django_resized.forms
-import uuid
-from django.conf import settings
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('adventures', '0001_initial'),
-        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
-    ]
-
-    operations = [
-        migrations.CreateModel(
-            name='AdventureImage',
-            fields=[
-                ('id', models.UUIDField(default=uuid.uuid4, editable=False, primary_key=True, serialize=False, unique=True)),
-                ('image', django_resized.forms.ResizedImageField(crop=None, force_format='WEBP', keep_meta=True, quality=75, scale=None, size=[1920, 1080], upload_to='images/')),
-                ('adventure', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='adventures.adventure')),
-                ('user_id', models.ForeignKey(default=1, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
-            ],
-        ),
-    ]
diff --git a/backend/server/adventures/migrations/0002_alter_adventureimage_adventure.py b/backend/server/adventures/migrations/0002_alter_adventureimage_adventure.py
deleted file mode 100644
index b03c3ad..0000000
--- a/backend/server/adventures/migrations/0002_alter_adventureimage_adventure.py
+++ /dev/null
@@ -1,19 +0,0 @@
-# Generated by Django 5.0.8 on 2024-08-15 23:31
-
-import django.db.models.deletion
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('adventures', '0001_adventure_image'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='adventureimage',
-            name='adventure',
-            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='images', to='adventures.adventure'),
-        ),
-    ]
diff --git a/backend/server/adventures/migrations/0007_remove_adventure_date_remove_adventure_end_date_and_more.py b/backend/server/adventures/migrations/0002_visit.py
similarity index 64%
rename from backend/server/adventures/migrations/0007_remove_adventure_date_remove_adventure_end_date_and_more.py
rename to backend/server/adventures/migrations/0002_visit.py
index 03f6b1e..8fa8f5d 100644
--- a/backend/server/adventures/migrations/0007_remove_adventure_date_remove_adventure_end_date_and_more.py
+++ b/backend/server/adventures/migrations/0002_visit.py
@@ -1,4 +1,4 @@
-# Generated by Django 5.0.8 on 2024-09-22 04:02
+# Generated by Django 5.0.8 on 2024-09-22 15:32
 
 import django.db.models.deletion
 import uuid
@@ -8,24 +8,16 @@ from django.db import migrations, models
 class Migration(migrations.Migration):
 
     dependencies = [
-        ('adventures', '0006_alter_adventure_link'),
+        ('adventures', '0001_initial'),
     ]
 
     operations = [
-        migrations.RemoveField(
-            model_name='adventure',
-            name='date',
-        ),
-        migrations.RemoveField(
-            model_name='adventure',
-            name='end_date',
-        ),
         migrations.CreateModel(
             name='Visit',
             fields=[
                 ('id', models.UUIDField(default=uuid.uuid4, editable=False, primary_key=True, serialize=False, unique=True)),
-                ('start_date', models.DateField()),
-                ('end_date', models.DateField()),
+                ('start_date', models.DateField(blank=True, null=True)),
+                ('end_date', models.DateField(blank=True, null=True)),
                 ('notes', models.TextField(blank=True, null=True)),
                 ('created_at', models.DateTimeField(auto_now_add=True)),
                 ('updated_at', models.DateTimeField(auto_now=True)),
diff --git a/backend/server/adventures/migrations/0003_adventure_end_date.py b/backend/server/adventures/migrations/0003_adventure_end_date.py
deleted file mode 100644
index a04bbdb..0000000
--- a/backend/server/adventures/migrations/0003_adventure_end_date.py
+++ /dev/null
@@ -1,18 +0,0 @@
-# Generated by Django 5.0.8 on 2024-08-18 16:16
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('adventures', '0002_alter_adventureimage_adventure'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='adventure',
-            name='end_date',
-            field=models.DateField(blank=True, null=True),
-        ),
-    ]
diff --git a/backend/server/adventures/migrations/0003_remove_adventure_date_remove_adventure_end_date_and_more.py b/backend/server/adventures/migrations/0003_remove_adventure_date_remove_adventure_end_date_and_more.py
new file mode 100644
index 0000000..7e4e6d6
--- /dev/null
+++ b/backend/server/adventures/migrations/0003_remove_adventure_date_remove_adventure_end_date_and_more.py
@@ -0,0 +1,26 @@
+# Generated by Django 5.0.8 on 2024-09-22 15:14
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('adventures', 'migrate_visits_and_categories'),
+        ('adventures', 'migrate_images'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='adventure',
+            name='date',
+        ),
+        migrations.RemoveField(
+            model_name='adventure',
+            name='end_date',
+        ),
+        migrations.RemoveField(
+            model_name='adventure',
+            name='image',
+        ),
+    ]
diff --git a/backend/server/adventures/migrations/0004_transportation_end_date.py b/backend/server/adventures/migrations/0004_transportation_end_date.py
deleted file mode 100644
index 04f3b57..0000000
--- a/backend/server/adventures/migrations/0004_transportation_end_date.py
+++ /dev/null
@@ -1,18 +0,0 @@
-# Generated by Django 5.0.8 on 2024-08-19 20:04
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('adventures', '0003_adventure_end_date'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='transportation',
-            name='end_date',
-            field=models.DateTimeField(blank=True, null=True),
-        ),
-    ]
diff --git a/backend/server/adventures/migrations/0005_collection_shared_with.py b/backend/server/adventures/migrations/0005_collection_shared_with.py
deleted file mode 100644
index c3ee3ac..0000000
--- a/backend/server/adventures/migrations/0005_collection_shared_with.py
+++ /dev/null
@@ -1,20 +0,0 @@
-# Generated by Django 5.0.8 on 2024-09-02 13:21
-
-from django.conf import settings
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('adventures', '0004_transportation_end_date'),
-        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='collection',
-            name='shared_with',
-            field=models.ManyToManyField(blank=True, related_name='shared_with', to=settings.AUTH_USER_MODEL),
-        ),
-    ]
diff --git a/backend/server/adventures/migrations/0006_alter_adventure_link.py b/backend/server/adventures/migrations/0006_alter_adventure_link.py
deleted file mode 100644
index 1a78820..0000000
--- a/backend/server/adventures/migrations/0006_alter_adventure_link.py
+++ /dev/null
@@ -1,18 +0,0 @@
-# Generated by Django 5.0.8 on 2024-09-17 14:18
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('adventures', '0005_collection_shared_with'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='adventure',
-            name='link',
-            field=models.URLField(blank=True, max_length=2083, null=True),
-        ),
-    ]
diff --git a/backend/server/adventures/migrations/migrate_images.py b/backend/server/adventures/migrations/migrate_images.py
index 9ef361f..d2b4a91 100644
--- a/backend/server/adventures/migrations/migrate_images.py
+++ b/backend/server/adventures/migrations/migrate_images.py
@@ -17,13 +17,8 @@ class Migration(migrations.Migration):
 
     dependencies = [
         ('adventures', '0001_initial'),
-        ('adventures', '0002_adventureimage'),
     ]
 
     operations = [
         migrations.RunPython(move_images_to_new_model),
-        migrations.RemoveField(
-            model_name='Adventure',
-            name='image',
-        ),
     ]
\ No newline at end of file
diff --git a/backend/server/adventures/migrations/migrate_visits_and_categories.py b/backend/server/adventures/migrations/migrate_visits_and_categories.py
new file mode 100644
index 0000000..1322712
--- /dev/null
+++ b/backend/server/adventures/migrations/migrate_visits_and_categories.py
@@ -0,0 +1,31 @@
+from django.db import migrations
+from django.db import migrations, models
+
+def move_images_to_new_model(apps, schema_editor):
+    Adventure = apps.get_model('adventures', 'Adventure')
+    Visit = apps.get_model('adventures', 'Visit')
+
+    for adventure in Adventure.objects.all():
+        # if the type is visited and there is no date, set note to 'No date provided.'
+        note = 'No date provided.' if adventure.type == 'visited' and not adventure.date else ''
+        if adventure.date or adventure.type == 'visited':
+            Visit.objects.create(
+                adventure=adventure,
+                start_date=adventure.date,
+                end_date=adventure.end_date,
+                notes=note,
+            )
+        if adventure.type == 'visited' or adventure.type == 'planned':
+            adventure.type = 'general'
+            adventure.save()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('adventures', '0002_visit')
+    ]
+
+    operations = [
+        migrations.RunPython(move_images_to_new_model),
+    ]
\ No newline at end of file
diff --git a/backend/server/adventures/models.py b/backend/server/adventures/models.py
index b9ecb72..5312d86 100644
--- a/backend/server/adventures/models.py
+++ b/backend/server/adventures/models.py
@@ -50,8 +50,8 @@ User = get_user_model()
 class Visit(models.Model):
     id = models.UUIDField(default=uuid.uuid4, editable=False, unique=True, primary_key=True)
     adventure = models.ForeignKey('Adventure', on_delete=models.CASCADE, related_name='visits')
-    start_date = models.DateField()
-    end_date = models.DateField()
+    start_date = models.DateField(null=True, blank=True)
+    end_date = models.DateField(null=True, blank=True)
     notes = models.TextField(blank=True, null=True)
     created_at = models.DateTimeField(auto_now_add=True)
     updated_at = models.DateTimeField(auto_now=True)
@@ -67,7 +67,7 @@ class Adventure(models.Model):
     id = models.UUIDField(default=uuid.uuid4, editable=False, unique=True, primary_key=True)
     user_id = models.ForeignKey(
         User, on_delete=models.CASCADE, default=default_user_id)
-    type = models.CharField(max_length=100, choices=ADVENTURE_TYPES)
+    type = models.CharField(max_length=100, choices=ADVENTURE_TYPES, default='general')
     name = models.CharField(max_length=200)
     location = models.CharField(max_length=200, blank=True, null=True)
     activity_types = ArrayField(models.CharField(
@@ -75,7 +75,6 @@ class Adventure(models.Model):
     description = models.TextField(blank=True, null=True)
     rating = models.FloatField(blank=True, null=True)
     link = models.URLField(blank=True, null=True, max_length=2083)
-    image = ResizedImageField(force_format="WEBP", quality=75, null=True, blank=True, upload_to='images/')
     is_public = models.BooleanField(default=False)
     longitude = models.DecimalField(max_digits=9, decimal_places=6, null=True, blank=True)
     latitude = models.DecimalField(max_digits=9, decimal_places=6, null=True, blank=True)
@@ -83,6 +82,12 @@ class Adventure(models.Model):
     created_at = models.DateTimeField(auto_now_add=True)
     updated_at = models.DateTimeField(auto_now=True)
 
+    # DEPRECATED FIELDS - TO BE REMOVED IN FUTURE VERSIONS
+    # Migrations performed in this version will remove these fields
+    # image = ResizedImageField(force_format="WEBP", quality=75, null=True, blank=True, upload_to='images/')
+    # date = models.DateField(blank=True, null=True)
+    # end_date = models.DateField(blank=True, null=True)
+
     def clean(self):
 
         if self.collection: