From 6c544172e3f8ee43df8b8594cca3e50dd5eeff74 Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Thu, 4 Apr 2024 13:02:51 +0200 Subject: [PATCH] fix(files): Do not show files from hidden folders in "Recent"-view if hidden files are disabled by user Needed to adjust the store creation to be able to inject pinia before the vue app is initialized. Signed-off-by: Ferdinand Thiessen --- apps/files/src/main.ts | 6 +- apps/files/src/services/Recent.ts | 24 ++++- apps/files/src/store/index.ts | 25 ++++++ apps/files/src/views/FilesList.vue | 8 ++ apps/files/src/views/recent.ts | 2 +- cypress/e2e/files/files-settings.cy.ts | 116 +++++++++++++++++++++++++ 6 files changed, 175 insertions(+), 6 deletions(-) create mode 100644 apps/files/src/store/index.ts create mode 100644 cypress/e2e/files/files-settings.cy.ts diff --git a/apps/files/src/main.ts b/apps/files/src/main.ts index 7b25c88a697..97b3b2f9651 100644 --- a/apps/files/src/main.ts +++ b/apps/files/src/main.ts @@ -1,8 +1,9 @@ -import Vue from 'vue' -import { createPinia, PiniaVuePlugin } from 'pinia' +import { PiniaVuePlugin } from 'pinia' import { getNavigation } from '@nextcloud/files' import { getRequestToken } from '@nextcloud/auth' +import Vue from 'vue' +import { pinia } from './store/index.ts' import router from './router/router' import RouterService from './services/RouterService' import SettingsModel from './models/Setting.js' @@ -30,7 +31,6 @@ Object.assign(window.OCP.Files, { Router }) // Init Pinia store Vue.use(PiniaVuePlugin) -const pinia = createPinia() // Init Navigation Service // This only works with Vue 2 - with Vue 3 this will not modify the source but return just a oberserver diff --git a/apps/files/src/services/Recent.ts b/apps/files/src/services/Recent.ts index 26c4babb5a2..47f31466dd2 100644 --- a/apps/files/src/services/Recent.ts +++ b/apps/files/src/services/Recent.ts @@ -20,17 +20,37 @@ * along with this program. If not, see . * */ -import type { ContentsWithRoot } from '@nextcloud/files' +import type { ContentsWithRoot, Node } from '@nextcloud/files' import type { FileStat, ResponseDataDetailed } from 'webdav' import { getCurrentUser } from '@nextcloud/auth' import { Folder, Permission, davGetRecentSearch, davGetClient, davResultToNode, davRootPath, davRemoteURL } from '@nextcloud/files' +import { useUserConfigStore } from '../store/userconfig.ts' +import { pinia } from '../store/index.ts' const client = davGetClient() const lastTwoWeeksTimestamp = Math.round((Date.now() / 1000) - (60 * 60 * 24 * 14)) +/** + * Get recently changed nodes + * + * This takes the users preference about hidden files into account. + * If hidden files are not shown, then also recently changed files *in* hidden directories are filtered. + * + * @param path Path to search for recent changes + */ export const getContents = async (path = '/'): Promise => { + const store = useUserConfigStore(pinia) + /** + * Filter function that returns only the visible nodes - or hidden if explicitly configured + * @param node The node to check + */ + const filterHidden = (node: Node) => + path !== '/' // We need to hide files from hidden directories in the root if not configured to show + || store.userConfig.show_hidden // If configured to show hidden files we can early return + || !node.dirname.split('/').some((dir) => dir.startsWith('.')) // otherwise only include the file if non of the parent directories is hidden + const contentsResponse = await client.getDirectoryContents(path, { details: true, data: davGetRecentSearch(lastTwoWeeksTimestamp), @@ -53,6 +73,6 @@ export const getContents = async (path = '/'): Promise => { owner: getCurrentUser()?.uid || null, permissions: Permission.READ, }), - contents: contents.map((r) => davResultToNode(r)), + contents: contents.map((r) => davResultToNode(r)).filter(filterHidden), } } diff --git a/apps/files/src/store/index.ts b/apps/files/src/store/index.ts new file mode 100644 index 00000000000..c97a09122ec --- /dev/null +++ b/apps/files/src/store/index.ts @@ -0,0 +1,25 @@ +/** + * @copyright Copyright (c) 2024 Ferdinand Thiessen + * + * @author Ferdinand Thiessen + * + * @license AGPL-3.0-or-later + * + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with this program. If not, see . + * + */ + +import { createPinia } from 'pinia' + +export const pinia = createPinia() diff --git a/apps/files/src/views/FilesList.vue b/apps/files/src/views/FilesList.vue index 66c81714860..f5bb45ede1d 100644 --- a/apps/files/src/views/FilesList.vue +++ b/apps/files/src/views/FilesList.vue @@ -214,6 +214,8 @@ export default defineComponent({ loading: true, promise: null, Type, + + _unsubscribeStore: () => {}, } }, @@ -454,10 +456,16 @@ export default defineComponent({ subscribe('files:node:updated', this.onUpdatedNode) subscribe('nextcloud:unified-search.search', this.onSearch) subscribe('nextcloud:unified-search.reset', this.onSearch) + + // reload on settings change + this._unsubscribeStore = this.userConfigStore.$subscribe(() => this.fetchContent(), { deep: true }) }, unmounted() { unsubscribe('files:node:updated', this.onUpdatedNode) + unsubscribe('nextcloud:unified-search.search', this.onSearch) + unsubscribe('nextcloud:unified-search.reset', this.onSearch) + this._unsubscribeStore() }, methods: { diff --git a/apps/files/src/views/recent.ts b/apps/files/src/views/recent.ts index bb536674ec1..257aaf30dfd 100644 --- a/apps/files/src/views/recent.ts +++ b/apps/files/src/views/recent.ts @@ -19,11 +19,11 @@ * along with this program. If not, see . * */ +import { View, getNavigation } from '@nextcloud/files' import { translate as t } from '@nextcloud/l10n' import HistorySvg from '@mdi/svg/svg/history.svg?raw' import { getContents } from '../services/Recent' -import { View, getNavigation } from '@nextcloud/files' export default () => { const Navigation = getNavigation() diff --git a/cypress/e2e/files/files-settings.cy.ts b/cypress/e2e/files/files-settings.cy.ts new file mode 100644 index 00000000000..497f7c8782e --- /dev/null +++ b/cypress/e2e/files/files-settings.cy.ts @@ -0,0 +1,116 @@ +/** + * @copyright Copyright (c) 2024 Ferdinand Thiessen + * + * @author Ferdinand Thiessen + * + * @license AGPL-3.0-or-later + * + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with this program. If not, see . + * + */ + +import type { User } from '@nextcloud/cypress' +import { getRowForFile } from './FilesUtils' + +const showHiddenFiles = () => { + // Open the files settings + cy.get('[data-cy-files-navigation-settings-button] a').click({ force: true }) + // Toggle the hidden files setting + cy.get('[data-cy-files-settings-setting="show_hidden"]').within(() => { + cy.get('input').should('not.be.checked') + cy.get('input').check({ force: true }) + }) + // Close the dialog + cy.get('[data-cy-files-navigation-settings] button[aria-label="Close"]').click() +} + +describe('files: Hide or show hidden files', { testIsolation: true }, () => { + let user: User + + const setupFiles = () => cy.createRandomUser().then(($user) => { + user = $user + + cy.uploadContent(user, new Blob([]), 'text/plain', '/.file') + cy.uploadContent(user, new Blob([]), 'text/plain', '/visible-file') + cy.mkdir(user, '/.folder') + cy.login(user) + }) + + context('view: All files', { testIsolation: false }, () => { + before(setupFiles) + + it('hides dot-files by default', () => { + cy.visit('/apps/files') + + getRowForFile('visible-file').should('be.visible') + getRowForFile('.file').should('not.exist') + getRowForFile('.folder').should('not.exist') + }) + + it('can show hidden files', () => { + showHiddenFiles() + // Now the files should be visible + getRowForFile('.file').should('be.visible') + getRowForFile('.folder').should('be.visible') + }) + }) + + context('view: Personal files', { testIsolation: false }, () => { + before(setupFiles) + + it('hides dot-files by default', () => { + cy.visit('/apps/files/personal') + + getRowForFile('visible-file').should('be.visible') + getRowForFile('.file').should('not.exist') + getRowForFile('.folder').should('not.exist') + }) + + it('can show hidden files', () => { + showHiddenFiles() + // Now the files should be visible + getRowForFile('.file').should('be.visible') + getRowForFile('.folder').should('be.visible') + }) + }) + + context('view: Recent files', { testIsolation: false }, () => { + before(() => { + setupFiles().then(() => { + // also add hidden file in hidden folder + cy.uploadContent(user, new Blob([]), 'text/plain', '/.folder/other-file') + cy.login(user) + }) + }) + + it('hides dot-files by default', () => { + cy.visit('/apps/files/recent') + + getRowForFile('visible-file').should('be.visible') + getRowForFile('.file').should('not.exist') + getRowForFile('.folder').should('not.exist') + getRowForFile('other-file').should('not.exist') + }) + + it('can show hidden files', () => { + showHiddenFiles() + + getRowForFile('visible-file').should('be.visible') + // Now the files should be visible + getRowForFile('.file').should('be.visible') + getRowForFile('.folder').should('be.visible') + getRowForFile('other-file').should('be.visible') + }) + }) +}) -- 2.39.5