From e03c74a61c905b5540e6f96e0b2ea5f670048157 Mon Sep 17 00:00:00 2001
From: Eduardo Morales <emoral435@gmail.com>
Date: Sat, 9 Mar 2024 14:51:26 -0600
Subject: [PATCH] feat: author displayname and avatar on versions

Signed-off-by: Eduardo Morales <emoral435@gmail.com>
---
 .../files_versions/src/components/Version.vue | 42 ++++++++++++++++++-
 apps/files_versions/src/utils/versions.ts     |  2 +
 2 files changed, 42 insertions(+), 2 deletions(-)

diff --git a/apps/files_versions/src/components/Version.vue b/apps/files_versions/src/components/Version.vue
index 694d10e1032..6c60f07ca7d 100644
--- a/apps/files_versions/src/components/Version.vue
+++ b/apps/files_versions/src/components/Version.vue
@@ -39,6 +39,23 @@
 			</div>
 		</template>
 
+		<!-- author -->
+		<template #name>
+			<div class="version__info">
+				<div v-if="versionLabel" class="version__info__label">{{ versionLabel }}</div>
+				<div v-if="versionAuthor" class="version__info version__info__author">
+					<div>•</div>
+					<div>{{ versionAuthor }}</div>
+					<NcAvatar class="avatar"
+						:user="version.author"
+						:size="24"
+						:disable-menu="true"
+						:disable-tooltip="true"
+						:show-user-status="false" />
+				</div>
+			</div>
+		</template>
+
 		<!-- Version file size as subline -->
 		<template #subname>
 			<div class="version__info">
@@ -113,11 +130,13 @@ import Pencil from 'vue-material-design-icons/Pencil.vue'
 
 import NcActionButton from '@nextcloud/vue/dist/Components/NcActionButton.js'
 import NcActionLink from '@nextcloud/vue/dist/Components/NcActionLink.js'
+import NcAvatar from '@nextcloud/vue/dist/Components/NcAvatar.js'
 import NcListItem from '@nextcloud/vue/dist/Components/NcListItem.js'
 import Tooltip from '@nextcloud/vue/dist/Directives/Tooltip.js'
 
 import { defineComponent, type PropType } from 'vue'
-import { getRootUrl } from '@nextcloud/router'
+import axios from '@nextcloud/axios'
+import { getRootUrl, generateOcsUrl } from '@nextcloud/router'
 import { joinPaths } from '@nextcloud/paths'
 import { loadState } from '@nextcloud/initial-state'
 import { Permission, formatFileSize } from '@nextcloud/files'
@@ -132,6 +151,7 @@ export default defineComponent({
 	components: {
 		NcActionLink,
 		NcActionButton,
+		NcAvatar,
 		NcListItem,
 		BackupRestore,
 		Download,
@@ -145,6 +165,10 @@ export default defineComponent({
 		tooltip: Tooltip,
 	},
 
+	created() {
+		this.fetchDisplayName()
+	},
+
 	filters: {
 		humanReadableSize(bytes: number): string {
 			return formatFileSize(bytes)
@@ -193,6 +217,7 @@ export default defineComponent({
 			previewLoaded: false,
 			previewErrored: false,
 			capabilities: loadState('core', 'capabilities', { files: { version_labeling: false, version_deletion: false } }),
+			versionAuthor: '',
 		}
 	},
 
@@ -279,6 +304,14 @@ export default defineComponent({
 			this.$emit('delete', this.version)
 		},
 
+		async fetchDisplayName() {
+			// check to make sure that we have a valid author - in case database did not migrate, null author, etc.
+			if (this.version.author) {
+				const { data } = await axios.get(generateOcsUrl(`/cloud/users/${this.version.author}`))
+				this.versionAuthor = data.ocs.data.displayname
+			}
+		},
+
 		click() {
 			if (!this.canView) {
 				window.location = this.downloadURL
@@ -310,8 +343,13 @@ export default defineComponent({
 		align-items: center;
 		gap: 0.5rem;
 
-		&__size {
+		&__label {
+			font-weight: 700;
+		}
+
+		&__size, &__author {
 			color: var(--color-text-lighter);
+			font-weight: 500;
 		}
 	}
 
diff --git a/apps/files_versions/src/utils/versions.ts b/apps/files_versions/src/utils/versions.ts
index e3555339b27..4048f23c327 100644
--- a/apps/files_versions/src/utils/versions.ts
+++ b/apps/files_versions/src/utils/versions.ts
@@ -35,6 +35,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 ''
+	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
 	mime: string, // Empty for the current version, else the actual mime type of the version
@@ -107,6 +108,7 @@ function formatVersion(version: any, fileInfo: any): Version {
 	return {
 		fileId: fileInfo.id,
 		label: version.props['version-label'],
+		author: version.props['version-author'] ?? null,
 		filename: version.filename,
 		basename: moment(mtime).format('LLL'),
 		mime: version.mime,
-- 
2.39.5