Don't get fooled by an empty prefix with a valid namespaceURI -- in

gvanrossum · gvanrossum · commit 795ad56b319d · 2001-02-05T18:50:15.000Z
this case, the code used to generate invalid tags and attribute names
with a leading colon, e.g. &lt;:tag&gt; or &lt;tag :attr="foo"&gt;.
diff --git a/Lib/xml/dom/pulldom.py b/Lib/xml/dom/pulldom.py
@@ -56,7 +56,11 @@ def startElementNS(self, name, tagName , attrs):
             # provide us with the original name. If not, create
             # *a* valid tagName from the current context.
             if tagName is None:
-                tagName = self._current_context[uri] + ":" + localname
+                prefix = self._current_context[uri]
+                if prefix:
+                    tagName = prefix + ":" + localname
+                else:
+                    tagName = localname
             node = self.document.createElementNS(uri, tagName)
         else:
             # When the tagname is not prefixed, it just appears as
@@ -66,7 +70,11 @@ def startElementNS(self, name, tagName , attrs):
         for aname,value in attrs.items():
             a_uri, a_localname = aname
             if a_uri:
-                qname = self._current_context[a_uri] + ":" + a_localname
+                prefix = self._current_context[a_uri]
+                if prefix:
+                    qname = prefix + ":" + a_localname
+                else:
+                    qname = a_localname
                 attr = self.document.createAttributeNS(a_uri, qname)
             else:
                 attr = self.document.createAttribute(a_localname)
