From cf57687068df5c26841367e4b399b40cd1f98b32 Mon Sep 17 00:00:00 2001
From: Hoffelhas <xela@live.nl>
Date: Sun, 15 Jan 2023 00:24:17 +0100
Subject: [PATCH] Implemented the filtered lists. Significant decrease of api
 requests and noticable speed-up!

---
 autodoist.py | 22 +++++-----------------
 1 file changed, 5 insertions(+), 17 deletions(-)

diff --git a/autodoist.py b/autodoist.py
index 72296dd..622d5e1 100644
--- a/autodoist.py
+++ b/autodoist.py
@@ -981,26 +981,14 @@ def autodoist_magic(args, api, connection):
 
     # Get all todoist info
     try:
-        projects = api.get_projects() # To save on request to stay under the limit
-        # all_sections = api.get_sections() # To save on request to stay under the limit
+        all_projects = api.get_projects() # To save on request to stay under the limit
+        all_sections = api.get_sections() # To save on request to stay under the limit
         all_tasks = api.get_tasks()
 
-        # Build a dict of all project and task numbers
-        all_projects = [x.project_id for x in all_tasks]
-        all_sections = [x.section_id for x in all_tasks]
-        h = defaultdict(list)
-        for k, v in zip(all_projects, all_sections):
-            h[k].append(v)
-        dict_project_section = dict(h)
-
-        # Store only unique values:
-        for key in dict_project_section:
-            dict_project_section[key] = list(set(dict_project_section[key]))
-
     except Exception as error:
         print(error)
 
-    for project in projects:
+    for project in all_projects:
 
         # Skip processing inbox as intended feature
         if project.is_inbox_project:
@@ -1023,7 +1011,7 @@ def autodoist_magic(args, api, connection):
 
         # Get all tasks for the project
         try:
-            project_tasks = api.get_tasks(project_id=project.id)  # TODO: call them all once, and use filter here instead.
+            project_tasks = [t for t in all_tasks if t.project_id == project.id]
         except Exception as error:
             print(error)
 
@@ -1054,7 +1042,7 @@ def autodoist_magic(args, api, connection):
 
         # Get all sections and add the 'None' section too.
         try:
-            sections = api.get_sections(project_id=project.id) # TODO: call them all once, and use filter here instead.
+            sections = [s for s in all_sections if s.project_id == project.id]
             sections.insert(0, Section(None, None, 0, project.id))
         except Exception as error:
             print(error)