fix(attachments): Open non-image attachments in viewer/new tab

When viewer is available, not in use and supports the mimetype, let's open the attachment in viewer. Otherwise, open the file app URL in a new tab. Fixes: #3849, #4723 Signed-off-by: Jonas <jonas@freesources.org>
nextcloud · Nov 27, 2023 · d2dfb69 · d2dfb69
1 parent 210e458
commit d2dfb69
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 15 deletions.
diff --git a/lib/Service/AttachmentService.php b/lib/Service/AttachmentService.php
@@ -252,17 +252,13 @@ public function getAttachmentList(int $documentId, ?string $userId = null, ?Sess
 				'mimetype' => $node->getMimeType(),
 				'mtime' => $node->getMTime(),
 				'isImage' => $isImage,
+				'davPath' => '/' . implode('/', array_slice(explode('/', $node->getPath()), 3)),
 				'fullUrl' => $isImage
 					? $this->urlGenerator->linkToRouteAbsolute('text.Attachment.getImageFile') . $urlParamsBase . '&imageFileName=' . urlencode($name) . '&preferRawImage=1'
 					: $this->urlGenerator->linkToRouteAbsolute('text.Attachment.getMediaFile') . $urlParamsBase . '&mediaFileName=' . urlencode($name),
 				'previewUrl' => $isImage
 					? $this->urlGenerator->linkToRouteAbsolute('text.Attachment.getImageFile') . $urlParamsBase . '&imageFileName=' . urlencode($name)
 					: $this->urlGenerator->linkToRouteAbsolute('text.Attachment.getMediaFilePreview') . $urlParamsBase . '&mediaFileName=' . urlencode($name),
-				/*
-					: ($isImage
-						? $this->urlGenerator->linkTo('', 'remote.php') . '/dav/files/' . $userId . '/' . implode('/', array_map('rawurlencode', array_slice(explode('/', $node->getPath()), 3)))
-						: ''),
-				*/
 			];
 		}
 

diff --git a/src/nodes/ImageView.vue b/src/nodes/ImageView.vue
@@ -36,7 +36,7 @@
 						<div v-if="isMediaAttachment"
 							contenteditable="false"
 							class="media"
-							@click="handleImageClick">
+							@click="handleAttachmentClick">
 							<div class="media__wrapper">
 								<img v-show="loaded"
 									:src="imageUrl"
@@ -201,14 +201,6 @@ export default {
 
 			return this.loaded && this.imageLoaded
 		},
-		/*
-		internalLinkOrImage() {
-			if (this.attachment.fileId) {
-				return generateUrl('/f/' + this.attachment.fileId)
-			}
-			return this.src
-		},
-		 */
 		src: {
 			get() {
 				return this.node.attrs.src || ''
@@ -277,6 +269,17 @@ export default {
 		onLoaded() {
 			this.loaded = true
 		},
+		async handleAttachmentClick() {
+			// Open in viewer
+			if (OCA.Viewer && !OCA.Viewer.file && OCA.Viewer.mimetypes.indexOf(this.attachment.mimetype) !== -1 && this.attachment.davPath) {
+				// Viewer exists, is not in use and supports mimetype
+				OCA.Viewer.open({ path: this.attachment.davPath })
+				return
+			}
+
+			// Download file
+			window.location.assign(this.attachment.fullUrl)
+		},
 		async handleImageClick() {
 			this.imageIndex = this.imageAttachments.findIndex(i => (i.isImage && i.fileId === this.attachment.fileId))
 			if (this.imageIndex !== -1) {

diff --git a/src/services/AttachmentResolver.js b/src/services/AttachmentResolver.js
@@ -76,7 +76,10 @@ export default class AttachmentResolver {
 				attachment = findAttachment(imageFileName)
 			}
 
-			return attachment
+			if (attachment) {
+				return attachment
+			}
+
 		}
 
 		// Direct URLs