From 75bcc9103778b8c78e0a9e063339578db50b4040 Mon Sep 17 00:00:00 2001
From: reverend <reverend@reverend2048.de>
Date: Sun, 13 Sep 2020 14:39:21 +0200
Subject: [PATCH] Refactoring places images related name

---
 lostplaces/lostplaces_app/models.py           | 31 ++++++++++++++++---
 lostplaces/lostplaces_app/templates/home.html |  2 +-
 .../lostplaces_app/templates/home_unauth.html |  2 +-
 .../templates/place/place_detail.html         |  6 ++--
 .../templates/place/place_list.html           |  2 +-
 5 files changed, 33 insertions(+), 10 deletions(-)

diff --git a/lostplaces/lostplaces_app/models.py b/lostplaces/lostplaces_app/models.py
index 27b60cc..c6b3ca1 100644
--- a/lostplaces/lostplaces_app/models.py
+++ b/lostplaces/lostplaces_app/models.py
@@ -21,18 +21,24 @@ from taggit.managers import TaggableManager
 # Create your models here.
 
 class Taggable(models.Model):
-    
+    '''
+    This abstract model represtens an object that is taggalble
+    using django-taggit
+    '''
     class Meta:
         abstract = True
         
     tags = TaggableManager(blank=True)
     
 class MapablePoint(models.Model):
+    '''
+    This abstract model class represents an object that can be
+    displayed on a map.
+    '''
     class Meta:
         abstract = True
         
     name = models.CharField(max_length=50)
-    
     latitude = models.FloatField(
         validators=[
             MinValueValidator(-90),
@@ -46,6 +52,23 @@ class MapablePoint(models.Model):
         ]
     )
 
+class Submittable(models.Model):
+    '''
+    This abstract model class represents an object that can be submitted by
+    an explorer.
+    '''
+    class Meta:
+        abstract = True
+
+    submitted_when = models.DateTimeField(auto_now_add=True, null=True)
+    submitted_by = models.ForeignKey(
+        Explorer,
+        on_delete=models.SET_NULL,
+        null=True,
+        blank=True,
+        related_name='%(class)s'
+    )
+
 class Explorer(models.Model):
     """
     Profile that is linked to the a User.
@@ -147,7 +170,7 @@ class PlaceImage (models.Model):
     place = models.ForeignKey(
         Place,
         on_delete=models.CASCADE,
-        related_name='images'
+        related_name='placeimages'
     )
     submitted_when = models.DateTimeField(auto_now_add=True, null=True)
     submitted_by = models.ForeignKey(
@@ -155,7 +178,7 @@ class PlaceImage (models.Model):
         on_delete=models.SET_NULL,
         null=True,
         blank=True,
-        related_name='images'
+        related_name='placeimages'
     )
 
     def __str__(self):
diff --git a/lostplaces/lostplaces_app/templates/home.html b/lostplaces/lostplaces_app/templates/home.html
index c16bd0e..c71bed0 100644
--- a/lostplaces/lostplaces_app/templates/home.html
+++ b/lostplaces/lostplaces_app/templates/home.html
@@ -23,7 +23,7 @@
             <a href="{% url 'place_detail' pk=place.pk %}" class="LP-Link">
                 <article class="LP-PlaceTeaser">
                     <div class="LP-PlaceTeaser__Image">
-                        <img class="LP-Image" src="{{ place.images.first.filename.thumbnail.url}}" />
+                        <img class="LP-Image" src="{{ place.placeimages.first.filename.thumbnail.url}}" />
                     </div>
                     <div class="LP-PlaceTeaser__Meta">
                         <div class="LP-PlaceTeaser__Info">
diff --git a/lostplaces/lostplaces_app/templates/home_unauth.html b/lostplaces/lostplaces_app/templates/home_unauth.html
index 7fb0e00..753a729 100644
--- a/lostplaces/lostplaces_app/templates/home_unauth.html
+++ b/lostplaces/lostplaces_app/templates/home_unauth.html
@@ -34,7 +34,7 @@
             <a href="{% url 'place_detail' pk=place.pk %}" class="LP-Link">
                 <article class="LP-PlaceTeaser">
                     <div class="LP-PlaceTeaser__Image">
-                        <img class="LP-Image" src="{{ place.images.first.filename.thumbnail.url}}" />
+                        <img class="LP-Image" src="{{ place.placeimages.first.filename.thumbnail.url}}" />
                     </div>
                     <div class="LP-PlaceTeaser__Meta">
                         <div class="LP-PlaceTeaser__Info">
diff --git a/lostplaces/lostplaces_app/templates/place/place_detail.html b/lostplaces/lostplaces_app/templates/place/place_detail.html
index d4b1cc9..0d8f191 100644
--- a/lostplaces/lostplaces_app/templates/place/place_detail.html
+++ b/lostplaces/lostplaces_app/templates/place/place_detail.html
@@ -23,9 +23,9 @@
 
     <header class="LP-PlaceDetail__Header">
         <h1 class="LP-Headline">{{ place.name }}</h1>
-        {% if place.images.first.filename.hero.url %}
+        {% if place.placeimages.first.filename.hero.url %}
         <figure class="LP-PlaceDetail__Image">
-            <img src="{{ place.images.first.filename.hero.url }}" class="LP-Image" />
+            <img src="{{ place.placeimages.first.filename.hero.url }}" class="LP-Image" />
         </figure>
         {% endif %}
     </header>
@@ -92,7 +92,7 @@
         <h1 class="LP-Headline">Bilder</h1>
         <div class="LP-ImageGrid">
             <ul class="LP-ImageGrid__Container">
-                {% for place_image in place.images.all %}
+                {% for place_image in place.placeimages.all %}
                 <li class="LP-ImageGrid__Item">
                     <a href="{{ place_image.filename.large.url }}" class="LP-Link"><img class="LP-Image" src="{{ place_image.filename.thumbnail.url }}"></a>
                 </li>
diff --git a/lostplaces/lostplaces_app/templates/place/place_list.html b/lostplaces/lostplaces_app/templates/place/place_list.html
index f0bcae8..fd619c7 100644
--- a/lostplaces/lostplaces_app/templates/place/place_list.html
+++ b/lostplaces/lostplaces_app/templates/place/place_list.html
@@ -19,7 +19,7 @@
             <a href="{% url 'place_detail' pk=place.pk %}" class="LP-Link">
                 <article class="LP-PlaceTeaser LP-PlaceTeaser--extended">
                     <div class="LP-PlaceTeaser__Image">
-                        <img class="LP-Image" src="{{ place.images.first.filename.thumbnail.url }}" />
+                        <img class="LP-Image" src="{{ place.placeimages.first.filename.thumbnail.url }}" />
                     </div>
                     <div class="LP-PlaceTeaser__Meta">
                         <div class="LP-PlaceTeaser__Info">