Skip to content

fix(order): return error when status save fails#392

Open
Ibochkarev wants to merge 1 commit into
betafrom
fix/376-order-status-save-false
Open

fix(order): return error when status save fails#392
Ibochkarev wants to merge 1 commit into
betafrom
fix/376-order-status-save-false

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

OrderStatusService::change при msOrder::save() === false больше не возвращает true. Callers (OrderSubmitHandler, finalize, mgr, customer cancel) получают ms3_err_unknown и не идут по success-path.

Логика выровнена early-return: лог/события/уведомления только после успешного save.

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

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

Связанные Issues

Closes #376

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

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

php -l на OrderStatusService.php. Симуляцию fail save на живом MODX не гонял.

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

  • MiniShop3: ветка fix/376-order-status-save-false
  • MODX: —
  • PHP: локальный php -l

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

До После

Чеклист

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

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

Не #207. Атомарность submit (create event до status) — вне scope, как в issue. Callers уже проверяют $result !== true.

OrderStatusService::change no longer returns true after a failed
msOrder::save(); callers get ms3_err_unknown instead of a false success.
@Ibochkarev Ibochkarev requested a review from biz87 July 15, 2026 18:56
@Ibochkarev Ibochkarev changed the title fix(order): return error when status save fails (#376) fix(order): return error when status save fails Jul 16, 2026
@Ibochkarev Ibochkarev added priority: high Важно исправить в ближайшее время bug Something isn't working and removed priority: high Важно исправить в ближайшее время 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] OrderStatusService::change возвращает true при неудачном save()

1 participant