Review: Needs Fixing

The name is mandatory, so you can simplify this:
    ${line.name and line.name.replace('\n','<br/>') or '' | n}
by that:
    ${line.name.replace('\n','<br/>') | n}
-- 
https://code.launchpad.net/~camptocamp/purchase-report/purchase_report_carriage_return_vre/+merge/214030
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to     : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp

Reply via email to