[Bug] Не обработанный Exception при вызове $modx->services->get(...)

[createit-ru]

Описание проблемы

Не обрабатывается исключение при вызове $modx->services->get(...).

$ms3 = $modx->services->get('ms3');
if ($ms3) {
    // код
}

Шаги воспроизведения

Например, если ms3 удален, а где-то в коде остался такой код - будет 500 ошибка.

Ожидаемое поведение

Вызов services->get() порождает Exception, нужно обрабатывать его, а не проверять результат выполнения функции.

https://github.com/modxcms/revolution/blob/3.x/core/src/Revolution/Services/Container.php#L30

[biz87]

И часто на твоей практике кто-то удалял минишоп с сайта?

[biz87]

Вариант решения использовать метод has()

  if (!$modx->services->has('ms3')) {
      $modx->log(modX::LOG_LEVEL_ERROR, '[MiniShop3] Service not registered');
      return '';  // или соответствующая обработка
  }
  $ms3 = $modx->services->get('ms3');

Почему не критично:

1. Условия возникновения редки:
   - Сервис ms3 регистрируется в ServiceRegistry при загрузке компонента
   - В нормальной работе сервис всегда доступен
   - Проблема возникает только когда:
   • Компонент удалён, но где-то остался вызывающий код
   • Битая установка (сломался autoloader)
   • Кастомный код вызывает сервис до его регистрации
2. Это не security issue — просто некрасивая 500 ошибка вместо понятного сообщения
3. Не влияет на нормальную работу — если MiniShop3 установлен корректно, проблемы нет

будет реализовано по мере свободного времени

[createit-ru]

И часто на твоей практике кто-то удалял минишоп с сайта?
Я, прям при первой же попытке его поставить. Т.к. поставил на php 8.1, в результате чего он установился некорректно. ну и далее уже словил проблему с отсутствующим сервисом.

has() да, должен помочь)