← Back to team overview

openerp-community team mailing list archive

Re: [Merge] lp:~akretion-team/openerp-product-attributes/no-stock-dep into lp:openerp-product-attributes

 

I can't say I like this construct of you using your xpath with a translate call in it.
The xpath always works on the default english language because the view has not been translated yet.

It would be simpler and save a little bit of cpu time to just keep your xpath on line 138/139 of the diff in the English version. Have you ever had an issue with it not matching because of translations ?
-- 
https://code.launchpad.net/~akretion-team/openerp-product-attributes/no-stock-dep/+merge/147797
Your team OpenERP Community is subscribed to branch lp:openerp-product-attributes.


Follow ups

References