Skip to content

fix(cart): add missing get success lexicon#223

Merged
biz87 merged 1 commit intobetafrom
fix/222-cart-get-success-lexicon
Apr 26, 2026
Merged

fix(cart): add missing get success lexicon#223
biz87 merged 1 commit intobetafrom
fix/222-cart-get-success-lexicon

Conversation

@Ibochkarev
Copy link
Copy Markdown
Member

Описание

Добавляет отсутствующий лексикон ms3_cart_get_success в ru/en cart-лексиконы, чтобы вызов Cart::get() не записывал в лог сообщение Language string not found.

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

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

Связанные Issues

Closes #222

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

Проверил, что изменение ограничено двумя лексиконами и PHP-синтаксис в обоих файлах валиден.

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

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

  • MiniShop3: beta
  • MODX: не указано
  • PHP: локальная проверка php -l

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

До После
Не применимо Не применимо

Чеклист

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

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

Причина бага: ms3_cart_get_success уже использовался в core/components/minishop3/src/Controllers/Cart/Cart.php, но отсутствовал в:

  • core/components/minishop3/lexicon/ru/cart.inc.php
  • core/components/minishop3/lexicon/en/cart.inc.php

Изменение не затрагивает бизнес-логику и устраняет только пропущенные переводы.

Add the missing cart get success messages in both RU and EN lexicons so cart retrieval no longer logs missing language string errors.
@Ibochkarev Ibochkarev requested a review from biz87 April 24, 2026 02:53
@Ibochkarev Ibochkarev marked this pull request as ready for review April 24, 2026 02:53
@biz87 biz87 merged commit 93e429f into beta Apr 26, 2026
@biz87 biz87 deleted the fix/222-cart-get-success-lexicon branch April 26, 2026 12:06
@biz87 biz87 mentioned this pull request Apr 26, 2026
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.

[Bug] Отсутствует лексикон ms3_cart_get_success для API корзины

2 participants