fix: Report download

baupen · Feb 25, 2023 · 2ade807 · 2ade807
1 parent cd09620
commit 2ade807
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/public/build/images/.gitkeep b/public/build/images/.gitkeep
diff --git a/src/Api/CustomController/IssuesReport.php b/src/Api/CustomController/IssuesReport.php
@@ -81,7 +81,7 @@ public function __invoke($data)
 
         $path = $this->router->generate('public_download', ['filename' => $filename]);
 
-        return new Response($path, Response::HTTP_FOUND);
+        return new Response($path);
     }
 
     private function getAuthor(?TokenInterface $token): ?string

diff --git a/tests/Api/IssueTest.php b/tests/Api/IssueTest.php
@@ -424,8 +424,8 @@ public function testReport()
         $this->loginApiConstructionManager($client);
 
         $constructionSite = $this->getTestConstructionSite();
-        $this->assertApiGetStatusCodeSame(StatusCode::HTTP_FOUND, $client, '/api/issues/report?constructionSite='.$constructionSite->getId(), 'application/pdf');
-        $this->assertResponseRedirects();
+        $response = $this->assertApiGetStatusCodeSame(StatusCode::HTTP_OK, $client, '/api/issues/report?constructionSite='.$constructionSite->getId(), 'application/pdf');
+        $this->assertTrue(str_contains($response->getContent(), '/download'));
     }
 
     public function testRender()