| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.4.5 |
| Bug #4950 | Incorrect CDATA serializing | ||
|---|---|---|---|
| Submitted: | 2005-07-30 00:09 UTC | ||
| From: | ja dot doma at gmail dot com | Assigned: | ashnazg |
| Status: | Closed | Package: | XML_Util |
| PHP Version: | 4.3.11 | OS: | |
| Roadmaps: | 1.2.0a1 | ||
[2005-07-30 00:09 UTC] ja dot doma at gmail dot com
Description:
------------
This code:
require_once "XML/Util.php";
// creating an XML tag
$tag = XML_Util::createTag("test", array(), "Content ]]></test> here!",null, XML_UTIL_CDATA_SECTION);
Will result in:
<test><![CDATA[Content ]]></test> here!]]></test>
As you can see it is wrong bechavior. I'm not sure, but the only one solution is to create two CDATA sections there:
<test><![CDATA[Content ]]>]]><![CDATA[</test> here!]]></test>
Note: the ]]> sequence is outside CDATA sections
Normal(DOM) parser may merge sequence of CDATA and text into one DomNode.text field.
Sorry for my badly english =)
Test script:
---------------
require_once "XML/Util.php";
// creating an XML tag
$tag = XML_Util::createTag("test", array(), "Content ]]></test> here!",null, XML_UTIL_CDATA_SECTION);
echo $tag;
Expected result:
----------------
<test><![CDATA[Content ]]>]]><![CDATA[</test> here!]]></test>
Actual result:
--------------
<test><![CDATA[Content ]]></test> here!]]></test>
Comments
[2005-07-30 00:20 UTC] ja dot doma at gmail dot com
The solution might be(line 642):
function createCDataSection($data) {
return sprintf("<![CDATA[%s]]>", preg_replace('/\]\]>/', "]]>]]><![CDATA[", strval($data)));
}
[2005-07-30 00:34 UTC] ja dot doma at gmail dot com
Oops! Since XML can not contain > symbol it must be escaped with >
So the proper fix to bug is:
function createCDataSection($data)
{
return sprintf("<![CDATA[%s]]>", preg_replace('/\]\]>/', "]]>]]><![CDATA[", strval($data)));
}
[2008-05-04 18:09 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2008-05-04 23:13 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2008-05-05 06:33 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2008-05-05 16:32 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2008-05-05 16:39 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2008-06-10 10:07 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!