Skip to content

fix(category): persist hide_children_in_tree checkbox#162

Open
Ibochkarev wants to merge 1 commit intobetafrom
fix/hide-children-in-tree-161
Open

fix(category): persist hide_children_in_tree checkbox#162
Ibochkarev wants to merge 1 commit intobetafrom
fix/hide-children-in-tree-161

Conversation

@Ibochkarev
Copy link
Member

Описание

Исправлено сохранение настройки «Скрыть дочерние ресурсы» (hide_children_in_tree) при редактировании и создании категории товаров. В процессорах Category/Update и Category/Create добавлена обработка чекбокса через handleCheckBoxes() и setCheckbox('hide_children_in_tree'), чтобы значение корректно записывалось в БД при включении и выключении переключателя (неотмеченные чекбоксы в HTML не передают значение в запросе).

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

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

Связанные Issues

Closes #161

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

  • Ручное тестирование: открытие категории, включение «Скрыть дочерние ресурсы», сохранение, перезагрузка страницы — значение сохраняется; дочерние элементы не отображаются в дереве.
  • Автоматические тесты (PHPStan, ESLint)
  • Тестирование на разных версиях PHP/MODX

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

  • MiniShop3: актуальная из репозитория
  • MODX: —
  • PHP: —

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

До После
Значение не сохранялось после «Сохранить» Переключатель сохраняет положение, дочерние скрыты в дереве

Чеклист

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

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

Оранжевый цвет переключателя в интерфейсе задаётся стилями MODX/VueTools или темой и не связан с логикой сохранения; исправление касается только персистентности значения.

Add handleCheckBoxes() with setCheckbox('hide_children_in_tree') in
Category/Update and Category/Create so the setting is saved correctly.

Fixes #161
@Ibochkarev Ibochkarev changed the title fix(category): persist hide_children_in_tree checkbox (Closes #161) fix(category): persist hide_children_in_tree checkbox Mar 17, 2026
@Ibochkarev Ibochkarev marked this pull request as ready for review March 17, 2026 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant