Skip to content

Fix crash to screen when a previously available provider has become unavailable#149

Open
steffenWi wants to merge 1 commit intobfabiszewski:masterfrom
steffenWi:FixCrashWithoutGradleStuff
Open

Fix crash to screen when a previously available provider has become unavailable#149
steffenWi wants to merge 1 commit intobfabiszewski:masterfrom
steffenWi:FixCrashWithoutGradleStuff

Conversation

@steffenWi
Copy link

The app currently crashes when a provider was selected that is no longer available and an attempt is made to start logging. In my case I had previously selected Network and GPS, but after a major system upgrade the "Network" provider had, unbeknownst to me, become unavailable. When I then tried to start logging the application just crashed.

  • to provoke this issue, make sure that a provider is unavailable, then in ProviderPreferenceDialogFragment.java change the body of public boolean isEnabled(int position) to return true;, start the application and select the provider that is unavailable, then try to switch on logging.

Note: Yes, this is an extremely rare issue, though if it occurs a user is left with a crashing app after a major system update, which might leave some to believe the app has become incompatible or that something went wrong during the update.

… when a provider was selected that is no longer available and an attempt is made to start logging. In my case I had previously selected Network and GPS, but after a major system upgrade the "Network" provider had, unbeknownst to me, become unavailable. When I then tried to start logging the application just crashed.

- to provoke this issue, make sure that a provider is unavailable, then in ProviderPreferenceDialogFragment.java change the body of public boolean isEnabled(int position) to "return true;", start the application and select the provider that is unavailable, then try to switch on logging.

Note: Yes, this is an extremely rare issue, though if it occurs a user is left with a crashing app after a major system update, which might leave some to believe the app has become incompatible or that something went wrong during the update.
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