Make listing inventories/playbooks look nicer; add the Jinja2 template;

ansible.py

@@ -81,14 +81,12 @@ def ansible_list(self, mess, objects=None):
         playbooks = []
         inventories = []
         # walk() comes from package "os"
-        for (dirpath, dirnames, filenames) in walk(self.config['PLAYBOOK_DIR']):
-            playbooks.extend(filenames)
-            break
-        for (dirpath, dirnames, filenames) in walk(self.config['INVENTORY_DIR']):
-            inventories.extend(filenames)
-            break
-        # TODO: below code sucks rocks, but it's 11pm and I have a bottle of
-        # cold beer waiting for me.
-        if objects == 'playbooks': return { 'objects': objects, 'objlist': playbooks }
-        elif objects == 'inventories': return { 'objects': objects, 'objlist': inventories }
-        return { 'objects': 'all', 'objlist': playbooks + inventories }
+        if objects is 'playbooks' or objects is 'all':
+            for (dirpath, dirnames, filenames) in walk(self.config['PLAYBOOK_DIR']):
+                playbooks.extend(filenames)
+                break
+        if objects is 'inventories' or objects is 'all':
+            for (dirpath, dirnames, filenames) in walk(self.config['INVENTORY_DIR']):
+                inventories.extend(filenames)
+                break
+        return { 'playbooks': playbooks, 'inventories': inventories }

templates/list_objects.md

@@ -0,0 +1,19 @@
+{%- if playbooks is defined %}
+Playbooks:
+
+```
+{% for obj in playbooks %}
+    * {{ obj }}
+{%- endfor %}
+```
+{%- endif %}
+{%- if inventories is defined %}
+Inventory files:
+
+```
+{% for obj in inventories %}
+    * {{ obj }}
+{%- endfor %}
+```
+{%- endif %}
+