[RaphaelFakhri]

Related Issues

App: Refs #4451

Folders picked as the automatic uploads destination store their remote path with a trailing separator (for example, /Bbbbb/). AutomaticUploadsWorker then appends another separator before the file name, so every picture or video upload targets a path such as /Bbbbb//PXL_20240819_120404334.jpg. That doubled path is stored in the transfers table and sent in the PUT request, as seen in the logs in the issue.

This change composes the remote path through FolderBackUpConfiguration.getRemotePathForFile(), which trims a trailing separator from the upload path before appending the file name. Configurations saved before this change (with or without a trailing separator, and the root folder /) now produce a single separator.

• Add changelog files for the fixed issues in folder changelog/unreleased. More info here
• Add feature to Release Notes in ReleaseNotesViewModel.kt creating a new ReleaseNote() with String resources (if required): not required

---

QA

• New unit test FolderBackUpConfigurationTest covers upload paths without a trailing separator, with a trailing separator, and the root folder. The last two fail without the fix.
• ./gradlew detekt :owncloudDomain:testDebugUnitTest :owncloudApp:compileOriginalDebugKotlin passes.
• Manual check: pick a destination folder for picture uploads, take a picture, and confirm the upload in the uploads list and the server request use /<folder>/<file> with a single separator.

[CLAassistant]

All committers have signed the CLA.

[jesmrec]

@RaphaelFakhri thanks a lot for your contribution! we'll check it as soon as we have some time for it 😅 . Keep tuned, we could ask you for code review changes or fixes in the code! It's cool to check that the commits are verified and follow the conventional commits convention 💯

Just one consideration: you must sign the CLA, otherwise the PR will not move forward.