From ea13dd0c7895c5aff612f1cb16f9b6c728df7b59 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Oct 20 2017 10:08:42 +0000
Subject: Add small icon showing if issues are blocked or blocking in the issue list


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

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

---

diff --git a/pagure/templates/issues.html b/pagure/templates/issues.html
index 9561b9f..a201724 100644
--- a/pagure/templates/issues.html
+++ b/pagure/templates/issues.html
@@ -215,7 +215,13 @@
                 <span class="oi" data-glyph="comment-square"
                     title="Comments on the ticket"></span>
                   {{issue.user_comments|count}}
-            </span>
+              </span>
+              {% endif %}
+              {% if issue.parents %}
+                <span class="oi" data-glyph="ban" title="Issue blocked by one or more issue(s)"></span>
+              {% endif %}
+              {% if issue.children %}
+                <span class="oi" data-glyph="lock-unlocked" title="Issue blocking one or more issue(s)"></span>
               {% endif %}
               {% for tag in issue.tags %}
                 <a class="label label-info"
diff --git a/tests/test_pagure_flask_ui_issues.py b/tests/test_pagure_flask_ui_issues.py
index eec2912..fe31702 100644
--- a/tests/test_pagure_flask_ui_issues.py
+++ b/tests/test_pagure_flask_ui_issues.py
@@ -1664,6 +1664,19 @@ class PagureFlaskIssuestests(tests.Modeltests):
         self.assertEqual(issue.depending_text, [2])
         self.assertEqual(issue.blocking_text, [])
 
+        # Check the icons showing if the issues are blocking/blocked
+        output = self.app.get('/test/issues/')
+        self.assertEqual(
+            output.data.count(
+                '<span class="oi" data-glyph="lock-unlocked" title="Issue '
+                'blocking one or more issue(s)"></span>'),
+            1)
+        self.assertEqual(
+            output.data.count(
+                '<span class="oi" data-glyph="ban" title="Issue blocked '
+                'by one or more issue(s)"></span>'),
+            1)
+
     @patch('pagure.lib.git.update_git')
     @patch('pagure.lib.notify.send_email')
     def test_update_issue_block(self, p_send_email, p_ugt):