Don't show unassigned action items when there are none.

- I had left this here since "explicit is better than implicit", but
  because of requests (and also "less is better than more"), I'm
  removing it.

darcs-hash:20090911212639-82ea9-eb11d665119f434148f4362a2082f6cf9fd9112f.gz

1 files changed, 7 insertions, 4 deletions

diff --git a/writers.py b/writers.py
index 92e7300..66ea738 100644
--- a/writers.py
+++ b/writers.py
@@ -516,14 +516,17 @@ class HTML2(_BaseWriter, _CSSmanager):
             doActionItemsPerson = False
         else:
             doActionItemsPerson = True
-        ActionItemsPerson.append("  <li><b>UNASSIGNED</b><ol>")
+        Unassigned = [ ]
+        Unassigned.append("  <li><b>UNASSIGNED</b><ol>")
         numberUnassigned = 0
         for m in self.iterActionItemsUnassigned():
-            ActionItemsPerson.append("    <li>%s</li>"%html(m.line))
+            Unassigned.append("    <li>%s</li>"%html(m.line))
             numberUnassigned += 1
         if numberUnassigned == 0:
-            ActionItemsPerson.append("    <li>(none)</li>")
-        ActionItemsPerson.append('  </ol>\n</li>')
+            Unassigned.append("    <li>(none)</li>")
+        Unassigned.append('  </ol>\n</li>')
+        if numberUnassigned > 1:
+            ActionItemsPerson.extend(Unassigned)
         ActionItemsPerson.append('</ol>')
         ActionItemsPerson = "\n".join(ActionItemsPerson)