fix: sync engine readonly folder file deletion

[guruz]

Resolves

This is for #7797 and #10099

It would probably be good to backport this (if it's the correct fix and is merged...)

Summary

Show stray file in UI instead of agressively deleting it (= potential data loss)

Checklist

• Sign-off message is added to all commits.
• The commit history is clean with no merge commits.
  • How to rebase your commits
  • How to squash commits with rebase
• Uploaded screenshots from before and after for UI changes.
• Documentation has been updated or is not required.
• Backports requested where applicable (critical bugfixes).
• Labels added where applicable (bug/enhancement).
• Milestone added for target version.

AI (if applicable)

• The content of this PR was partly or fully generated using AI
  • I have read the guidelines for AI-assisted contributions.
  • I have read Nextcloud's AI Contribution Policy.

[guruz]

I'll look at the test failure..

[guruz]

I've made the code distinct between totally new file and move of existing file from same sync folder to the readonly dir.
In the latter case, the file will be deleted and restored at original location.
(EDIT: Also added explicit test for this restore case into PermissionsTest)

I've also enabled the PermissionsTest on MacOS since they now all pass on my machine. Let's see if the CI sees it differently?

(I also rebased on master)

[github-actions]

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

Digest: sha256:d09f4f55cf3c4bf7aade303b1b0cc26ccd4b5099f2e1e011ce8ace6796b0e4e5

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.

[mid1221213]

Tested OK on my setup (binary built on commit a910031 on a Debian unstable and using a kinda "custom" NC server): the Desktop Client patched with the content of this MR behaves exactly as told.

[guruz]

CI failure seems unrelated:
15:22:48.566 ERROR Not authorized or project not found. Please check the 'SONAR_TOKEN' environment variable, the 'sonar.projectKey' and 'sonar.organization' properties, or contact the project administrator to verify the token's permissions.

[github-actions]

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)