Skip to content

fix(oc-file-list): local file indicator edge cases#16705

Merged
alperozturk96 merged 2 commits into
masterfrom
fix/empty-media-paths
Mar 18, 2026
Merged

fix(oc-file-list): local file indicator edge cases#16705
alperozturk96 merged 2 commits into
masterfrom
fix/empty-media-paths

Conversation

@alperozturk96

@alperozturk96 alperozturk96 commented Mar 18, 2026

Copy link
Copy Markdown
Collaborator

Demo

Around 4K record (500-1000 byte files) takes max 5 milliseconds to check. @tobiasKaminsky

Screen.Recording.2026-03-18.at.14.04.26.mp4

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 added this to the Nextcloud App 33.1.0 milestone Mar 18, 2026
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 merged commit 9f51730 into master Mar 18, 2026
15 of 19 checks passed
@alperozturk96 alperozturk96 deleted the fix/empty-media-paths branch March 18, 2026 13:14
@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/16705.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice4040
Correctness7272
Dodgy code237237
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3434
Performance4343
Security1818
Total454454

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant