Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ed525fb

Browse files
committed
Fix getElementsByTagNameNS:
- actually return a result - Compare with tag.localName in getElementsByTagNameNSHelper
1 parent 453163d commit ed525fb

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

Lib/xml/dom/minidom.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ def _getElementsByTagNameHelper(parent, name, rc):
279279
def _getElementsByTagNameNSHelper(parent, nsURI, localName, rc):
280280
for node in parent.childNodes:
281281
if node.nodeType == Node.ELEMENT_NODE:
282-
if ((localName == "*" or node.tagName == localName) and
282+
if ((localName == "*" or node.localName == localName) and
283283
(nsURI == "*" or node.namespaceURI == nsURI)):
284284
rc.append(node)
285285
_getElementsByTagNameNSHelper(node, nsURI, localName, rc)
@@ -551,7 +551,9 @@ def getElementsByTagName(self, name):
551551
return _getElementsByTagNameHelper(self, name, [])
552552

553553
def getElementsByTagNameNS(self, namespaceURI, localName):
554-
_getElementsByTagNameNSHelper(self, namespaceURI, localName, [])
554+
rc = []
555+
_getElementsByTagNameNSHelper(self, namespaceURI, localName, rc)
556+
return rc
555557

556558
def __repr__(self):
557559
return "<DOM Element: %s at %s>" % (self.tagName, id(self))
@@ -880,7 +882,9 @@ def createAttributeNS(self, namespaceURI, qualifiedName):
880882
return a
881883

882884
def getElementsByTagNameNS(self, namespaceURI, localName):
883-
_getElementsByTagNameNSHelper(self, namespaceURI, localName)
885+
rc = []
886+
_getElementsByTagNameNSHelper(self, namespaceURI, localName, rc)
887+
return rc
884888

885889
def getElementsByTagName(self, name):
886890
rc = []

0 commit comments

Comments
 (0)