fix(product): удаление опции из товара при сохранении формы

[Ibochkarev]

Описание

При сохранении товара в менеджере опции из POST (options-*) синхронизировались с removeOther=false, поэтому строки в msProductOption для ключей, которых больше нет в отправленной форме (после удаления опции пользователем, в том числе после копирования товара), не удалялись и снова появлялись после перезагрузки.

Вызовы saveOptions из процессоров Product/Create и Product/Update переведены на removeOther=true. Путь saveOptions(null) из msProductData::save() не менялся: при автосборе только JSON-полей по-прежнему принудительно removeOther=false, чтобы не повторить регрессию #153/#158 (кастомные опции категорий).

Дополнительно уточнён PHPDoc ProductDataService::saveOptions() и комментарии в процессорах.

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

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

Связанные Issues

Closes #199

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

Локально не запускалось в этом PR; рекомендуется проверить сценарий: копирование товара → удаление опции на вкладке опций → сохранение → перезагрузка карточки — опция не должна возвращаться. Дополнительно — сохранение товара с опциями только из категории (не в JSON), без открытия полной формы опций, чтобы убедиться в отсутствии регрессии #153.

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

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

• MiniShop3:
• MODX:
• PHP:

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

До	После

Чеклист

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

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

Ревьюеру: сравнить с логикой OptionSyncService::removeUnusedOptions и принудительным removeOther=false в ProductDataService при options === null.

[biz87]

Ревью

Точечный и правильный фикс. Изменение минимальное — 4 файла, суть в одном символе: false → true.

Анализ

Проблема: при сохранении товара из формы опции передаются как options-color, options-size и т.д. Если пользователь удалил опцию, её ключ отсутствует в POST. Но saveOptions($options, false) не удалял строки из msProductOption для отсутствующих ключей — опция возвращалась после перезагрузки.

Фикс: saveOptions($options, true) — POST из формы содержит полный набор опций. Отсутствующие ключи = удалённые пользователем → нужно удалить из БД.

Защита от регрессии #153/#158: при saveOptions(null) (автосбор JSON-полей из msProductData::save()) — removeOther принудительно false на строке 144 ProductDataService.php:

$removeOther = $optionsExplicit ? $removeOther : false;

Кастомные опции категорий не затрагиваются. Защита работает корректно.

Замечания

Нет. Фикс логичный, комментарии подробные, PHPDoc обновлён.

LGTM 👍