gpesquero/organicmaps
1
Fork 0
forked from organicmaps/organicmaps
Code Pull requests Activity

[android] Fix a crash on rotation when location dialog is displayed

Browse source 
Closes #4570

Signed-off-by: Roman Tsisyk <roman@tsisyk.com>
This commit is contained in:
Roman Tsisyk 2023-03-01 11:26:44 +02:00
parent e49708808e
commit d4bf7a73da
2 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
android/src/app/organicmaps/MwmActivity.java
View file

@ -1028,6 +1028,7 @@ public class MwmActivity extends BaseMwmFragmentActivity
if (mOnmapDownloader != null)
mOnmapDownloader.onPause();
mNavigationController.onActivityPaused(this);
LocationHelper.INSTANCE.closeLocationDialog();
super.onPause();
}

13
android/src/app/organicmaps/location/LocationHelper.java
View file

@ -161,6 +161,13 @@ public enum LocationHelper implements Initializable<Context>, AppBackgroundTrack
}
}
public void closeLocationDialog()
{
if (mErrorDialog != null && mErrorDialog.isShowing())
mErrorDialog.dismiss();
mErrorDialog = null;
}
void notifyCompassUpdated(double north)
{
mSavedNorth = north;
@ -179,8 +186,7 @@ public enum LocationHelper implements Initializable<Context>, AppBackgroundTrack
if (mSavedLocation == null)
throw new IllegalStateException("No saved location");
if (mErrorDialog != null && mErrorDialog.isShowing())
mErrorDialog.dismiss();
closeLocationDialog();
for (LocationListener listener : mListeners)
listener.onLocationUpdated(mSavedLocation);
@ -250,8 +256,7 @@ public enum LocationHelper implements Initializable<Context>, AppBackgroundTrack
}
// Cancel our dialog in favor of system dialog.
if (mErrorDialog != null && mErrorDialog.isShowing())
mErrorDialog.dismiss();
closeLocationDialog();
// Launch system permission resolution dialog.
IntentSenderRequest intentSenderRequest = new IntentSenderRequest.Builder(pendingIntent.getIntentSender())