fix: Renamed getMetadataPhotosSizeForFileId to getImageMetadataForFileId

Signed-off-by: skalidindi53 <s.teja2004@gmail.com>
nextcloud · Aug 20, 2024 · 51d419b · 51d419b
1 parent dfc2b70
commit 51d419b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/lib/Chat/Parser/SystemMessage.php b/lib/Chat/Parser/SystemMessage.php
@@ -812,7 +812,7 @@ protected function getFileFromShare(Room $room, ?Participant $participant, strin
 		// If a preview is available, check if we can get the dimensions of the file from the metadata API
 		if ($isPreviewAvailable && str_starts_with($node->getMimeType(), 'image/')) {
 			try {
-				$sizeMetadata = $this->metadataCache->getMetadataPhotosSizeForFileId($fileId);
+				$sizeMetadata = $this->metadataCache->getImageMetadataForFileId($fileId);
 				if (isset($sizeMetadata['width'], $sizeMetadata['height'])) {
 					$data['width'] = (string) $sizeMetadata['width'];
 					$data['height'] = (string) $sizeMetadata['height'];

diff --git a/lib/Share/Helper/FilesMetadataCache.php b/lib/Share/Helper/FilesMetadataCache.php
@@ -44,7 +44,7 @@ public function preloadMetadata(array $fileIds): void {
 	 * @psalm-return array{width: int, height: int, blurhash?: string}
 	 * @throws FilesMetadataNotFoundException
 	 */
-	public function getMetadataPhotosSizeForFileId(int $fileId): array {
+	public function getImageMetadataForFileId(int $fileId): array {
 		if (!array_key_exists($fileId, $this->filesSizeData)) {
 			try {
 				$this->cachePhotosSize($fileId, $this->filesMetadataManager->getMetadata($fileId, true));

diff --git a/tests/php/Chat/Parser/SystemMessageTest.php b/tests/php/Chat/Parser/SystemMessageTest.php
@@ -635,7 +635,7 @@ public function testGetFileFromShareForGuest(): void {
 			->willReturn(true);
 
 		$this->filesMetadataCache->expects($this->once())
-			->method('getMetadataPhotosSizeForFileId')
+			->method('getImageMetadataForFileId')
 			->with(54)
 			->willReturn(['width' => 1234, 'height' => 4567]);
 
@@ -706,7 +706,7 @@ public function testGetFileFromShareForGuestWithBlurhash(): void {
 			->willReturn(true);
 
 		$this->filesMetadataCache->expects($this->once())
-			->method('getMetadataPhotosSizeForFileId')
+			->method('getImageMetadataForFileId')
 			->with(54)
 			->willReturn([
 				'width' => 1234,
@@ -786,7 +786,7 @@ public function testGetFileFromShareForOwner(): void {
 			->willReturn('absolute-link-owner');
 
 		$this->filesMetadataCache->expects($this->never())
-			->method('getMetadataPhotosSizeForFileId');
+			->method('getImageMetadataForFileId');
 
 		$participant = $this->createMock(Participant::class);
 		$attendee = Attendee::fromRow([
@@ -873,7 +873,7 @@ public function testGetFileFromShareForRecipient(): void {
 			->willReturn(false);
 
 		$this->filesMetadataCache->expects($this->never())
-			->method('getMetadataPhotosSizeForFileId');
+			->method('getImageMetadataForFileId');
 
 		$this->url->expects($this->once())
 			->method('linkToRouteAbsolute')