Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve read only virtual files support on Windows #7477

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

mgallien
Copy link
Collaborator

@mgallien mgallien commented Nov 8, 2024

No description provided.

mgallien and others added 21 commits November 12, 2024 23:15
ensure that even if move to trash is enabled, we do not erroneously try
to move online only files or folders to trash bin

Signed-off-by: Matthieu Gallien <[email protected]>
we need to make shortcut files be proper virtual placeholder files

let the sync engine use the proper item type for that

Signed-off-by: Matthieu Gallien <[email protected]>
we cannot do a single call to convert the plain old file to be a
dehydrated virtual file placeholder

split the conversion into a separate initial API call

fix transition from releases < 3.14 to current releases that have first
class support for windows shortcuts (so called link files)

Signed-off-by: Matthieu Gallien <[email protected]>
should make the existing log more useful when tracking the
self requested implicit hydration bugs

Signed-off-by: Matthieu Gallien <[email protected]>
we were using specialized API to set the file permissions on the windows
shortcut and later falling back to the generic code path due to the
mising return

Signed-off-by: Matthieu Gallien <[email protected]>
will prevent some type of activity being added to the list from
triggering implicit hydration during synchronization

most probably breaks existing open actions on those activities

Signed-off-by: Matthieu Gallien <[email protected]>
change the way we evaluate a folder permissions to be read-only when no
new file or new folders can be created

if those permissions are lacking, client will make the local folder
read-only

if those permissions are granted, client will make local folder
read-write

Signed-off-by: Matthieu Gallien <[email protected]>
for now unable to dynamically switch between light and dark mode

Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien mgallien force-pushed the bugfix/readOnlyVirtualFilesFixes branch from 45faa94 to c5d66c9 Compare November 12, 2024 22:15
@nextcloud-desktop-bot
Copy link

AppImage file: nextcloud-PR-7477-c5d66c94e9d858a197c231f728e13ffd3c4ed95c-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

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

Copy link

@Mika-6-9 Mika-6-9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi.
Would it be possible to take into consideration with this addition (8652238) the icons of locking and sharing files in Windows Explorer?
#6070
#5644
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants