From c30730fa871f89bfff0918b8d4ad77e3be13fb87 Mon Sep 17 00:00:00 2001
From: Paul W. Frields <stickster@gmail.com>
Date: Apr 04 2016 14:12:50 +0000
Subject: [PATCH 1/3] Remove reply button for non-authn users


---

diff --git a/pagure/templates/_formhelper.html b/pagure/templates/_formhelper.html
index d9dd2e7..0877448 100644
--- a/pagure/templates/_formhelper.html
+++ b/pagure/templates/_formhelper.html
@@ -138,9 +138,11 @@
       {% endif %}
       <div class="issue_action icon pull-xs-right p-b-1">
         <div class="btn-group" role="group" aria-label="Basic example">
-          <a class="reply btn btn-secondary btn-sm" data-toggle="tooltip" title="Reply to this comment - loose formating">
-            <span class="oi" data-glyph="share-boxed"></span>
-          </a>
+          {% if id != 0 and g.fas_user and comment.parent.status in [True, 'Open'] or repo_admin %}
+            <a class="reply btn btn-secondary btn-sm" data-toggle="tooltip" title="Reply to this comment - loose formating">
+              <span class="oi" data-glyph="share-boxed"></span>
+            </a>
+          {% endif %}
           {% if id != 0 and g.fas_user and (
               (comment.parent.status in [True, 'Open'] and g.fas_user.username == comment.user.username)
               or repo_admin) %}
@@ -176,10 +178,12 @@
   </section>
   <div class="issue_action icon pull-xs-right">
     <div class="btn-group" role="group">
-      <a class="btn btn-secondary btn-sm reply"
-        title="Reply to the initial comment - loose formatting">
-        <span class="oi" data-glyph="share-boxed"></span>
-      </a>
+      {% if repo_admin or g.fas_user %}
+        <a class="btn btn-secondary btn-sm reply"
+          title="Reply to the initial comment - loose formatting">
+          <span class="oi" data-glyph="share-boxed"></span>
+        </a>
+      {% endif %}
       {% if repo_admin or (
         g.fas_user and g.fas_user.username == comment.user.username) %}
       <a class="btn btn-primary btn-sm" href="{{

From f3a30c5bf26f47c24c29bbdcba0a93efd5629097 Mon Sep 17 00:00:00 2001
From: Paul W. Frields <stickster@gmail.com>
Date: Apr 04 2016 14:19:52 +0000
Subject: [PATCH 2/3] repo_admin not needed due to g.fas_user


---

diff --git a/pagure/templates/_formhelper.html b/pagure/templates/_formhelper.html
index 0877448..a871424 100644
--- a/pagure/templates/_formhelper.html
+++ b/pagure/templates/_formhelper.html
@@ -138,7 +138,7 @@
       {% endif %}
       <div class="issue_action icon pull-xs-right p-b-1">
         <div class="btn-group" role="group" aria-label="Basic example">
-          {% if id != 0 and g.fas_user and comment.parent.status in [True, 'Open'] or repo_admin %}
+          {% if id != 0 and g.fas_user and comment.parent.status in [True, 'Open'] %}
             <a class="reply btn btn-secondary btn-sm" data-toggle="tooltip" title="Reply to this comment - loose formating">
               <span class="oi" data-glyph="share-boxed"></span>
             </a>
@@ -178,7 +178,7 @@
   </section>
   <div class="issue_action icon pull-xs-right">
     <div class="btn-group" role="group">
-      {% if repo_admin or g.fas_user %}
+      {% if g.fas_user %}
         <a class="btn btn-secondary btn-sm reply"
           title="Reply to the initial comment - loose formatting">
           <span class="oi" data-glyph="share-boxed"></span>

From 741e7eb5088d56c0cda60c72e0be073b08d69682 Mon Sep 17 00:00:00 2001
From: Paul W. Frields <stickster@gmail.com>
Date: Apr 04 2016 14:42:20 +0000
Subject: [PATCH 3/3] Remove other unnecessary repo_admin


---

diff --git a/pagure/templates/_formhelper.html b/pagure/templates/_formhelper.html
index a871424..774f6e0 100644
--- a/pagure/templates/_formhelper.html
+++ b/pagure/templates/_formhelper.html
@@ -144,8 +144,7 @@
             </a>
           {% endif %}
           {% if id != 0 and g.fas_user and (
-              (comment.parent.status in [True, 'Open'] and g.fas_user.username == comment.user.username)
-              or repo_admin) %}
+              comment.parent.status in [True, 'Open'] and g.fas_user.username == comment.user.username) %}
           <a class="btn btn-secondary btn-sm edit_btn" href="{{
             '%s/comment/%s/edit' % (request.base_url, comment.id) }}"
             data-comment="{{ comment.id }}" data-objid="{{ issueid }}">
@@ -153,8 +152,7 @@
           </a>
           {% endif %}
           {% if id != 0 and g.fas_user and (
-              (comment.parent.status in [True, 'Open'] and g.fas_user.username == comment.user.username)
-              or repo_admin) %}
+              comment.parent.status in [True, 'Open'] and g.fas_user.username == comment.user.username) %}
           <button class="btn btn-secondary btn-sm" type="submit" name="drop_comment" value="{{ comment.id }}"
             onclick="return confirm('Do you really want to remove this comment?');"
             title="Remove comment">