X-Git-Url: https://apps.delaporte.us/gitweb/matrix-goal-bot.git/blobdiff_plain/08aa0bb297ebc364cd6b0bc725169c9116454baa..6c0064a26d3facaea4590bc9d02558e6e6fdb937:/matrix_goal_bot/db.py

diff --git a/matrix_goal_bot/db.py b/matrix_goal_bot/db.py
index 8301733..bc8cdbc 100644
--- a/matrix_goal_bot/db.py
+++ b/matrix_goal_bot/db.py
@@ -4,6 +4,7 @@ from datetime import datetime as dt
 
 from sqlmodel import Field, Session, SQLModel, create_engine, select, text, Relationship
 
+logger = logging.getLogger(__name__)
 
 DATE_FORMAT = '%Y-%m-%d'
 
@@ -11,10 +12,6 @@ def today():
     return dt.now().strftime(DATE_FORMAT)
 
 
-logger = logging.getLogger(__name__)
-logging.basicConfig(filename='goals.log', encoding='utf-8', level=logging.DEBUG)
-
-
 class Vision(SQLModel, table=True):
     id: int | None = Field(default=None, primary_key=True)
     name: str = Field(default=None, unique=True)
@@ -70,7 +67,7 @@ def pretty_print(goal, tasks):
 class Data():
     def __init__(self, db_file=None):
         if not db_file:
-            db_file = os.path.join(os.environ.get('MATRIX_DATA_DIR', ''), 'goals.db')
+            db_file = os.path.join(os.environ.get('MATRIX_DATA_DIR', ''), 'goal.db')
 
         self.file = db_file
         self.engine = self.get_engine()
@@ -197,16 +194,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):
-        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'''