Fixup later command

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

diff --git a/matrix_goal_bot/db.py b/matrix_goal_bot/db.py
index 5056f551da56fd04c83953a540e2591262684305..a9d1b879d6b9b6f3d5005378b13deccd4d10484c 100644 (file)
--- a/matrix_goal_bot/db.py
+++ b/matrix_goal_bot/db.py
@@ -2,6 +2,7 @@ import logging
 import os
 from datetime import datetime as dt
 from datetime import timedelta
 import os
 from datetime import datetime as dt
 from datetime import timedelta

+from typing import List

 
 from sqlmodel import Field, Session, SQLModel, create_engine, select, text, Relationship
 
 
 from sqlmodel import Field, Session, SQLModel, create_engine, select, text, Relationship
 


@@ -83,6 +84,22 @@ def _get_with_name(cls, name):
     return select(cls).where(func.lower(cls.name) == func.lower(name))
 
 
     return select(cls).where(func.lower(cls.name) == func.lower(name))
 
 

+def _get_goals_by_vision(goals) -> List[str]:
+    results = []
+    visions = {}
+
+    for goal in goals:
+        vision = str(goal.vision)
+        if not vision in visions:
+            visions[vision] = []
+        visions[vision].append(str(goal))
+    for vision in visions:
+        results.append(f' ## {vision}')
+        for item in visions[vision]:
+            results.append(item)
+    return results
+
+

 class Data():
     def __init__(self, db_file=None):
         if not db_file:
 class Data():
     def __init__(self, db_file=None):
         if not db_file:


@@ -136,26 +153,30 @@ class Data():
             cmd = select(Vision).where(Vision.complete_date == None)
             return session.exec(cmd).all()
 
             cmd = select(Vision).where(Vision.complete_date == None)
             return session.exec(cmd).all()
 

-    def get_all_goals(self):
+    def get_all_goals(self) -> List[str]:

         '''List all goals'''
         '''List all goals'''

+        results = []

         with Session(self.engine) as session:
             cmd = select(Goal).where(Goal.complete_date == None)
         with Session(self.engine) as session:
             cmd = select(Goal).where(Goal.complete_date == None)

-            return session.exec(cmd).all()
+            goals = session.exec(cmd).all()
+            results.append(_get_goals_by_vision(goals))
+        return results
+

 
     def get_current_goals(self):
         '''List current goals'''
         with Session(self.engine) as session:
             cmd = select(Goal).where(Goal.complete_date == None).where(Goal.start_date <= dt.today())
 
     def get_current_goals(self):
         '''List current goals'''
         with Session(self.engine) as session:
             cmd = select(Goal).where(Goal.complete_date == None).where(Goal.start_date <= dt.today())

-            results = session.exec(cmd).all()
-            logger.debug(f'Current goals: {results}')
-            return results
+            goals = session.exec(cmd).all()
+            logger.debug(f'Current goals: {goals}')
+            return _get_goals_by_vision(goals)

 
     def get_done_goals(self):
         '''List completed goals'''
         with Session(self.engine) as session:
             cmd = select(Goal).where(Goal.complete_date != None)
 
     def get_done_goals(self):
         '''List completed goals'''
         with Session(self.engine) as session:
             cmd = select(Goal).where(Goal.complete_date != None)

-            return session.exec(cmd).all()
-
+            goals = session.exec(cmd).all()
+            return _get_goals_by_vision(goals)

 
     def get_goal(self, goal_id):
         self._set_pref_goal_id(goal_id)
 
     def get_goal(self, goal_id):
         self._set_pref_goal_id(goal_id)


@@ -203,15 +224,15 @@ class Data():
         return vision_id
 
 
         return vision_id
 
 

-    def put_off_goal(self, goal_id):
-        vision_id = -1
+    def put_off_vision(self, vision_id):

         with Session(self.engine) as session:
         with Session(self.engine) as session:

-            cmd = select(Goal).where(Goal.id == goal_id)
-            goal = session.exec(cmd).first()
-            goal.start_date = dt.today() + timedelta(days=14)
-            vision_id = goal.vision_id
+            # vision = session.exec(cmd).first()
+            cmd = select(Goal).where(Goal.vision_id == vision_id)
+            goals = session.exec(cmd).all()
+            for goal in goals:
+                goal.start_date = dt.today() + timedelta(days=14)

             session.commit()
             session.commit()

-        return vision_id
+        return True

 
     def start_goal(self, goal_id):
         vision = None
 
     def start_goal(self, goal_id):
         vision = None


@@ -299,11 +320,12 @@ class Data():
             goal = session.exec(cmd).first()
             goal.vision_id = vision_id
             session.commit()
             goal = session.exec(cmd).first()
             goal.vision_id = vision_id
             session.commit()

+        return True

 
     def visionless(self):
         '''Get goals with no vision'''
         with Session(self.engine) as session:
 
     def visionless(self):
         '''Get goals with no vision'''
         with Session(self.engine) as session:

-            cmd = select(Goal).where(Goal.vision_id  == None)
+            cmd = select(Goal).where(Goal.vision_id  == None).where(Goal.complete_date == None)

             return session.exec(cmd).all()
 
 
             return session.exec(cmd).all()