avoid deadlock: cancel hydration requests coming from itself

our current infrastructure will cause a deadlock for any hydration
requests coming from desktop files client itself

the main thread is responsible for executing the hydration request but
if the same thread is already blocked waiting for the hydration to
happen, both (open system call and hydration request handling) process
will never be completed and wait for each over in a stuck cycle

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6865 from nextcloud/backport/6864/stable-3.13

[stable-3.13] avoid logs too verbose and with binary data

avoid logs too verbose and with binary data

with those logs, hydrating is causing too much log to be written and
after some time (with big fixles), hydration is failing

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

prepare for 3.13.1 release

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6858 from nextcloud/backport/6802/stable-3.13

[stable-3.13] Fix sharePath for virtual paths using suffix vfs mode

Fix sharePath for virtual paths using suffix vfs mode

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6857 from nextcloud/backport/6773/stable-3.13

[stable-3.13] Bugfix nextcloudcmd when syncing results in deleting all files

Add tests to remove all files when sycing with or without nextcloudcmd.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Add sync option for nextcloudcmd.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Use readable variable names.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Make warning text before deleting all files more clear and concise.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Fix #3144: check for promptDeleteAllFiles config setting before emitting signal to display warning.

The signal is not connected to any slot when running nextcloudcmd.
The callback to finish the sync was never being called because the check for the config was done in the slot,
which was never called.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Merge pull request #6856 from nextcloud/backport/6821/stable-3.13

[stable-3.13] Allow deletion of virtual suffix placeholder files

Properly reset fakefolder test state post-placeholder file delete in testsyncvirtualfiles

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Ensure stripping of suffix vfs placeholder file's suffix when issuing delete command to server

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Amend behaviour of suffix virtual files test to reflect new expected behaviour

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Allow deletion of virtual suffix placeholder files

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6853 from nextcloud/backport/6839/stable-3.13

[stable-3.13] Bugfix/rw folders broken

Do not wrap macOS ifdefs in filesystem when parent function already wrapped in ifdef

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Do not re-generate standard string path twice

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Ensure we are re-adding same permissions as removed

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6850 from nextcloud/backport/6846/stable-3.13

[stable-3.13] Add cfapi debug logs

Print desktop client process id.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Add new cfapi callbacks for logging.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Add more logs to track file hydration.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Merge pull request #6842 from nextcloud/backport/6841/stable-3.13

[stable-3.13] Bugfix/update encryption state when server change

ignore encrypted items during discovery when encryption is disabled

when we know a folder should be encrypted but encryption client side app
is disabled, ignore any items in the encrypted folders

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

do not allow to select e2ee folders if the server support is missing

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

maintain e2ee state even if the server does not support e2ee

might be that the server previously supported e2ee and e2ee folders
exist

so we have to fetch the info to properly handle it on client

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6838 from nextcloud/backport/6833/stable-3.13

[stable-3.13] Fix crash when updating enabled File Provider domains (macOS VFS)

Remove no longer existing account id from configured vfs accounts list

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Prevent crash when updating file provider domains if the account for which a file provider domain was configured no longer exists

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Use accountfwd header for account forward declaration in account header itself, preventing conflicting decls

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Assert validity of accountstate and its accountptr when adding it

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Remove overwrought loadFromSettings method from AccountState

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Take AccountPtr by reference in accountstate constructor

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6834 from nextcloud/backport/6832/stable-3.13

[stable-3.13] Ci/test is mount root property

automated test: allow rename of the child items of a mount point

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

automated test: block rename of the top folder of a mount point

encode current behavior in a new automated test

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6827 from nextcloud/bugfix/doNotUpdateVfsMetadataWhenSomethingElseIsNeeded

only use VFS update instruction when sync engine would do nothing

only use VFS update instruction when sync engine would do nothing

will ensure we do not erroneously use this instruction when we should
not

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6812 from nextcloud/backport/6811/stable-3.13

[stable-3.13] Fix text colour for NCInputTextEdit

Fix text colour for NCInputTextEdit

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6795 from nextcloud/backport/6714/stable-3.13

stable-3.13: Backport #6714

Update build-appimage.sh to use $VERSION_SUFFIX

Make sure that we can pass the value of MIRALL_VERSION_SUFFIX via parameter.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Merge pull request #6765 from nextcloud/ci/fixCompilation-stable-3.13

you cannot call non const method on a temp const QString

you cannot call non const method on a temp const QString

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6747 from nextcloud/backport/6742/stable-3.13

[stable-3.13] Bugfix/lockownertype setreadonly basedoncaps

Fix incorrect date parsing.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Bugfix. Files lock. Check lock owner type for setting readonly based on server capabilities (NC27/28 compatibility issue).

Signed-off-by: alex-z <blackslayer4@gmail.com>

Merge pull request #6759 from nextcloud/backport/6706/stable-3.13

Censor account details string in file provider logging

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6755 from nextcloud/backport/6753/stable-3.13

[stable-3.13] Do not open client on install as this breaks MDM deployments

Do not open client on install as this breaks MDM deployments

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6709 from nextcloud/backport/6703/stable-3.13

[stable-3.13] chore(i18n): Improve grammar

chore(i18n): Improve grammar

Fixes #6408

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

Merge pull request #6698 from nextcloud/backport/6691/stable-3.13

[stable-3.13] Bugfix/slow sync with tray open

Merge pull request #6697 from nextcloud/backport/6696/stable-3.13

[stable-3.13] Bugfix. Conflict dialog for multiple files. Fix checkbox border colors for dark mode on Windows.

Merge pull request #6692 from nextcloud/backport/6660/stable-3.13

[stable-3.13] Bugfix. Files lock. Fix incorrect readonly state.

Hotfix. Slow download speed while tray is open. Do not run rotation animation in syncstatus in tray.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Just for test. Disable status update to avoid havin a sync status animation in tray.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Bugfix. Conflict dialog for multiple files. Fix checkbox border colors for dark mode on Windows.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Bugfix. Files lock. Fix incorrect readonly state.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Bump version to 3.13.0.

Signed-off-by: Camila Ayres <hello@camilasan.com>

Always correctly set values for all fields in File Provider sharing UI

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Fetch macOS VFS package in autoupdater if the client is using the file provider module

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Prevent use of invalid characters for file provider domain names

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Only show successful debug archive creation dialog if it has indeed been successful

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Prevent crash on creating debug archive in non-writeable location

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Give debug archive save location dialog a default location

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Fix "false" error about bad applying of nodiscard to value type

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Re-run update sync paused state slot when folder list has changed

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Make sure to emit syncIsPausedChanged in syncIsPaused setter

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Make sure to emit relevant signals and set sync is paused to true if relevant

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Extract syncIsPaused initialisation into new updater slot

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

do not cause implicit hydration of virtual files during sync

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6675 from nextcloud/backport/6621/stable-3.13

[stable-3.13] if a virtual file change but nothing changed: set it as in sync

if a virtual file change but bothing changed: set it as in sync

some software (at least outlook native software) may fiddle with CfApi
placeholder metadta and set a .msg file to be out of sync state when
opening it

in that case, we will let the sync engine go over it and decide what to
do

it is then possible in that case that we would just put it back in "in
sync" state

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6626 from nextcloud/backport/6613/stable-3.13

[stable-3.13] Feature/office files lock newly created. Plus refactoring.

Detect office files for locking on new upload. Notify FolderWatcher.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Merge pull request #6674 from nextcloud/backport/6670/stable-3.13

[stable-3.13] Use davUser instead of direct credentials user in file provider

Use davUser instead of direct credentials user in file provider

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6667 from nextcloud/backport/6663/stable-3.13

[stable-3.13] create registry keys needed for edit locally in local machine category

create registry keys needed for edit locally in local machine category

creating the registry keys via installer for the current user is not
enough to have the feature enabled for all user accounts on a given
machine

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

Merge pull request #6664 from nextcloud/backport/6655/stable-3.13

[stable-3.13] Bugfix/folder conflict disappear

Fix tests failure. Refactoring.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Fix CI errors.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Unit tests for diverse conflicts in one folder.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Also support nested folder scenarios.

Signed-off-by: alex-z <blackslayer4@gmail.com>

 Bugfix. Folder invalid char conflict. Do not update parent folder record if it contains conflicted subfolders.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Bugfix. Folder case clash conflict. Do not update parent folder record if it contains conflicted subfolers. Also fix crash with local rename.

Signed-off-by: alex-z <blackslayer4@gmail.com>

Merge pull request #6649 from nextcloud/backport/6648/stable-3.13

[stable-3.13] Fix possible issues with item metadata acquisition required for macOS VFS file sharing

Wrap access of itemUrl in security scoping

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Explicitly set bundle name and identifiers from env vars in FileProviderUIExt

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Unify FileProviderUIExt entitlements

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Make sure network error is shown in UI instead of generic error

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Improve logging across file provider sharing

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Merge pull request #6645 from nextcloud/backport/6642/stable-3.13

[stable-3.13] Make use of NextcloudFileProviderKit in File Provider Module

Pin NextcloudFileProviderKit version to 0.9.0 (up to next major version)

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

Remove now-unneeded Realm dependency from FileProviderExt

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>