Merge pull request #22 from yamilramilev/main

FabianBeiner · web-flow · commit 2e6fcfdbe501 · 2022-12-20T17:52:53.000+01:00
Fixed `getAllSections` to use `projectId` correctly
diff --git a/src/FabianBeiner/Todoist/TodoistSectionsTrait.php b/src/FabianBeiner/Todoist/TodoistSectionsTrait.php
@@ -30,10 +30,10 @@ public function getAllSections(?string $projectId = null): bool|array
     {
         if ( ! $this->validateId($projectId)) {
             /** @var object $result Result of the GET request. */
-            $result = $this->get('sections?project_id=' . $projectId);
+            $result = $this->get('sections');
         } else {
             /** @var object $result Result of the GET request. */
-            $result = $this->get('sections');
+            $result = $this->get('sections?project_id=' . $projectId);
         }
 
         return $this->handleResponse($result->getStatusCode(), $result->getBody()->getContents());

Original file line number	Diff line number	Diff line change
`@@ -30,10 +30,10 @@ public function getAllSections(?string $projectId = null): bool\|array`
`30`	`30`	`{`
`31`	`31`	`if ( ! $this->validateId($projectId)) {`
`32`	`32`	`/** @var object $result Result of the GET request. */`
`33`		`- $result = $this->get('sections?project_id=' . $projectId);`
	`33`	`+ $result = $this->get('sections');`
`34`	`34`	`} else {`
`35`	`35`	`/** @var object $result Result of the GET request. */`
`36`		`- $result = $this->get('sections');`
	`36`	`+ $result = $this->get('sections?project_id=' . $projectId);`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`return $this->handleResponse($result->getStatusCode(), $result->getBody()->getContents());`