Skip to content

fix: stop running gallery background job when user leave#16133

Merged
alperozturk96 merged 2 commits into
masterfrom
fix/cancel-gallery-thumbnail-generation-when-leave-media-tab
Dec 17, 2025
Merged

fix: stop running gallery background job when user leave#16133
alperozturk96 merged 2 commits into
masterfrom
fix/cancel-gallery-thumbnail-generation-when-leave-media-tab

Conversation

@alperozturk96

@alperozturk96 alperozturk96 commented Dec 15, 2025

Copy link
Copy Markdown
Collaborator
  • Tests written, or not not needed

Issue

When user opens the media tab, gallery background job starts but when user opens another screen background jobs still runs.

@alperozturk96

Copy link
Copy Markdown
Collaborator Author

/backport to stable-3.35

@alperozturk96 alperozturk96 force-pushed the fix/cancel-gallery-thumbnail-generation-when-leave-media-tab branch from 3f8c1fa to 6ce47fa Compare December 15, 2025 08:26
@github-actions

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice4343
Correctness7474
Dodgy code260260
Experimental11
Internationalization77
Malicious code vulnerability33
Multithreaded correctness3434
Performance4848
Security1818
Total488488

Signed-off-by: alperozturk <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 force-pushed the fix/cancel-gallery-thumbnail-generation-when-leave-media-tab branch from 6ce47fa to 1ba8b80 Compare December 17, 2025 10:24
Signed-off-by: alperozturk <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 merged commit ec57fec into master Dec 17, 2025
6 of 9 checks passed
@alperozturk96 alperozturk96 deleted the fix/cancel-gallery-thumbnail-generation-when-leave-media-tab branch December 17, 2025 10:30
@github-actions

Copy link
Copy Markdown

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/16133.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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants