Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.When switching from a text format that is not configured to use CKEditor to a text format configured to use CKEditor, a recursion error occurs caused by media_wysiwyg's media_ckeditor/plugin.js when processing a list element (ul, ol).
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | media_wysiwyg-list-recursion-on-upcast-2342729-1.patch | 1007 bytes | recrit |
Comments
Comment #1
recrit CreditAttribution: recrit commentedI found that if I changed the widget's upcast function to only set the HTML if it replaced something then it would not cause the recursion error. The attached patch adds this check.
This could be due to the fact that setting the HTML for a list item results in markup not equivalent to the original which then causes another DOM changed event.
Comment #2
Chris Matthews CreditAttribution: Chris Matthews as a volunteer commentedThe 4 year old patch in #1 media_ckeditor/plugin.js does not apply to the latest media 7.x-2.x-dev and if still relevant needs a reroll.
Comment #3
Chris Matthews CreditAttribution: Chris Matthews as a volunteer commentedClosing as outdated per joseph.olstad.