CHANGELOG.md

@@ -6,6 +6,19 @@
 
 All notable changes to this project will be documented in this file.
 
+## 5.3.5
+[Full Changelog](https://github.com/nextcloud-libraries/nextcloud-dialogs/compare/v5.3.4...v5.3.5)
+
+### Fixed
+* fix(FilePicker): Cleanup DAV handling and properly handle `currentFolder` [\#1378](https://github.com/nextcloud-libraries/nextcloud-dialogs/pull/1378) \([susnux](https://github.com/susnux)\)
+
+### Changed
+* Updated translations
+* chore(deps): Bump @nextcloud/typings from 1.8.0 to 1.9.0
+* chore(deps-dev): Bump typedoc from 0.26.0 to 0.26.3
+* chore(deps-dev): Bump @zamiell/typedoc-plugin-not-exported from 0.2.0 to 0.3.0
+* chore(deps-dev): Bump vite from 5.3.1 to 5.3.2
+
 ## 5.3.4
 [Full Changelog](https://github.com/nextcloud-libraries/nextcloud-dialogs/compare/v5.3.3...v5.3.4)
 

l10n/fi_FI.pot

@@ -1,15 +1,126 @@
 # 
 # Translators:
-# Joas Schilling, 2023
+# John Molakvoæ <skjnldsv@protonmail.com>, 2023
+# Jiri Grönroos <jiri.gronroos@iki.fi>, 2024
+# thingumy, 2024
 # 
 msgid ""
 msgstr ""
-"Last-Translator: Joas Schilling, 2023\n"
+"Last-Translator: thingumy, 2024\n"
 "Language-Team: Finnish (Finland) (https://app.transifex.com/nextcloud/teams/64236/fi_FI/)\n"
 "Content-Type: text/plain; charset=UTF-8\n"
 "Language: fi_FI\n"
 "Plural-Forms: nplurals=2; plural=(n != 1);\n"
 
-#: lib/toast.ts:223
+msgid "\"{name}\" is an invalid folder name."
+msgstr "\"{name}\" on virheellinen kansion nimi."
+
+msgid "\"{name}\" is not an allowed folder name"
+msgstr "\"{name}\" ei ole sallittu kansion nimi"
+
+msgid "\"/\" is not allowed inside a folder name."
+msgstr "\"/\" ei ole sallittu kansion nimessä."
+
+msgid "All files"
+msgstr "Kaikki tiedostot"
+
+msgid "Choose"
+msgstr "Valitse"
+
+msgid "Choose {file}"
+msgstr "Valitse {file}"
+
+msgid "Choose %n file"
+msgid_plural "Choose %n files"
+msgstr[0] "Valitse %n tiedosto"
+msgstr[1] "Valitse %n tiedostoa"
+
+msgid "Copy"
+msgstr "Kopioi"
+
+msgid "Copy to {target}"
+msgstr "Kopioi sijaintiin {target}"
+
+msgid "Could not create the new folder"
+msgstr "Uutta kansiota ei voitu luoda"
+
+msgid "Could not load files settings"
+msgstr "Tiedoston asetuksia ei saa ladattua"
+
+msgid "Could not load files views"
+msgstr "Tiedoston näkymiä ei saa ladattua"
+
+msgid "Create directory"
+msgstr "Luo kansio"
+
+msgid "Current view selector"
+msgstr "Nykyisen näkymän valinta"
+
+msgid "Favorites"
+msgstr "Suosikit"
+
+msgid "Files and folders you mark as favorite will show up here."
+msgstr "Tiedostot ja kansiot, jotka merkitset suosikkeihisi, näkyvät täällä."
+
+msgid "Files and folders you recently modified will show up here."
+msgstr "Tiedostot ja kansiot, joita muokkasit äskettäin, näkyvät täällä."
+
+msgid "Filter file list"
+msgstr "Suodata tiedostolistaa"
+
+msgid "Folder name cannot be empty."
+msgstr "Kansion nimi ei voi olla tyhjä."
+
+msgid "Home"
+msgstr "Koti"
+
+msgid "Modified"
+msgstr "Muokattu"
+
+msgid "Move"
+msgstr "Siirrä"
+
+msgid "Move to {target}"
+msgstr "Siirrä sijaintiin {target}"
+
+msgid "Name"
+msgstr "Nimi"
+
+msgid "New"
+msgstr "Uusi"
+
+msgid "New folder"
+msgstr "Uusi kansio"
+
+msgid "New folder name"
+msgstr "Uuden kansion nimi"
+
+msgid "No files in here"
+msgstr "Täällä ei ole tiedostoja"
+
+msgid "No files matching your filter were found."
+msgstr "Suodatinta vastaavia tiedostoja ei löytynyt."
+
+msgid "No matching files"
+msgstr "Ei vastaavia tiedostoja"
+
+msgid "Recent"
+msgstr "Viimeisimmät"
+
+msgid "Select all entries"
+msgstr "Valitse kaikki tietueet"
+
+msgid "Select entry"
+msgstr "Valitse tietue"
+
+msgid "Select the row for {nodename}"
+msgstr "Valitse rivi {nodename}:lle"
+
+msgid "Size"
+msgstr "Koko"
+
 msgid "Undo"
 msgstr "Kumoa"
+
+msgid "Upload some content or sync with your devices!"
+msgstr "Lähetä jotain sisältöä tai synkronoi laitteidesi kanssa!"

lib/components/FilePicker/FilePicker.vue

@@ -146,13 +146,19 @@ const isOpen = ref(true)
  * Map buttons to Dialog buttons by wrapping the callback function to pass the selected files
  */
 const dialogButtons = computed(() => {
-	const nodes = selectedFiles.value.length === 0 && props.allowPickDirectory && currentFolder.value ? [currentFolder.value] : selectedFiles.value
+	const nodes = selectedFiles.value.length === 0
+		&& props.allowPickDirectory
+		&& currentFolder.value
+		? [currentFolder.value]
+		: selectedFiles.value
+
 	const buttons = typeof props.buttons === 'function'
 		? props.buttons(nodes, currentPath.value, currentView.value)
 		: props.buttons
 
 	return buttons.map((button) => ({
 		...button,
+		disabled: button.disabled || isLoading.value,
 		callback: () => {
 			// lock default close handling
 			isHandlingCallback = true
@@ -203,9 +209,9 @@ const navigatedPath = ref('')
 watch([navigatedPath], () => {
 	if (props.path === undefined && navigatedPath.value) {
 		window.sessionStorage.setItem('NC.FilePicker.LastPath', navigatedPath.value)
-		// Reset selected files
-		selectedFiles.value = []
 	}
+	// Reset selected files
+	selectedFiles.value = []
 })
 
 /**

lib/composables/dav.spec.ts

@@ -71,7 +71,6 @@ describe('dav composable', () => {
 		expect(Array.isArray(vue.vm.files)).toBe(true)
 		expect(vue.vm.files.length).toBe(0)
 		// functions
-		expect(typeof vue.vm.getFile === 'function').toBe(true)
 		expect(typeof vue.vm.loadFiles === 'function').toBe(true)
 	})
 
@@ -153,23 +152,6 @@ describe('dav composable', () => {
 		expect(client.search).toBeCalledTimes(1)
 	})
 
-	it('getFile works', async () => {
-		const client = {
-			stat: vi.fn((v) => ({ data: { path: v } })),
-			getDirectoryContents: vi.fn(() => ({ data: [] })),
-		}
-		nextcloudFiles.davGetClient.mockImplementation(() => client)
-		nextcloudFiles.davResultToNode.mockImplementation((v) => v)
-
-		const { getFile } = useDAVFiles(ref('files'), ref('/'))
-
-		const node = await getFile('/some/path/file.ext')
-		expect(node).toEqual({ path: `${nextcloudFiles.davRootPath}/some/path/file.ext` })
-		// Check mock usage
-		expect(client.stat).toBeCalledWith(`${nextcloudFiles.davRootPath}/some/path/file.ext`, { details: true })
-		expect(nextcloudFiles.davResultToNode).toBeCalledWith({ path: `${nextcloudFiles.davRootPath}/some/path/file.ext` })
-	})
-
 	it('createDirectory works', async () => {
 		const client = {
 			stat: vi.fn((v) => ({ data: { path: v } })),
@@ -189,11 +171,12 @@ describe('dav composable', () => {
 	it('loadFiles work', async () => {
 		const client = {
 			stat: vi.fn((v) => ({ data: { path: v } })),
-			getDirectoryContents: vi.fn((p, o) => ({ data: [] })),
-			search: vi.fn((p, o) => ({ data: { results: [], truncated: false } })),
+			getDirectoryContents: vi.fn((_p, _o) => ({ data: [] })),
+			search: vi.fn((_p, _o) => ({ data: { results: [], truncated: false } })),
 		}
 		nextcloudFiles.davGetClient.mockImplementationOnce(() => client)
 		nextcloudFiles.davResultToNode.mockImplementationOnce((v) => v)
+		nextcloudFiles.getFavoriteNodes.mockImplementationOnce(() => Promise.resolve([]))
 
 		const view = ref<'files' | 'recent' | 'favorites'>('files')
 		const path = ref('/')
@@ -216,8 +199,8 @@ describe('dav composable', () => {
 	it('request cancelation works', async () => {
 		const client = {
 			stat: vi.fn((v) => ({ data: { path: v } })),
-			getDirectoryContents: vi.fn((p, o) => ({ data: [] })),
-			search: vi.fn((p, o) => ({ data: { results: [], truncated: false } })),
+			getDirectoryContents: vi.fn((_p, _o) => ({ data: [] })),
+			search: vi.fn((_p, _o) => ({ data: { results: [], truncated: false } })),
 		}
 		nextcloudFiles.davGetClient.mockImplementationOnce(() => client)
 		nextcloudFiles.davResultToNode.mockImplementationOnce((v) => v)

lib/composables/dav.ts

@@ -2,14 +2,14 @@
  * SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
-import type { Folder, Node } from '@nextcloud/files'
+import type { ContentsWithRoot, Folder, Node } from '@nextcloud/files'
 import type { ComputedRef, Ref } from 'vue'
-import type { FileStat, ResponseDataDetailed, SearchResult } from 'webdav'
 
-import { davGetClient, davGetDefaultPropfind, davGetRecentSearch, davResultToNode, davRootPath, getFavoriteNodes } from '@nextcloud/files'
-import { join } from 'path'
-import { onMounted, ref, shallowRef, watch } from 'vue'
+import { davGetClient, davRootPath, getFavoriteNodes } from '@nextcloud/files'
 import { CancelablePromise } from 'cancelable-promise'
+import { join } from 'node:path'
+import { onMounted, ref, shallowRef, watch } from 'vue'
+import { getFile, getNodes, getRecentNodes } from '../utils/dav'
 
 /**
  * Handle file loading using WebDAV
@@ -27,48 +27,6 @@ export const useDAVFiles = function(
 	 */
 	const client = davGetClient()
 
-	const resultToNode = (result: FileStat) => davResultToNode(result)
-
-	const getRecentNodes = (): CancelablePromise<Node[]> => {
-		const controller = new AbortController()
-		// unix timestamp in seconds, two weeks ago
-		const lastTwoWeek = Math.round(Date.now() / 1000) - (60 * 60 * 24 * 14)
-		return new CancelablePromise(async (resolve, reject, onCancel) => {
-			onCancel(() => controller.abort())
-			try {
-				const { data } = await client.search('/', {
-					signal: controller.signal,
-					details: true,
-					data: davGetRecentSearch(lastTwoWeek),
-				}) as ResponseDataDetailed<SearchResult>
-				const nodes = data.results.map(resultToNode)
-				resolve(nodes)
-			} catch (error) {
-				reject(error)
-			}
-		})
-	}
-
-	const getNodes = (): CancelablePromise<Node[]> => {
-		const controller = new AbortController()
-		return new CancelablePromise(async (resolve, reject, onCancel) => {
-			onCancel(() => controller.abort())
-			try {
-				const results = await client.getDirectoryContents(join(davRootPath, currentPath.value), {
-					signal: controller.signal,
-					details: true,
-					data: davGetDefaultPropfind(),
-				}) as ResponseDataDetailed<FileStat[]>
-				let nodes = results.data.map(resultToNode)
-				// Hack for the public endpoint which always returns folder itself
-				nodes = nodes.filter((file) => file.path !== currentPath.value)
-				resolve(nodes)
-			} catch (error) {
-				reject(error)
-			}
-		})
-	}
-
 	/**
 	 * All files in current view and path
 	 */
@@ -77,49 +35,33 @@ export const useDAVFiles = function(
 	/**
 	 * The current folder
 	 */
-	const folder = shallowRef<Folder>()
-	watch([currentPath], async () => {
-		folder.value = (files.value.find(({ path }) => path === currentPath.value) ?? await getFile(currentPath.value)) as Folder
-	}, { immediate: true })
+	const folder = shallowRef<Folder|null>(null)
 
 	/**
 	 * Loading state of the files
 	 */
 	const isLoading = ref(true)
 
 	/**
-	 * The cancelable promise
+	 * The cancelable promise used internally to cancel on fast navigation
 	 */
-	const promise = ref<null | CancelablePromise<unknown>>(null)
+	const promise = ref<null | CancelablePromise<Node[] | ContentsWithRoot>>(null)
 
 	/**
 	 * Create a new directory in the current path
+	 * The directory will be added to the current file list
 	 * @param name Name of the new directory
 	 * @return {Promise<Folder>} The created directory
 	 */
 	async function createDirectory(name: string): Promise<Folder> {
 		const path = join(currentPath.value, name)
 
 		await client.createDirectory(join(davRootPath, path))
-		const directory = await getFile(path) as Folder
+		const directory = await getFile(client, path) as Folder
 		files.value = [...files.value, directory]
 		return directory
 	}
 
-	/**
-	 * Get information for one file
-	 *
-	 * @param path The path of the file or folder
-	 * @param rootPath DAV root path, defaults to '/files/USERID'
-	 */
-	async function getFile(path: string, rootPath: string = davRootPath) {
-		const { data } = await client.stat(join(rootPath, path), {
-			details: true,
-			data: davGetDefaultPropfind(),
-		}) as ResponseDataDetailed<FileStat>
-		return resultToNode(data)
-	}
-
 	/**
 	 * Force reload files using the DAV client
 	 */
@@ -132,11 +74,18 @@ export const useDAVFiles = function(
 		if (currentView.value === 'favorites') {
 			promise.value = getFavoriteNodes(client, currentPath.value)
 		} else if (currentView.value === 'recent') {
-			promise.value = getRecentNodes()
+			promise.value = getRecentNodes(client)
+		} else {
+			promise.value = getNodes(client, currentPath.value)
+		}
+		const content = await promise.value
+		if ('folder' in content) {
+			folder.value = content.folder
+			files.value = content.contents
 		} else {
-			promise.value = getNodes()
+			folder.value = null
+			files.value = content
 		}
-		files.value = await promise.value as Node[]
 
 		promise.value = null
 		isLoading.value = false
@@ -157,7 +106,6 @@ export const useDAVFiles = function(
 		files,
 		folder,
 		loadFiles: loadDAVFiles,
-		getFile,
 		createDirectory,
 	}
 }