Skip to content

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Oct 17, 2025

  • Tests written, or not not needed

Changes

Uses DAO functions instead of ContentResolvers

Fixes UI state handling in the uploads list: since UploadGroupLoadRunnable performs asynchronous operations, we now wait for all upload groups to complete their refresh before updating the UI. So that list reflects the latest data and avoids inconsistent results.

@alperozturk96 alperozturk96 force-pushed the use-dao-in-upload-list-adapter-for-consistency branch from 707f3e6 to 9d41097 Compare October 21, 2025 09:25
@github-actions
Copy link

Codacy

Lint

TypemasterPR
Warnings4542
Errors1111

SpotBugs

CategoryBaseNew
Bad practice4444
Correctness7373
Dodgy code266267
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3535
Performance4848
Security1818
Total494495

SpotBugs increased!

@mhzawadi

This comment was marked as off-topic.

@jkufner
Copy link

jkufner commented Oct 21, 2025

See #15812 (comment)

@alperozturk96 alperozturk96 force-pushed the use-dao-in-upload-list-adapter-for-consistency branch from 9d41097 to 2f58e63 Compare October 27, 2025 07:53
@alperozturk96 alperozturk96 merged commit 3453a7a into master Oct 27, 2025
12 of 19 checks passed
@alperozturk96 alperozturk96 deleted the use-dao-in-upload-list-adapter-for-consistency branch October 27, 2025 07:57
@alperozturk96 alperozturk96 linked an issue Oct 27, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deep dive Auto Upload

4 participants