organicmaps/organicmaps-tmp

Archived

Fork 1

[android] Refactor permission handling #2089

Merged

root merged 1 commit from rt-android-permission-refactoring into master

2022-02-12 13:44:30 +00:00

rtsisyk commented

2022-02-12 06:02:09 +00:00

Owner

Signed-off-by: Roman Tsisyk roman@tsisyk.com

Signed-off-by: Roman Tsisyk <roman@tsisyk.com>

pastk reviewed 2022-02-12 06:02:09 +00:00

biodranik (Migrated from github.com) approved these changes 2022-02-12 08:20:23 +00:00

android/src/com/mapswithme/util/PermissionsUtils.java

biodranik (Migrated from github.com) commented

2022-02-12 08:20:04 +00:00

Если пользователь (случайно) выбрал только грубую позицию, как мы ему потом ненавязчиво напомним, чтобы он выбрал точную для нормальной работы с программой?

rtsisyk reviewed 2022-02-12 08:34:57 +00:00

android/src/com/mapswithme/util/PermissionsUtils.java

rtsisyk commented

2022-02-12 08:34:57 +00:00

Author

Owner

Экспериментально проверил, что можно из COARSE в FINE повышать точно через специальной системный диалог:

Ща допилю тогда.

Экспериментально проверил, что можно из COARSE в FINE повышать точно через специальной системный диалог: ![2022-02-12 11 34 14](https://user-images.githubusercontent.com/1799054/153703970-86a23da8-4625-492a-ac08-c7775d05fb54.jpg) Ща допилю тогда.

rtsisyk reviewed 2022-02-12 10:04:14 +00:00

android/src/com/mapswithme/util/PermissionsUtils.java

rtsisyk commented

2022-02-12 10:04:14 +00:00

Author

Owner

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

vng (Migrated from github.com) reviewed 2022-02-12 10:36:00 +00:00

android/src/com/mapswithme/maps/intent/Factory.java

vng (Migrated from github.com) commented

2022-02-12 10:33:00 +00:00

Here and below, are we sure that LocationHelper.INSTANCE.getMyPosition() always returns not null?
Or it doesn't matter and RoutingController.prepare does it good?

Here and below, are we sure that LocationHelper.INSTANCE.getMyPosition() always returns not null? Or it doesn't matter and RoutingController.prepare does it good?

rtsisyk reviewed 2022-02-12 11:33:58 +00:00

android/src/com/mapswithme/maps/intent/Factory.java

rtsisyk commented

2022-02-12 11:33:58 +00:00

Author

Owner

Both arguments are nullable:

  public void prepare(@Nullable MapObject startPoint, @Nullable MapObject endPoint)
  {
    prepare(startPoint, endPoint, false);
  }

Both arguments are nullable: ``` public void prepare(@Nullable MapObject startPoint, @Nullable MapObject endPoint) { prepare(startPoint, endPoint, false); } ```

biodranik (Migrated from github.com) reviewed 2022-02-12 11:57:58 +00:00

android/src/com/mapswithme/maps/MwmActivity.java

biodranik (Migrated from github.com) commented

2022-02-12 11:57:58 +00:00

Это может привести к странным последствиям при некоторых нестандартных действиях пользователя. Предлагаю сделать по-другому: если есть грубая позиция, всегда строить от неё (без позиции вообще по идее мы в этот код и не попадём, верно?). При этом параллельно всегда показывать диалог "включите точную позицию для навигации", пока пользователь её таки не разрешит. Если пользователь сам себе крокодил, и откажется, тогда у него навигатор будет показывать маршрут, но не будет двигаться дальше. Если разрешит точную позицию, то она через какое-то время прилетит и обновит текущий маршрут в навигаторе, и дальше всё будет работать как положено.

vng (Migrated from github.com) approved these changes 2022-02-12 12:29:45 +00:00

rtsisyk reviewed 2022-02-12 13:32:34 +00:00

android/src/com/mapswithme/maps/MwmActivity.java

rtsisyk commented

2022-02-12 13:32:34 +00:00

Author

Owner

Ок. Я уберу эту оптимизацию с сохранением локации. Будет запускаться построение маршрута и тут же параллельно вылетать диалог с настройками. В целом это одноразовый кейс, нет смысла под него так костылить.

rtsisyk reviewed 2022-02-12 13:44:15 +00:00

android/src/com/mapswithme/maps/MwmActivity.java

rtsisyk commented

2022-02-12 13:44:15 +00:00

Author

Owner

Убрал костыль. UX хуже не стал ввиду того, что первом построении роутинга всё равно показывается сначала disclaimer диалог, поверх которого вылетает диалог с локацией. Выглядит вполне ок.

This repo is archived. You cannot comment on pull requests.

No reviewers

pastk

biodranik

vng