From a225965cd09ccc16a82c37da0f45a298912a73ed Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Jan 31 2019 20:04:48 +0000
Subject: Fix not showing the edit and delete buttons when they won't work


If there is no identifier (which is the case when the comment is added
using ajax rather than being received by SSE), the edit and delete
buttons on the comment won't work, so there is no point in showing them,
the users will just have to refresh the page to see them.

In addition, this commit removes the link to the comment itself, for the
same reason.

Fixes https://pagure.io/pagure/issue/4210

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure/static/request_ev.js b/pagure/static/request_ev.js
index 7c39e6d..4fb56b7 100644
--- a/pagure/static/request_ev.js
+++ b/pagure/static/request_ev.js
@@ -41,23 +41,25 @@ add_comment = function(data, username) {
     + '      <a href="/user/' + data.comment_user + '"'
     + '        class="notblue font-weight-bold">'+data.comment_user+'</a>'
     + '      <a class="notblue" title="Permalink to this headline"'
-    + '          href="#comment-' + data.comment_id + '">'
-    + '        <span>commented seconds ago</span>'
+    if (data.comment_id) {
+      _data += ' href="#comment-' + data.comment_id + '"';
+    }
+    _data += '><span>commented seconds ago</span>'
     + '      </a>'
     + '    </div>';
 
-    if ( data.comment_user == username && data.comment_id !== 'undefined') {
-        _data = _data
-    + '    <div class="issue_actions ml-auto">'
-    + '      <a href="/' + data.project + '/pull-request/' + data.request_id + '/comment/' + data.comment_id + '/edit"'
-    + '         class="btn btn-outline-primary border-0" data-comment="' + data.comment_id + '" data-objid="' + data.request_id + '">'
-    + '        <i class="fa fa-pencil"  title="Edit comment"></i>'
-    + '      </a>'
-    + '      <button class="btn btn-outline-primary border-0" title="Remove comment" name="drop_comment" '
-    + '         value="' + data.comment_id + '" type="submit" onclick="return confirm(\'Do you really want to remove this comment?\');" >'
-    + '        <i class="fa fa-trash"></i>'
-    + '      </button>'
-    + '    </div>'
+    if ( data.comment_user == username && data.comment_id !== undefined) {
+      _data = _data
+      + '    <div class="issue_actions ml-auto">'
+      + '      <a href="/' + data.project + '/pull-request/' + data.request_id + '/comment/' + data.comment_id + '/edit"'
+      + '         class="btn btn-outline-primary border-0" data-comment="' + data.comment_id + '" data-objid="' + data.request_id + '">'
+      + '        <i class="fa fa-pencil"  title="Edit comment"></i>'
+      + '      </a>'
+      + '      <button class="btn btn-outline-primary border-0" title="Remove comment" name="drop_comment" '
+      + '         value="' + data.comment_id + '" type="submit" onclick="return confirm(\'Do you really want to remove this comment?\');" >'
+      + '        <i class="fa fa-trash"></i>'
+      + '      </button>'
+      + '    </div>';
     }
 
     _data = _data