fix(AttachmentController): Set fileName of returned attachments

Signed-off-by: Jonas <[email protected]>
nextcloud · Nov 27, 2023 · d19ee78 · d19ee78
1 parent fd7a8f6
commit d19ee78
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/Controller/AttachmentController.php b/lib/Controller/AttachmentController.php
@@ -207,7 +207,7 @@ public function getImageFile(string $imageFileName, ?string $shareToken = null,
 			return $imageFile !== null
 				? new DataDownloadResponse(
 					$imageFile->getContent(),
-					(string) Http::STATUS_OK,
+					$imageFile->getName(),
 					$this->getSecureMimeType($imageFile->getMimeType())
 				)
 				: new DataResponse('', Http::STATUS_NOT_FOUND);
@@ -241,7 +241,7 @@ public function getMediaFile(string $mediaFileName, ?string $shareToken = null):
 			return $mediaFile !== null
 				? new DataDownloadResponse(
 					$mediaFile->getContent(),
-					(string) Http::STATUS_OK,
+					$mediaFile->getName(),
 					$this->getSecureMimeType($mediaFile->getMimeType())
 				)
 				: new DataResponse('', Http::STATUS_NOT_FOUND);
@@ -275,7 +275,7 @@ public function getMediaFilePreview(string $mediaFileName, ?string $shareToken =
 			if ($preview['type'] === 'file') {
 				return new DataDownloadResponse(
 					$preview['file']->getContent(),
-					(string) Http::STATUS_OK,
+					$mediaFileName,
 					$this->getSecureMimeType($preview['file']->getMimeType())
 				);
 			} elseif ($preview['type'] === 'icon') {