fix: auto upload collect changed files

[alperozturk96]

• Tests written, or not not needed

Issue

The application collects modified files from ContentObserverWork as an array of strings, for example: content://media/external/images/media/2281. This array is then optionally passed to the AutoUploadWorker. The goal is to avoid scanning the entire local folder and instead only process the files that have changed.

However, errors can occur during this operation—for instance, the file path could be null, or the file may not exist. Such cases can result in upload information being lost, which may cause the auto-upload to fail.

Example crash log:

2025-10-09 15:49:24.520 17281-17324 FileSyncHelper          com.nextcloud.client                 D  getFileFromURI, URI: content://media/external/images/media/2275
2025-10-09 15:49:24.523 17281-17281 WM-SystemFgDispatcher   com.nextcloud.client                 D  Notifying with (id:266, workSpecId: 11ef7815-eaeb-4c0d-ad0e-363921909fdc, notificationType :1)
2025-10-09 15:49:24.527 17281-17324 FileSyncHelper          com.nextcloud.client                 E  cant get file from URI
2025-10-09 15:49:24.530 17281-17322 🔄📤AutoUpload          com.nextcloud.client                 D  Exception collectFileChangesFromContentObserverWork: java.lang.NullPointerException

Changes

Instead of failing the entire worker when encountering such issues, we should handle all potential edge cases. As a fallback, the worker should scan the entire local folder to ensure no files are missed for upload.

Documentation and links have been added to the functions, and function and variable names have been changed to prevent further confusions.

How to test?

1. Set up a folder for auto-upload, e.g., the Pictures directory.
2. Capture a photo using the Camera app that saves it to the configured folder.
3. Verify that the new or changed file appears in the auto-upload folder and is processed correctly.

[github-actions]

test-Unit test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/15764-Unit-test-10-49

[github-actions]

Codacy

Lint

Type	master	PR
Warnings	45	45
Errors	11	11

SpotBugs

Category	Base	New
Bad practice	46	46
Correctness	74	74
Dodgy code	267	267
Experimental	1	1
Internationalization	7	7
Malicious code vulnerability	2	2
Multithreaded correctness	35	35
Performance	49	49
Security	18	18
Total	499	499