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

[android] Added unconditional refresing of place page view, when map object is selected, i.e. for the same object refresh will be updated as well

Browse source
This commit is contained in:
Александр Зацепин 2019-03-07 20:59:45 +03:00 • committed by Aleksandr Zatsepin
parent ddd7951e88
commit be2cccc50e
2 changed files with 2 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
android/src/com/mapswithme/maps/widget/placepage/BottomSheetPlacePageController.java
View file

@ -269,17 +269,8 @@ public class BottomSheetPlacePageController implements PlacePageController, Loca
mPlacePage.setMapObject(object, (policy, isSameObject) -> {
@AnchorBottomSheetBehavior.State
int state = mPlacePageBehavior.getState();
// The method openFor could be invoked many times, e.g. when we leave the map and come back
// on it. So, we should do nothing if the map object is not changed or place page is already
// opened, but we have to refresh place page views, since the map object data could be changed
// while we were on the another screen (e.g. editor, ugc, settings, etc.). This behavior is
// produced by the framework 'UpdatePlacePageInfoForCurrentSelection' method that in turn calls
// method 'ActivateMapSelection'.
if (isSameObject && !isHiddenState(state))
{
mPlacePage.refreshViews(policy);
return;
}
mPlacePage.resetScroll();
mPlacePage.resetWebView();

3
android/src/com/mapswithme/maps/widget/placepage/PlacePageView.java
View file

@ -1008,9 +1008,10 @@ public class PlacePageView extends NestedScrollView
if (MapObject.same(mMapObject, mapObject))
{
mMapObject = mapObject;
NetworkPolicy policy = NetworkPolicy.newInstance(NetworkPolicy.getCurrentNetworkUsageStatus());
refreshViews(policy);
if (listener != null)
{
NetworkPolicy policy = NetworkPolicy.newInstance(NetworkPolicy.getCurrentNetworkUsageStatus());
listener.onSetMapObjectComplete(policy, true);
}
return;