TILES-3943 Add route case #556

Kuznecoff · 2021-12-09T08:13:40Z

Кнопка "Открыть в 2ГИС" - стала динамической если есть один маркер и он сечас виден на карте она меняеться на "Доехать с 2ГИС" и открывает 2гис онлайн для построения маршрута к точке.
Для локального тестирования нужно создать файл config.local.json
{
"baseUrl": ""
}

Тесты руками:

Карта без маркеров, на русском, на экране территория россии или любая другая кроме эмиратов!:
✓ есть кнопка Открыть в 2гис
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ru
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ru открываеться с актуальным положением и маштабом
Карта без маркеров, на русском, на экране территория эмиратов!:
✓ есть кнопка Открыть в 2гис
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ae
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ae открываеться с актуальным положением и маштабом
Карта с маркером, на русском, на экране территория россии или любая другая кроме эмиратов!:
✓ есть кнопка Доехать с 2гис
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ru в режиме построить маршрут до точки, точка совпадает с маркером на карте
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ru в режиме построить маршрут до точки, точка совпадает с маркером на карте
Карта с маркером, на русском, на экране территория эмиратов!:
✓ есть кнопка Доехать с 2гис
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ae в режиме построить маршрут до точки, точка совпадает с маркером на карте
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ae в режиме построить маршрут до точки, точка совпадает с маркером на карте
Карта без маркеров, на арабском, на экране территория россии или любая другая кроме эмиратов!:
✓ есть кнопка Open in 2gis
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ru
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ru открываеться с актуальным положением и маштабом
Карта без маркеров, на арабском, на экране территория эмиратов!:
✓ есть кнопка Open in 2gis
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ae
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ae открываеться с актуальным положением и маштабом
Карта с маркером, на арабском, на экране территория россии или любая другая кроме эмиратов!:
✓ есть кнопка Route in 2gis
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ru в режиме построить маршрут до точки, точка совпадает с маркером на карте
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ru в режиме построить маршрут до точки, точка совпадает с маркером на карте
Карта с маркером, на арабском, на экране территория эмиратов!:
✓ есть кнопка Route in 2gis
✓ клик по конопке открывает с текущим положением карты и маштабом 2gis.ae в режиме построить маршрут до точки, точка совпадает с маркером на карте
✓ клик по кнопке после изменение маштаба и перемещение карты - карта на 2gis.ae в режиме построить маршрут до точки, точка совпадает с маркером на карте
Смена языка карты когда нет слоя с маркером:
✓ map.setLang('en') - нет кнопки вместо нее логотип
✓ map.setLang('ar') - кнопка Open in 2gis
✓ map.setLang('ru') - кнопка Открыть в 2гис
✓ map.setLang('xx') - неизвестный язык карта по дефолту на русском - кнопка Открыть в 2гис
Смена языка карты когда есть слой с маркером:
✓ map.setLang('en') - нет кнопки вместо нее логотип
✓ map.setLang('ar') - кнопка Route in 2gis
✓ map.setLang('ru') - кнопка Доехать с 2гис
✓ map.setLang('xx') - неизвестный язык карта по дефолту на русском - кнопка Доехать с 2гис
На карте несколько маркеров:
markerTest = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
✓ Добавили единственный маркер на карту - кнопка Открыть в 2гис изменилась на Доехать с 2гис

markerTest.remove()
✓ Удалили единственный маркер на карте - кнопка Доехать с 2гис изменилась на Открыть в 2гис
```
markerTest = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest2 = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
```
✓ Добавили несколько маркеров на карту - на карте кнопка Открыть в 2гис
```
markerTest = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest2 = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest3 = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest2.remove();markerTest3.remove();
```
✓ Удалили несколько маркеров на карте пока не остался один маркер - на карте кнопка Доехать с 2гис
```
markerTest = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest2 = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest3 = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest2.remove();markerTest3.remove();markerTest.remove();
```
✓ Удалили все маркеры - на карте кнопка Открыть в 2гис
```
markerTest = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest2 = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest3 = DG.marker([54.980156831455, 82.897440725094]).addTo(map)
markerTest2.remove();
```
✓ Удалили маркер но на карте еще больше одного маркера - на карте кнопка Открыть в 2гис
Нажали на кнопку Доехать с 2гис на зуме меньше 12
✓ карта откроется на 2gis в режиме построить маршрут до точки проезд на машине
Нажали на кнопку Доехать с 2гис на зуме больше или равно 12
✓ карта откроется на 2gis в режиме построить маршрут до точки проезд на автобусе
Карта инициализированна с опцией logotype:true:
✓ нет кнопки вместо нее логотип
✓ смена языка - нет кнопки вместо нее логотип
✓ добавить маркер - нет кнопки вместо нее логотип
✓ удалить маркер - нет кнопки вместо нее логотип
✓ двигать карту - нет кнопки вместо нее логотип
Карта открыта в режиме мобильного телефона:
✓ кнопка Доехать с 2гис правильно показанна в альбомной и книжной ориентации
✓ кнопка Открыть в 2гис правильно показанна в альбомной и книжной ориентации

Trufi · 2021-12-10T05:02:49Z

Добавь описание и как посмотреть. Я открыл нашу стандартную демку и у меня ничего не нарисовалось, хотя там как раз 1 маркер.

src/DGAttribution/src/DGAttribution.js

artem-malko · 2021-12-10T10:09:08Z

config.main.json

-
-    "openLink":  "http://2gis.{domain}/{projectCode}/center/{center}/zoom/{zoom}/openLink=true",
+    "ppnotLink2gis": "{gislink}/center/{center}/zoom/{zoom}/routeTab/rsType/{rsType}/to/{point}/openLink=true",
+    "openLink2gis":  "{gislink}/center/{center}/zoom/{zoom}/openLink=true",


Не совсем понял, какая ссылка в итоге получится?

Просто в онайлне координаты и зум в query-части. Слагов center и zoom нет

тот же вопрос про ссылку выше

Примеры:

http://2gis.ru/center/82.89689093828201,54.97947773210846/zoom/18/routeTab/rsType/bus/to/82.897440725094,54.980156831455/openLink=true
http://2gis.ru/center/82.89838761091234,54.97849876432055/zoom/18/openLink=true
Так работает зум и центр - и темплейт скопировал из темплейтов ниже и чуть допилил

"ppnotLink": "http://2gis.{domain}/{projectCode}/center/{center}/zoom/{zoom}/routeTab/rsType/{rsType}/to/{point}╎{name}",

Давай все же сделаем норм сслыки? Сейчас просто у нас где-то глубоко есть редиректор с таких урлов. Но, сколько он еще проживет — пес его знает.

https://2gis.ru/novosibirsk?m=82.898388%2C54.978499%2F18

https://2gis.ru/novosibirsk/directions/tab/bus/points/%7C82.897441%2C54.980157?m=82.896891%2C54.979478%2F18

ок мои линки поправил, там есть еще два старых линка - ссыкотно их править вдруг кто завязался на их структуру.

itanka9 · 2021-12-10T10:17:57Z

src/DGAttribution/src/DGAttribution.js

+
+        layerremove: function (e) {
+            if (e.layer instanceof DG.Marker) {
+                this._markers = [];


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

переделал упростил до одного маркера. удаление переделал, но под старый JS тут еще все на var тащить filter не стал

itanka9 · 2021-12-10T10:21:30Z

src/DGAttribution/src/DGAttribution.js

+        layerremove: function (e) {
+            if (e.layer instanceof DG.Marker) {
+                this._markers = [];
+                this._map.eachLayer(function (layer) {


И вот этот код зачем? Тут же, по-моему вовсе нет переменной markerLayers, куда пушатся маркеры.

оу тут ошибка! this._markers должно было быть. Спасибо

itanka9 · 2021-12-10T10:23:02Z

config.main.json

@@ -7,8 +7,8 @@

    "trafficLayerMinZoom": 10,
    "trafficLayerUpdateInterval": 300000,
-
-    "openLink":  "http://2gis.{domain}/{projectCode}/center/{center}/zoom/{zoom}/openLink=true",
+    "ppnotLink2gis": "{gislink}/center/{center}/zoom/{zoom}/routeTab/rsType/{rsType}/to/{point}/openLink=true",


А договорились давать ссылку именно на маршрут общественного транспорта? Тут просто используется аббревиатура ppnot.

там выбор на зуме до 12 будет для машины выше 12 для автобусов

Лучше назвать тогда этот параметр ppLink2gis, а то ppnotLink2gis введет в заблуждение что это только про маршрут на ОТ.

переименовал и линк переделал на современный как Артем писал

itanka9 · 2021-12-10T10:24:14Z

src/DGAttribution/src/DGAttribution.js

        }
+        var isNotTranslate = btn.label == 'open_on' || btn.label == 'route_on';


Уточни, плиз, что делает эта переменная?

проверка есть перевод для этой кнопки для текущего язка или нету, потом проверяем если нету то кнопку не показываем вообще, например для чехии и украины нету кнопки открыть.
Было решение в ТЗ показывать только для России и для Эмиратов , для Эмиратов на английском кнопку написать.

Назови тогда хоть buttonNotTranslated и эту логику комментом опиши, а то непонятно.

назвал комент дописал

itanka9 · 2021-12-10T10:46:23Z

config.main.json

@@ -7,8 +7,8 @@

    "trafficLayerMinZoom": 10,
    "trafficLayerUpdateInterval": 300000,
-
-    "openLink":  "http://2gis.{domain}/{projectCode}/center/{center}/zoom/{zoom}/openLink=true",
+    "ppnotLink2gis": "{gislink}/center/{center}/zoom/{zoom}/routeTab/rsType/{rsType}/to/{point}/openLink=true",


Лучше назвать тогда этот параметр ppLink2gis, а то ppnotLink2gis введет в заблуждение что это только про маршрут на ОТ.

itanka9 · 2021-12-10T10:48:10Z

src/DGAttribution/src/DGAttribution.js

        }
+        var isNotTranslate = btn.label == 'open_on' || btn.label == 'route_on';


Назови тогда хоть buttonNotTranslated и эту логику комментом опиши, а то непонятно.

src/DGAttribution/src/DGAttribution.js

Co-authored-by: itanka9 <itano@yandex.ru>

src/DGAttribution/src/DGAttribution.js

Co-authored-by: Mstislav Zhivodkov <stevemyz@gmail.com>

Trufi · 2021-12-15T03:36:00Z

А теперь самый главный вопрос — как ты тестировал финальный код? :)

Andrey Kuznecov added 2 commits December 9, 2021 15:11

TILES-3943 Add route case

57f9ddb

fix UML label

efc7060

Trufi suggested changes Dec 10, 2021

View reviewed changes

src/DGAttribution/src/DGAttribution.js Outdated Show resolved Hide resolved

Andrey Kuznecov added 2 commits December 10, 2021 14:12

Fix review

a8b6721

fix remove project link

c1557c4

artem-malko reviewed Dec 10, 2021

View reviewed changes

itanka9 reviewed Dec 10, 2021

View reviewed changes

Trufi reviewed Dec 10, 2021

View reviewed changes

src/DGAttribution/src/DGAttribution.js Outdated Show resolved Hide resolved

fix review

5925ee4

itanka9 self-requested a review December 13, 2021 10:13

itanka9 reviewed Dec 13, 2021

View reviewed changes

src/DGAttribution/src/DGAttribution.js Outdated Show resolved Hide resolved

itanka9 approved these changes Dec 13, 2021

View reviewed changes

Update src/DGAttribution/src/DGAttribution.js

8ba90aa

Co-authored-by: itanka9 <itano@yandex.ru>

Trufi suggested changes Dec 13, 2021

View reviewed changes

src/DGAttribution/src/DGAttribution.js Outdated Show resolved Hide resolved

fix update

43f3c48

Trufi reviewed Dec 14, 2021

View reviewed changes

src/DGAttribution/src/DGAttribution.js Outdated Show resolved Hide resolved

Update src/DGAttribution/src/DGAttribution.js

5e7865a

Co-authored-by: Mstislav Zhivodkov <stevemyz@gmail.com>

Trufi approved these changes Dec 14, 2021

View reviewed changes

Kuznecoff and others added 3 commits December 14, 2021 14:06

Merge branch 'master' into TILES-3943-Add-route-case

1d094e6

fix

7d9005b

fix init link

5d14003

Kuznecoff merged commit c9953d6 into master Dec 16, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

TILES-3943 Add route case #556

TILES-3943 Add route case #556

Kuznecoff commented Dec 9, 2021 •

edited

Loading

Trufi commented Dec 10, 2021

artem-malko Dec 10, 2021

artem-malko Dec 10, 2021

Kuznecoff Dec 10, 2021

Kuznecoff Dec 10, 2021

artem-malko Dec 10, 2021

Kuznecoff Dec 13, 2021

itanka9 Dec 10, 2021

Kuznecoff Dec 13, 2021

itanka9 Dec 10, 2021

Kuznecoff Dec 10, 2021

itanka9 Dec 10, 2021

Kuznecoff Dec 10, 2021

itanka9 Dec 10, 2021

Kuznecoff Dec 13, 2021

itanka9 Dec 10, 2021

Kuznecoff Dec 10, 2021

itanka9 Dec 10, 2021

Kuznecoff Dec 13, 2021

itanka9 Dec 10, 2021

itanka9 Dec 10, 2021

Trufi commented Dec 15, 2021

		}
		var isNotTranslate = btn.label == 'open_on' \|\| btn.label == 'route_on';

TILES-3943 Add route case #556

TILES-3943 Add route case #556

Conversation

Kuznecoff commented Dec 9, 2021 • edited Loading

Trufi commented Dec 10, 2021

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Trufi commented Dec 15, 2021

Kuznecoff commented Dec 9, 2021 •

edited

Loading