cdek 0.3.14 → 0.3.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 876ccf8ab0ab45c5d2ea73d3a2abd8d396e5abcedf8c30d7fbdd3c18fbaf9f00
-  data.tar.gz: af5b8ea84efcfd064cf7a265405ecc5d0460bb338ec12f4fc25d3c4ac2e885e1
+  metadata.gz: 1dd8710b9122c71addd999aab3d16759f3a919916a11e3d1715d88483d6aa21b
+  data.tar.gz: 50b97380e716f8b23c4d75165552caff3c93856b814a3f64b2ac725e6a61ff02
 SHA512:
-  metadata.gz: 6da66c243aeba6ee2c00c6f151b1873c9b5c47924eb89cf5f9a6031d9edc002d04320a7f71b419fcad64190c25c3968efcf1161538f228798a8953c360bf4502
-  data.tar.gz: d49a1accd14e41e2b4cf199e3f6a42892a61a32400e3f683519b12aa81a0b51d1d84ef47489bd35b47261aa3fc30c3da75df2e885604c436125d14e7f9f552ce
+  metadata.gz: b4adea92e439961fce478880f1792bb9790809bdf1073e3bc1ba45a4468652558f59278bae279bd0bf40643876bcf9ea96e14f9c6c7e4f779d150baa299b444b
+  data.tar.gz: be21d7a9b78b4bdefc1c624742b6a52e5de70228b93497571731ff714379f936f035792fa5dad2daa7963096195eab58e7bcdb86a54add254df46f24e3dde87d

data/CHANGELOG.md

@@ -1,5 +1,46 @@
 # 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` теперь держит ровно один

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.14"
+  VERSION = "0.3.16"
 end

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

@@ -26,15 +26,6 @@ import { Controller } from "@hotwired/stimulus"
 // виджет будет открыт без расчётных packages, чтобы не считать доставку
 // по выдуманным размерам.
 
-let _scriptPromise = null
-
-// @cdek-it/widget@3 — Vue-приложение с ГЛОБАЛЬНЫМ стором (singleton). Два
-// живых инстанса делят один стор и конфликтуют: монтаж нового перенастраивает
-// общий стор, и старый перезапрашивает офисы и пере-центрирует карту на
-// прежний город (в server-логах — повторные widget_service по старому городу
-// сразу после нового). Поэтому держим ровно один живой виджет на странице и
-// гасим предыдущий ПЕРЕД монтажом нового. destroy() у виджета бросает, если он
-// ещё не домонтировался, — глушим, ссылку всё равно чистим.
 let _activeWidget = null
 
 function destroyWidget(widget) {
@@ -46,20 +37,16 @@ function destroyWidget(widget) {
     if (_activeWidget === widget) _activeWidget = null
 }
 
-function ensureWidgetScript(url) {
-    if (typeof window.CDEKWidget !== "undefined") {
-        return Promise.resolve()
-    }
-    if (_scriptPromise) return _scriptPromise
-    _scriptPromise = new Promise((resolve, reject) => {
+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) {
@@ -104,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" }
     }
@@ -131,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 })
     }
@@ -225,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.14
+  version: 0.3.16
 platform: ruby
 authors:
 - Sergey Korolyov