chore(attachments): Remove obsolete metadata API endpoint

Signed-off-by: Jonas <[email protected]>

appinfo/routes.php

@@ -41,8 +41,6 @@
 		['name' => 'Attachment#getMediaFile', 'url' => '/media', 'verb' => 'GET'],
 		/** @see Controller\AttachmentController::getMediaFilePreview() */
 		['name' => 'Attachment#getMediaFilePreview', 'url' => '/mediaPreview', 'verb' => 'GET'],
-		/** @see Controller\AttachmentController::getMediaFileMetadata() */
-		['name' => 'Attachment#getMediaFileMetadata', 'url' => '/mediaMetadata', 'verb' => 'GET'],
 
 		/** @see Controller\SessionController::create() */
 		['name' => 'Session#create', 'url' => '/session/{documentId}/create', 'verb' => 'PUT'],

lib/Controller/AttachmentController.php

@@ -278,32 +278,6 @@ public function getMediaFilePreview(string $mediaFileName, ?string $shareToken =
 		return new DataResponse('', Http::STATUS_NOT_FOUND);
 	}
 
-	/**
-	 * Serve the media files metadata in the editor
-	 */
-	#[NoAdminRequired]
-	#[PublicPage]
-	#[NoCSRFRequired]
-	#[RequireDocumentSession]
-	public function getMediaFileMetadata(string $mediaFileName, ?string $shareToken = null): DataResponse {
-		$documentId = $this->getSession()->getDocumentId();
-		try {
-			if ($shareToken) {
-				$metadata = $this->attachmentService->getMediaFileMetadataPublic($documentId, $mediaFileName, $shareToken);
-			} else {
-				$userId = $this->getSession()->getUserId();
-				$metadata = $this->attachmentService->getMediaFileMetadataPrivate($documentId, $mediaFileName, $userId);
-			}
-			if ($metadata === null) {
-				return new DataResponse('', Http::STATUS_NOT_FOUND);
-			}
-			return new DataResponse($metadata);
-		} catch (Exception $e) {
-			$this->logger->error('getMediaFileMetadata error', ['exception' => $e]);
-			return new DataResponse('', Http::STATUS_NOT_FOUND);
-		}
-	}
-
 	/**
 	 * Allow all supported mimetypes
 	 * Use mimetype detector for the other ones

lib/Service/AttachmentService.php

@@ -202,60 +202,6 @@ private function getMediaFilePreviewFile(string $mediaFileName, File $textFile):
 		return null;
 	}
 
-	/**
-	 * @param int $documentId
-	 * @param string $mediaFileName
-	 * @param string $userId
-	 *
-	 * @return array|null
-	 * @throws NotFoundException
-	 * @throws NotPermittedException
-	 * @throws InvalidPathException
-	 * @throws NoUserException
-	 */
-	public function getMediaFileMetadataPrivate(int $documentId, string $mediaFileName, string $userId): ?array {
-		$textFile = $this->getTextFile($documentId, $userId);
-		return $this->getMediaFileMetadata($mediaFileName, $textFile);
-	}
-
-	/**
-	 * @param int $documentId
-	 * @param string $mediaFileName
-	 * @param string $shareToken
-	 *
-	 * @return array|null
-	 * @throws NotFoundException
-	 * @throws NotPermittedException
-	 * @throws InvalidPathException
-	 * @throws NoUserException
-	 */
-	public function getMediaFileMetadataPublic(int $documentId, string $mediaFileName, string $shareToken): ?array {
-		$textFile = $this->getTextFilePublic($documentId, $shareToken);
-		return $this->getMediaFileMetadata($mediaFileName, $textFile);
-	}
-
-	/**
-	 * @param string $mediaFileName
-	 * @param File $textFile
-	 *
-	 * @return array|null
-	 * @throws NotFoundException
-	 * @throws NotPermittedException
-	 * @throws InvalidPathException
-	 * @throws NoUserException
-	 */
-	private function getMediaFileMetadata(string $mediaFileName, File $textFile): ?array {
-		$attachmentFolder = $this->getAttachmentDirectoryForFile($textFile, true);
-		$mediaFile = $attachmentFolder->get($mediaFileName);
-		if ($mediaFile instanceof File) {
-			return [
-				'size' => Util::humanFileSize($mediaFile->getSize()),
-				'mtime' => $mediaFile->getMTime(),
-			];
-		}
-		return null;
-	}
-
 	/**
 	 * @param int          $documentId
 	 * @param string|null  $userId