Merge pull request #41687 from nextcloud/fix/unified-search-view

fix(core): Subscribe to navigation changes on mounted for Unified search
nextcloud · Nov 23, 2023 · a9c627c · a9c627c
2 parents 5f159d4 + fca05f4
commit a9c627c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/core/src/views/UnifiedSearch.vue b/core/src/views/UnifiedSearch.vue
@@ -334,7 +334,6 @@ export default {
 	},
 
 	async created() {
-		subscribe('files:navigation:changed', this.onNavigationChange)
 		this.types = await getTypes()
 		this.logger.debug('Unified Search initialized with the following providers', this.types)
 	},
@@ -344,6 +343,9 @@ export default {
 	},
 
 	mounted() {
+		// subscribe in mounted, as onNavigationChange relys on $el
+		subscribe('files:navigation:changed', this.onNavigationChange)
+
 		if (OCP.Accessibility.disableKeyboardShortcuts()) {
 			return
 		}
@@ -383,7 +385,7 @@ export default {
 		},
 
 		onNavigationChange() {
-			this.$el.querySelector('form[role="search"]').reset()
+			this.$el?.querySelector?.('form[role="search"]')?.reset?.()
 		},
 
 		/**