fix(category): restore visual editor (TinyMCE) for product category content#156
Merged
fix(category): restore visual editor (TinyMCE) for product category content#156
Conversation
…ontent - Include modx-resource-content block in category form so RTE can attach - Fix getContent() to not overwrite content value in update mode - Uncomment loadRichTextEditor() in category update controller
biz87
approved these changes
Mar 16, 2026
Member
biz87
left a comment
biz87
left a comment
There was a problem hiding this comment.
Code Review
Чистый, точный багфикс. Всё корректно.
Анализ бага
Строка 88 в текущем коде:
item_j.value = "<p></p>"Выполняется безусловно, перезаписывая:
- значение
ms3_category_content_defaultизifвыше - существующий контент при редактировании категории
В результате: контент всегда <p></p>, TinyMCE не инициализируется (блок modx-resource-content был исключён из формы, loadRichTextEditor() закомментирован).
Проверка исправлений
getContent()—<p></p>теперь только приcreateбез дефолта. Вupdateсуществующий контент не трогается. Корректно.modx-resource-contentблок — возвращён в форму, чтобы MODX мог повесить RTE. Корректно.loadRichTextEditor()— раскомментирован. Был отключён вместе с блоком контента. Корректно.- Порядок условий
config['mode'] === 'create' && MODx.config[...]— функционально эквивалентен, читается лучше.
Дублирования контент-блока нет (проверено: getFields итерирует оригинальные поля один раз).
LGTM
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Описание
Восстановлена работа визуального редактора (TinyMCE) для поля «Содержимое» при редактировании ресурса-категории товаров. Раньше при переводе ресурса в статус категории товаров блок с полем контента исключался из формы, из-за чего отображалась только разметка HTML вместо редактора, при том что в настройках ресурса визуальный редактор был включён.
Изменения:
modx-resource-content, чтобы MODX мог инициализировать TinyMCE на поле контента.getContent()значение контента задаётся только в режиме создания; в режиме редактирования существующее содержимое не перезаписывается.loadRichTextEditor().Тип изменений
Связанные Issues
(Сообщение пользователя на форуме: при переводе ресурса в категорию товаров визуальный редактор отключался, отображался только raw markup; редактор — TinyMCE.)
Как это было протестировано?
Конфигурация тестирования:
Скриншоты (если применимо)
Чеклист
Дополнительные заметки
Category.getFields()и также получает блок контента с возможностью RTE.