[android]: Update GoogleFusedLocationProvider to use the latest API

Разве для fused провайдера нужна эта проверка? Он сам должен внутри миксовать и всегда выдавать лучшие позиции (или худшие по одному параметру но лучшие по другому). Я бы убрал этот фильтр.

vng commented

2022-02-05 14:10:38 +00:00

(Migrated from github.com)

Я бы в этом PR не менял логику фильтрации которая была до этого. Лучше потом отдельны PR. Иначе когда посыпется, будет непонятно из-за чего ... Пусть mLocationFixChecker логика остается как есть тут.

rtsisyk commented

2022-02-06 07:08:44 +00:00

Author

Owner

LocationResult.getLastLocation() is declared as NonNull.

  @NonNull
  public Location getLastLocation() {
    int var1 = this.zzb.size();
    return var1 == 0 ? null : (Location)this.zzb.get(var1 + -1);
  }

For mFusedLocationClient.getLastLocation() this check exists:

    mFusedLocationClient.getLastLocation().addOnSuccessListener(location -> {
      if (location == null)
        return;

LocationResult.getLastLocation() is declared as NonNull. ``` @NonNull public Location getLastLocation() { int var1 = this.zzb.size(); return var1 == 0 ? null : (Location)this.zzb.get(var1 + -1); } ``` For `mFusedLocationClient.getLastLocation()` this check exists: ``` mFusedLocationClient.getLastLocation().addOnSuccessListener(location -> { if (location == null) return; ```

rtsisyk commented

2022-02-06 07:09:23 +00:00

Author

Owner

Это старый код перенесенный во время рефакторинга.

rtsisyk commented

2022-02-06 07:10:15 +00:00

Author

Owner

mLocationFixChecker перенесен из старого кода как есть без изменений. В целом данный код выглядит логичным и я не вижу смысла его трогать.

rtsisyk commented

2022-02-06 07:10:58 +00:00

Author

Owner

  @NonNull
  public static FusedLocationProviderClient getFusedLocationProviderClient(@NonNull Context var0) {

  @NonNull
  public static SettingsClient getSettingsClient(@NonNull Activity var0) {

``` @NonNull public static FusedLocationProviderClient getFusedLocationProviderClient(@NonNull Context var0) { ``` ``` @NonNull public static SettingsClient getSettingsClient(@NonNull Activity var0) { ```

rtsisyk commented

2022-02-06 07:13:13 +00:00

Author

Owner

  public void onLocationUpdated(@NonNull Location location)
  {
    mSavedLocation = location;
    mMyPosition = null;
    mSavedLocationTime = System.currentTimeMillis();
  }

  void notifyLocationUpdated()
  {
    if (mSavedLocation == null)
    {

Да, весь этот LocationHelper выглядит очень странно. Нагромождение лишних методов и классов. Но в этом патче я там вообще ничего не трогал и оставил старую логику. Любой рефакторинг здесь надо делать аккуратно чтобы не сломать что-либо.

``` public void onLocationUpdated(@NonNull Location location) { mSavedLocation = location; mMyPosition = null; mSavedLocationTime = System.currentTimeMillis(); } ``` ``` void notifyLocationUpdated() { if (mSavedLocation == null) { ``` Да, весь этот LocationHelper выглядит очень странно. Нагромождение лишних методов и классов. Но в этом патче я там вообще ничего не трогал и оставил старую логику. Любой рефакторинг здесь надо делать аккуратно чтобы не сломать что-либо.

rtsisyk commented

2022-02-06 07:14:19 +00:00

Author

Owner

See my answer above: #2023

See my answer above: https://git.omaps.dev/organicmaps/organicmaps/pulls/2023#discussion_r800137883

rtsisyk commented

2022-02-06 07:14:49 +00:00

Author

Owner

requested review from @vng

requested review from `@vng`

rtsisyk commented

2022-02-06 07:14:51 +00:00

Author

Owner

requested review from @biodranik

requested review from `@biodranik`

biodranik commented

2022-02-06 07:39:10 +00:00

(Migrated from github.com)

Documentation says, that it may return null. It should crash on the very first cold start.
In your first code sample it returns null. Where did you get it?
How is the second code sample relevant?

Please fix.

1. Documentation [says](https://developers.google.com/android/reference/com/google/android/gms/location/LocationResult?hl=en#getLastLocation()), that it may return null. It should crash on the very first cold start. 2. In your first code sample it returns null. Where did you get it? 3. How is the second code sample relevant? Please fix.

biodranik commented

2022-02-06 07:48:22 +00:00

(Migrated from github.com)

Вот и давай сделаем аккуратный рефакторинг. Надо решать проблему.

biodranik commented

2022-02-06 07:48:54 +00:00

(Migrated from github.com)

Уже очевидно, что старая логика не работает. Пользователи жалуются. Не вижу смысла оставлять старый код.

vng commented

2022-02-06 08:42:18 +00:00

(Migrated from github.com)

Я предлагаю закатить бету сначала с этими изменениями и собрать фидбек работает ли на том же Pixel 6.
Потом уже коцать рефакторинг фильтрации и прочее.

Я предлагаю закатить бету сначала с этими изменениями и собрать фидбек работает ли на том же Pixel 6. Потом уже коцать рефакторинг фильтрации и прочее.

biodranik commented

2022-02-06 08:45:10 +00:00

(Migrated from github.com)

Эти правки вообще не релевантны проблеме пикселя. Там у людей чистый оффлайн, без всяких плей сервисов. И у нас не работает. Начинает работать только после османда.

vng commented

2022-02-06 09:05:48 +00:00

(Migrated from github.com)

Ок, есть же issues которые релевантны или близки к этому фиксу? Проверить сначала там.

biodranik commented

2022-02-06 09:11:48 +00:00

(Migrated from github.com)

Ок, есть же issues которые релевантны или близки к этому фиксу? Проверить сначала там.

Нет. На google play services никто не жаловался, просто мы решили обновить старый код в рамках акции "починим локейшн везде раз и навсегда".

> Ок, есть же issues которые релевантны или близки к этому фиксу? Проверить сначала там. Нет. На google play services никто не жаловался, просто мы решили обновить старый код в рамках акции "починим локейшн везде раз и навсегда".

rtsisyk commented

2022-02-06 10:42:25 +00:00

Author

Owner

Данный делает ровно то, что написано в заголовке - переводит код на использование актуальной версии Google Location Services API вместо deprecated API. Логика по выбору наиболее точной локации в данном патче не меняется.

Данный делает ровно то, что написано в заголовке - переводит код на использование актуальной версии [Google Location Services API](https://developer.android.com/reference/android/location/package-summary.html) вместо deprecated API. Логика по выбору наиболее точной локации в данном патче не меняется.

rtsisyk commented

2022-02-06 10:47:08 +00:00

Author

Owner

It looks like a bug in their SDK and/or documentation. I added an extra check for null. Thanks.