Skip to content

fix(import): read processor object in startImport#390

Open
Ibochkarev wants to merge 1 commit into
betafrom
fix/370-import-start-response-object
Open

fix(import): read processor object in startImport#390
Ibochkarev wants to merge 1 commit into
betafrom
fix/370-import-start-response-object

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

startImport читал scheduled / total с корня ответа connector, хотя процессор кладёт их в object. Из‑за этого при Scheduler importRunning зависал в true, а в sync-режиме счётчики были нулями.

Исправление: const data = response.object || response (как в loadAvailableFields / previewFile / upload) и finally { importRunning = false }. Флаг success берётся с корня envelope.

Тип изменений

  • Исправление бага (non-breaking change)
  • Новая функциональность (non-breaking change)
  • Breaking change (изменение, ломающее обратную совместимость)
  • Рефакторинг (без изменения функциональности)
  • Документация
  • Другое (опишите):

Связанные Issues

Closes #370

Как это было протестировано?

  • Ручное тестирование
  • Автоматические тесты (PHPStan, ESLint)
  • Тестирование на разных версиях PHP/MODX

ESLint: vueManager/src/components/ImportProducts.vue. Ручной прогон импорта (Scheduler / sync) в mgr не гонял в этой сессии.

Конфигурация тестирования:

  • MiniShop3: ветка fix/370-import-start-response-object
  • MODX: —
  • PHP: —

Скриншоты (если применимо)

До После

Чеклист

  • Код соответствует стилю проекта
  • Добавлены/обновлены комментарии в сложных местах
  • Изменения не ломают существующую функциональность
  • Лексиконы добавлены на двух языках (ru/en)
  • PHPStan проходит без новых ошибок
  • ESLint проходит без ошибок (для JS/Vue изменений)
  • Обновлён CHANGELOG.md (для значимых изменений)

Дополнительные заметки

Не #291. После merge стоит вручную: CSV → mapping → Start с Scheduler (ожидается сообщение «запланировано», ProgressBar гаснет) и sync без Scheduler (ненулевые счётчики из object).

Unwrap response.object like other import steps and clear importRunning
in finally so Scheduler mode no longer leaves the ProgressBar stuck.
@Ibochkarev Ibochkarev requested a review from biz87 July 15, 2026 18:44
@Ibochkarev Ibochkarev changed the title fix(import): read processor object in startImport (#370) fix(import): read processor object in startImport Jul 16, 2026
@Ibochkarev Ibochkarev added priority: critical Блокирует работу, требует немедленного исправления javascript Pull requests that update javascript code bug Something isn't working and removed priority: critical Блокирует работу, требует немедленного исправления labels Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] ImportProducts: startImport не читает response.object — hang Scheduler / нули

1 participant