Skip to content

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Dec 18, 2025

  • Tests written, or not not needed

Issue

When the user navigates to a subdirectory from the Favorites or Shared tab, the receivers are unregistered, which causes the folder to not refresh correctly.

Changes

Receivers needs to be registered and unregistered on onStart/onStop.

Documentation:

Screenshot 2025-12-18 at 13 51 01

Result

Screenshot 2025-12-18 at 13 47 37

@alperozturk96
Copy link
Collaborator Author

/backport to stable-3.35

val fileDisplayActivity = activity as FileDisplayActivity
fileDisplayActivity.updateActionBarTitleAndHomeButtonByString(getString(R.string.drawer_item_shared))
fileDisplayActivity.setMainFabVisible(false)
fileDisplayActivity.initSyncBroadcastReceiver()
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to register from here again since SharedListFragment child of OCFileListFragment and OCFileListFragment used in FileDisplayActivity.

Co-authored-by: Tom <[email protected]>
Signed-off-by: Alper Öztürk <[email protected]>
@alperozturk96 alperozturk96 merged commit 2af5184 into master Dec 18, 2025
13 of 20 checks passed
@alperozturk96 alperozturk96 deleted the fix/listen-broadcast-manager branch December 18, 2025 14:54
@github-actions
Copy link

Codacy

SpotBugs

CategoryBaseNew
Bad practice4343
Correctness7373
Dodgy code257257
Experimental11
Internationalization77
Malicious code vulnerability33
Multithreaded correctness3434
Performance4343
Security1818
Total479479

@github-actions
Copy link

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

@github-actions
Copy link

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

@alperozturk96 alperozturk96 added this to the Nextcloud App 3.35.0 milestone Dec 19, 2025
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.

3 participants