Skip to content

refactor(upload-list-adapter): performance optimizations#16761

Open
alperozturk96 wants to merge 11 commits intomasterfrom
upload-list-adapter-kt
Open

refactor(upload-list-adapter): performance optimizations#16761
alperozturk96 wants to merge 11 commits intomasterfrom
upload-list-adapter-kt

Conversation

@alperozturk96
Copy link
Copy Markdown
Collaborator

Changes

  • Converts UploadListAdapter.java to Kotlin
  • Uses separated files for actions, data classes
  • Splits UI logic into functions and regions
  • Replaces Comparator with extensions
  • Simplifes logics
  • Increase performance

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 added this to the Nextcloud App 33.1.0 milestone Mar 27, 2026
@alperozturk96 alperozturk96 added technical debt performance 🚀 Performance improvement opportunities (non-crash related) labels Mar 27, 2026
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions
Copy link
Copy Markdown

APK file: https://github.com/nextcloud/android/actions/runs/23654154564/artifacts/6147162264
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@github-actions
Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice3939
Correctness7070
Dodgy code235232
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3434
Performance4242
Security1616
Total446443

@github-actions
Copy link
Copy Markdown

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review performance 🚀 Performance improvement opportunities (non-crash related) technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants