As you can see on the following page: http://larsolesen.dk/node/134, I get an DOM error after implementing a youtube video using this module.

* Warning: DOMDocumentFragment::appendXML(): Entity: line 20: parser error : StartTag: invalid element name in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): //--> in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): ^ in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): Entity: line 20: parser error : Sequence ']]>' not allowed in content in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): //--> in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): ^ in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): Entity: line 20: parser error : Sequence ']]>' not allowed in content in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): //--> in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): ^ in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): Entity: line 20: parser error : internal error in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): //--> in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMDocumentFragment::appendXML(): ^ in filter_dom_serialize_escape_cdata_element() (line 1084 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).
* Warning: DOMNode::appendChild(): Document Fragment is empty in filter_dom_serialize_escape_cdata_element() (line 1085 of /home/lsolesen/larsolesen.dk/modules/filter/filter.module).

CommentFileSizeAuthor
#2 cdata_stuff.patch955 bytesmfer

Comments

mfer’s picture

mfer commented

Sadly, I can confirm them.

mfer’s picture

mfer commented
Status: Active » Needs review
StatusFileSize
new cdata_stuff.patch955 bytes

Drupals filtering system now adds the cdata wrappers inside a script element. If they are in the module then as well they end up in there twice and the parser doesn't like that.

lsolesen’s picture

lsolesen commented
Status: Needs review » Reviewed & tested by the community

I can confirm that this patch solves the problem.

aaron’s picture

aaron commented
Status: Reviewed & tested by the community » Fixed

got it, thanks mfer!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.