Code review comment for lp:~camptocamp/account-invoice-report/semantic-fix

Why do you need to display the commercial_partner_id?

I thought that the normal behavior was to display the contact, which should have the correct name and address related to its company.

In the normal report of the 'account' module, the displayed address is only:

          <para style="terp_default_8">[[ (o.partner_id and o.partner_id.title and o.partner_id.title.name) or '' ]] [[ (o.partner_id and o.partner_id.name) or '' ]]</para>
          <para style="terp_default_8">[[ display_address(o.partner_id) ]]</para>

So what is the rationale for this?

Thanks