Produits Entr'ouvert » Passerelle

Il y a pas moyen de circonscrire ça au connecteur concerné ?

Ou une version de pyexcel-xls ou xlrd qui corrigerait ?

Là ça fait un peu banzai sur tout la satck python.

Malheureusement on utilise pyexcel-xls qui est limité à la version < 2 de xlrd :

Requires-Dist: pyexcel-io (>=0.6.2)
Requires-Dist: xlrd (<2)
Requires-Dist: xlwt

T'as le choix soit on se prive de pyexcel-xls soit d'un xlrd compatible py39 :

• xlrd changelog 2.x.x : https://github.com/python-excel/xlrd/blob/master/CHANGELOG.rst#200-11-december-2020
• pyexcel-xls, souci avec xlrd 2.x.x : https://github.com/pyexcel/pyexcel-xls/pull/41 https://github.com/pyexcel/pyexcel-xls/issues/45

Le truc c'est que xlrd a abandonné le support de OpenXML (.xlsx) et conseille d'utiliser openpyxl dans ce cas mais pyexcel-xls n'a pas fait la migration.

Je comprends pas pourquoi ça marche pas en fait, il y a un commit dans xlrd qui essaie de gérer ça https://github.com/python-excel/xlrd/commit/93673f98637359f1be41181e7d9ca00f48c99624.

Tu pourrais mettre la trace ?

Emmanuel Cazenave a écrit :

Je comprends pas pourquoi ça marche pas en fait, il y a un commit dans xlrd qui essaie de gérer ça https://github.com/python-excel/xlrd/commit/93673f98637359f1be41181e7d9ca00f48c99624.

C'est un commit de xlrd 2.x.x, je t'ai pointé la contrainte de pyexcel-xls qui dit xlrd<2. Comment veux-tu installer xlrd>=2 si pyexcel-xls l'interdit (à fortiori à raison) ? Pour information (c'est peut-être l'information qui te manque) on n'utilise pas xlrd directement dans passerelle, c'est une dépendance de pyexcel-xls.

Tu pourrais mettre la trace ?

Ben non j'ai expliqué déjà.

PS: le problème vient bien de l'absence de getiterator() utilisé par xlrd<2, et sur python>3.7 mais vu le patch et le commentaire qu'il contient ça me paraissait évident.

Benjamin Dauvergne a écrit :

C'est un commit de xlrd 2.x.x,

Non, 0.9.4.

Emmanuel Cazenave a écrit :

Benjamin Dauvergne a écrit :

C'est un commit de xlrd 2.x.x,

Non, 0.9.4.

Ok et bien leur commit ne marche pas :

self = <xlrd.xlsx.X12Book object at 0x7f32542c1280>, stream = <zipfile.ZipExtFile name='xl/workbook.xml' mode='r' compress_type=deflate>, heading = 'Workbook'

    def process_stream(self, stream, heading=None):
        if self.verbosity >= 2 and heading is not None:
            fprintf(self.logfile, "\n=== %s ===\n", heading)
        self.tree = ET.parse(stream)
        getmethod = self.tag2meth.get
>       for elem in self.tree.iter() if Element_has_iter else self.tree.getiterator():
E       AttributeError: 'ElementTree' object has no attribute 'getiterator'

/tmp/tox-bdauvergne/passerelle/py3-django111-coverage/lib/python3.9/site-packages/xlrd/xlsx.py:266: AttributeError
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> entering PDB >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>

>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> PDB post_mortem (IO-capturing turned off) >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
> /tmp/tox-bdauvergne/passerelle/py3-django111-coverage/lib/python3.9/site-packages/xlrd/xlsx.py(266)process_stream()
-> for elem in self.tree.iter() if Element_has_iter else self.tree.getiterator():
(Pdb) Element_has_iter
False
(Pdb) 

et ça ne marche parce que ça utilise finalement defusedxml au lieu du ElementTree natif :

>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> PDB set_trace (IO-capturing turned off) >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
> /tmp/tox-bdauvergne/passerelle/py3-django111-coverage/lib/python3.9/site-packages/xlrd/xlsx.py(61)ensure_elementtree_imported()
-> Element_has_iter = hasattr(ET, 'ElementTree') and hasattr(ET.ElementTree, 'iter')
(Pdb) hasattr(ET, 'ElementTree')
False
(Pdb) ET.Element
*** AttributeError: module 'defusedxml.cElementTree' has no attribute 'Element'
(Pdb) dir(ET)
['DefusedXMLParser', 'XML', 'XMLParse', 'XMLParser', 'XMLTreeBuilder', '_TreeBuilder', '__all__', '__builtins__', '__cached__', '__doc__', '__file__', '__loader__', '__name__', '__origin__', '__package__', '__spec__', '_generate_etree_functions', '_iterparse', '_parse', 'absolute_import', 'fromstring', 'iterparse', 'parse', 'tostring']

Et je vais arrêter là mes investigations parce que le code est juste affreux.