From 1494ff21202e65296bea20283f5cfc7888cacb3e Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Tue, 4 Jul 2023 12:13:55 +0200
Subject: [PATCH 1/3] move NewMessageUploadEditor.vue up to parent component

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 src/components/ChatView.vue                          | 6 ++++++
 src/components/NewMessage/NewMessage.vue             | 5 -----
 src/components/NewMessage/NewMessageUploadEditor.vue | 4 ++--
 3 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/components/ChatView.vue b/src/components/ChatView.vue
index a1fb7b83180..09fd38d7f92 100644
--- a/src/components/ChatView.vue
+++ b/src/components/ChatView.vue
@@ -51,6 +51,10 @@
 			:container="containerId"
 			has-typing-indicator
 			:aria-label="t('spreed', 'Post message')" />
+
+		<!-- File upload dialog -->
+		<NewMessageUploadEditor />
+
 		<TransitionWrapper name="fade">
 			<NcButton v-show="!isChatScrolledToBottom"
 				type="secondary"
@@ -74,6 +78,7 @@ import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
 import GuestWelcomeWindow from './GuestWelcomeWindow.vue'
 import MessagesList from './MessagesList/MessagesList.vue'
 import NewMessage from './NewMessage/NewMessage.vue'
+import NewMessageUploadEditor from './NewMessage/NewMessageUploadEditor.vue'
 import TransitionWrapper from './TransitionWrapper.vue'
 
 import { CONVERSATION } from '../constants.js'
@@ -88,6 +93,7 @@ export default {
 		ChevronDoubleDown,
 		MessagesList,
 		NewMessage,
+		NewMessageUploadEditor,
 		TransitionWrapper,
 		GuestWelcomeWindow,
 	},
diff --git a/src/components/NewMessage/NewMessage.vue b/src/components/NewMessage/NewMessage.vue
index 0918e855f87..dd9ef675340 100644
--- a/src/components/NewMessage/NewMessage.vue
+++ b/src/components/NewMessage/NewMessage.vue
@@ -137,9 +137,6 @@
 			</template>
 		</form>
 
-		<!-- File upload dialog -->
-		<NewMessageUploadEditor />
-
 		<!-- Poll creation dialog -->
 		<NewMessagePollEditor v-if="showPollEditor"
 			:token="token"
@@ -175,7 +172,6 @@ import NewMessageAudioRecorder from './NewMessageAudioRecorder.vue'
 import NewMessageNewFileDialog from './NewMessageNewFileDialog.vue'
 import NewMessagePollEditor from './NewMessagePollEditor.vue'
 import NewMessageTypingIndicator from './NewMessageTypingIndicator.vue'
-import NewMessageUploadEditor from './NewMessageUploadEditor.vue'
 
 import { CONVERSATION, PARTICIPANT, PRIVACY } from '../../constants.js'
 import { EventBus } from '../../services/EventBus.js'
@@ -211,7 +207,6 @@ export default {
 		NewMessageNewFileDialog,
 		NewMessagePollEditor,
 		NewMessageTypingIndicator,
-		NewMessageUploadEditor,
 		Quote,
 		// Icons
 		BellOff,
diff --git a/src/components/NewMessage/NewMessageUploadEditor.vue b/src/components/NewMessage/NewMessageUploadEditor.vue
index 236b92b48dd..2837cb3913e 100644
--- a/src/components/NewMessage/NewMessageUploadEditor.vue
+++ b/src/components/NewMessage/NewMessageUploadEditor.vue
@@ -139,14 +139,14 @@ export default {
 		},
 
 		voiceMessageName() {
-			if (!this.firstFile.file.name) {
+			if (!this.firstFile?.file?.name) {
 				return ''
 			}
 			return this.firstFile.file.name
 		},
 
 		voiceMessageLocalURL() {
-			if (!this.firstFile.file.localURL) {
+			if (!this.firstFile?.file?.localURL) {
 				return ''
 			}
 			return this.firstFile.file.localURL

From 3b8e6e9db7dfdc5bc386d2347a43097c1835d5fb Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Sun, 15 Oct 2023 20:02:31 +0200
Subject: [PATCH 2/3] fix name overlapping for shared items tab

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 src/components/RightSidebar/SharedItems/SharedItems.vue   | 2 +-
 .../RightSidebar/SharedItems/SharedItemsTab.vue           | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/components/RightSidebar/SharedItems/SharedItems.vue b/src/components/RightSidebar/SharedItems/SharedItems.vue
index 7b86f402c16..66818ebdf1a 100644
--- a/src/components/RightSidebar/SharedItems/SharedItems.vue
+++ b/src/components/RightSidebar/SharedItems/SharedItems.vue
@@ -40,7 +40,7 @@
 			<div v-else-if="isOther"
 				:key="item.id"
 				class="shared-items__other">
-				<a v-if="item.messageParameters.object.link"
+				<a v-if="item.messageParameters.object?.link"
 					:href="item.messageParameters.object.link"
 					target="_blank">
 					{{ item.messageParameters.object.name }}
diff --git a/src/components/RightSidebar/SharedItems/SharedItemsTab.vue b/src/components/RightSidebar/SharedItems/SharedItemsTab.vue
index c9cf5667a1f..c10b74ae811 100644
--- a/src/components/RightSidebar/SharedItems/SharedItemsTab.vue
+++ b/src/components/RightSidebar/SharedItems/SharedItemsTab.vue
@@ -20,8 +20,8 @@
 -->
 
 <template>
-	<div class="shared-items">
-		<LoadingComponent v-if="loading" class="shared-items__loading" />
+	<div class="shared-items-tab">
+		<LoadingComponent v-if="loading" class="shared-items-tab__loading" />
 
 		<template v-else>
 			<!-- Shared items grouped by type -->
@@ -64,7 +64,7 @@
 
 			<!-- No shared content -->
 			<NcEmptyContent v-else-if="!hasSharedItems && !hasRelatedResources"
-				class="shared-items__empty-content"
+				class="shared-items-tab__empty-content"
 				:name="t('spreed', 'No shared items')">
 				<template #icon>
 					<FolderMultipleImage :size="20" />
@@ -227,7 +227,7 @@ export default {
 	}
 }
 
-.shared-items {
+.shared-items-tab {
 	display: flex;
 	flex-direction: column;
 	height: 100%;

From 9c86af40ac148b1a702f3dbf7b3dd7b4f282f2d3 Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Mon, 16 Oct 2023 17:03:38 +0200
Subject: [PATCH 3/3] remove checks like 'message === {file}' or {object}'

removed in favor of checking property existence
matches /file/ and /file.*/ for multiple files support

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 .../Message/MessageButtonsBar/MessageButtonsBar.vue |  9 +++++----
 src/components/Quote.vue                            |  3 +--
 src/store/messagesStore.js                          | 13 ++++++++-----
 src/utils/getItemTypeFromMessage.js                 |  4 ++--
 4 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageButtonsBar.vue b/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageButtonsBar.vue
index f9d4745cd3e..77a4bb07e19 100644
--- a/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageButtonsBar.vue
+++ b/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageButtonsBar.vue
@@ -514,13 +514,15 @@ export default {
 
 		linkToFile() {
 			if (this.isFileShare) {
-				return this.messageParameters?.file?.link
+				const firstFileKey = (Object.keys(this.messageParameters).find(key => key.startsWith('file')))
+				return this.messageParameters?.[firstFileKey]?.link
+			} else {
+				return ''
 			}
-			return ''
 		},
 
 		isFileShare() {
-			return this.message === '{file}' && this.messageParameters?.file
+			return Object.keys(Object(this.messageParameters)).some(key => key.startsWith('file'))
 		},
 
 		isCurrentGuest() {
@@ -542,7 +544,6 @@ export default {
 
 		isPollMessage() {
 			return this.messageType === 'comment'
-				&& this.message === '{object}'
 				&& this.messageParameters?.object?.type === 'talk-poll'
 		},
 
diff --git a/src/components/Quote.vue b/src/components/Quote.vue
index e16ed39890f..2e503962dd6 100644
--- a/src/components/Quote.vue
+++ b/src/components/Quote.vue
@@ -172,8 +172,7 @@ export default {
 		},
 
 		isFileShareMessage() {
-			return this.message === '{file}'
-				&& 'file' in this.messageParameters
+			return Object.keys(Object(this.messageParameters)).some(key => key.startsWith('file'))
 		},
 
 		richParameters() {
diff --git a/src/store/messagesStore.js b/src/store/messagesStore.js
index 1905c55c164..25a2fb18da8 100644
--- a/src/store/messagesStore.js
+++ b/src/store/messagesStore.js
@@ -567,10 +567,13 @@ const actions = {
 
 		context.commit('addMessage', message)
 
-		if ((message.messageType === 'comment' && message.message === '{file}' && message.messageParameters?.file)
-			|| (message.messageType === 'voice-message' && message.message === '{file}' && message.messageParameters?.file)
-			|| (message.messageType === 'comment' && message.message === '{object}' && message.messageParameters?.object)) {
-			sharedItemsStore.addSharedItemFromMessage(message)
+		if (message.messageParameters && (message.messageType === 'comment' || message.messageType === 'voice-message')) {
+			if (message.messageParameters?.object || message.messageParameters?.file) {
+				// Handle voice messages, shares with single file, polls, deck cards, e.t.c
+				sharedItemsStore.addSharedItemFromMessage(message)
+			} else if (Object.keys(message.messageParameters).some(key => key.startsWith('file'))) {
+				// Handle shares with multiple files
+			}
 		}
 	},
 
@@ -1297,7 +1300,7 @@ const actions = {
 			delete message.parent
 		}
 
-		if (message.message === '{object}' && message.messageParameters.object) {
+		if (message.messageParameters?.object) {
 			const richObject = message.messageParameters.object
 			const response = await postRichObjectToConversation(
 				targetToken,
diff --git a/src/utils/getItemTypeFromMessage.js b/src/utils/getItemTypeFromMessage.js
index 69d0fa1410f..0c41aa38ab5 100644
--- a/src/utils/getItemTypeFromMessage.js
+++ b/src/utils/getItemTypeFromMessage.js
@@ -1,7 +1,7 @@
 import { SHARED_ITEM } from '../constants.js'
 
 export const getItemTypeFromMessage = function(message) {
-	if (message.message === '{object}') {
+	if (message.messageParameters?.object) {
 		if (message.messageParameters.object.type === 'geo-location') {
 			return SHARED_ITEM.TYPES.LOCATION
 		} else if (message.messageParameters.object.type === 'deck-card') {
@@ -11,7 +11,7 @@ export const getItemTypeFromMessage = function(message) {
 		} else {
 			return SHARED_ITEM.TYPES.OTHER
 		}
-	} else if (message.message === '{file}') {
+	} else if (message.messageParameters?.file) {
 		const messageType = message.messageType || ''
 		const mimetype = message.messageParameters.file?.mimetype || ''
 		if (messageType === 'record-audio' || messageType === 'record-video') {