Fix for add task

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

diff --git a/matrix_goal_bot/bot.py b/matrix_goal_bot/bot.py
index 9c0bec0e754a6965220a7e4154d6a3ef274b21ea..fd8fdcad380bed6d9e23f56c694f2157958a9cca 100644 (file)
--- a/matrix_goal_bot/bot.py
+++ b/matrix_goal_bot/bot.py
@@ -34,19 +34,23 @@ async def do_goal(match):
 
     Commands:
       help - show this help
+      extra.help - show extended help
       add - add a goal
-      add-task ID - add a task under the goal specified by ID
-      add-vision - add a new vision (collection of goals)
-      close-goal ID - mark goal done by ID
-      close-vision ID - mark vision as complete by ID
+      task ID - add a task under the goal specified by ID
+      close.goal ID - mark goal done by ID
       done ID - mark task done by ID
       list - list incomplete goals
-      list-all - list everything
-      list-done - list completed goals
       print GOAL_ID - print a goal and all tasks
-      print-vision ID - print a vision and all goals
-      set-vision GOAL_ID VISION_ID- associate GOAL_ID with VISION_ID
+    '''
+    ex_help = '''
+      add.vision - add a new vision (collection of goals)
+      close.vision ID - mark vision as complete by ID
+      list.all - list everything
+      print.vision ID - print a vision and all goals
+      set.vision GOAL_ID VISION_ID- associate GOAL_ID with VISION_ID
       visions - list incomplete visions
+      list.done - list completed goals
+      visionless - list goals with no set vision
     '''
 
     if match.command('setup'):
@@ -66,65 +70,68 @@ async def do_goal(match):
         return results
 
 
-    if match.command('list-done'):
+    if match.command('list.done'):
         goals = Data().get_done_goals()
         results = ['# Completed Goals...']
         for goal in goals:
             results.append(str(goal))
         return results
 
-
-    if match.command('add-task'):
-        goal_id = int(match.args()[1])
-        new = ' '.join(match.args()[2:])
-        result = Data().add_task(goal_id=goal_id, name=new)
+    if match.command('task'):
+        new = ' '.join(match.args()[1:])
+        goal_id  = Data().quick_add_task(new)
         goal,tasks = Data().get_goal(goal_id)
         results = pretty_print(goal, tasks)
         return results
-    # return [f'Added task with id {result}']
 
-    if match.command('add-vision'):
+    if match.command('add.vision'):
         new = ' '.join(match.args()[1:])
         result = Data().add_vision(new)
         return [f'Vision added with id {result}']
 
     if match.command('print'):
-        goal_id = int(match.args()[1])
+        if len(match.args()) > 1:
+            goal_id = int(match.args()[1])
+        else:
+            goal_id = Data()._get_pref_goal_id()
+
         logger.debug(f'Searching for goal {goal_id}...')
         goal,tasks = Data().get_goal(goal_id)
         results = pretty_print(goal, tasks)
+
         return results
 
 
-    if match.command('print-vision'):
+    if match.command('print.vision'):
         get_id = int(match.args()[1])
         vision,goals = Data().get_vision(get_id)
         results = pretty_print(vision, goals)
         return results
 
-    if match.command('close-goal'):
+    if match.command('close.goal'):
         goal_id = int(match.args()[1])
         vision_id = Data().complete_goal(goal_id)
         vision,goals = Data().get_vision(vision_id)
         results = pretty_print(vision, goals)
         return results
 
-    if match.command('close-vision'):
+    if match.command('close.vision'):
         vision_id = int(match.args()[1])
         Data().complete_vision(vision_id)
         vision,goals = Data().get_vision(vision_id)
         results = pretty_print(vision, goals)
         return results
 
-
     if match.command('done'):
-        task_id = int(match.args()[1])
-        goal_id = Data().complete_task(task_id)
+        ids = match.args()[1:]
+        for task_id in ids:
+            goal_id = Data().complete_task(int(task_id))
+        # Print the goal of the last closed task
         goal,tasks = Data().get_goal(goal_id)
         results = pretty_print(goal, tasks)
         return results
 
-    if match.command('list-all'):
+    if match.command('list.all'):
         goals = Data().get_all_goals()
         results = ['# Everything is...']
         for goal in goals:
@@ -143,8 +150,17 @@ async def do_goal(match):
             results.append(item)
         return results
 
+    if match.command('visionless'):
+        items = Data().visionless()
+        results = ['# Goals with no vision']
+        if not items:
+            results.append('-- No visionless goals --')
+
+        for item in items:
+            results.append(item)
+        return results
 
-    if match.command('set-vision'):
+    if match.command('set.vision'):
         goal_id = int(match.args()[1])
         vision_id = int(match.args()[2])
         Data().assign_vision(goal_id=goal_id, vision_id=vision_id)
@@ -152,6 +168,19 @@ async def do_goal(match):
         results = pretty_print(vision, goals)
         return results
 
+
+    if match.command('rename'):
+        goal_id = int(match.args()[1])
+        name = ' '.join(match.args()[2:])
+        Data().rename_goal(goal_id=goal_id, name=name)
+        goal,tasks = Data().get_goal(goal_id)
+        results = pretty_print(goal, tasks)
+        return results
+
+    if match.command('extra.help'):
+        return [ex_help]
+
+    # Help text
     return [do_goal.__doc__]
 
 async def goal_bot(room, message, bot):
@@ -167,6 +196,10 @@ async def goal_bot(room, message, bot):
                 await bot.api.send_text_message(
                     room.room_id, str(line))
     except Exception as ex:
-        return ['Ye cannot get ye flask.']
-
+        line ='Ye cannot get ye flask.'
+        await bot.api.send_text_message(
+                    room.room_id, str(line))
+        await bot.api.send_text_message(
+                    room.room_id, str(ex))
+