fix: listen to dialog triggers from URL hash only when conversation i…

…s open

Signed-off-by: Maksim Sukharev <[email protected]>

src/App.vue

@@ -474,10 +474,6 @@ export default {
 			} else if (to.name === 'notfound') {
 				this.setPageTitle('')
 			}
-
-			if (to.hash === '#direct-call') {
-				emit('talk:media-settings:show')
-			}
 		})
 
 		if (getCurrentUser()) {
@@ -502,10 +498,6 @@ export default {
 			})
 		}
 
-		if (this.$route.hash === '#direct-call') {
-			emit('talk:media-settings:show')
-		}
-
 		subscribe('notifications:action:execute', this.interceptNotificationActions)
 		subscribe('notifications:notification:received', this.interceptNotificationReceived)
 	},

src/components/ConversationSettings/ConversationSettingsDialog.vue

@@ -267,10 +267,9 @@ export default {
 		}
 	},
 
-	mounted() {
+	beforeMount() {
 		subscribe('show-conversation-settings', this.handleShowSettings)
 		subscribe('hide-conversation-settings', this.handleHideSettings)
-
 	},
 
 	beforeDestroy() {

src/components/MediaSettings/MediaSettings.vue

@@ -466,7 +466,7 @@ export default {
 		},
 	},
 
-	mounted() {
+	beforeMount() {
 		subscribe('talk:media-settings:show', this.showModal)
 		subscribe('talk:media-settings:hide', this.closeModalAndApplySettings)
 

src/views/MainView.vue

@@ -16,13 +16,16 @@
 </template>
 
 <script>
+import { emit } from '@nextcloud/event-bus'
+
 import CallView from '../components/CallView/CallView.vue'
 import ChatView from '../components/ChatView.vue'
 import LobbyScreen from '../components/LobbyScreen.vue'
 import PollViewer from '../components/PollViewer/PollViewer.vue'
 import TopBar from '../components/TopBar/TopBar.vue'
 
 import { useIsInCall } from '../composables/useIsInCall.js'
+import Router from '../router/router.js'
 
 export default {
 	name: 'MainView',
@@ -68,6 +71,20 @@ export default {
 		},
 	},
 
+	mounted() {
+		const handleRouteHashChange = (token, route) => {
+			if (route?.hash === '#direct-call') {
+				emit('talk:media-settings:show')
+				Router.replace({ ...route, hash: '' })
+			} else if (route?.hash === '#settings') {
+				emit('show-conversation-settings', { token })
+				Router.replace({ ...route, hash: '' })
+			}
+		}
+
+		handleRouteHashChange(this.token, Router.currentRoute)
+		Router.afterEach((to) => handleRouteHashChange(this.token, to))
+	},
 }
 </script>