Update list formatting

[matrix-goal-bot.git] / matrix_goal_bot / bot.py

diff --git a/matrix_goal_bot/bot.py b/matrix_goal_bot/bot.py
index d62d1881f6d74a061445dae84a3adfd7af5636f2..3da43ce9cef81f0fac125708f20c44d9d2700ca8 100644 (file)
--- a/matrix_goal_bot/bot.py
+++ b/matrix_goal_bot/bot.py
@@ -21,7 +21,7 @@ More commands:
     add.vision - add a new vision (collection of goals)
     close.vision ID - mark vision as complete by ID
     dump - dump the full data set - BEWARE - so many messages
     add.vision - add a new vision (collection of goals)
     close.vision ID - mark vision as complete by ID
     dump - dump the full data set - BEWARE - so many messages

-    later GOAL_ID - put off goal for 14 days
+    later VISION_ID - put off all goals in VISION for 14 days

     list.later - list all 'later' goals
     print.vision ID - print a vision and all goals
     set.vision GOAL_ID VISION_ID- associate GOAL_ID with VISION_ID
     list.later - list all 'later' goals
     print.vision ID - print a vision and all goals
     set.vision GOAL_ID VISION_ID- associate GOAL_ID with VISION_ID


@@ -90,9 +90,16 @@ class GoalBot():
         if match.prefix():
             try:
                 result_lines = await self._handle_match(match)
         if match.prefix():
             try:
                 result_lines = await self._handle_match(match)

+
+                message_lines = []

                 for line in result_lines:
                 for line in result_lines:

-                    await self.bot.api.send_text_message(
-                        room.room_id, str(line))
+                    message_lines.append(str(line))
+
+                message = '\n'.join(message_lines)
+
+                await self.bot.api.send_text_message(
+                    room.room_id, message)
+

             except Exception as ex:
                 logger.warn(f'Caught exception: {str(ex)}')
                 dumb_msg_idx = len(str(ex)) % len(snark)
             except Exception as ex:
                 logger.warn(f'Caught exception: {str(ex)}')
                 dumb_msg_idx = len(str(ex)) % len(snark)


@@ -126,7 +133,8 @@ class GoalBot():
         if match.command('list'):
             logger.debug('Asked to list current goals...')
             goals = self.data.get_current_goals()
         if match.command('list'):
             logger.debug('Asked to list current goals...')
             goals = self.data.get_current_goals()

-            results = ['# Goals list...']
+            results = ['# Goals list']
+            results.append('------------']

             for goal in goals:
                 results.append(str(goal))
             return results
             for goal in goals:
                 results.append(str(goal))
             return results


@@ -197,10 +205,13 @@ class GoalBot():
             return results
 
         if match.command('later'):
             return results
 
         if match.command('later'):

-            goal_id = int(match.args()[1])
-            vision_id = self.data.put_off_goal(goal_id)
-            vision,goals = self.data.get_vision(vision_id)
-            results = pretty_print(vision, goals)
+            vision_id = int(match.args()[1])
+            self.data.put_off_vision(vision_id)
+            # Want to see what we just cleaned up
+            goals = self.data.get_current_goals()
+            results = ['# Goals list...']
+            for goal in goals:
+                results.append(str(goal))

             return results
 
 
             return results