cdek 0.3.12 → 0.3.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5dfa6bb78c45fc7a0bf40bf04cdf59999b7598766cf0ef5c3b086b699db45f38
-  data.tar.gz: c9f43b02cfc8b5314ad3b4338f8786e319216b0e72878e2f82a817f4cbca76f9
+  metadata.gz: 876ccf8ab0ab45c5d2ea73d3a2abd8d396e5abcedf8c30d7fbdd3c18fbaf9f00
+  data.tar.gz: af5b8ea84efcfd064cf7a265405ecc5d0460bb338ec12f4fc25d3c4ac2e885e1
 SHA512:
-  metadata.gz: 8e532a826e6bcacae86aaf2f1f4ee23f14ae1358889c344a632cc76107deb7e359d16b61983921a7fdc2f4b7e274f262d7fa10f010af9b2e276dec5c6af96267
-  data.tar.gz: ed305863d0c728ba013f3f6b7e363e54e6e973dd0e67d76c28468750f685157af106a6a13a2aa7cfa43e8d6eff0f6d389f4f0701c51680f368f35a3562572780
+  metadata.gz: 6da66c243aeba6ee2c00c6f151b1873c9b5c47924eb89cf5f9a6031d9edc002d04320a7f71b419fcad64190c25c3968efcf1161538f228798a8953c360bf4502
+  data.tar.gz: d49a1accd14e41e2b4cf199e3f6a42892a61a32400e3f683519b12aa81a0b51d1d84ef47489bd35b47261aa3fc30c3da75df2e885604c436125d14e7f9f552ce

data/CHANGELOG.md

@@ -1,5 +1,31 @@
 # Changelog
 
+## 0.3.14
+
+* Шаблон генератора `cdek_widget_controller.js` теперь держит ровно один
+  живой инстанс виджета на странице: предыдущий уничтожается синхронно
+  перед монтированием нового. `@cdek-it/widget` — Vue-приложение с
+  глобальным стором, поэтому два одновременно живых инстанса делили стор
+  и конфликтовали — при смене города (пересоздание модалки) старый инстанс
+  перезапрашивал офисы и пере-центрировал карту на прежний город (точки
+  выбранного города мелькали и пропадали). Добавлены module-level
+  `_activeWidget` и `destroyWidget()`; `disconnect` и `_mountWidget`
+  переведены на них. Хост-приложениям, сгенерировавшим контроллер на
+  ≤0.3.13, нужно перегенерировать его (`bin/rails generate cdek:install`)
+  или внести правку вручную.
+
+## 0.3.13
+
+* Виджет ПВЗ больше не откатывает центр карты на Москву при выборе
+  не-московского города. `defaultLocation` передавался строкой-названием,
+  центр ставился асинхронным геокодингом и при сбое оставался дефолт
+  [37.62,55.75]. `CityResolver` теперь из одного кэшированного
+  `/location/cities` отдаёт и `code`, и координаты `[lng,lat]`;
+  `cdek_widget_tag` прокидывает `default_coords`, JS-контроллер использует
+  синхронную ветку без геокодинга. При отсутствии координат — fallback на
+  строку (нулевая регрессия). Перегенерировать контроллер:
+  `bin/rails generate cdek:install`.
+
 ## 0.3.9
 
 * В `cdek.gemspec` добавлена development-зависимость `rake`, потому что

data/app/helpers/cdek/widget_helper.rb

@@ -51,6 +51,7 @@ module Cdek
       data = cdek_widget_data(
         api_key:           api_key.presence || ENV["YANDEX_MAPS_API_KEY"].to_s,
         default_city:      default_city,
+        default_coords:    cdek_default_coords(default_city),
         sender_city:       sender_city,
         sender_city_code:  sender_city_code.to_s,
         goods_json:        JSON.generate(goods_payload),
@@ -91,7 +92,7 @@ module Cdek
 
     # Хэш data-* атрибутов для Stimulus-контроллера cdek-widget.
     # Ключи в snake_case — Rails сам конвертит "_" в "-" в HTML.
-    def cdek_widget_data(api_key:, default_city:, sender_city:, sender_city_code:,
+    def cdek_widget_data(api_key:, default_city:, default_coords:, sender_city:, sender_city_code:,
                          goods_json:,
                          service_path:, script_url:, modal_id:,
                          field_code:, field_name:, field_address:, field_city_code:,
@@ -102,6 +103,7 @@ module Cdek
         cdek_widget_script_url_value:        script_url,
         cdek_widget_api_key_value:           api_key,
         cdek_widget_default_location_value:  default_city,
+        cdek_widget_default_coords_value:    default_coords,
         cdek_widget_sender_city_value:       sender_city,
         cdek_widget_sender_city_code_value:  sender_city_code,
         cdek_widget_goods_value:             goods_json,
@@ -115,6 +117,21 @@ module Cdek
       }
     end
 
+    # Координаты города «по умолчанию» строкой "долгота,широта" для data-атрибута
+    # (или "" если город пуст / не нашёлся / CDEK недоступен). JS-контроллер
+    # передаёт их виджету как defaultLocation = [lng, lat]; при пустой строке
+    # он откатывается на defaultLocation = название города (прежнее поведение).
+    def cdek_default_coords(city_name)
+      coords =
+        begin
+          Cdek.city_coordinates(city_name)
+        rescue Cdek::Error
+          nil
+        end
+
+      coords.is_a?(Array) && coords.length == 2 ? coords.join(",") : ""
+    end
+
     # Путь до UMD-бандла виджета, вшитого в гем. Используется JS-контроллером
     # для динамической загрузки скрипта по требованию.
     def cdek_widget_asset_path

data/lib/cdek/city_resolver.rb

@@ -1,18 +1,27 @@
 # frozen_string_literal: true
 
 module Cdek
-  # Resolves a human-readable city name to a CDEK city code.
+  # Resolves a human-readable city name to a CDEK city record — both the CDEK
+  # city code and the city coordinates — in one API-backed, cached lookup.
   #
   # This is intentionally small and API-backed: host applications can pass
   # user-entered city names to the widget proxy without duplicating lookup
-  # logic or loading all delivery points for the whole country.
+  # logic or loading all delivery points for the whole country. The widget
+  # proxy uses #code to filter offices, while the widget helper uses
+  # #coordinates to center the map on the chosen city (passing coordinates to
+  # the JS widget instead of a name avoids a flaky async geocode that could
+  # otherwise drop the map back to its built-in default center).
   class CityResolver
     DEFAULT_COUNTRY_CODES = "RU"
     CACHE_EXPIRES_IN = 43_200
 
     class << self
       def call(name, **options)
-        new(name, **options).call
+        new(name, **options).code
+      end
+
+      def coordinates(name, **options)
+        new(name, **options).coordinates
       end
     end
 
@@ -23,27 +32,45 @@ module Cdek
       @cache = cache
     end
 
-    def call
-      normalized_name.empty? ? nil : cached_city_code
+    # CDEK city code or nil when the city can't be resolved.
+    def code
+      city = cached_city
+      city.is_a?(Hash) ? (city["code"] || city[:code]) : nil
+    end
+
+    # [longitude, latitude] (floats) or nil when coordinates are unknown.
+    # The order matches what the CDEK JS widget expects ([lng, lat]).
+    def coordinates
+      city = cached_city
+      lng = city_coordinate(city, "longitude")
+      lat = city_coordinate(city, "latitude")
+
+      (lng && lat) ? [lng, lat] : nil
     end
 
     private
 
       attr_reader :name, :client, :country_codes, :cache
 
-      def cached_city_code
-        if cache
-          cache.fetch(cache_key, expires_in: CACHE_EXPIRES_IN) { fetch_city_code }
+      def cached_city
+        if normalized_name.empty?
+          nil
+        elsif cache
+          cache.fetch(cache_key, expires_in: CACHE_EXPIRES_IN) { fetch_city }
         else
-          fetch_city_code
+          fetch_city
         end
       end
 
-      def fetch_city_code
-        city = Cdek.locations(client).find_city(normalized_name, country_codes: country_codes)
+      def fetch_city
+        Cdek.locations(client).find_city(normalized_name, country_codes: country_codes)
+      end
 
+      def city_coordinate(city, key)
         if city.is_a?(Hash)
-          city["code"] || city[:code]
+          raw = city.key?(key) ? city[key] : city[key.to_sym]
+          number = Float(raw, exception: false)
+          number&.finite? ? number : nil
         end
       end
 
@@ -52,7 +79,7 @@ module Cdek
       end
 
       def cache_key
-        ["cdek", "city_code", country_codes, normalized_name.downcase].join(":")
+        ["cdek", "city", country_codes, normalized_name.downcase].join(":")
       end
 
       def default_cache

data/lib/cdek/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Cdek
-  VERSION = "0.3.12"
+  VERSION = "0.3.14"
 end

data/lib/cdek.rb

@@ -68,6 +68,13 @@ module Cdek
       CityResolver.call(name, **options)
     end
 
+    # Координаты [долгота, широта] города по пользовательскому названию
+    # (или nil, если город/координаты не нашлись). Нужны виджету, чтобы
+    # центрировать карту по выбранному городу координатами, а не названием.
+    def city_coordinates(name, **options)
+      CityResolver.coordinates(name, **options)
+    end
+
     # Нормализованные подсказки городов для autocomplete в хост-приложении.
     def city_suggestions(query, **options)
       CitySuggestions.call(query, **options)

data/lib/generators/cdek/install/templates/cdek_widget_controller.js

@@ -28,6 +28,24 @@ import { Controller } from "@hotwired/stimulus"
 
 let _scriptPromise = null
 
+// @cdek-it/widget@3 — Vue-приложение с ГЛОБАЛЬНЫМ стором (singleton). Два
+// живых инстанса делят один стор и конфликтуют: монтаж нового перенастраивает
+// общий стор, и старый перезапрашивает офисы и пере-центрирует карту на
+// прежний город (в server-логах — повторные widget_service по старому городу
+// сразу после нового). Поэтому держим ровно один живой виджет на странице и
+// гасим предыдущий ПЕРЕД монтажом нового. destroy() у виджета бросает, если он
+// ещё не домонтировался, — глушим, ссылку всё равно чистим.
+let _activeWidget = null
+
+function destroyWidget(widget) {
+    if (!widget) return
+    try {
+        if (typeof widget.destroy === "function") widget.destroy()
+        else if (typeof widget.close === "function") widget.close()
+    } catch (_) { /* виджет мог не домонтироваться — это ок */ }
+    if (_activeWidget === widget) _activeWidget = null
+}
+
 function ensureWidgetScript(url) {
     if (typeof window.CDEKWidget !== "undefined") {
         return Promise.resolve()
@@ -77,6 +95,7 @@ export default class extends Controller {
         scriptUrl:       String,
         apiKey:          String,
         defaultLocation: { type: String, default: "Москва" },
+        defaultCoords:   { type: String, default: "" },
         senderCity:      { type: String, default: "Москва" },
         senderCityCode:  { type: String, default: "" },
         goods:           { type: String, default: "" },
@@ -100,13 +119,8 @@ export default class extends Controller {
             window.clearTimeout(this._mountTimer)
             this._mountTimer = null
         }
-        if (this._widget) {
-            try {
-                if (typeof this._widget.destroy === "function") this._widget.destroy()
-                else if (typeof this._widget.close === "function") this._widget.close()
-            } catch (_) { /* no-op */ }
-            this._widget = null
-        }
+        destroyWidget(this._widget)
+        this._widget = null
     }
 
     _mountWhenReady() {
@@ -138,6 +152,22 @@ export default class extends Controller {
         return url.pathname + url.search
     }
 
+    // Координаты выбранного города ([lng, lat]) приходят из data-cdek-widget-default-coords-value.
+    // Передаём их виджету как defaultLocation — тогда он центрирует карту по
+    // координатам синхронно и не делает асинхронный геокодинг названия, который
+    // при сбое оставляет карту на встроенном дефолтном центре (Москве). Если
+    // координат нет (город не нашёлся / пусто) — откатываемся на название города.
+    _resolveDefaultLocation() {
+        const raw = this.defaultCoordsValue.trim()
+        const parts = raw === "" ? [] : raw.split(",")
+        const lng = Number(parts[0])
+        const lat = Number(parts[1])
+
+        return (parts.length === 2 && Number.isFinite(lng) && Number.isFinite(lat))
+            ? [lng, lat]
+            : this.defaultLocationValue
+    }
+
     _mountWidget() {
         if (!this.hasRootTarget) return
         if (typeof window.CDEKWidget !== "function") {
@@ -164,7 +194,7 @@ export default class extends Controller {
             root:                this.rootTarget.id,
             servicePath:         this._servicePath(),
             apiKey:              this.apiKeyValue,
-            defaultLocation:     this.defaultLocationValue,
+            defaultLocation:     this._resolveDefaultLocation(),
             from:                from,
             hideDeliveryOptions: { door: true, office: false },
             lang:                "rus",
@@ -176,8 +206,13 @@ export default class extends Controller {
             widgetOptions.goods = goods
         }
 
+        // Гасим любой предыдущий инстанс ДО создания нового — иначе два виджета
+        // дерутся за глобальный стор (старый город перезапрашивается, карта прыгает).
+        destroyWidget(_activeWidget)
+
         try {
             this._widget = new window.CDEKWidget(widgetOptions)
+            _activeWidget = this._widget
         } catch (err) {
             this._showError(err && err.message ? err.message : "ошибка инициализации виджета")
         }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cdek
 version: !ruby/object:Gem::Version
-  version: 0.3.12
+  version: 0.3.14
 platform: ruby
 authors:
 - Sergey Korolyov