diff options
| author | Christoph M. Becker <cmb@php.net> | 2015-07-07 14:55:00 +0200 |
|---|---|---|
| committer | Christoph M. Becker <cmb@php.net> | 2015-09-05 01:17:50 +0200 |
| commit | b2954c64ab67bca980070c63f43cb7afb7ed6d16 (patch) | |
| tree | 0ac81a08bcc5683a33d7a7232a386fc412fa4b07 | |
| parent | b59ea797f53f560ae79c55c1ca4956b5e77203d2 (diff) | |
| download | php-git-b2954c64ab67bca980070c63f43cb7afb7ed6d16.tar.gz | |
Fix #70001: Assigning to DOMNode::textContent does additional entity encoding
Assigning to DOMNode::textContent encodes entities, what does not match the
behavior of DOMText::__construct() and DOMDocument::createTextNode. This patch
changes the behavior of DOMNode::textContent in this regard.
| -rw-r--r-- | ext/dom/node.c | 7 | ||||
| -rw-r--r-- | ext/dom/tests/bug70001.phpt | 17 |
2 files changed, 20 insertions, 4 deletions
diff --git a/ext/dom/node.c b/ext/dom/node.c index 887d6d8009..5b955c06f3 100644 --- a/ext/dom/node.c +++ b/ext/dom/node.c @@ -930,7 +930,6 @@ int dom_node_text_content_write(dom_object *obj, zval *newval TSRMLS_DC) { xmlNode *nodep = dom_object_get_node(obj); zval value_copy; - xmlChar *enc_str; if (nodep == NULL) { php_dom_throw_error(INVALID_STATE_ERR, 0 TSRMLS_CC); @@ -938,9 +937,9 @@ int dom_node_text_content_write(dom_object *obj, zval *newval TSRMLS_DC) } convert_to_string_copy(newval, value_copy); - enc_str = xmlEncodeEntitiesReentrant(nodep->doc, Z_STRVAL_P(newval)); - xmlNodeSetContent(nodep, enc_str); - xmlFree(enc_str); + /* we have to use xmlNodeAddContent() to get the same behavior as with xmlNewText() */ + xmlNodeSetContent(nodep, (xmlChar *) ""); + xmlNodeAddContent(nodep, Z_STRVAL_P(newval)); if (newval == &value_copy) { zval_dtor(newval); } diff --git a/ext/dom/tests/bug70001.phpt b/ext/dom/tests/bug70001.phpt new file mode 100644 index 0000000000..7a8c291d4b --- /dev/null +++ b/ext/dom/tests/bug70001.phpt @@ -0,0 +1,17 @@ +--TEST-- +Bug #70001 (Assigning to DOMNode::textContent does additional entity encoding) +--SKIPIF-- +<?php require_once('skipif.inc'); ?> +--FILE-- +<?php +$element = new DOMText('<p>foo & bar</p>'); +var_dump($element->textContent); +$element = (new DOMDocument())->createTextNode('<p>foo & bar</p>'); +var_dump($element->textContent); +$element->textContent = ('<p>foo & bar</p>'); +var_dump($element->textContent); +?> +--EXPECT-- +string(16) "<p>foo & bar</p>" +string(16) "<p>foo & bar</p>" +string(16) "<p>foo & bar</p>" |