cdek 0.3.11 → 0.3.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7c67295d1a6a2b9199d0c751a230ab6dc48ecda34fe855ec74eca1276efccf05
-  data.tar.gz: f43d4696e5f063d284329a2fb34fed6f9775807a2a79490623b7f1b6d0ef7526
+  metadata.gz: ad8868fd8f6e296c1439f90772bf3c16e724d551ed095d6627250b8182bd486c
+  data.tar.gz: 29cabd18b3071e083f7ff8964bf8005f218da9985e35f6bcc80f030ac2677924
 SHA512:
-  metadata.gz: 6ffcc0ddbfc9046c56bcc24246922a8755b492f8932d267298602dd7a49bcc447c88f2400ae562214759ff52441855889f67dd9508b7e56e50a7e4cdf11666c1
-  data.tar.gz: d6b4f68d0f9d992fb22b34f6c31298a0394b08072ae4165414f735d7586dff05e36d54b06e86cd65fd68040d62f8742109780a8fbad2d9c4a2a67a22edf9d4aa
+  metadata.gz: bb520764c8df5cfa6b6b012d5e28e219c0a664124caaa07f58ce46ed7a56d079290d7bb8bf1e7f27c02805fd7faca35b051b4df60e3fb94a40112066ad258bf8
+  data.tar.gz: 9f832dae5fbb1443e52a9fb4f621b3a8052d308cc88d05d08089e6f2e423f27c815f4e9ecaac3f1c153ce828e470ee5947d528175b4d83c20f7b70eb05bd71ee

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 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/city_suggestions.rb

@@ -41,7 +41,8 @@ module Cdek
       def suggestion_payload(city)
         if city.is_a?(Hash)
           code = city["code"] || city[:code]
-          city_name = city["city"] || city[:city] || city["name"] || city[:name]
+          full_name = city["full_name"] || city[:full_name]
+          city_name = city["city"] || city[:city] || city["name"] || city[:name] || city_name_from_full_name(full_name)
           region = city["region"] || city[:region]
           country = city["country"] || city[:country]
           country_code_value = city["country_code"] || city[:country_code]
@@ -50,17 +51,30 @@ module Cdek
             {
               code: code,
               city: city_name,
+              full_name: full_name,
               region: region,
               country: country,
               country_code: country_code_value,
-              label: label_for(city_name, region)
+              label: label_for(city_name, region, full_name)
             }
           end
         end
       end
 
-      def label_for(city_name, region)
-        [city_name, region].compact.map(&:to_s).reject(&:empty?).uniq.join(", ")
+      def city_name_from_full_name(full_name)
+        value = full_name.to_s.split(",").first.to_s.strip
+
+        unless value.empty?
+          value
+        end
+      end
+
+      def label_for(city_name, region, full_name)
+        if full_name.to_s.strip.empty?
+          [city_name, region].compact.map(&:to_s).reject(&:empty?).uniq.join(", ")
+        else
+          full_name.to_s.strip
+        end
       end
 
       def normalized_query

data/lib/cdek/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Cdek
-  VERSION = "0.3.11"
+  VERSION = "0.3.13"
 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

@@ -77,6 +77,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: "" },
@@ -138,6 +139,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 +181,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",

metadata

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