Vauxoo / pylint-odoo

1 stars 4 forks source link

[ADD] xpath-ref-string: Add check to validate a xpath referenced to string #114

Closed moylop260 closed 4 years ago

moylop260 commented 7 years ago

If odoo use a xml view inherited with:

Then we need a new lint check to avoid this case.

Thanks @oscarolar Reference https://github.com/Vauxoo/abastotal/pull/147/files

NOTE: The version 10.0 has this validation now. TODO: If is a check for just one version maybe is not a good idea but I will let open this issue in order to discuss it

luisg123v commented 4 years ago

I think this issue may be closed now, as this validation is included in Odoo itself from a long time ago, which makes not possible to base xpaths on string attributes.