search

OCA / odoo-pre-commit-hooks

Linters of Odoo addons that complement pylint-odoo
GNU Affero General Public License v3.0
10 stars 12 forks source link

Add column number to xml checks #64

Closed moylop260 closed 1 year ago

moylop260 commented 1 year ago

lxml supports xml_node.sourceline but what about column number?

R&D - Check if it is possible to add it

antonag32 commented 1 year ago

I looked around the entire documentation, google and mailing list for LXML. Column number sometimes appear on exceptions but apparently there is no API to access it so I think we should close this for now, it seems like it is not possible (easily).

Subclassing the XML parser to access internals is also not as straightforward since it is all Cython,