X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=php5%2Fxml-template.php;h=e6a4e1909d2f0dc4153fbd644ec6797b0040587c;hb=83a41af3311ec494205205ec093ffff0aad56fd0;hp=f4581ff9c4b08e78c016773815c7869211649e3e;hpb=86bc564142b279461bd97cc752b7138fa03c6417;p=xml-template

diff --git a/php5/xml-template.php b/php5/xml-template.php
index f4581ff..e6a4e19 100644
--- a/php5/xml-template.php
+++ b/php5/xml-template.php
@@ -25,16 +25,20 @@ function XML_Template_process($node, $obj, $clean = 1)
 		XML_Template_process($frag, array(), $clean);
 		$node->appendChild($frag);
 	} else if (!is_array($obj)) {                         # overwrite
-		for ($i = 0; $i < $node->childNodes->length; ++$i) {
-			$child = $node->childNodes->item($i);
-			$node->removeChild($child);
+		while ($node->childNodes->length > 0) {
+			$node->removeChild($node->firstChild);
 		}
 		$doc = $node->ownerDocument;
 		$node->appendChild($doc->createTextNode($obj));
 	} else if (is_associative_array($obj)) {              # substitute
 		$num_children = ($node->childNodes == null) ? 0 : $node->childNodes->length;
+		$children = array();
 		for ($i = 0; $i < $num_children; ++$i) {
-			$child = $node->childNodes->item($i);
+			$children[] = $node->childNodes->item($i);
+		}
+
+		for ($i = 0; $i < $num_children; ++$i) {
+			$child = $children[$i];
 			$processed = false;
 
 			if ($child->nodeType == XML_ELEMENT_NODE) {
@@ -45,14 +49,13 @@ function XML_Template_process($node, $obj, $clean = 1)
 
 				if (isset($attrs)) {
 					$replace_child = 0;
-					foreach ($child->attributes as $attr) {
-						if ($attr->namespaceURI == 'http://template.sesse.net/' && $attr->name == 'id') {
-							$id = $attr->value;
-							if ($clean) {
-								$child->removeAttributeNode($attr);
-								$replace_child = 1;
-							}
-						}
+					$id_node = $child->getAttributeNodeNS('http://template.sesse.net/', 'id');
+					if (isset($id_node)) {
+						$id = $id_node->value;
+						if ($clean) {
+							$child->removeAttributeNode($id_node);
+							$replace_child = 1;
+						}       
 					}
 
 					# This seems to be the only way to reliably get rid of the excess
@@ -135,7 +138,7 @@ function XML_Template_clean($node)
 		while ($node->childNodes->length > 0) {
 			$child = $node->childNodes->item(0);
 			$node->removeChild($child);
-			$parent->insertBefore($child, $node);
+			$parent->insertBefore(own_clone_node($child, $doc), $node);
 		}
 		$parent->removeChild($node);
 	}
@@ -203,7 +206,9 @@ function own_clone_element($node, $doc)
 			} else {
 				$attr2 = $doc->createAttribute($attr->localName);
 			}
-			$attr2->value = $attr->value;
+
+			# You've got to be kidding me...
+			$attr2->value = preg_replace("/&/", "&amp;", $attr->value);
 			$newnode->appendChild($attr2);
 		}
 	}