Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-834004-ksa

its very strange that we need to use to_xml 2 times on the values
and the report is being correctly printed with & only if we have it like "&",
we need to find whats the problem,

also I don't like the variable names,
uom_id is actually name of uom, please change it,
also do the same changes on other variables in it where you find its not understandable properly

thanks