Skip to content

fix: use msPayment type hints in Payment Sort processor#177

Open
Ibochkarev wants to merge 1 commit intobetafrom
fix/payment-sort-mspayment-typehints-175
Open

fix: use msPayment type hints in Payment Sort processor#177
Ibochkarev wants to merge 1 commit intobetafrom
fix/payment-sort-mspayment-typehints-175

Conversation

@Ibochkarev
Copy link
Copy Markdown
Member

Описание

В процессоре MiniShop3\Processors\Settings\Payment\Sort метод sort() и связанные @var в process() ошибочно указывали msDelivery (копипаста из Delivery\Sort), хотя $classKey = msPayment::class. Исправлены сигнатура метода, PHPDoc @param и @var на msPayment, удалён неиспользуемый use MiniShop3\Model\msDelivery. Логика и поведение в рантайме не меняются; восстанавливается корректная статическая типизация для IDE и PHPStan.

Связано с обнаружением в PR #174, см. issue #175.

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

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

Связанные Issues

Closes #175

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

Проверка синтаксиса PHP (php -l) для изменённого файла. Убедился, что sort() вызывается только из process() этого процессора и объекты загружаются через $this->classKey (то есть msPayment). Status/Sort.php использует msOrderStatus корректно, правки не требуются.

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

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

  • MiniShop3: ветка fix/payment-sort-mspayment-typehints-175
  • MODX: —
  • PHP: проверка php -l

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

До После

Чеклист

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

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

Изменения только в PHP-процессоре: лексиконы, ESLint и CHANGELOG не затрагиваются. PHPStan на стороне ревьюера/CI.

@Ibochkarev Ibochkarev requested a review from biz87 March 30, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: Payment\Sort использует msDelivery вместо msPayment в type hints

1 participant