Skip to content

Conversation

mgallien
Copy link
Collaborator

@mgallien mgallien commented Jun 6, 2025

will enable to dynamically fetch the list of files inside a folder

for now, only basic infrastructure plugging into CfApi is there

@mgallien mgallien added this to the 3.18.0 milestone Jun 6, 2025
@mgallien mgallien force-pushed the feature/onDemandFolderFetchVirtualFilesWindows branch 4 times, most recently from 969881f to 045c43e Compare June 13, 2025 13:08
@mgallien mgallien force-pushed the feature/onDemandFolderFetchVirtualFilesWindows branch 4 times, most recently from b30e9ed to 2e25647 Compare June 24, 2025 11:36
@mgallien mgallien force-pushed the feature/onDemandFolderFetchVirtualFilesWindows branch 3 times, most recently from 02daeaa to f4e6894 Compare July 7, 2025 13:00
@mgallien mgallien force-pushed the feature/onDemandFolderFetchVirtualFilesWindows branch from 6d9c4ce to 9a911c7 Compare July 9, 2025 09:26
@mgallien mgallien force-pushed the feature/onDemandFolderFetchVirtualFilesWindows branch 7 times, most recently from 00394d5 to 339046a Compare September 23, 2025 17:00
@Rello Rello moved this to 🏗️ In progress in 💻 Desktop Clients team Sep 24, 2025
@mgallien mgallien force-pushed the feature/onDemandFolderFetchVirtualFilesWindows branch 4 times, most recently from e69eecc to d3a874f Compare September 26, 2025 07:37
should make the logic much more reliable

Signed-off-by: Matthieu Gallien <[email protected]>
should enable proper type handling for virtual directories (i.e.
on-demand populated folders)

Signed-off-by: Matthieu Gallien <[email protected]>
after having populated the entries in a on-demand folder, update its
type in database to ensure updated state

only virtual folders have the on-demand feature enabled and that should
happen only once as we always fully populated a folder

Signed-off-by: Matthieu Gallien <[email protected]>
do not forget that a virtual folder is being also a folder

Signed-off-by: Matthieu Gallien <[email protected]>
when a folder is being hydrated (is no longer virtual), let's change the
DB record type to be an usual folder part of the normal synchronization

Signed-off-by: Matthieu Gallien <[email protected]>
in autotests, we use some QList
ensure we do not blindly access items in an empty list

Signed-off-by: Matthieu Gallien <[email protected]>
prevent recursive discovery inside a virtual folder but do not prevent
recursive discovery when needed

Signed-off-by: Matthieu Gallien <[email protected]>
add a static method in LsColJob class

moves some declaration to enable to reuse the code outside of its former
implementation location

Signed-off-by: Matthieu Gallien <[email protected]>
only add a Q_ASSERT for debug builds

Signed-off-by: Matthieu Gallien <[email protected]>
on-demand populating of folders is breaking some tests

for now, we expect the failures

Signed-off-by: Matthieu Gallien <[email protected]>
avoid duplicating the list of common webdav properties

Signed-off-by: Matthieu Gallien <[email protected]>
Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien mgallien force-pushed the feature/onDemandFolderFetchVirtualFilesWindows branch from c3caa4f to ef184cf Compare October 2, 2025 09:45
Copy link

github-actions bot commented Oct 2, 2025

Artifact containing the AppImage: nextcloud-appimage-pr-8359.zip

Digest: sha256:4bdc6a7f9cca1af94d17099c61e501382d4b73c8069d8482e5ad1bea8ef9f43e

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@mgallien mgallien merged commit 5f369e7 into master Oct 2, 2025
18 of 20 checks passed
@mgallien mgallien deleted the feature/onDemandFolderFetchVirtualFilesWindows branch October 2, 2025 11:08
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💻 Desktop Clients team Oct 2, 2025
@szaimen
Copy link
Contributor

szaimen commented Oct 2, 2025

🎉🎉🎉🎉🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ☑️ Done
Development

Successfully merging this pull request may close these issues.

3 participants