Fix bug of same dict being used for different Classes' caches.

[plomtask] / plomtask / days.py

diff --git a/plomtask/days.py b/plomtask/days.py
index 5fe984b7c49596121246bb7e822c8b4bb1536139..258d38dbbf1d7f920f36ca06c9f8292506d16807 100644 (file)
--- a/plomtask/days.py
+++ b/plomtask/days.py
@@ -28,7 +28,6 @@ class Day(BaseModel[str]):
     """Individual days defined by their dates."""
     table_name = 'days'
     to_save = ['comment']
-    id_type = str
 
     def __init__(self, date: str, comment: str = '') -> None:
         self.id_: str = valid_date(date)
@@ -72,13 +71,11 @@ class Day(BaseModel[str]):
         """Retrieve Day by date if in DB (prefer cache), else return None."""
         day, _ = super()._by_id(db_conn, date)
         if day:
-            assert isinstance(day, Day)
             return day
         if not create:
             raise NotFoundException(f'Day not found for date: {date}')
         day = cls(date)
         day.cache()
-        assert isinstance(day, Day)
         return day
 
     @property