From 80ac5e86ad9672725466dcb3339a9b43995eb4c6 Mon Sep 17 00:00:00 2001
From: Zambom <erikzambom@hotmail.com>
Date: Wed, 5 Oct 2016 11:59:51 -0300
Subject: [PATCH] Adding post answer deletion [Issue: #165]

---
 forum/static/js/forum.js                             | 29 +++++++++++++++++++++++++++++
 forum/templates/post_answers/post_answer_list.html   |  4 ++--
 forum/templates/post_answers/post_answer_render.html |  4 ++--
 forum/urls.py                                        |  2 ++
 forum/views.py                                       | 13 ++++++++++++-
 5 files changed, 47 insertions(+), 5 deletions(-)

diff --git a/forum/static/js/forum.js b/forum/static/js/forum.js
index 0ccd594..244ff27 100644
--- a/forum/static/js/forum.js
+++ b/forum/static/js/forum.js
@@ -212,6 +212,11 @@ function delete_post(url, post) {
     });
 }
 
+/*
+*
+* Function to load answer post form and set the submit function
+*
+*/
 function answer(id, url) {
     $.ajax({
         url: url, 
@@ -241,4 +246,28 @@ function answer(id, url) {
     });
 
     $("#post_"+id).find(".answer_post").show();
+}
+
+/*
+*
+* Function to delete an answer
+*
+*/
+function delete_answer(url, answer, message) {
+    alertify.confirm(message, function(){
+        var csrftoken = getCookie('csrftoken');
+        
+        $.ajax({
+            method: 'post',
+            beforeSend: function (request) {
+                request.setRequestHeader('X-CSRFToken', csrftoken);
+            },
+            url: url, 
+            success: function(data) {
+                alertify.alert('Amadeus', data, function(){
+                    $("#answer_"+answer).remove();
+                });
+            }
+        });
+    });
 }
\ No newline at end of file
diff --git a/forum/templates/post_answers/post_answer_list.html b/forum/templates/post_answers/post_answer_list.html
index b634bd2..aeb80e7 100644
--- a/forum/templates/post_answers/post_answer_list.html
+++ b/forum/templates/post_answers/post_answer_list.html
@@ -2,7 +2,7 @@
 
 {% if answers|length > 0 %}
     {% for answer in answers %}
-        <div id="answer_{{ answer_id }}" class="row" style="background-color: #e0e0e0">
+        <div id="answer_{{ answer.id }}" class="row" style="background-color: #e0e0e0">
             <div class="col-sm-12 col-xs-12">
                 <h3 class="user-name">
                     {{ answer.user }}
@@ -14,7 +14,7 @@
                                 </a>
                                 <ul class="dropdown-menu" aria-labelledby="dropdownMenu1">
                                     <li><a href="javascript:void(0)"><i class="material-icons">create</i> {% trans 'Edit' %}</a></li>
-                                    <li><a href="javascript:void(0)"><i class="material-icons">delete_sweep</i> {% trans 'Remove' %}</a></li>
+                                    <li><a href="javascript:delete_answer('{% url 'course:forum:delete_answer' answer.id %}', '{{ answer.id }}', '{% trans "Are you sure you want to delete this answer?" %}')"><i class="material-icons">delete_sweep</i> {% trans 'Remove' %}</a></li>
                                 </ul>
                             </div>
                         </div>
diff --git a/forum/templates/post_answers/post_answer_render.html b/forum/templates/post_answers/post_answer_render.html
index 69ea77a..6981968 100644
--- a/forum/templates/post_answers/post_answer_render.html
+++ b/forum/templates/post_answers/post_answer_render.html
@@ -1,6 +1,6 @@
 {% load i18n permission_tags %}
 
-<div id="answer_{{ answer_id }}" class="row" style="background-color: #e0e0e0">
+<div id="answer_{{ answer.id }}" class="row" style="background-color: #e0e0e0">
     <div class="col-sm-12 col-xs-12">
         <h3 class="user-name">
             {{ answer.user }}
@@ -12,7 +12,7 @@
                         </a>
                         <ul class="dropdown-menu" aria-labelledby="dropdownMenu1">
                             <li><a href="javascript:void(0)"><i class="material-icons">create</i> {% trans 'Edit' %}</a></li>
-                            <li><a href="javascript:void(0)"><i class="material-icons">delete_sweep</i> {% trans 'Remove' %}</a></li>
+                            <li><a href="javascript:delete_answer('{% url 'course:forum:delete_answer' answer.id %}', '{{ answer.id }}', '{% trans "Are you sure you want to delete this answer?" %}')"><i class="material-icons">delete_sweep</i> {% trans 'Remove' %}</a></li>
                         </ul>
                     </div>
                 </div>
diff --git a/forum/urls.py b/forum/urls.py
index a3ad25f..f5b07a6 100644
--- a/forum/urls.py
+++ b/forum/urls.py
@@ -19,5 +19,7 @@ urlpatterns = [
 	url(r'^post_answers/$', views.PostAnswerIndex.as_view(), name='post_answers'),
 	url(r'^reply_post/$', views.CreatePostAnswerView.as_view(), name='reply_post'),
 	url(r'^render_post_answer/([\w_-]+)/$', views.render_post_answer, name='render_post_answer'),
+	url(r'^delete_post_answer/(?P<pk>[\w_-]+)/$', views.PostAnswerDeleteView.as_view(), name='delete_answer'),
+	url(r'^post_answer_deleted/$', views.answer_deleted, name='deleted_answer'),
 	url(r'^(?P<slug>[\w_-]+)/$', views.ForumDetailView.as_view(), name='view'),
 ]
diff --git a/forum/views.py b/forum/views.py
index 572da42..58c0ae1 100644
--- a/forum/views.py
+++ b/forum/views.py
@@ -208,4 +208,15 @@ def render_post_answer(request, answer):
 	context = {}
 	context['answer'] = last_answer
 
-	return render(request, "post_answers/post_answer_render.html", context)
\ No newline at end of file
+	return render(request, "post_answers/post_answer_render.html", context)
+
+class PostAnswerDeleteView(LoginRequiredMixin, generic.DeleteView):
+	login_url = reverse_lazy("core:home")	
+	redirect_field_name = 'next'
+
+	model = PostAnswer
+	pk_url_kwarg = 'pk'	
+	success_url = reverse_lazy('course:forum:deleted_answer')
+
+def answer_deleted(request):
+	return HttpResponse(_("Post answer deleted successfully."))
\ No newline at end of file
--
libgit2 0.21.2