From 9a1733effa1ed838b25b2a4e6750aa0b316e730f Mon Sep 17 00:00:00 2001
From: DorraJaouad <dorra.jaoued7@gmail.com>
Date: Mon, 29 Apr 2024 14:36:33 +0200
Subject: [PATCH] feat(OpenConversationsList): expose description

Signed-off-by: DorraJaouad <dorra.jaoued7@gmail.com>
---
 lib/Service/RoomFormatter.php                    |  1 +
 .../ConversationsList/Conversation.vue           |  2 +-
 .../ConversationSearchResult.vue                 |  9 +++------
 .../ConversationsSearchListVirtual.vue           |  7 ++-----
 .../OpenConversationsList.vue                    |  5 +++++
 src/composables/useConversationInfo.js           | 16 +++++++++-------
 6 files changed, 21 insertions(+), 19 deletions(-)

diff --git a/lib/Service/RoomFormatter.php b/lib/Service/RoomFormatter.php
index c850bc54e663..54630f9e0a42 100644
--- a/lib/Service/RoomFormatter.php
+++ b/lib/Service/RoomFormatter.php
@@ -162,6 +162,7 @@ public function formatRoomV4(
 			return array_merge($roomData, [
 				'name' => $room->getName(),
 				'displayName' => $room->getDisplayName($isListingBreakoutRooms || $isSIPBridgeRequest || $this->userId === null ? '' : $this->userId, $isListingBreakoutRooms || $isSIPBridgeRequest),
+				'description' => $room->getListable() !== Room::LISTABLE_NONE ? $room->getDescription() : '',
 				'objectType' => $room->getObjectType(),
 				'objectId' => $room->getObjectId(),
 				'readOnly' => $room->getReadOnly(),
diff --git a/src/components/LeftSidebar/ConversationsList/Conversation.vue b/src/components/LeftSidebar/ConversationsList/Conversation.vue
index 06a9159a7849..e935fb3e010c 100644
--- a/src/components/LeftSidebar/ConversationsList/Conversation.vue
+++ b/src/components/LeftSidebar/ConversationsList/Conversation.vue
@@ -182,7 +182,7 @@ export default {
 
 	setup(props) {
 		const { item, isSearchResult } = toRefs(props)
-		const { counterType, conversationInformation } = useConversationInfo({ item, isSearchResult })
+		const { counterType, conversationInformation } = useConversationInfo({ item, isSearchResult, exposeMessages: true, exposeDescription: true })
 
 		return {
 			counterType,
diff --git a/src/components/LeftSidebar/ConversationsList/ConversationSearchResult.vue b/src/components/LeftSidebar/ConversationsList/ConversationSearchResult.vue
index 2e47ed7f0d9e..25877bdc2194 100644
--- a/src/components/LeftSidebar/ConversationsList/ConversationSearchResult.vue
+++ b/src/components/LeftSidebar/ConversationsList/ConversationSearchResult.vue
@@ -39,10 +39,6 @@ export default {
 	},
 
 	props: {
-		exposeMessages: {
-			type: Boolean,
-			default: false,
-		},
 		item: {
 			type: Object,
 			default() {
@@ -66,9 +62,10 @@ export default {
 	emits: ['click'],
 
 	setup(props) {
-		const { item, exposeMessages } = toRefs(props)
+		const { item } = toRefs(props)
 		const selectedRoom = inject('selectedRoom', null)
-		const { counterType, conversationInformation } = useConversationInfo({ item, exposeMessages })
+		const exposeDescription = inject('exposeDescription', false)
+		const { counterType, conversationInformation } = useConversationInfo({ item, exposeDescription })
 
 		return {
 			selectedRoom,
diff --git a/src/components/LeftSidebar/ConversationsList/ConversationsSearchListVirtual.vue b/src/components/LeftSidebar/ConversationsList/ConversationsSearchListVirtual.vue
index 2db1931d80f3..191411516539 100644
--- a/src/components/LeftSidebar/ConversationsList/ConversationsSearchListVirtual.vue
+++ b/src/components/LeftSidebar/ConversationsList/ConversationsSearchListVirtual.vue
@@ -10,7 +10,7 @@
 		:item-size="CONVERSATION_ITEM_SIZE"
 		key-field="token">
 		<template #default="{ item }">
-			<ConversationSearchResult :item="item" :expose-messages="exposeMessages" @click="onClick" />
+			<ConversationSearchResult :item="item" @click="onClick" />
 		</template>
 		<template #after>
 			<LoadingPlaceholder v-if="loading" type="conversations" />
@@ -42,10 +42,7 @@ export default {
 			type: Array,
 			required: true,
 		},
-		exposeMessages: {
-			type: Boolean,
-			default: false,
-		},
+
 		loading: {
 			type: Boolean,
 			default: false,
diff --git a/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue b/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue
index a88fe300073f..a32cd13356d8 100644
--- a/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue
+++ b/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue
@@ -14,6 +14,7 @@
 </template>
 
 <script>
+import { provide } from 'vue'
 
 import RoomSelector from '../../RoomSelector.vue'
 
@@ -25,6 +26,10 @@ export default {
 		RoomSelector,
 	},
 
+	setup() {
+		provide('exposeDescription', true)
+	},
+
 	data() {
 		return {
 			modal: false,
diff --git a/src/composables/useConversationInfo.js b/src/composables/useConversationInfo.js
index aac39fc060dd..bf520b713c08 100644
--- a/src/composables/useConversationInfo.js
+++ b/src/composables/useConversationInfo.js
@@ -12,15 +12,17 @@ import { ATTENDEE, CONVERSATION } from '../constants.js'
  * @param {object} payload function payload
  * @param {import('vue').Ref} payload.item conversation item
  * @param {import('vue').Ref} [payload.isSearchResult] whether conversation item appears as search result
- * @param {import('vue').Ref} [payload.exposeMessages] whether to show messages in conversation item
+ * @param {boolean} [payload.exposeMessages] whether to show messages in conversation item
+ * @param {boolean} [payload.exposeDescription] whether to show description in conversation item
  */
 export function useConversationInfo({
 	item,
 	isSearchResult = ref(null),
-	exposeMessages = ref(null),
+	exposeMessages = false,
+	exposeDescription = false,
 }) {
 	const counterType = computed(() => {
-		if (exposeMessages.value === false) {
+		if (exposeMessages === false) {
 			return ''
 		} else if (item.value.unreadMentionDirect || (item.value.unreadMessages !== 0
 			&& [CONVERSATION.TYPE.ONE_TO_ONE, CONVERSATION.TYPE.ONE_TO_ONE_FORMER].includes(item.value.type)
@@ -43,7 +45,7 @@ export function useConversationInfo({
 	 * e.g. no avatars on mentions.
 	 */
 	const simpleLastChatMessage = computed(() => {
-		if (exposeMessages.value === false || !hasLastMessage.value) {
+		if (exposeMessages === false || !hasLastMessage.value) {
 			return ''
 		}
 
@@ -62,7 +64,7 @@ export function useConversationInfo({
 	 * @return {string} Part of the name until the first space
 	 */
 	const shortLastChatMessageAuthor = computed(() => {
-		if (exposeMessages.value === false || !hasLastMessage.value || item.value.lastMessage.systemMessage.length) {
+		if (exposeMessages === false || !hasLastMessage.value || item.value.lastMessage.systemMessage.length) {
 			return ''
 		}
 
@@ -81,8 +83,8 @@ export function useConversationInfo({
 			return t('spreed', 'Joining conversation …')
 		}
 
-		if (exposeMessages.value === false || !hasLastMessage.value) {
-			return ''
+		if (exposeMessages === false || !hasLastMessage.value) {
+			return exposeDescription ? item.value?.description : ''
 		}
 
 		if (shortLastChatMessageAuthor.value === '') {