Merge pull request #41617 from nextcloud/artonge/fix/error_files_vers…

…ions_previews Handle error in previews for files versions
nextcloud · Nov 21, 2023 · ff92a0a · ff92a0a
2 parents 24c2c09 + 6896724
commit ff92a0a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/apps/files_versions/src/components/Version.vue b/apps/files_versions/src/components/Version.vue
@@ -24,14 +24,15 @@
 			@click="click">
 			<template #icon>
 				<div v-if="!(loadPreview || previewLoaded)" class="version__image" />
-				<img v-else-if="isCurrent || version.hasPreview"
+				<img v-else-if="(isCurrent || version.hasPreview) && !previewErrored"
 					:src="version.previewUrl"
 					alt=""
 					decoding="async"
 					fetchpriority="low"
 					loading="lazy"
 					class="version__image"
-					@load="previewLoaded = true">
+					@load="previewLoaded = true"
+					@error="previewErrored = true">
 				<div v-else
 					class="version__image">
 					<ImageOffOutline :size="20" />
@@ -212,6 +213,7 @@ export default {
 	data() {
 		return {
 			previewLoaded: false,
+			previewErrored: false,
 			showVersionLabelForm: false,
 			formVersionLabelValue: this.version.label,
 			capabilities: loadState('core', 'capabilities', { files: { version_labeling: false, version_deletion: false } }),

diff --git a/dist/files_versions-files_versions.js b/dist/files_versions-files_versions.js
diff --git a/dist/files_versions-files_versions.js.map b/dist/files_versions-files_versions.js.map