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

[android] Fix IllegalStateException in AndroidNativeProvider

Browse source 
Closes #3588

Signed-off-by: Roman Tsisyk <roman@tsisyk.com>
This commit is contained in:
Roman Tsisyk 2022-10-12 10:14:18 +03:00
parent 16adc0b47f
commit 61412a7b30
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
android/src/com/mapswithme/maps/location/LocationHelper.java
View file

@ -2,6 +2,7 @@ package com.mapswithme.maps.location;
import static android.Manifest.permission.ACCESS_COARSE_LOCATION;
import static android.Manifest.permission.ACCESS_FINE_LOCATION;
import static com.mapswithme.util.concurrency.UiThread.runLater;
import android.app.Activity;
import android.app.Dialog;
@ -247,16 +248,19 @@ public enum LocationHelper implements Initializable<Context>, AppBackgroundTrack
mUiCallback.onLocationUpdated(mSavedLocation);
// TODO: consider to create callback mechanism to transfer 'ROUTE_IS_FINISHED' event from
// the core to the platform code (https://jira.mail.ru/browse/MAPSME-3675),
// the core to the platform code (https://github.com/organicmaps/organicmaps/issues/3589),
// because calling the native method 'nativeIsRouteFinished'
// too often can result in poor UI performance.
if (RoutingController.get().isNavigating() && Framework.nativeIsRouteFinished())
{
Logger.d(TAG, "End point is reached");
restart();
if (mUiCallback != null)
mUiCallback.onRoutingFinish();
RoutingController.get().cancel();
// Break onLocationChanged() -> start() -> onLocationChanged() -> start() loop.
// https://github.com/organicmaps/organicmaps/issues/3588
runLater(this::restart);
}
}