[Python-checkins] bpo-43399: Fix ElementTree.extend not working on iterators (GH-24751)

serhiy-storchaka webhook-mailer at python.org
Tue Mar 30 17:11:50 EDT 2021


https://github.com/python/cpython/commit/51a85ddce8b336addcb61b96f04c9c5edef07296
commit: 51a85ddce8b336addcb61b96f04c9c5edef07296
branch: master
author: Alex Prengère <2138730+alexprengere at users.noreply.github.com>
committer: serhiy-storchaka <storchaka at gmail.com>
date: 2021-03-31T00:11:29+03:00
summary:

bpo-43399: Fix ElementTree.extend not working on iterators (GH-24751)

files:
A Misc/NEWS.d/next/Library/2021-03-04-17-53-46.bpo-43399.Wn95u-.rst
M Lib/test/test_xml_etree.py
M Lib/xml/etree/ElementTree.py
M Misc/ACKS

diff --git a/Lib/test/test_xml_etree.py b/Lib/test/test_xml_etree.py
index fcb1f7fdfbbde..553529a300170 100644
--- a/Lib/test/test_xml_etree.py
+++ b/Lib/test/test_xml_etree.py
@@ -330,6 +330,9 @@ def test_simpleops(self):
         elem.extend([e])
         self.serialize_check(elem, '<body><tag /><tag2 /></body>')
         elem.remove(e)
+        elem.extend(iter([e]))
+        self.serialize_check(elem, '<body><tag /><tag2 /></body>')
+        elem.remove(e)
 
         element = ET.Element("tag", key="value")
         self.serialize_check(element, '<tag key="value" />') # 1
diff --git a/Lib/xml/etree/ElementTree.py b/Lib/xml/etree/ElementTree.py
index 168418e466c45..99246800b9866 100644
--- a/Lib/xml/etree/ElementTree.py
+++ b/Lib/xml/etree/ElementTree.py
@@ -252,7 +252,7 @@ def extend(self, elements):
         """
         for element in elements:
             self._assert_is_element(element)
-        self._children.extend(elements)
+            self._children.append(element)
 
     def insert(self, index, subelement):
         """Insert *subelement* at position *index*."""
diff --git a/Misc/ACKS b/Misc/ACKS
index 5d3f75a4165b1..42f0efdd536dd 100644
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -1381,6 +1381,7 @@ Matheus Vieira Portela
 Davin Potts
 Guillaume Pratte
 Florian Preinstorfer
+Alex Prengère
 Amrit Prem
 Paul Prescod
 Donovan Preston
diff --git a/Misc/NEWS.d/next/Library/2021-03-04-17-53-46.bpo-43399.Wn95u-.rst b/Misc/NEWS.d/next/Library/2021-03-04-17-53-46.bpo-43399.Wn95u-.rst
new file mode 100644
index 0000000000000..0b8dffb2312e4
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2021-03-04-17-53-46.bpo-43399.Wn95u-.rst
@@ -0,0 +1,2 @@
+Fix ``ElementTree.extend`` not working on iterators when using the
+Python implementation



More information about the Python-checkins mailing list