Skip to content

fix(service-registry): skip logs for missing optional defaults#225

Merged
biz87 merged 1 commit intobetafrom
fix/224-service-registry-optional-config-logs
Apr 26, 2026
Merged

fix(service-registry): skip logs for missing optional defaults#225
biz87 merged 1 commit intobetafrom
fix/224-service-registry-optional-config-logs

Conversation

@Ibochkarev
Copy link
Copy Markdown
Member

Описание

Убирает лишние debug-сообщения из ServiceRegistry, когда отсутствуют дефолтные опциональные пути core/config/ms3.services.php и core/config/ms3.services.d/.

Логирование сохраняется для реально проблемных случаев: если путь был явно задан через системные настройки ms3_services_config или ms3_services_addons_dir, но не существует.

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

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

Связанные Issues

Closes #224

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

Проверил поведение до и после изменения на минимальном воспроизведении ServiceRegistry.

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

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

  • MiniShop3: beta
  • MODX: stub-окружение для изолированной проверки ServiceRegistry
  • PHP: локальная проверка php -l + выполнение PHP-скриптов проверки

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

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

Чеклист

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

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

Сейчас отсутствие дефолтных override-путей является штатным сценарием, потому что они используются только при кастомизации сервисов. До фикса это приводило к постоянному debug-шуму в логах на обычной установке.

Изменение намеренно не отключает диагностику для явно настроенных путей: если пользователь задал кастомный путь и он отсутствует, ServiceRegistry по-прежнему пишет debug-сообщение.

Avoid debug noise when the default service override config paths are absent, while still logging missing paths that were explicitly configured by the user.
@Ibochkarev Ibochkarev marked this pull request as ready for review April 24, 2026 03:59
@Ibochkarev Ibochkarev requested a review from biz87 April 24, 2026 03:59
@biz87 biz87 merged commit 4bafd3f into beta Apr 26, 2026
@biz87 biz87 deleted the fix/224-service-registry-optional-config-logs branch April 26, 2026 12:02
@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] ServiceRegistry logs missing optional service override config paths

2 participants