From 795ad56b319daa610766327b99a74e5ba1ae41b2 Mon Sep 17 00:00:00 2001 From: Guido van Rossum Date: Mon, 5 Feb 2001 18:50:15 +0000 Subject: [PATCH] Don't get fooled by an empty prefix with a valid namespaceURI -- in this case, the code used to generate invalid tags and attribute names with a leading colon, e.g. <:tag> or . --- Lib/xml/dom/pulldom.py | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/Lib/xml/dom/pulldom.py b/Lib/xml/dom/pulldom.py index 66794f34e3f..c400742e423 100644 --- 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)