From f7c9ac8b543bce3869dde2d7fb52482275b2722c Mon Sep 17 00:00:00 2001 From: Louis Chemineau Date: Wed, 19 Jun 2024 11:18:57 +0200 Subject: fix(files): Rename openFileInfo to fileInfo And check if openFile is set in the URL before opening a file. Signed-off-by: Louis Chemineau --- apps/files/lib/Controller/ViewController.php | 2 +- apps/files/src/components/FilesListVirtual.vue | 7 +++++-- 2 files changed, 6 insertions(+), 3 deletions(-) (limited to 'apps') diff --git a/apps/files/lib/Controller/ViewController.php b/apps/files/lib/Controller/ViewController.php index 84bede683c5..ed01efe0ebb 100644 --- a/apps/files/lib/Controller/ViewController.php +++ b/apps/files/lib/Controller/ViewController.php @@ -292,7 +292,7 @@ class ViewController extends Controller { } $this->initialState->provideInitialState( - 'openFileInfo', [ + 'fileInfo', [ 'id' => $node->getId(), 'name' => $isRoot ? '' : $node->getName(), 'path' => $path, diff --git a/apps/files/src/components/FilesListVirtual.vue b/apps/files/src/components/FilesListVirtual.vue index 12f4642f227..7c74f2d2d7e 100644 --- a/apps/files/src/components/FilesListVirtual.vue +++ b/apps/files/src/components/FilesListVirtual.vue @@ -206,8 +206,7 @@ export default defineComponent({ const mainContent = window.document.querySelector('main.app-content') as HTMLElement mainContent.addEventListener('dragover', this.onDragOver) - // handle initially opening a given file - const { id } = loadState<{ id?: number }>('files', 'openFileInfo', {}) + const { id } = loadState<{ id?: number }>('files', 'fileInfo', {}) this.scrollToFile(id ?? this.fileId) this.openSidebarForFile(id ?? this.fileId) this.handleOpenFile(id ?? null) @@ -248,6 +247,10 @@ export default defineComponent({ * @param fileId File to open */ handleOpenFile(fileId: number|null) { + if (!this.openFile) { + return + } + if (fileId === null || this.openFileId === fileId) { return } -- cgit v1.2.3