[alperozturk96]

Changes

• Uses DAO functions
• The provider query no longer appends AND null, so deleting a folder removes its children at every depth.
• deleteAllFiles() deletes only by file_owner. The provider's folder delete without an id now respects where.
• adds tests

How to test?

• Have folders in root and delete a folder web and refresh from client, it should refresh correctly
• Have a nested folder hierarchy and delete folder all child files must be removed.

check via app inspection

[github-actions]

📱 QA build

Download	app-qa-debug.apk
QR code	Open the QR code for this download
Commit	e5d38cf
Version	17941
Available until	7 days after this build

The QA build installs alongside a released Nextcloud app, so you can keep
using your existing install while testing.

Downloading the file requires a GitHub account, so open this link on the
device you want to test on, or transfer the APK to it.

[codecov]

Codecov Report

❌ Patch coverage is 97.07113% with 7 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...ils/extensions/FileDataStorageManagerExtensions.kt	90.90%	1 Missing and 3 partials ⚠️
...wncloud/android/providers/FileContentProvider.java	91.30%	0 Missing and 2 partials ⚠️
...com/nextcloud/utils/extensions/OCFileExtensions.kt	98.92%	0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[github-actions]

SpotBugs

Category	Base	New
Bad practice	24	24
Correctness	45	45
Dodgy code	145	145
Experimental	1	1
Malicious code vulnerability	1	1
Multithreaded correctness	13	13
Performance	28	28
Security	15	15
Total	272	272