cdek 0.3.13 → 0.3.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad8868fd8f6e296c1439f90772bf3c16e724d551ed095d6627250b8182bd486c
-  data.tar.gz: 29cabd18b3071e083f7ff8964bf8005f218da9985e35f6bcc80f030ac2677924
+  metadata.gz: 1dd8710b9122c71addd999aab3d16759f3a919916a11e3d1715d88483d6aa21b
+  data.tar.gz: 50b97380e716f8b23c4d75165552caff3c93856b814a3f64b2ac725e6a61ff02
 SHA512:
-  metadata.gz: bb520764c8df5cfa6b6b012d5e28e219c0a664124caaa07f58ce46ed7a56d079290d7bb8bf1e7f27c02805fd7faca35b051b4df60e3fb94a40112066ad258bf8
-  data.tar.gz: 9f832dae5fbb1443e52a9fb4f621b3a8052d308cc88d05d08089e6f2e423f27c815f4e9ecaac3f1c153ce828e470ee5947d528175b4d83c20f7b70eb05bd71ee
+  metadata.gz: b4adea92e439961fce478880f1792bb9790809bdf1073e3bc1ba45a4468652558f59278bae279bd0bf40643876bcf9ea96e14f9c6c7e4f779d150baa299b444b
+  data.tar.gz: be21d7a9b78b4bdefc1c624742b6a52e5de70228b93497571731ff714379f936f035792fa5dad2daa7963096195eab58e7bcdb86a54add254df46f24e3dde87d

data/CHANGELOG.md

@@ -1,5 +1,60 @@
 # Changelog
 
+## 0.3.16
+
+* Полная изоляция инстансов виджета между открытиями карты. Singleton-гард
+  0.3.14 (один живой инстанс) оказался недостаточным: `@cdek-it/widget@3`
+  держит ГЛОБАЛЬНЫЙ pinia-стор на уровне модуля, а `destroy()` размонтирует
+  Vue-приложение, но не очищает этот стор и не снимает подписки, заведённые
+  в бутстрапе виджета (вне дерева компонентов). После смены города
+  (пересоздания модалки) подписки уничтоженного инстанса оставались на общем
+  сторе и, реагируя на запись нового инстанса, перезапрашивали офисы и
+  пере-центрировали карту на ПРЕЖНИЙ город — точки нового города уходили
+  «за границы» и пропадали навсегда (в server-логах — `widget_service` по
+  старому городу сразу после нового).
+* Решение: шаблон `cdek_widget_controller.js` теперь перед каждым монтажом
+  удаляет `window.CDEKWidget` и заново выполняет бандл, поэтому каждый виджет
+  получает свежий модуль со своим стором, а подписки прежнего инстанса висят
+  на осиротевшем сторе и на новый виджет не влияют. Повторное выполнение
+  бандла безопасно: Yandex Maps (ymaps3) грузится идемпотентно, дублирование
+  `<style>` безвредно, custom elements не регистрируются. Прежний
+  `ensureWidgetScript` (один общий `_scriptPromise`) заменён на
+  `loadFreshWidgetScript`.
+* Шаблон генератора снова пишет координаты выбранного ПВЗ (`office.location`
+  → строка `"lon lat"`) в скрытое поле `order_cdek_point_coors`. Поддержка
+  `fieldCoors` была в host-копиях ≤0.3.12, но потерялась при переносе фиксов
+  0.3.13/0.3.14 в шаблон — из-за этого после синхронизации контроллера
+  хост-приложение переставало получать координаты пункта и не могло
+  отрисовать карту заказа. Значение `fieldCoors` (по умолчанию
+  `order_cdek_point_coors`) и его data-атрибут `cdek_widget_field_coors_value`
+  добавлены и в шаблон, и в `cdek_widget_tag` (новый необязательный параметр
+  `field_coors:`), консистентно с остальными `field_*`.
+* Шаблон: после асинхронной загрузки скрипта виджета монтирование
+  выполняется только если контроллер ещё подключён
+  (`if (this._connected) this._mountWidget()`). Раньше промис загрузки мог
+  доехать уже после `disconnect()` (быстрое закрытие/переоткрытие модалки),
+  и виджет монтировался в уже отсоединённый узел.
+* Совокупно с 0.3.13 (центрирование карты по координатам вместо асинхронного
+  геокодинга названия) и 0.3.14 (ровно один живой инстанс) это полностью
+  убирает «мелькание» и пропадание точек ПВЗ при открытии карты после смены
+  города. Хост-приложениям, сгенерировавшим контроллер на ≤0.3.15, нужно
+  перегенерировать его (`bin/rails generate cdek:install`) или внести правку
+  вручную из шаблона.
+
+## 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
 
 * Виджет ПВЗ больше не откатывает центр карты на Москву при выборе

data/README.md

@@ -115,9 +115,11 @@ suggestions = Cdek.city_suggestions("мос")
    * `#order_cdek_point_name`
    * `#order_cdek_point_address`
    * `#order_cdek_city_code`
+   * `#order_cdek_point_coors` — координаты ПВЗ строкой `"долгота широта"`
+     (из `office.location`), чтобы хост мог отрисовать карту заказа.
 
    DOM-id переопределяются именованными аргументами `field_code`,
-   `field_name`, `field_address`, `field_city_code` хелпера.
+   `field_name`, `field_address`, `field_city_code`, `field_coors` хелпера.
 
 5. Также диспатчится событие `cdek-widget:chosen` с `detail.office` —
    можно слушать в собственных Stimulus-контроллерах.

data/app/helpers/cdek/widget_helper.rb

@@ -43,6 +43,7 @@ module Cdek
                         field_name:      "order_cdek_point_name",
                         field_address:   "order_cdek_point_address",
                         field_city_code: "order_cdek_city_code",
+                        field_coors:     "order_cdek_point_coors",
                         label_selector:    "[data-cdek-widget-label]",
                         address_selector:  "#order_cdek_point_address_view",
                         height: "600px")
@@ -62,6 +63,7 @@ module Cdek
         field_name:        field_name,
         field_address:     field_address,
         field_city_code:   field_city_code,
+        field_coors:       field_coors,
         label_selector:    label_selector,
         address_selector:  address_selector
       )
@@ -95,7 +97,7 @@ module Cdek
     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:,
+                         field_code:, field_name:, field_address:, field_city_code:, field_coors:,
                          label_selector:, address_selector:)
       {
         controller: "cdek-widget",
@@ -112,6 +114,7 @@ module Cdek
         cdek_widget_field_name_value:        field_name,
         cdek_widget_field_address_value:     field_address,
         cdek_widget_field_city_code_value:   field_city_code,
+        cdek_widget_field_coors_value:       field_coors,
         cdek_widget_label_selector_value:    label_selector,
         cdek_widget_address_selector_value:  address_selector
       }

data/lib/cdek/version.rb

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

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

@@ -26,22 +26,27 @@ import { Controller } from "@hotwired/stimulus"
 // виджет будет открыт без расчётных packages, чтобы не считать доставку
 // по выдуманным размерам.
 
-let _scriptPromise = null
+let _activeWidget = null
 
-function ensureWidgetScript(url) {
-    if (typeof window.CDEKWidget !== "undefined") {
-        return Promise.resolve()
-    }
-    if (_scriptPromise) return _scriptPromise
-    _scriptPromise = new Promise((resolve, reject) => {
+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 loadFreshWidgetScript(url) {
+    return new Promise((resolve, reject) => {
+        try { delete window.CDEKWidget } catch (_) { window.CDEKWidget = undefined }
         const s = document.createElement("script")
         s.src = url
         s.async = true
         s.onload = () => resolve()
-        s.onerror = () => { _scriptPromise = null; reject(new Error("Не удалось загрузить скрипт виджета СДЭК")) }
+        s.onerror = () => reject(new Error("Не удалось загрузить скрипт виджета СДЭК"))
         document.head.appendChild(s)
     })
-    return _scriptPromise
 }
 
 function setFieldValue(id, value) {
@@ -86,6 +91,7 @@ export default class extends Controller {
         fieldName:       { type: String, default: "order_cdek_point_name" },
         fieldAddress:    { type: String, default: "order_cdek_point_address" },
         fieldCityCode:   { type: String, default: "order_cdek_city_code" },
+        fieldCoors:      { type: String, default: "order_cdek_point_coors" },
         labelSelector:   { type: String, default: "[data-cdek-widget-label]" },
         addressSelector: { type: String, default: "#order_cdek_point_address_view" }
     }
@@ -101,13 +107,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() {
@@ -118,8 +119,8 @@ export default class extends Controller {
         }
 
         this._mounting = true
-        ensureWidgetScript(this.scriptUrlValue)
-            .then(() => this._mountWidget())
+        loadFreshWidgetScript(this.scriptUrlValue)
+            .then(() => { if (this._connected) this._mountWidget() })
             .catch((err) => this._showError(err && err.message ? err.message : "ошибка"))
             .finally(() => { this._mounting = false })
     }
@@ -193,8 +194,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 : "ошибка инициализации виджета")
         }
@@ -207,10 +213,16 @@ export default class extends Controller {
         const address  = office.address || office.address_full || ""
         const cityCode = office.city_code || ""
 
+        const loc      = Array.isArray(office.location) ? office.location : []
+        const lon      = Number(loc[0])
+        const lat      = Number(loc[1])
+        const coors    = (Number.isFinite(lon) && Number.isFinite(lat)) ? `${lon} ${lat}` : ""
+
         setFieldValue(this.fieldCodeValue,     code)
         setFieldValue(this.fieldNameValue,     name)
         setFieldValue(this.fieldAddressValue,  address)
         setFieldValue(this.fieldCityCodeValue, cityCode)
+        setFieldValue(this.fieldCoorsValue,    coors)
 
         const label = name ? (code ? `${name} (${code})` : name) : "не выбран"
         setText(this.labelSelectorValue,   label)

metadata

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