From f6a0788afaba3a1a26352517d5901d62a5a3a9d0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89lie=20Bouttier?= <elie@bouttier.eu>
Date: Sun, 17 Aug 2014 17:42:22 -0700
Subject: [PATCH] post comment directly from issue page

---
 issue/templates/issue/issue.html | 49 +++++++++++++++++++-------------
 issue/views.py                   |  6 ++++
 2 files changed, 35 insertions(+), 20 deletions(-)

diff --git a/issue/templates/issue/issue.html b/issue/templates/issue/issue.html
index 7e6d723..2e38d11 100644
--- a/issue/templates/issue/issue.html
+++ b/issue/templates/issue/issue.html
@@ -2,10 +2,15 @@
 
 {% load humanize %}
 {% load django_markdown %}
+{% load bootstrap_tags %}
 {% load issue_tags %}
 
 {% block issuetab %} class="active"{% endblock %}
 
+{% block media %}
+{{ form.media }}
+{% endblock %}
+
 {% block content %}
 
 
@@ -67,28 +72,32 @@
     </div>
     {% endfor %}
 
-    <div class="row">
-
-      <div class="col-md-3">
-      <a href="{% url 'comment-issue' project.name issue.id %}" class="btn btn-default btn-block">Add a comment</a>
+    <div class="panel panel-default">
+      <div class="panel-heading">
+        <b>Comment</b>
       </div>
-
-      <div class="col-md-3">
-        <a href="{% url 'edit-issue' project.name issue.id %}" class="btn btn-primary btn-block"><span class="glyphicon glyphicon-edit"></span> Edit this issue</a>
+      <div class="panel-body">
+        {% if perm.add_comment %}
+        <form action="{% url 'comment-issue' project.name issue.id %}" method="post" role="form">
+          {% csrf_token %}
+          <div class="form-group">
+            {{ form.comment }}
+          </div>
+          <div class="text-center">
+            {% if perm.manage_issue %}
+            {% if issue.closed %}
+            <a href="{% url 'reopen-issue' project.name issue.id %}" class="btn btn-default"><span class="glyphicon glyphicon-refresh"></span> Reopen this issue</a>
+            {% else %}
+            <a href="{% url 'close-issue' project.name issue.id %}" class="btn btn-default"><span class="glyphicon glyphicon-ok-circle"></span> Close this issue</a>
+            {% endif %}
+            {% endif %}
+            <button type="submit" class="btn btn-success"><span class="glyphicon glyphicon-pencil"></span> Add a comment</button>
+          </div>
+        </form>
+        {% else %}
+        To post a comment, please <a href="{% url 'login' %}?prev={{ request.path }}&next={{ request.path }}">login</a>.
+        {% endif %}
       </div>
-
-      <div class="col-md-3">
-      {% if issue.closed %}
-      <a href="{% url 'reopen-issue' project.name issue.id %}" class="btn btn-info btn-block"><span class="glyphicon glyphicon-refresh"></span> Reopen this issue</a>
-      {% else %}
-      <a href="{% url 'close-issue' project.name issue.id %}" class="btn btn-success btn-block"><span class="glyphicon glyphicon-ok-circle"></span> Close this issue</a>
-      {% endif %}
-      </div>
-
-      <div class="col-md-3">
-        <a href="#" data-item="issue" data-action="{% url 'delete-issue' project.name issue.id %}" data-toggle="modal" data-target="#confirm-delete" class="btn btn-danger btn-block"><span class="glyphicon glyphicon-trash"></span> Delete this issue</a>
-      </div>
-
     </div>
 
   </div>
diff --git a/issue/views.py b/issue/views.py
index 6b44af7..2f5b170 100644
--- a/issue/views.py
+++ b/issue/views.py
@@ -432,12 +432,18 @@ def issue(request, project, issue):
 
     events = issue.events.all()
 
+    if request.user.has_perm('create_comment', project):
+        form = CommentForm(request.POST or None)
+    else:
+        form = None
+
     c = {
         'labels': labels,
         'milestones': milestones,
         'project': project,
         'issue': issue,
         'events': events,
+        'form': form,
     }
 
     return render(request, 'issue/issue.html', c)