lis-dev / nova-poshta-api-2

PHP class for API 2.0 ukrainian delivery company "Nova Poshta"
144 stars 85 forks source link

Создаются не в тот город 3 #45

Open RemizovAnton opened 4 years ago

RemizovAnton commented 4 years ago

Добрый день, все таки наверное существует еще один баг с городами, попытаюсь объяснить:

когда я заметил ошибку с созданиями городов первый раз, например если создать в город:

Кам'янське Дніпропетровська обл Відділення №12 (до 30 кг): бульв. Будівельників, 27а (маг.VARUS, ТЦ

то создаст вот сюда:

Кам'янське (Нікопольський р-н), Пункт приймання-видачі (до 30 кг): вул. Будівників, 3а

Эту проблему пофиксили в релизе 2 - 0.1.4 (вот ссылка Fixed: https://github.com/lis-dev/nova-poshta-api-2/releases/tag/0.1.4)

Но в этом релизе некорректно создавалась накладная , если выбрать город:

Покровськ, Покровська міськрада, Донецька обл. Відділення №2 (до 30 кг на одне місце): мкрн. Шахтарський, ТЦ

получалась сюда:

Покровське (Мангушський р-н), Відділення №1: вул. Азовської флотилії, 2 (пос. Моряків)

Эту пофиксили в релизе 2-0.1.5 , теперь опять моя посылка поехала в другой город( Вернулась проблема что была пофиксина в релизе 2-0.1.4 с городом : Кам'янське Дніпропетровська обл Відділення №12 (до 30 кг): бульв. Будівельників, 27а (маг.VARUS, ТЦ опять создалось в : Кам'янське (Нікопольський р-н), Пункт приймання-видачі (до 30 кг): вул. Будівників, 3а Спасибо что помогаете в решении проблемы!

lis-dev commented 4 years ago

Добрый день, спасибо, что помогаете. С населёнными пунктами, на подобие Кам'янське следующая ситуация: чтобы однозначно определить город при создании нового документа нужно сделать одно из перечисленного:

  1. Полностью указать название города, как он отдаётся методом getCity (Кам'янське(Дніпропетровська обл)), тогда склад можно указывать только частично (например, Відділення №12)
  2. Полностью указать адрес склада, как он отдаётся методом getWarehouses (Відділення №12 (до 30 кг): бульв. Будівельників, 27а (маг.VARUS, ТЦ"Терра")), тогда город можно указать частично (Кам'янське). Этот функционал был добавлен в версии 0.1.5. Если указывать неполные названия и в поле города и в поле склада, то библиотека пытается угадать город, иногда ошибочно.

Добавил тесты для обоих городов Кам'янське: https://github.com/lis-dev/nova-poshta-api-2/pull/46/commits/a23c3280674efb329b28fd95ab2ca8e4679e125f#diff-89ba1607e8f69e25ef5d8d61baa11cecR733