fix(files_versions): fix undefined success message

apps/files_versions/src/components/Version.vue

@@ -209,20 +209,28 @@ export default defineComponent({
 		},
 
 		versionLabel(): string {
-			const label = this.version.label ?? ''
+			const label = this.version?.label ?? ''
 
+			// Current but empty label
+			if (this.isCurrent && label === '') {
+				return t('files_versions', 'Current version')
+			}
+
+			// Current with label
 			if (this.isCurrent) {
-				if (label === '') {
-					return t('files_versions', 'Current version')
-				} else {
-					return `${label} (${t('files_versions', 'Current version')})`
-				}
+				return `${label} (${t('files_versions', 'Current version')})`
 			}
 
+			// First version with empty label
 			if (this.isFirstVersion && label === '') {
 				return t('files_versions', 'Initial version')
 			}
 
+			// Unknown version with empty label
+			if (label === '') {
+				return t('files_versions', 'Unnamed version')
+			}
+
 			return label
 		},
 

apps/files_versions/src/utils/versions.ts

@@ -18,7 +18,7 @@ import logger from '../utils/logger.js'
 
 export interface Version {
 	fileId: string, // The id of the file associated to the version.
-	label: string, // 'Current version' or ''
+	label?: string, // 'Current version' or ''
 	author: string|null, // UID for the author of the version
 	filename: string, // File name relative to the version DAV endpoint
 	basename: string, // A base name generated from the mtime

apps/files_versions/src/views/VersionTab.vue

@@ -200,7 +200,7 @@ export default {
 
 			try {
 				await restoreVersion(version)
-				if (version.label !== '') {
+				if (version?.label && version?.label !== '') {
 					showSuccess(t('files_versions', `${version.label} restored`))
 				} else if (version.mtime === this.initialVersionMtime) {
 					showSuccess(t('files_versions', 'Initial version restored'))