From 6d89bca033de2e708daeab60792d41ee289d53c0 Mon Sep 17 00:00:00 2001
From: reverend <reverend@reverend2048.de>
Date: Wed, 2 Sep 2020 00:08:00 +0200
Subject: [PATCH] Deletion of tags

---
 lostplaces/lostplaces_app/forms.py            |  6 +++++-
 lostplaces/lostplaces_app/static/main.css     |  3 ++-
 .../templates/partials/tagging.html           | 19 +++++++++++++++----
 .../templates/place/place_detail.html         |  2 +-
 lostplaces/lostplaces_app/urls.py             |  4 +++-
 .../lostplaces_app/views/place_views.py       |  9 +++++++--
 lostplaces/lostplaces_app/views/views.py      | 13 ++++++++++++-
 7 files changed, 45 insertions(+), 11 deletions(-)

diff --git a/lostplaces/lostplaces_app/forms.py b/lostplaces/lostplaces_app/forms.py
index 6ac307a..9b97388 100644
--- a/lostplaces/lostplaces_app/forms.py
+++ b/lostplaces/lostplaces_app/forms.py
@@ -51,4 +51,8 @@ class PlaceImageCreateForm(forms.ModelForm):
 
 
 class TagSubmitForm(forms.Form):
-	tag_list = forms.CharField(max_length=500, required=False)
\ No newline at end of file
+	tag_list = forms.CharField(
+        max_length=500,
+        required=False, 
+        widget=forms.TextInput(attrs={'autocomplete':'off'})
+        )
\ No newline at end of file
diff --git a/lostplaces/lostplaces_app/static/main.css b/lostplaces/lostplaces_app/static/main.css
index 8489e0c..fef1375 100644
--- a/lostplaces/lostplaces_app/static/main.css
+++ b/lostplaces/lostplaces_app/static/main.css
@@ -1472,7 +1472,8 @@ body {
   line-height: inherit;
   position: relative;
   white-space: pre-wrap;
-  margin-left: 15px; }
+  margin-left: 15px;
+  height: 1em; }
 
 .tagify__tag__removeBtn {
   order: 5;
diff --git a/lostplaces/lostplaces_app/templates/partials/tagging.html b/lostplaces/lostplaces_app/templates/partials/tagging.html
index d9db359..38a274e 100644
--- a/lostplaces/lostplaces_app/templates/partials/tagging.html
+++ b/lostplaces/lostplaces_app/templates/partials/tagging.html
@@ -3,14 +3,25 @@
         {% for tag in tag_list %}
         <li class="LP-TagList__Item">
             <div class="LP-Tag">
-                <p class="LP-Paragraph">{{tag}}</p>
+                <a href="#" class="LP-Link">
+                </a>
+                    <span class="LP-Link__Text">{{tag}}</span>
+                <a href="{% url config.delete_url_name tagged_id=config.tagged_item.id tag_id=tag.id %}" class="LP-Link">
+                    <span class="LP-Tag__Remove RV-Iconized__Container RV-Iconized__Container--extraSmall">
+                        <svg class="RV-Iconized__Icon" height="24" viewBox="0 0 24 24" width="24" xmlns="http://www.w3.org/2000/svg">
+                            <path d="M0 0h24v24H0z" fill="none"></path>
+                            <path d="M19 6.4L17.6 5 12 10.6 6.4 5 5 6.4 10.6 12 5 17.6 6.4 19 12 13.4 17.6 19 19 17.6 13.4 12z">
+                            </path>
+                        </svg>
+                    </span>
+                </a>
             </div>
         </li>
         {% endfor %}
     </ul>
 </div>
 
-<form id="id_tag_submit_form" class="LP-Form LP-Form--inline LP-Form--tagging" method="POST" action="{{url}}">
+<form id="id_tag_submit_form" class="LP-Form LP-Form--inline LP-Form--tagging" method="POST" action="{{config.submit_url}}">
     <fieldset class="LP-Form__Fieldset">
         <legend class="LP-Form__Legend">Tags hinzufügen</legend>
         {% csrf_token %}
@@ -19,14 +30,14 @@
                 <button id="id_tag_submit_button" class="LP-Button"> Tags hinzufügen</button>
             </div>
             <div class="LP-Form__Field">
-                {% include 'partials/form/inputField.html' with field=input_field classes="LP-Input--tagging" %}
+                {% include 'partials/form/inputField.html' with field=config.submit_form.tag_list classes="LP-Input--tagging" %}
             </div>
         </div>
     </fieldset>
 </form>
 
 <script>
-    const input = document.getElementById('{{input_field.auto_id}}')
+    const input = document.getElementById('{{config.submit_form.tag_list.auto_id}}')
     const submit_form = document.getElementById('id_tag_submit_form')
     const submit_button = document.getElementById('id_tag_submit_button')
 
diff --git a/lostplaces/lostplaces_app/templates/place/place_detail.html b/lostplaces/lostplaces_app/templates/place/place_detail.html
index b687a1e..452b38e 100644
--- a/lostplaces/lostplaces_app/templates/place/place_detail.html
+++ b/lostplaces/lostplaces_app/templates/place/place_detail.html
@@ -40,7 +40,7 @@
     <section class="LP-Section">
 
         {% url 'place_tag_submit' place_id=place.id as tag_submit_url%}
-        {% include 'partials/tagging.html' with tag_list=place.tags.all url=tag_submit_url input_field=tagging_form.tag_list%}
+        {% include 'partials/tagging.html' with tag_list=place.tags.all config=tagging_config all_tags=all_tags %}
 
     </section>
 
diff --git a/lostplaces/lostplaces_app/urls.py b/lostplaces/lostplaces_app/urls.py
index d5d7b15..c660f2f 100644
--- a/lostplaces/lostplaces_app/urls.py
+++ b/lostplaces/lostplaces_app/urls.py
@@ -9,7 +9,8 @@ from .views import (
     PlaceDeleteView,
 	PhotoAlbumCreateView,
 	PhotoAlbumDeleteView,
-	PlaceTagSubmitView
+	PlaceTagSubmitView,
+    PlaceTagDeleteView
 )
 
 urlpatterns = [
@@ -23,4 +24,5 @@ urlpatterns = [
     path('place/delete/<int:pk>/', PlaceDeleteView.as_view(), name='place_delete'),
     path('place/', PlaceListView.as_view(), name='place_list'),
 	path('place/tag/<int:place_id>', PlaceTagSubmitView.as_view(), name='place_tag_submit'),
+    path('place/tag/delete/<int:tagged_id>/<int:tag_id>', PlaceTagDeleteView.as_view(), name='place_tag_delete')
 ]
diff --git a/lostplaces/lostplaces_app/views/place_views.py b/lostplaces/lostplaces_app/views/place_views.py
index 30a7123..f538187 100644
--- a/lostplaces/lostplaces_app/views/place_views.py
+++ b/lostplaces/lostplaces_app/views/place_views.py
@@ -33,8 +33,13 @@ class PlaceDetailView(IsAuthenticated, View):
             'place': place,
             'place_list': [ place ],
             'place_map_center': [ place.latitude, place.longitude ],
-            'tagging_form': TagSubmitForm(),
-            'all_tags': Tag.objects.all()
+            'all_tags': Tag.objects.all(),
+            'tagging_config': {
+                'submit_url': reverse_lazy('place_tag_submit', kwargs={'place_id': place.id}),
+                'submit_form': TagSubmitForm(),
+                'tagged_item': place,
+                'delete_url_name': 'place_tag_delete'
+            }
         }
         return render(request, 'place/place_detail.html', context)
 
diff --git a/lostplaces/lostplaces_app/views/views.py b/lostplaces/lostplaces_app/views/views.py
index 5725b58..b1d86b1 100644
--- a/lostplaces/lostplaces_app/views/views.py
+++ b/lostplaces/lostplaces_app/views/views.py
@@ -5,6 +5,7 @@ from django.contrib.messages.views import SuccessMessageMixin
 from django.contrib import messages
 from django.urls import reverse_lazy
 from django.shortcuts import render, redirect
+from django.http import HttpResponseForbidden
 
 from lostplaces_app.forms import ExplorerCreationForm, TagSubmitForm
 from lostplaces_app.models import Place, PhotoAlbum
@@ -12,8 +13,11 @@ from lostplaces_app.views.base_views import IsAuthenticated
 
 from lostplaces_app.views.base_views import (
     PlaceAssetCreateView, 
-    PlaceAssetDeleteView
+    PlaceAssetDeleteView,
 )
+
+from taggit.models import Tag
+
 class SignUpView(SuccessMessageMixin, CreateView):
     form_class = ExplorerCreationForm
     success_url = reverse_lazy('login')
@@ -56,3 +60,10 @@ class PlaceTagSubmitView(IsAuthenticated, View):
 			place.save()
 
 		return redirect(reverse_lazy('place_detail', kwargs={'pk': place.id}))
+
+class PlaceTagDeleteView(IsAuthenticated, View):
+    def get(self, request, tagged_id, tag_id, *args, **kwargs):
+        place = Place.objects.get(pk=tagged_id)
+        tag = Tag.objects.get(pk=tag_id)
+        place.tags.remove(tag)
+        return redirect(reverse_lazy('place_detail', kwargs={'pk': tagged_id}))
\ No newline at end of file