Restore compact_dict usage in update_task payload builders

Reapply compact_dict in sync/async update_task to keep payload construction consistent with the rest of the API surface.

Author: goncalossilva

todoist_api_python/api.py

@@ -350,7 +350,7 @@ def add_task_quick(
         )
         return Task.from_dict(task_data)
 
-    def update_task(  # noqa: PLR0912
+    def update_task(
         self,
         task_id: str,
         *,
@@ -396,37 +396,27 @@ def update_task(  # noqa: PLR0912
         """
         endpoint = get_api_url(f"{TASKS_PATH}/{task_id}")
 
-        data: dict[str, Any] = {}
-        if content is not None:
-            data["content"] = content
-        if description is not None:
-            data["description"] = description
-        if labels is not None:
-            data["labels"] = labels
-        if priority is not None:
-            data["priority"] = priority
-        if due_string is not None:
-            data["due_string"] = due_string
-        if due_lang is not None:
-            data["due_lang"] = due_lang
-        if due_date is not None:
-            data["due_date"] = format_date(due_date)
-        if due_datetime is not None:
-            data["due_datetime"] = format_datetime(due_datetime)
-        if assignee_id is not None:
-            data["assignee_id"] = assignee_id
-        if day_order is not None:
-            data["day_order"] = day_order
-        if collapsed is not None:
-            data["collapsed"] = collapsed
-        if duration is not None:
-            data["duration"] = duration
-        if duration_unit is not None:
-            data["duration_unit"] = duration_unit
-        if deadline_date is not None:
-            data["deadline_date"] = format_date(deadline_date)
-        if deadline_lang is not None:
-            data["deadline_lang"] = deadline_lang
+        data = compact_dict(
+            content=content,
+            description=description,
+            labels=labels,
+            priority=priority,
+            due_string=due_string,
+            due_lang=due_lang,
+            due_date=format_date(due_date) if due_date is not None else None,
+            due_datetime=(
+                format_datetime(due_datetime) if due_datetime is not None else None
+            ),
+            assignee_id=assignee_id,
+            day_order=day_order,
+            collapsed=collapsed,
+            duration=duration,
+            duration_unit=duration_unit,
+            deadline_date=(
+                format_date(deadline_date) if deadline_date is not None else None
+            ),
+            deadline_lang=deadline_lang,
+        )
 
         task_data: dict[str, Any] = post(
             self._client,

todoist_api_python/api_async.py

@@ -374,7 +374,7 @@ async def add_task_quick(
         )
         return Task.from_dict(task_data)
 
-    async def update_task(  # noqa: PLR0912
+    async def update_task(
         self,
         task_id: str,
         *,
@@ -420,37 +420,27 @@ async def update_task(  # noqa: PLR0912
         """
         endpoint = get_api_url(f"{TASKS_PATH}/{task_id}")
 
-        data: dict[str, Any] = {}
-        if content is not None:
-            data["content"] = content
-        if description is not None:
-            data["description"] = description
-        if labels is not None:
-            data["labels"] = labels
-        if priority is not None:
-            data["priority"] = priority
-        if due_string is not None:
-            data["due_string"] = due_string
-        if due_lang is not None:
-            data["due_lang"] = due_lang
-        if due_date is not None:
-            data["due_date"] = format_date(due_date)
-        if due_datetime is not None:
-            data["due_datetime"] = format_datetime(due_datetime)
-        if assignee_id is not None:
-            data["assignee_id"] = assignee_id
-        if day_order is not None:
-            data["day_order"] = day_order
-        if collapsed is not None:
-            data["collapsed"] = collapsed
-        if duration is not None:
-            data["duration"] = duration
-        if duration_unit is not None:
-            data["duration_unit"] = duration_unit
-        if deadline_date is not None:
-            data["deadline_date"] = format_date(deadline_date)
-        if deadline_lang is not None:
-            data["deadline_lang"] = deadline_lang
+        data = compact_dict(
+            content=content,
+            description=description,
+            labels=labels,
+            priority=priority,
+            due_string=due_string,
+            due_lang=due_lang,
+            due_date=format_date(due_date) if due_date is not None else None,
+            due_datetime=(
+                format_datetime(due_datetime) if due_datetime is not None else None
+            ),
+            assignee_id=assignee_id,
+            day_order=day_order,
+            collapsed=collapsed,
+            duration=duration,
+            duration_unit=duration_unit,
+            deadline_date=(
+                format_date(deadline_date) if deadline_date is not None else None
+            ),
+            deadline_lang=deadline_lang,
+        )
 
         task_data: dict[str, Any] = await post_async(
             self._client,