Vauxoo / odoo

Fork of Odoo (formerly OpenERP). [This project is not publically mantained just born for internal usage with some little patches] go to official repository on github.com/odoo/odoo
https://www.odoo.com
Other
9 stars 9 forks source link

[IMP] account: set element names for easier xpaths #387

Closed R4Alex closed 3 years ago

R4Alex commented 3 years ago

Description of the issue/feature this PR addresses: Make it easier to xpath to an element in the invoice report adding a name.

Current behavior before PR:

Some elements are not easy to reach with an xpath and are more risky then they should be. There where two elements with the name note, which is no longer the case after this PR. Desired behavior after PR is merged: You can do an xpath to the element itself which is safer and cleaner.

E.g: <xpath expr=//p[@name="payment_communication"]" position="replace">...

-- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr