X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=python%2Fxmltemplate.py;h=b24d7642ccda30a0173ccd345fc413a313e19dbd;hb=83a41af3311ec494205205ec093ffff0aad56fd0;hp=19531496f7d08af8d76df86c515c63cb1dcfe337;hpb=a1cfd19c4c66e6c78740427fc8bb905037b409d9;p=xml-template

diff --git a/python/xmltemplate.py b/python/xmltemplate.py
index 1953149..b24d764 100644
--- a/python/xmltemplate.py
+++ b/python/xmltemplate.py
@@ -8,7 +8,7 @@ def process_file(filename, obj, clean = True):
 	return doc
 
 def process(node, obj, clean = True):
-	if isinstance(obj, str):			# overwrite
+	if isinstance(obj, basestring):			# overwrite
 		while not node.firstChild is None:
 			node.removeChild(node.firstChild)
 		doc = _get_document_element(node)
@@ -35,8 +35,8 @@ def process(node, obj, clean = True):
 				attrs_to_remove = []
 				if not attrs is None:
 					for i in range(attrs.length):
-						attr = attrs.item(0)
-						if attr.namespaceURI == "http://template.sesse.net/" or attr.name == "id":
+						attr = attrs.item(i)
+						if attr.namespaceURI == "http://template.sesse.net/" and attr.localName == "id":
 							id = attr.value
 							if clean:
 								attrs_to_remove.append(attr.name)
@@ -73,11 +73,12 @@ def process(node, obj, clean = True):
 			frag.appendChild(child)
 
 		for instance in obj:
-			newnode = frag.cloneNode(True)
-			node.appendChild(newnode)
-			process(newnode, instance, clean)
-			if clean:
-				_clean(newnode)
+			if instance is not None:
+				newnode = frag.cloneNode(True)
+				node.appendChild(newnode)
+				process(newnode, instance, clean)
+				if clean:
+					_clean(newnode)
 
 		# remove all the <fragment> tags
 
@@ -99,8 +100,9 @@ def process(node, obj, clean = True):
 def alternate(tag, array, *elems):
 	i = 0
 	for ref in array:
-		ref[tag] = elems[i % len(elems)]
-		i = i + 1
+		if ref is not None:
+			ref[tag] = elems[i % len(elems)]
+			i = i + 1
 		
 	return array