From 642c452984784943c044beabcbc54aeaba6bb0dc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?V=C3=AD=C3=B0ir=20Valberg=20Gu=C3=B0mundsson?=
 <valberg@orn.li>
Date: Sun, 6 May 2018 21:55:50 +0200
Subject: [PATCH] Add option to delete info item.

---
 .../templates/info_item_delete_confirm.html   | 30 +++++++++++++++++++
 src/teams/templates/info_item_form.html       | 20 ++++++++-----
 src/teams/urls.py                             |  7 ++++-
 src/teams/views/info.py                       | 15 +++++++++-
 4 files changed, 63 insertions(+), 9 deletions(-)
 create mode 100644 src/teams/templates/info_item_delete_confirm.html

diff --git a/src/teams/templates/info_item_delete_confirm.html b/src/teams/templates/info_item_delete_confirm.html
new file mode 100644
index 00000000..fe9f2403
--- /dev/null
+++ b/src/teams/templates/info_item_delete_confirm.html
@@ -0,0 +1,30 @@
+{% extends 'base.html' %}
+{% load commonmark %}
+{% load bootstrap3 %}
+
+{% block title %}
+{% if form.instance.id %}
+Edit Info Item: {{ form.instance.headline }}
+{% else %}
+Create Info item
+{% endif %}
+in {{ form.instance.category.headline }}
+{% endblock %}
+
+{% block content %}
+<div class="panel panel-default">
+    <div class="panel-heading">
+        <h4>
+            Delete info item {{ object.name }}
+            in {{ form.instance.category.headline }}
+        </h4>
+    </div>
+    <div class="panel-body">
+        <form method="POST">
+            {% csrf_token %}
+            <p>Are you sure you want to delete "{{ object.headline }}" in the "{{ object.category.headline }}" category.</p>
+            <button type="submit" class="btn btn-danger">Delete</button>
+        </form>
+    </div>
+</div>
+{% endblock %}
diff --git a/src/teams/templates/info_item_form.html b/src/teams/templates/info_item_form.html
index 3a15e8e8..03d56ddf 100644
--- a/src/teams/templates/info_item_form.html
+++ b/src/teams/templates/info_item_form.html
@@ -3,31 +3,37 @@
 {% load bootstrap3 %}
 
 {% block title %}
-{% if form.instance.id %}
-Edit Info Item: {{ form.instance.headline }}
+{% if object %}
+Editing "{{ object.headline }}"
 {% else %}
 Create Info item
 {% endif %}
-in {{ form.instance.category.headline }}
+in "{{ form.instance.category.headline }}"
 {% endblock %}
 
 {% block content %}
 <div class="panel panel-default">
     <div class="panel-heading">
         <h4>
-            {% if form.instance.id %}
-            Edit Info Item: {{ form.instance.name }}
+            {% if object %}
+            Editing "{{ object.headline }}"
             {% else %}
             Create Info Item
             {% endif %}
-            in {{ form.instance.category.headline }}
+            in "{{ object.category.headline }}"
         </h4>
     </div>
     <div class="panel-body">
         <form method="POST">
             {% csrf_token %}
             {% bootstrap_form form %}
-            <button type="submit" class="btn btn-primary">{% if form.instance.id %}Save{% else %}Create{% endif %}</button>
+            <button type="submit" class="btn btn-primary">{% if object.id %}Save{% else %}Create{% endif %}</button>
+            {% if object %}
+            <a href="{% url 'teams:info_item_delete' camp_slug=camp.slug team_slug=object.category.team.slug category_anchor=object.category.anchor item_anchor=object.anchor %}"
+               class="btn btn-danger">
+                <i class="fa fa-remove"></i> Delete
+            </a>
+            {% endif %}
         </form>
     </div>
     <div class="panel-footer"><i>This info item belongs to the <a href="{% url 'teams:detail' team_slug=team.slug camp_slug=team.camp.slug %}">{{ team.name }} Team</a></i></div>
diff --git a/src/teams/urls.py b/src/teams/urls.py
index 4de29253..d528b2c1 100644
--- a/src/teams/urls.py
+++ b/src/teams/urls.py
@@ -10,7 +10,7 @@ from teams.views.base import (
     TeamManageView,
     FixIrcAclView,
 )
-from teams.views.info import InfoItemUpdateView, InfoItemCreateView
+from teams.views.info import InfoItemUpdateView, InfoItemCreateView, InfoItemDeleteView
 
 from teams.views.tasks import (
     TaskCreateView,
@@ -105,6 +105,11 @@ urlpatterns = [
                                 InfoItemUpdateView.as_view(),
                                 name='info_item_update',
                             ),
+                            url(
+                                r'^delete/$',
+                                InfoItemDeleteView.as_view(),
+                                name='info_item_delete',
+                            ),
                         ]),
                     ),
                 ])
diff --git a/src/teams/views/info.py b/src/teams/views/info.py
index 740fdd9f..5a6efaaa 100644
--- a/src/teams/views/info.py
+++ b/src/teams/views/info.py
@@ -1,6 +1,6 @@
 from django.contrib.auth.mixins import LoginRequiredMixin
 from django.http import HttpResponseRedirect
-from django.views.generic import CreateView, UpdateView
+from django.views.generic import CreateView, UpdateView, DeleteView
 from reversion.views import RevisionMixin
 
 from camps.mixins import CampViewMixin
@@ -47,3 +47,16 @@ class InfoItemUpdateView(LoginRequiredMixin, CampViewMixin, EnsureTeamResponsibl
         if next:
             return next
         return self.team.get_absolute_url()
+
+
+class InfoItemDeleteView(LoginRequiredMixin, CampViewMixin, EnsureTeamResponsibleMixin, RevisionMixin, DeleteView):
+    model = InfoItem
+    template_name = "info_item_delete_confirm.html"
+    slug_field = 'anchor'
+    slug_url_kwarg = 'item_anchor'
+
+    def get_success_url(self):
+        next = self.request.GET.get('next')
+        if next:
+            return next
+        return self.team.get_absolute_url()