Close handling

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

apps/files/src/services/Sidebar.js

@@ -75,15 +75,6 @@ export default class Sidebar {
 		return false
 	}
 
-	/**
-	 * Close the sidebar
-	 *
-	 * @memberof Sidebar
-	 */
-	close() {
-		this.#state.file = ''
-	}
-
 	/**
 	 * Return current opened file
 	 *

apps/files/src/sidebar.js

@@ -57,4 +57,5 @@ window.addEventListener('DOMContentLoaded', () => {
 	})
 	AppSidebar.$mount('#app-sidebar')
 	window.OCA.Files.Sidebar.open = AppSidebar.open
+	window.OCA.Files.Sidebar.close = AppSidebar.close
 })

apps/files/src/views/Sidebar.vue

@@ -26,7 +26,7 @@
 		ref="sidebar"
 		v-bind="appSidebar"
 		:force-menu="true"
-		@close="onClose"
+		@close="close"
 		@update:active="setActiveTab"
 		@update:starred="toggleStarred"
 		@[defaultActionListener].stop.prevent="onDefaultAction">
@@ -251,10 +251,6 @@ export default {
 		canDisplay(tab) {
 			return tab.isEnabled(this.fileInfo)
 		},
-		onClose() {
-			this.resetData()
-			OCA.Files.Sidebar.close()
-		},
 		resetData() {
 			this.error = null
 			this.fileInfo = null
@@ -382,8 +378,9 @@ export default {
 		/**
 		 * Open the sidebar for the given file
 		 *
-		 * @memberof Sidebar
 		 * @param {string} path the file path to load
+		 * @returns {Promise}
+		 * @throws {Error} loading failure
 		 */
 		async open(path) {
 			// update current opened file
@@ -416,6 +413,14 @@ export default {
 				}
 			}
 		},
+
+		/**
+		 * Close the sidebar
+		 */
+		close() {
+			this.Sidebar.file = ''
+			this.resetData()
+		},
 	},
 }
 </script>