Allow users and funders to comment back and forth on their applications

app/views.py

@@ -247,13 +247,19 @@ def event_new(request):
 @requester_only
 @require_http_methods(["GET", "POST"])
 def event_edit(request, event_id):
+    user = request.user
     event = Event.objects.get(pk=event_id)
     if event.over:
         return redirect('event-show', event_id)
     if request.method == 'POST':
         form = EventForm(request.POST, instance=event)
         if form.is_valid():
             try:
+                if request.POST.get('new-comment'):
+                    messages.success(request, "Saved comment!")
+                    comment = Comment(comment=request.POST['new-comment'],
+                                funder=user.profile, event=event)
+                    comment.save()
                 with transaction.atomic():
                     event = form.save()
                     save_from_form(event, request.POST)
@@ -279,6 +285,11 @@ def event_show(request, event_id):
     user = request.user
     event = Event.objects.get(pk=event_id)
     if request.method == 'POST':  # TODO: should really be PUT
+        if request.POST.get('new-comment'):
+                messages.success(request, "Saved comment!")
+                comment = Comment(comment=request.POST['new-comment'],
+                                  funder=user.profile, event=event)
+                comment.save()
         if user.profile.is_funder:
             grants = []
             for item in event.item_set.all():
@@ -312,11 +323,6 @@ def event_show(request, event_id):
                     user.profile.notify_osa(event, grants)
                 except smtplib.SMTPException:
                     pass
-            if request.POST.get('new-comment'):
-                messages.success(request, "Saved comment!")
-                comment = Comment(comment=request.POST['new-comment'],
-                                  funder=user.profile, event=event)
-                comment.save()
             return redirect('event-show', event_id)
         else:
             return redirect(EVENTS_HOME)

penncfa/templates/app/application-show.html

@@ -39,20 +39,18 @@
     {% application user event None %}
 
     <div class="d-print-none">
-    {% if user and user.profile.is_funder %}
     <p>To submit your funding changes and any comments, click "Save." <br>
       To share a read-only copy of this application, click "Share."</p>
     <div class="form-group">
       <a class="btn btn-secondary" href="{% url 'events' %}">Cancel</a>
       <button class="btn btn-primary ml-1" type="submit" name="submit">Save</button>
-      <a class="btn btn-info ml-1" href="#" data-target="#share" data-toggle="modal">Share</a>
+      {% if user and user.profile.is_funder %}
       <button id="no-fund" class="btn btn-danger ml-1" type="submit" name="submit">Do Not Fund</button>
-    </div>
-    {% else %}
-    <div class="form-group">
+      <a class="btn btn-info ml-1" href="#" data-target="#share" data-toggle="modal">Share</a>
+      {% else %}
       <a class="btn btn-primary" href="{% url 'events' %}">Done</a>
+      {% endif %}
     </div>
-    {% endif %}
     </div>
   </form>
 </div>

penncfa/templates/app/templatetags/application.html

@@ -234,9 +234,13 @@ <h2>
     {% empty %}
     There are no comments yet.
     {% endfor %}
-    {% if user.profile.is_funder %}
+    {% if user == event.requester.user or user.profile.is_funder %}
+    {% if user and user.profile.is_funder %}
     <p>Specify any conditions and stipulations here.</p>
-    <textarea name="new-comment" class="span8 form-control"></textarea>
+    {% else %}
+    <p>Leave a comment for your funder here.</p>
+    {% endif %}
+    <textarea name="new-comment" class="span8 form-control"></textarea> 
     {% endif %}
   </div>
 </section>