Require explicit call to Day.save to have any Day committed to DB.

[plomtask] / plomtask / days.py

diff --git a/plomtask/days.py b/plomtask/days.py
index 8db9f15b7362ede61c70265e4d5eb8a4c2c0c626..17bb46329760beb078a6497d68bd7b0a39a05e3e 100644 (file)
--- a/plomtask/days.py
+++ b/plomtask/days.py
@@ -1,16 +1,46 @@
-#!/usr/bin/env python3
 """Collecting Day and date-related items."""
 from datetime import datetime
+from sqlite3 import Row
+from plomtask.misc import HandledException
+from plomtask.db import DatabaseConnection
 
 DATE_FORMAT = '%Y-%m-%d'
 
 
+def date_valid(date: str):
+    """Validate date against DATE_FORMAT, return Datetime or None."""
+    try:
+        result = datetime.strptime(date, DATE_FORMAT)
+    except (ValueError, TypeError):
+        return None
+    return result
+
+
 class Day:
     """Individual days defined by their dates."""
 
     def __init__(self, date: str):
         self.date = date
-        self.datetime = datetime.strptime(date, DATE_FORMAT)
+        self.datetime = date_valid(self.date)
+        if not self.datetime:
+            raise HandledException(f'Given date of wrong format: {self.date}')
+
+    def save(self, db_conn: DatabaseConnection):
+        """Add (or re-write) self to database."""
+        db_conn.exec('REPLACE INTO days VALUES (?)', (self.date,))
+
+    @classmethod
+    def from_table_row(cls, row: Row):
+        """Make new Day from database row."""
+        return cls(row[0])
+
+    @classmethod
+    def all(cls, db_conn: DatabaseConnection):
+        """Return list of all Days in database."""
+        days = []
+        for row in db_conn.exec('SELECT * FROM days'):
+            days += [cls.from_table_row(row)]
+        return days
 
     @property
     def weekday(self):