fixed copilot suggestions

Author: jessevz

src/inc/apiv2/helper/getTaskProgressImage.routes.php

@@ -50,7 +50,7 @@ public function getParamsSwagger(): array {
           "type" => "integer",
           "format" => "int32"
         ],
-        "required" => true,
+        "required" => false,
         "example" => 1,
         "description" => "The ID of the supertask where you want to create the progress image of."
       ],
@@ -61,7 +61,7 @@ public function getParamsSwagger(): array {
           "type" => "integer",
           "format" => "int32"
         ],
-        "required" => true,
+        "required" => false,
         "example" => 1,
         "description" => "The ID of the task where you want to create the progress image of."
       ]
@@ -86,15 +86,15 @@ public function handleGet(Request $request, Response $response): Response {
     if ($task_id) {
       $task = Factory::getTaskFactory()->get($task_id);
       if ($task == null) {
-        throw new HttpNotFoundException($request, "Invalid file");
+        throw new HttpNotFoundException($request, "Invalid task");
       }
       $taskWrapper = Factory::getTaskWrapperFactory()->get($task->getTaskWrapperId());
       if ($taskWrapper == null) {
         throw new HttpError("Inconsistency on task!");
       }
     }
     else if ($supertask_id) {
-      $taskWrapper = Factory::getTaskWrapperFactory()->get($_GET['supertask']);
+      $taskWrapper = Factory::getTaskWrapperFactory()->get($supertask_id);
       if ($taskWrapper == null) {
         throw new HttpError("Invalid task wrapper!");
       }
@@ -201,7 +201,11 @@ public function handleGet(Request $request, Response $response): Response {
     }
 
     //send image data to output
+    ob_start($image);
     imagepng($image);
+    $imageData = ob_get_clean();
+    imagedestroy($image);
+    $response->getBody()->write($imageData);
     return $response->withStatus(200)
       ->withHeader("Content-Type", "image/png")
       ->withHeader("Cache-Control", "no-cache");