Skip to content

chore(api): remove dead field-config override endpoints#383

Open
Ibochkarev wants to merge 1 commit into
betafrom
chore/remove-dead-field-config-overrides-347
Open

chore(api): remove dead field-config override endpoints#383
Ibochkarev wants to merge 1 commit into
betafrom
chore/remove-dead-field-config-overrides-347

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Удаляет deprecated override API после drop таблицы ms3_field_config_overrides: маршрут DELETE /api/mgr/config/page-fields/{page_key}/{field_name}, ConfigController::deleteFieldOverride, no-op ConfigService::removeFieldOverride и FieldConfigManager::saveFieldsConfig.

Актуальный путь сохранён: PUT /api/mgr/config/page-fields/{page_key}ConfigService::saveFieldsConfig(array $fields) пишет в ms3_product_fields. Убраны неиспользуемые параметры $pageKey/$contextKey у save.

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

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

Связанные Issues

Closes #347

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

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

Grep: callers deprecated API = 0. Vue зовёт только GET/PUT page-fields. php -l на изменённых PHP.

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

  • MiniShop3: ветка chore/remove-dead-field-config-overrides-347
  • MODX: —
  • PHP: локальный php -l

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

До После

Чеклист

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

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

DELETE …/page-fields/{page_key}/{field_name} больше нет (раньше был no-op success). Внешние клиенты получат 404 роутера. OrderService::save/remove @deprecated не трогали — отдельно, как в issue.

Drop no-op removeFieldOverride/saveFieldsConfig shims and the DELETE
page-fields route after ms3_field_config_overrides was removed.
@Ibochkarev Ibochkarev requested a review from biz87 July 15, 2026 18:41
@Ibochkarev Ibochkarev changed the title chore(api): remove dead field-config override endpoints (#347) chore(api): remove dead field-config override endpoints Jul 16, 2026
@Ibochkarev Ibochkarev added priority: low Низкий приоритет, когда будет время enhancement New feature or request tech-debt Maintainability / refactor / architecture debt and removed priority: low Низкий приоритет, когда будет время labels Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request tech-debt Maintainability / refactor / architecture debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Удалить dead API field-config overrides после drop таблицы

1 participant