Revert changes to DOMAttr::$value and DOMAttr::$nodeValue expansion

Closes phpGH-11469.

Author: ndossche

NEWS

@@ -13,6 +13,8 @@ PHP                                                                        NEWS
     (nielsdos)
   . Fix #77894 (DOMNode::C14N() very slow on generated DOMDocuments even after
     normalisation). (nielsdos)
+  . Revert changes to DOMAttr::$value and DOMAttr::$nodeValue expansion.
+    (nielsdos)
 
 - GD:
   . Removed imagerotate "ignore_transparent" argument since it has no effect.

UPGRADING

@@ -44,10 +44,6 @@ PHP 8.3 UPGRADE NOTES
   . Static variable initializers can now contain arbitrary expressions.
     RFC: https://wiki.php.net/rfc/arbitrary_static_variable_initializers
 
-- DOM:
-  . Assignment to DOMAttr::$value and DOMAttr::$nodeValue no longer expands
-    entities in the new value.
-
 - FFI:
   . C functions that have a return type of void now return null instead of
     returning the following object object(FFI\CData:void) { }

ext/dom/attr.c

@@ -148,8 +148,7 @@ int dom_attr_value_write(dom_object *obj, zval *newval)
 	}
 
 	dom_remove_all_children((xmlNodePtr) attrp);
-	xmlNodePtr node = xmlNewTextLen((xmlChar *) ZSTR_VAL(str), ZSTR_LEN(str));
-	xmlAddChild((xmlNodePtr) attrp, node);
+	xmlNodeSetContentLen((xmlNodePtr) attrp, (xmlChar *) ZSTR_VAL(str), ZSTR_LEN(str));
 
 	zend_string_release_ex(str, 0);
 	return SUCCESS;

ext/dom/node.c

@@ -178,9 +178,6 @@ int dom_node_node_value_write(dom_object *obj, zval *newval)
 	/* Access to Element node is implemented as a convenience method */
 	switch (nodep->type) {
 		case XML_ATTRIBUTE_NODE:
-			dom_remove_all_children(nodep);
-			xmlAddChild(nodep, xmlNewTextLen((xmlChar *) ZSTR_VAL(str), ZSTR_LEN(str)));
-			break;
 		case XML_ELEMENT_NODE:
 			dom_remove_all_children(nodep);
 			ZEND_FALLTHROUGH;