mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-08-03 16:39:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

Issue #17901: fix TreeBuilder construction for an explicit element_factory=None

Browse source 
Based on report and patch by Aaron Oakley.
This commit is contained in:
Eli Bendersky 2013-05-18 15:47:16 -07:00
parent 90a24270f8
commit 08231a9c6a
3 changed files with 8 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
Lib/test/test_xml_etree.py
View file

@ -1762,6 +1762,12 @@ class TreeBuilderTest(unittest.TestCase):
parser.feed(self.sample1)
self.assertIsNone(parser.close())
def test_treebuilder_elementfactory_none(self):
parser = ET.XMLParser(target=ET.TreeBuilder(element_factory=None))
parser.feed(self.sample1)
e = parser.close()
self._check_sample1_element(e)
def test_subclass(self):
class MyTreeBuilder(ET.TreeBuilder):
def foobar(self, x):

1
Misc/ACKS
View file

@ -887,6 +887,7 @@ John O'Connor
Kevin O'Connor
Tim O'Malley
Zooko O'Whielacronx
Aaron Oakley
James Oakley
Jon Oberheide
Pascal Oberndoerfer

2
Modules/_elementtree.c
View file

@ -2381,7 +2381,7 @@ treebuilder_handle_start(TreeBuilderObject* self, PyObject* tag,
self->data = NULL;
}
if (self->element_factory) {
if (self->element_factory && self->element_factory != Py_None) {
node = PyObject_CallFunction(self->element_factory, "OO", tag, attrib);
} else {
node = create_new_element(tag, attrib);