From 2fe192d12890b5cfcdf61fc1c5b3b67f87ed9be6 Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Mon, 13 May 2024 08:02:45 +0200
Subject: [PATCH] fix(settings): don't allow guests when breakout rooms are set

- keep error handling in the store only
- disable option in dialog window

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 .../LinkShareSettings.vue                     | 37 +++++--------------
 src/store/conversationsStore.js               |  7 +++-
 2 files changed, 16 insertions(+), 28 deletions(-)

diff --git a/src/components/ConversationSettings/LinkShareSettings.vue b/src/components/ConversationSettings/LinkShareSettings.vue
index f683f6098f3..1ab00442f50 100644
--- a/src/components/ConversationSettings/LinkShareSettings.vue
+++ b/src/components/ConversationSettings/LinkShareSettings.vue
@@ -10,8 +10,11 @@
 		</h4>
 
 		<template v-if="canModerate">
+			<p v-if="hasBreakoutRooms" class="app-settings-section__hint">
+				{{ t('spreed', 'Breakout rooms are not allowed in public conversations.') }}
+			</p>
 			<NcCheckboxRadioSwitch :checked="isSharedPublicly"
-				:disabled="isSaving"
+				:disabled="hasBreakoutRooms || isSaving"
 				type="switch"
 				aria-describedby="link_share_settings_hint"
 				@update:checked="toggleGuests">
@@ -133,18 +136,16 @@ export default {
 			return this.$store.getters.conversation(this.token) || this.$store.getters.dummyConversation
 		},
 
+		hasBreakoutRooms() {
+			return this.conversation.breakoutRoomMode !== CONVERSATION.BREAKOUT_ROOM_MODE.NOT_CONFIGURED
+		},
+
 		isPasswordProtectionChecked() {
 			return this.conversation.hasPassword || this.showPasswordField
 		},
 	},
 
 	methods: {
-		focus() {
-			this.$nextTick(() => {
-				this.$refs.toggleGuests.focus()
-			})
-		},
-
 		async setConversationPassword(newPassword) {
 			this.isSaving = true
 			try {
@@ -169,27 +170,9 @@ export default {
 		},
 
 		async toggleGuests() {
-			const enabled = this.conversation.type !== CONVERSATION.TYPE.PUBLIC
+			const allowGuests = this.conversation.type !== CONVERSATION.TYPE.PUBLIC
 			this.isSaving = true
-			try {
-				await this.$store.dispatch('toggleGuests', {
-					token: this.token,
-					allowGuests: enabled,
-				})
-
-				if (enabled) {
-					showSuccess(t('spreed', 'You allowed guests'))
-				} else {
-					showSuccess(t('spreed', 'You disallowed guests'))
-				}
-			} catch (e) {
-				if (enabled) {
-					showError(t('spreed', 'Error occurred while allowing guests'))
-				} else {
-					showError(t('spreed', 'Error occurred while disallowing guests'))
-				}
-				console.error('Error toggling guest mode', e)
-			}
+			await this.$store.dispatch('toggleGuests', { token: this.token, allowGuests })
 			this.isSaving = false
 		},
 
diff --git a/src/store/conversationsStore.js b/src/store/conversationsStore.js
index 884840b3666..0057f15c147 100644
--- a/src/store/conversationsStore.js
+++ b/src/store/conversationsStore.js
@@ -471,17 +471,22 @@ const actions = {
 		}
 
 		try {
-			const conversation = Object.assign({}, getters.conversations[token])
+			const conversation = Object.assign({}, getters.conversation(token))
 			if (allowGuests) {
 				await makeConversationPublic(token)
 				conversation.type = CONVERSATION.TYPE.PUBLIC
+				showSuccess(t('spreed', 'You allowed guests'))
 			} else {
 				await makeConversationPrivate(token)
 				conversation.type = CONVERSATION.TYPE.GROUP
+				showSuccess(t('spreed', 'You disallowed guests'))
 			}
 			commit('addConversation', conversation)
 		} catch (error) {
 			console.error('Error while changing the conversation public status: ', error)
+			showError(allowGuests
+				? t('spreed', 'Error occurred while allowing guests')
+				: t('spreed', 'Error occurred while disallowing guests'))
 		}
 	},