X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=php5%2Fxml-template.php;h=e6a4e1909d2f0dc4153fbd644ec6797b0040587c;hb=dd670f1b631ffb9531f7c58ed6410d8c043b03d6;hp=f69fbcaa082e7abb8234c70ce4957527332d4988;hpb=81500cd7f6be3775d5c5f7322a55874e9d691585;p=xml-template

diff --git a/php5/xml-template.php b/php5/xml-template.php
index f69fbca..e6a4e19 100644
--- a/php5/xml-template.php
+++ b/php5/xml-template.php
@@ -11,29 +11,34 @@ function XML_Template_process_file($filename, $obj, $clean = 1)
 function XML_Template_process($node, $obj, $clean = 1)
 {
 	if (is_a($obj, 'DOMNode')) {                          # overwrite
-		for ($i = 0; $i < $node->childNodes->length; ++$i) {
-			$child = $node->childNodes->item($i);
+		while ($node->childNodes->length > 0) {
+			$child = $node->childNodes->item(0);
 			$node->removeChild($child);
 		}
 
 		if (is_a($obj, 'DOMDocument')) {
-			$obj = $obj->document_element();
+			$obj = $obj->documentElement;
 		}
 
-		$newobj = $obj->cloneNode(true);
-		$node->appendChild($newobj);
-
-		XML_Template_process($newobj, array(), $clean);
+		$frag = $node->ownerDocument->createDocumentFragment();
+		$frag->appendChild($node->ownerDocument->importNode($obj, true));
+		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
-		for ($i = 0; $i < $node->childNodes->length; ++$i) {
-			$child = $node->childNodes->item($i);
+		$num_children = ($node->childNodes == null) ? 0 : $node->childNodes->length;
+		$children = array();
+		for ($i = 0; $i < $num_children; ++$i) {
+			$children[] = $node->childNodes->item($i);
+		}
+
+		for ($i = 0; $i < $num_children; ++$i) {
+			$child = $children[$i];
 			$processed = false;
 
 			if ($child->nodeType == XML_ELEMENT_NODE) {
@@ -43,13 +48,25 @@ function XML_Template_process($node, $obj, $clean = 1)
 				$attrs = $child->attributes;
 
 				if (isset($attrs)) {
-					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 = 0;
+					$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
+					# XML namespace declarations.
+					if ($replace_child == 1) {
+						$newchild = own_clone_element($child, $child->ownerDocument);
+						while ($child->childNodes->length > 0) {
+							$newchild->appendChild($child->firstChild);
 						}
+						$node->replaceChild($newchild, $child);
+						$child = $newchild;
 					}
 				}
 			
@@ -78,10 +95,11 @@ function XML_Template_process($node, $obj, $clean = 1)
 		}
 	} else {						# repeat
 		$doc = $node->ownerDocument;
-		$frag = $doc->createElement("temporary-fragment");    # ugh
+		$frag = $doc->createDocumentFragment();
 
 		while ($node->childNodes->length > 0) {
 			$child = $node->childNodes->item(0);
+			$node->removeChild($child);
 			$frag->appendChild($child);
 		}
 
@@ -90,27 +108,14 @@ function XML_Template_process($node, $obj, $clean = 1)
 				continue;
 			}
 
-			$newnode = $frag->cloneNode(true);
-			$node->appendChild($newnode);
+			$newnode = own_clone_node($frag, $frag->ownerDocument);
 			XML_Template_process($newnode, $instance, $clean);
+
+			$node->appendChild($newnode);
 			if ($clean) {
 				XML_Template_clean($newnode);
 			}
 		}
-
-		# remove all the <fragment> tags
-		for ($i = 0; $i < $node->childNodes->length; ++$i) {
-			$child = $node->childNodes->item($i);
-			if ($child->localName != 'temporary-fragment') {
-				continue;
-			}
-			while ($child->childNodes->length > 0) {
-				$child2 = $child->childNodes->item(0);
-				$node->appendChild($child2);
-			}
-			--$i;
-			$node->removeChild($child);
-		}	
 	}
 
 	if ($clean) {
@@ -133,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);
 	}
@@ -153,6 +158,63 @@ function XML_Template_alternate($tag, $array, $elems)
 	return $array;
 }
 		
+# Ideally, this would be "return $obj->clone_node(true)". But surprise,
+# surprise, PHP is buggy; it does not preserve the prefix information on
+# attributes properly during a clone. (PHP4 and PHP5 are broken in different
+# ways here...) IOW, we'll have to clone everything ourselves.
+function own_clone_node($node, $doc)
+{
+	// we only need these two
+	if ($node->nodeType == XML_DOCUMENT_FRAG_NODE) {
+		$newnode = $doc->createDocumentFragment();
+
+		for ($i = 0; $i < $node->childNodes->length; ++$i) {
+			$child = $node->childNodes->item($i);
+			$newnode->appendChild(own_clone_node($child, $doc));
+		}
+		return $newnode;
+	} else if ($node->nodeType == XML_ELEMENT_NODE) {
+		$newnode = own_clone_element($node, $doc);
+		for ($i = 0; $i < $node->childNodes->length; ++$i) {
+			$child = $node->childNodes->item($i);
+			$newnode->appendChild(own_clone_node($child, $doc));
+		}
+		return $newnode;
+	} else {
+		return $node->cloneNode(true);
+	}
+}
+		
+function own_clone_element($node, $doc)
+{
+	$nsuri = $node->namespaceURI;
+	if (isset($nsuri) && $node->prefix != "default") {
+		$newnode = $doc->createElementNS($node->namespaceURI, $node->nodeName, $node->prefix);
+	
+		// remove useless empty text child
+		$newnode->removeChild($newnode->firstChild);
+	} else {
+		$newnode = $doc->createElement($node->localName);
+	}
+	
+	$attrs = $node->attributes;
+	if (isset($attrs)) {
+		foreach ($node->attributes as $attr) {
+			$nsuri = $attr->namespaceURI;
+			if (isset($nsuri) && $attr->prefix != "default") {
+				$attr2 = $doc->createAttributeNS($nsuri, $attr->prefix . ":" . $attr->name);
+			} else {
+				$attr2 = $doc->createAttribute($attr->localName);
+			}
+
+			# You've got to be kidding me...
+			$attr2->value = preg_replace("/&/", "&amp;", $attr->value);
+			$newnode->appendChild($attr2);
+		}
+	}
+	return $newnode;
+}
+
 function is_associative_array($arr)
 {
 	if (!is_array($arr)) {