Merge pull request #7360 from nextcloud/bugfix/mac-vfs-storage-use-elide

Make sure storage headers do not collide with each other in macOS VFS settings

Make sure storage headers do not collide with each other in macOS VFS settings

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

Merge pull request #7323 from nextcloud/bugfix/mac-vfs-ci-swift6

Fix Swift 6-related CI errors with macOS VFS module

Run post-auth state check with main actor task

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

Fix swift 6 concurrency error regarding use of authAttemptState

Cannot pass a local variable into a concurrently executing Task. It's
not a real issue as we are using a semaphore to wait before using this
variable for anything outside of the Task block, but still.

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

Merge pull request #7328 from nextcloud/bugfix/global-progress-vfs-bigsur

Check for availability of Big Sur (under 11.3) before using VFS progress tracking method

Wrap use of NSFileProviderManager globalProgress in availability check

This selector was only made available on macOS 11.3
Just to be safe we should check for the availability

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

Merge pull request #6123 from nextcloud/bugfix/docs-format

Fix issues on manpages

Incorporate doc fixes from pull request #6122

Fix issues on manpages

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7351 from nextcloud/bugfix/alwaysRecordMovedItemsAfterServerReplied

on receiving reply of a remote MOVE: always record it

on receiving reply of a remote MOVE: always record it

even if local item has disappeared, record the result of a remote MOVE
when receiving the reply

enable better state tracking when syncing rename or move operations

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

Merge pull request #7346 from nextcloud/Rello-patch-1

remove ownCloud wording in logging

remove ownCloud wording in logging

remove ownCloud wording in logging

Signed-off-by: Rello <Rello@users.noreply.github.com>

Merge pull request #3014 from nextcloud/foreground-background

Show/Hide Menubar and Dock Icon on macOS

Show/Hide Menubar and Dock Icon

Signed-off-by: Elsie Hupp <9206310+elsiehupp@users.noreply.github.com>

Merge pull request #4367 from nextcloud/finder-sidebar-icon-fix

Fix Finder sidebar icon to work as a "template" image

Fix Finder sidebar icon to work as a "template" image and change to SF Symbols lineweights

Signed-off-by: Elsie Hupp <github@elsiehupp.com>

Merge pull request #7321 from nextcloud/dependabot/github_actions/cpp-linter/cpp-linter-action-2.13.2

Bump cpp-linter/cpp-linter-action from 2.13.1 to 2.13.2

Bump cpp-linter/cpp-linter-action from 2.13.1 to 2.13.2

Bumps [cpp-linter/cpp-linter-action](https://github.com/cpp-linter/cpp-linter-action) from 2.13.1 to 2.13.2.
- [Release notes](https://github.com/cpp-linter/cpp-linter-action/releases)
- [Commits](https://github.com/cpp-linter/cpp-linter-action/compare/v2.13.1...v2.13.2)

---
updated-dependencies:
- dependency-name: cpp-linter/cpp-linter-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #7334 from nextcloud/bugfix/useFileIdToTrackRenamedItems

Bugfix/use file id to track renamed items

remove the enforcement of identical etag for a server side item MOVE

orignally added by https://github.com/owncloud/client/pull/6632

most probably a too strong assumption on the behavior of the Nextcloud
server

better check real item metadata like fileid, size or modification time

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

handle NEW/NEW conflicts by being real conflicts

NEW/NEW conflicts could sometime be ignored and replaced by update
metadata instructions

we stop doing this and handle them like any other conflicts

that would cause more download from the server

those conflicts would be solved automatically in case this is not a real
conflict but the client was missing the server reply with the updated
metadata

will enable more changes to improve MOVE detection from server side

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

in automated tests, update modtime when modifying the content of a file

why one would not update the modification time when modifying a remote
file during automated tests

there is no reason to not update the modification time as this is what
is done by the server

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

Merge pull request #7291 from nextcloud/bugfix/improveLocalDiscoveryLogs

better logs to understand when a folder is locally discovered

better logs tounderstand when a folder is local scanned

should hopefully allows to understand when local file system discovery
is done or omitted

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

Merge pull request #7285 from nextcloud/bugfix/catchFilesystemExceptions

test syncing of read-only .lnk windows shorcuts

test syncing of read-only .lnk windows shorcuts

related to https://github.com/nextcloud/desktop/pull/7282

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7333 from nextcloud/bugfix/signature

Bugfix/signature

Fail with error before calling folder metada setup when received empty signature.

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

Report client status error when signature is empty.

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

Add test testFolderMetadataWithEmptySignatureDecryptFails.

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

Check if signature is empty.

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

Merge pull request #7322 from nextcloud/bugfix/vfs-crash-socketcontroller

Fix crash caused due to null accountstate in FileProviderSocketController

Fix crash caused by null accountstate due to bad domain identifier parsed from FPExt

Turns out since we are forced to replace colons with hyphens we need to
replace them back to colons when we receive the domain identifier from
the file provider extension. If we do not then we get a null
accountState pointer in the socket controller and this causes a crash

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

Merge pull request #7319 from nextcloud/bugfix/betterMessageForFilesDeletionDetection

improve wordings of confirmation dialog when many files are deleted

improve wordings of confirmation dialog when many files are deleted

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7320 from nextcloud/bugfix/text

Make string more readable for translators.

Make string more readable for translators.

Fix #7309.

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

Merge pull request #7316 from nextcloud/bugfix/blacklist-string-tr

Fix poor translation strings for blacklisted error reason strings

Fix poor translation strings for blacklisted error reason strings

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7240 from nextcloud/bugfix/mac-vfs-login-loop

Stop fetching user profile details now that is not necessary in file provider extension

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

Handle userId correctly in File Provider Extension client interface

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

Properly provide both user-provided username AND backing user ID to File Provider Extension

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

Fetch user profile (and user ID) when setting up domain account

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

Wait for task to complete synchronously after setting up domain account rather than wrapping everything in task

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

Retry authentication if connection timed out

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

Try to authenticate new account details before setting them

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

Merge pull request #7286 from nextcloud/bugfix/cmd-hosturl-parse

Properly check for webdav or dav path in provided host url for cmd.cpp

Properly check for webdav or dav path in provided host url for cmd.cpp

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

Encourage use of https in error message for cmd.cpp

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

Remove unused <random> import in cmd.cpp

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7066 from nextcloud/ci/addGithubActionMacOsTimeout

try to increase timeout of github actions for macOS

CI fix warning: inkscape was renamed to homebrew/cask/inkscape

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

no longer use deprecated Craft command line option

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

try to increase timeout of github actions for macOS

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

Merge pull request #7290 from nextcloud/bugfix/improveLogsAgainFromUsingThem

Cleaning logs

improve log when too many deleted files are detected

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

remove info level output from activity list model: not so usefull

they just are printed when everything is working fine

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

remove not so usefull log output

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

Merge pull request #7289 from nextcloud/bugfix/removeQStringArgWarning

remove extra argument from QString::arg: string has only 1 placeholder

remove extra argument from QString::arg: string has only 1 placeholder

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7284 from nextcloud/update/readmes

Small READMEs fix/updates

Fix permanent link to mac-crafter README.

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

Fix the list of steps order.

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

Merge pull request #7252 from nextcloud/add/readmes

Add README about mac-crafter.

Match the universal build instructions with what we actually do with mac-crafter and make_universal

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

Clarify architecture setting has automatic default in mac-crafter readme

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

Simplify build instructions in mac-crafter README

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

Add note about craft-master in the main README.

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

Add README about mac-crafter.

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

Merge pull request #7282 from nextcloud/bugfix/catchFilesystemExceptions

properly catch std::filesystem exceptions

properly catch std::filesystem exceptions

will prevent std::terminate from being called with uncatched exceptions

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7271 from nextcloud/bugfix/fixPermanentDeleteLogCrash

ensure log context category is valid before reading it

ensure log context category is valid before reading it

category of logs accessed via log context can be null
in such a case, strcmp will crash

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

Merge pull request #7270 from nextcloud/bugfix/useQmlFusionOnWindows

Bugfix/use qml fusion on windows

remove hard coded white color in elements of folder wizard

fix broken white backgrounds in folder wizard when using Windows dark
mode

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

switch folder wizar to modern style to avoid borken windows dark mode

the QWizard AeroStyle style is broken on windows dark mode, switch to
ModernStyle that is working fine

see upstream Qt bug: https://bugreports.qt.io/browse/QTBUG-123853

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

qml Windows native style is broken on dark mode: switch to Fusion

we will temporarily be using Fusion generic desktop style

Close https://github.com/nextcloud/desktop/issues/7215

see upstream Qt bug
https://bugreports.qt.io/browse/QTBUG-113469?focusedId=724929&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-724929

they apparently do not plan to fix Windows qml style to support dark
mode on Windows 11

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

Merge pull request #7263 from nextcloud/bugfix/cleanUpLogs

remove too verbose logs during discovery

downgrade severity level of logs during propagation

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

remove internal logs in propagation scheduler: not usefull

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

remove too verbose not so usefull logs

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

remove useless internal log about remote permissions management

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

remove too verbose logs during discovery

will make the logs less verbose and improve performance

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #7259 from nextcloud/bugfix/fixLogIssues

Bugfix/fix log issues

improve debugger logs on Windows to have same content than normal log

makes debugger logs more usefull

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