From: guillaume.lambert <guillaume.lambert@orange.com>
Date: Sun, 26 Sep 2021 09:04:42 +0000 (+0200)
Subject: Fix JIRA directives for empty query results
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=commitdiff_plain;h=c22f6321cddc0c9be14b08d8a12e805f6808f3cc;p=docs.git

Fix JIRA directives for empty query results

Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I1ce0b8f2fba38a8a42a6c43566e4307adc214196
---

diff --git a/docs/ext/odl-jira.py b/docs/ext/odl-jira.py
index d2d021ecd..a5d2470f7 100644
--- a/docs/ext/odl-jira.py
+++ b/docs/ext/odl-jira.py
@@ -76,20 +76,21 @@ class JiraFixedIssuesDirective(Directive):
             '     - Fix Version(s)',
         ]
 
-        for issue in issues:
-            table.append('   * - .. image:: %s' % issue.fields.issuetype.iconUrl)
-            table.append('          :align: center')
-            table.append('          :alt: %s' % issue.fields.issuetype.name)
-            table.append('     - `%s <https://jira.opendaylight.org/browse/%s>`_' % (issue.key, issue.key))
-            table.append('     - %s' % issue.fields.summary)
-            table.append('     - %s' % issue.fields.resolution)
-            table.append('     - %s' % format_versions(issue.fields.fixVersions))
-
-        table.append('')
-
-        for idx, line in enumerate(table):
-            self.content.data.insert(idx, line)
-            self.content.items.insert(idx, (None, idx))
+        if issues:
+            for issue in issues:
+                table.append('   * - .. image:: %s' % issue.fields.issuetype.iconUrl)
+                table.append('          :align: center')
+                table.append('          :alt: %s' % issue.fields.issuetype.name)
+                table.append('     - `%s <https://jira.opendaylight.org/browse/%s>`_' % (issue.key, issue.key))
+                table.append('     - %s' % issue.fields.summary)
+                table.append('     - %s' % issue.fields.resolution)
+                table.append('     - %s' % format_versions(issue.fields.fixVersions))
+
+            table.append('')
+
+            for idx, line in enumerate(table):
+                self.content.data.insert(idx, line)
+                self.content.items.insert(idx, (None, idx))
 
         node = nodes.container()
         self.state.nested_parse(self.content, self.content_offset, node)
@@ -129,23 +130,24 @@ class JiraKnownIssuesDirective(Directive):
             '     - Fix Version(s)',
         ]
 
-        for issue in issues:
-            fixVersions = format_versions(issue.fields.fixVersions)
-            affectvedVersions = format_versions(issue.fields.versions)
-            table.append('   * - .. image:: %s' % issue.fields.issuetype.iconUrl)
-            table.append('          :align: center')
-            table.append('          :alt: %s' % issue.fields.issuetype.name)
-            table.append('     - `%s <https://jira.opendaylight.org/browse/%s>`_' % (issue.key, issue.key))
-            table.append('     - %s' % issue.fields.summary)
-            table.append('     - %s' % issue.fields.status)
-            table.append('     - %s' % fixVersions)
-            table.append('     - %s' % affectvedVersions)
-
-        table.append('')
-
-        for idx, line in enumerate(table):
-            self.content.data.insert(idx, line)
-            self.content.items.insert(idx, (None, idx))
+        if issues:
+            for issue in issues:
+                fixVersions = format_versions(issue.fields.fixVersions)
+                affectvedVersions = format_versions(issue.fields.versions)
+                table.append('   * - .. image:: %s' % issue.fields.issuetype.iconUrl)
+                table.append('          :align: center')
+                table.append('          :alt: %s' % issue.fields.issuetype.name)
+                table.append('     - `%s <https://jira.opendaylight.org/browse/%s>`_' % (issue.key, issue.key))
+                table.append('     - %s' % issue.fields.summary)
+                table.append('     - %s' % issue.fields.status)
+                table.append('     - %s' % fixVersions)
+                table.append('     - %s' % affectvedVersions)
+
+            table.append('')
+
+            for idx, line in enumerate(table):
+                self.content.data.insert(idx, line)
+                self.content.items.insert(idx, (None, idx))
 
         node = nodes.container()
         self.state.nested_parse(self.content, self.content_offset, node)