Refactor save and remove methods of BaseObject subclasses.

[plomtask] / plomtask / http.py

diff --git a/plomtask/http.py b/plomtask/http.py
index 316fd0805588a00b59dc28fbd8094fc8634dcd87..adac957f5a1090267fb7807875eabcec2c8c37d3 100644 (file)
--- a/plomtask/http.py
+++ b/plomtask/http.py
@@ -212,6 +212,10 @@ class TaskHandler(BaseHTTPRequestHandler):
             todo = Todo.by_id(self.conn, todo_id)
             todo.is_done = True
             todo.save(self.conn)
+            for condition in todo.enables:
+                condition.save(self.conn)
+            for condition in todo.disables:
+                condition.save(self.conn)
         return f'/day?date={date}'
 
     def do_POST_todo(self) -> str:
@@ -259,7 +263,7 @@ class TaskHandler(BaseHTTPRequestHandler):
                                self.form_data.get_all_int('condition'))
         process.set_enables(self.conn, self.form_data.get_all_int('enables'))
         process.set_disables(self.conn, self.form_data.get_all_int('disables'))
-        process.save_core(self.conn)
+        process.save(self.conn)
         process.explicit_steps = []
         steps: list[tuple[int | None, int, int | None]] = []
         for step_id in self.form_data.get_all_int('steps'):
@@ -287,6 +291,7 @@ class TaskHandler(BaseHTTPRequestHandler):
             condition.remove(self.conn)
             return '/conditions'
         condition = Condition.by_id(self.conn, id_, create=True)
+        condition.is_active = self.form_data.get_all_str('is_active') != []
         condition.title.set(self.form_data.get_str('title'))
         condition.description.set(self.form_data.get_str('description'))
         condition.save(self.conn)