Skip to content

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Feb 3, 2026

  • Tests written, or not not needed

Issue

App is crashing when user trying to open the app after deleting data.

How to reproduce?

  1. Open the app and log in.
  2. Send the app to the background.
  3. Go to the app’s settings.
  4. Begin the process to delete the app’s data and confirm.
  5. Attempt to reopen the app afterward.

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
if (result) {
finishAffinity()
finishAndRemoveTask()
exitProcess(0)
Copy link
Collaborator Author

@alperozturk96 alperozturk96 Feb 3, 2026

Choose a reason for hiding this comment

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

Ideally, FileDisplayActivity should not be launched after all accounts are removed on APP_RESUME. However, the app uses quite a bit of complex logic during startup.

Currently, FirstRunActivity, OnboardingServiceImpl, LauncherActivity, and AuthenticatorActivity all play a role in triggering the login flow and eventually FileDisplayActivity. To be honest, I’m not entirely sure why the app has four different launcher-related activities.

If you have any suggestions or ideas on a better approach to handle this, I’d really appreciate your feedback.

@ZetaTom @tobiasKaminsky

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Codacy

SpotBugs

CategoryBaseNew
Bad practice4343
Correctness7474
Dodgy code258258
Experimental11
Internationalization77
Malicious code vulnerability33
Multithreaded correctness3434
Performance4343
Security1818
Total481481

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

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

github-actions bot commented Feb 3, 2026

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