X-Git-Url: https://apps.delaporte.us/gitweb/matrix-goal-bot.git/blobdiff_plain/bbacd38a568a3a580941390939bd590a9c6d87aa..913b65b598c1a233cce4f69724b4aca8ca6ef02a:/matrix_goal_bot/db.py?ds=sidebyside

diff --git a/matrix_goal_bot/db.py b/matrix_goal_bot/db.py
index 1e90bcf..6a73021 100644
--- a/matrix_goal_bot/db.py
+++ b/matrix_goal_bot/db.py
@@ -106,9 +106,6 @@ class Data():
             cmd = select(Vision).where(Vision.complete_date == None)
             return session.exec(cmd).all()
 
-    def print_ophan_tasks(self):
-        pass
-
     def get_all_goals(self):
         '''List all goals'''
         with Session(self.engine) as session:
@@ -200,17 +197,23 @@ class Data():
 
     def quick_add_task(self, name):
         goal_id = self._get_pref_goal_id()
-        return self.add_task(goal_id, name)
-
-    def add_task(self, goal_id, name):
-        self._set_pref_goal_id(goal_id)
-        result = self.add_with_name(Task, name)
+        task_id = self.add_with_name(Task, name)
         with Session(self.engine) as session:
-            cmd = select(Task).where(Task.name == name)
+            cmd = select(Task).where(Task.id == task_id)
             new = session.exec(cmd).first()
             new.goal_id = goal_id
             session.commit()
-        return result
+        return goal_id
+
+    def remove_task(self, task_id):
+        task_name = ''
+        with Session(self.engine) as session:
+            cmd = select(Task).where(Task.id  == task_id)
+            to_remove = session.exec(cmd).first()
+            task_name = to_remove.name
+            session.delete(to_remove)
+            session.commit()
+        return f'Removed task "{task_name}"'
 
     def assign_vision(self, goal_id, vision_id):
         '''Goals may move between visions'''