From 09369ecd8c1e02f8c33bf690a7b9cd0a4c9245e8 Mon Sep 17 00:00:00 2001
From: Vivek Anand <vivekanand1101@gmail.com>
Date: Aug 07 2017 20:04:10 +0000
Subject: my_issues: only show issues if the issue tracker is on for the project


Signed-off-by: Vivek Anand <vivekanand1101@gmail.com>

---

diff --git a/pagure/templates/user_issues.html b/pagure/templates/user_issues.html
index c25a0ad..e5150cc 100644
--- a/pagure/templates/user_issues.html
+++ b/pagure/templates/user_issues.html
@@ -42,7 +42,7 @@
         <tfoot>
         </tfoot>
         <tbody>
-        {% for issue in issues %}
+        {% for issue in issues if issue.project.settings.get('issue_tracker', True) %}
           <tr class="issue-status issue-status-{{issue.status|lower}} loop-index-{{outer_loop.index}}"
               {%- if issue.status|lower != "open" %} style="display:none;"{% endif %}>
                 <td class="nowrap"><span class="label label-default">#{{ issue.id }}</span></td>
diff --git a/tests/test_pagure_flask_ui_app.py b/tests/test_pagure_flask_ui_app.py
index 6dd97d4..8ab1813 100644
--- a/tests/test_pagure_flask_ui_app.py
+++ b/tests/test_pagure_flask_ui_app.py
@@ -1350,6 +1350,52 @@ class PagureFlaskApptests(tests.Modeltests):
             output.data.count('<tr class="issue-status issue-status-open'),
             3)
 
+    @patch(
+        'pagure.lib.git.update_git', MagicMock(return_value=True))
+    @patch(
+        'pagure.lib.notify.send_email', MagicMock(return_value=True))
+    def test_view_my_issues_disabled(self):
+        """Test the view_user_issues endpoint when the project disabled issue
+        tracking."""
+        # Create the issue
+        tests.create_projects(self.session)
+        repo = pagure.lib._get_project(self.session, 'test')
+        msg = pagure.lib.new_issue(
+            session=self.session,
+            repo=repo,
+            title='Test issue #1',
+            content='We should work on this for the second time',
+            user='pingou',
+            status='Open',
+            ticketfolder=None
+        )
+        self.session.commit()
+        self.assertEqual(msg.title, 'Test issue #1')
+
+        # Before
+        output = self.app.get('/user/pingou/issues')
+        self.assertEqual(output.status_code, 200)
+        self.assertIn('Test issue #1', output.data)
+        self.assertEqual(
+            output.data.count('<tr class="issue-status issue-status-open'),
+            1)
+
+        # Disable issue tracking
+        repo = pagure.lib._get_project(self.session, 'test')
+        settings = repo.settings
+        settings['issue_tracker'] = False
+        repo.settings = settings
+        self.session.add(repo)
+        self.session.commit()
+
+        # After
+        output = self.app.get('/user/pingou/issues')
+        self.assertEqual(output.status_code, 200)
+        self.assertNotIn('Test issue #1', output.data)
+        self.assertEqual(
+            output.data.count('<tr class="issue-status issue-status-open'),
+            0)
+
     def test_view_my_issues_tickets_turned_off(self):
         """Test the view_user_issues endpoint when the user exists and
         and ENABLE_TICKETS is False """