From 4b6e723f5cdafa64dbe7a00b0f7ebd1f3e483112 Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Wed, 26 Jun 2024 14:57:58 +0200
Subject: [PATCH 1/2] fix(RightSidebar): remove unused actions/getters/computed

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 src/components/RightSidebar/RightSidebar.vue | 36 ++------------------
 src/store/sidebarStore.js                    | 32 -----------------
 src/store/sidebarStore.spec.js               | 35 -------------------
 3 files changed, 3 insertions(+), 100 deletions(-)

diff --git a/src/components/RightSidebar/RightSidebar.vue b/src/components/RightSidebar/RightSidebar.vue
index 6ce9a815559..b8b7d7aa311 100644
--- a/src/components/RightSidebar/RightSidebar.vue
+++ b/src/components/RightSidebar/RightSidebar.vue
@@ -22,11 +22,10 @@
 
 <template>
 	<NcAppSidebar v-show="opened"
-		:name="name"
-		:title="name"
-		:active="activeTab"
+		:name="conversation.displayName"
+		:title="conversation.displayName"
+		:active.sync="activeTab"
 		:class="'active-tab-' + activeTab"
-		@update:active="handleUpdateActive"
 		@closed="handleClosed"
 		@close="handleClose">
 		<template #description>
@@ -208,10 +207,6 @@ export default {
 				|| (this.conversation.type === CONVERSATION.TYPE.PUBLIC && this.conversation.objectType !== CONVERSATION.OBJECT_TYPE.VIDEO_VERIFICATION))
 		},
 
-		isSearching() {
-			return this.searchText !== ''
-		},
-
 		participantType() {
 			return this.conversation.participantType
 		},
@@ -220,10 +215,6 @@ export default {
 			return this.participantType === PARTICIPANT.TYPE.OWNER || this.participantType === PARTICIPANT.TYPE.MODERATOR
 		},
 
-		canModerate() {
-			return !this.isOneToOne && (this.canFullModerate || this.participantType === PARTICIPANT.TYPE.GUEST_MODERATOR)
-		},
-
 		isModeratorOrUser() {
 			return this.$store.getters.isModeratorOrUser
 		},
@@ -232,23 +223,6 @@ export default {
 			return this.$store.getters.isInLobby
 		},
 
-		/**
-		 * The conversation name value passed into the NcAppSidebar component.
-		 *
-		 * @return {string} The conversation's name.
-		 */
-		name() {
-			if (this.isRenamingConversation) {
-				return this.conversationName
-			} else {
-				return this.conversation.displayName
-			}
-		},
-
-		isRenamingConversation() {
-			return this.$store.getters.isRenamingConversation
-		},
-
 		showSIPSettings() {
 			return this.conversation.sipEnabled !== WEBINAR.SIP.DISABLED
 				&& this.conversation.attendeePin
@@ -301,10 +275,6 @@ export default {
 
 	watch: {
 		conversation(newConversation, oldConversation) {
-			if (!this.isRenamingConversation) {
-				this.conversationName = this.conversation.displayName
-			}
-
 			if (newConversation.token === oldConversation.token || !this.showParticipantsTab) {
 				return
 			}
diff --git a/src/store/sidebarStore.js b/src/store/sidebarStore.js
index 454d049ed3d..f66f74280e4 100644
--- a/src/store/sidebarStore.js
+++ b/src/store/sidebarStore.js
@@ -22,18 +22,12 @@
 
 const state = {
 	show: true,
-	isRenamingConversation: false,
-	sidebarOpenBeforeEditing: null,
-
 }
 
 const getters = {
 	getSidebarStatus: (state) => {
 		return state.show
 	},
-	isRenamingConversation: (state) => {
-		return state.isRenamingConversation
-	},
 }
 
 const mutations = {
@@ -53,23 +47,6 @@ const mutations = {
 	hideSidebar(state) {
 		state.show = false
 	},
-	/**
-	 * Renaming state of the conversation
-	 *
-	 * @param {object} state current store state;
-	 * @param {boolean} boolean the state of the renaming action;
-	 */
-	isRenamingConversation(state, boolean) {
-		if (boolean) {
-			// Record sidebar status before starting editing process
-			state.sidebarOpenBeforeEditing = state.show
-			state.isRenamingConversation = true
-		} else {
-			state.isRenamingConversation = false
-			// Go back to the previous sidebar state
-			state.show = state.sidebarOpenBeforeEditing
-		}
-	},
 }
 
 const actions = {
@@ -90,15 +67,6 @@ const actions = {
 	hideSidebar(context) {
 		context.commit('hideSidebar')
 	},
-	/**
-	 * Renaming state of the conversation
-	 *
-	 * @param {object} context default store context;
-	 * @param {boolean} boolean the state of the renaming action;
-	 */
-	isRenamingConversation(context, boolean) {
-		context.commit('isRenamingConversation', boolean)
-	},
 }
 
 export default { state, mutations, getters, actions }
diff --git a/src/store/sidebarStore.spec.js b/src/store/sidebarStore.spec.js
index 0924ab208ca..e72e8499d42 100644
--- a/src/store/sidebarStore.spec.js
+++ b/src/store/sidebarStore.spec.js
@@ -22,7 +22,6 @@ describe('sidebarStore', () => {
 
 	test('defaults are off', () => {
 		expect(store.getters.getSidebarStatus).toBe(true)
-		expect(store.getters.isRenamingConversation).toBe(false)
 	})
 
 	test('toggle sidebar', () => {
@@ -34,38 +33,4 @@ describe('sidebarStore', () => {
 
 		expect(store.getters.getSidebarStatus).toBe(true)
 	})
-
-	test('toggling renaming mode and remembers sidebar hidden state', () => {
-		store.dispatch('hideSidebar')
-		store.dispatch('isRenamingConversation', true)
-
-		expect(store.getters.getSidebarStatus).toBe(false)
-		expect(store.getters.isRenamingConversation).toBe(true)
-
-		store.dispatch('showSidebar')
-
-		expect(store.getters.getSidebarStatus).toBe(true)
-
-		store.dispatch('isRenamingConversation', false)
-
-		expect(store.getters.getSidebarStatus).toBe(false)
-		expect(store.getters.isRenamingConversation).toBe(false)
-	})
-
-	test('toggling renaming mode and remembers sidebar shown state', () => {
-		store.dispatch('showSidebar')
-		store.dispatch('isRenamingConversation', true)
-
-		expect(store.getters.getSidebarStatus).toBe(true)
-		expect(store.getters.isRenamingConversation).toBe(true)
-
-		store.dispatch('hideSidebar')
-
-		expect(store.getters.getSidebarStatus).toBe(false)
-
-		store.dispatch('isRenamingConversation', false)
-
-		expect(store.getters.getSidebarStatus).toBe(true)
-		expect(store.getters.isRenamingConversation).toBe(false)
-	})
 })

From 8b5ec5d09c830a0fa697d10d2d25f78d674b9e40 Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Wed, 26 Jun 2024 15:38:32 +0200
Subject: [PATCH 2/2] feat(RightSidebar): show participants tab in 1-1 calls

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 src/components/RightSidebar/RightSidebar.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/RightSidebar/RightSidebar.vue b/src/components/RightSidebar/RightSidebar.vue
index b8b7d7aa311..551164a1b8e 100644
--- a/src/components/RightSidebar/RightSidebar.vue
+++ b/src/components/RightSidebar/RightSidebar.vue
@@ -253,7 +253,7 @@ export default {
 		},
 
 		showParticipantsTab() {
-			return (this.getUserId || this.isModeratorOrUser) && !this.isOneToOne && !this.isNoteToSelf
+			return (this.getUserId || this.isModeratorOrUser) && (!this.isOneToOne || this.isInCall) && !this.isNoteToSelf
 		},
 
 		showSharedItemsTab() {