Merge lp:~camptocamp/account-invoice-report/7.0-fix-1188224 into lp:~account-core-editors/account-invoice-report/7.0

Proposed by Yannick Vaucher @ Camptocamp
Status: Merged
Merged at revision: 34
Proposed branch: lp:~camptocamp/account-invoice-report/7.0-fix-1188224
Merge into: lp:~account-core-editors/account-invoice-report/7.0
Diff against target: 31 lines (+9/-2)
1 file modified
invoice_webkit/report/account_invoice.mako (+9/-2)
To merge this branch: bzr merge lp:~camptocamp/account-invoice-report/7.0-fix-1188224
Reviewer Review Type Date Requested Status
Alexandre Fayolle - camptocamp code review, test Approve
Guewen Baconnier @ Camptocamp Approve
Review via email: mp+167781@code.launchpad.net

Description of the change

Ensure we remove only the name and not the street from the formatted address.

To post a comment you must log in.
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

LGTM

review: Approve
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

works fine

review: Approve (code review, test)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'invoice_webkit/report/account_invoice.mako'
--- invoice_webkit/report/account_invoice.mako 2013-05-27 07:52:26 +0000
+++ invoice_webkit/report/account_invoice.mako 2013-06-06 15:19:24 +0000
@@ -178,7 +178,12 @@
178 %else:178 %else:
179 <tr><td class="name">${inv.partner_id.title and inv.partner_id.title.name or ''} ${inv.partner_id.name }</td></tr>179 <tr><td class="name">${inv.partner_id.title and inv.partner_id.title.name or ''} ${inv.partner_id.name }</td></tr>
180 %endif180 %endif
181 %for part in inv.partner_id.contact_address.split("\n")[1:]:181 %if inv.partner_id.parent_id:
182 <% address_lines = inv.partner_id.contact_address.split("\n")[1:] %>
183 %else:
184 <% address_lines = inv.partner_id.contact_address.split("\n") %>
185 %endif
186 %for part in address_lines:
182 %if part:187 %if part:
183 <tr><td>${part}</td></tr>188 <tr><td>${part}</td></tr>
184 %endif189 %endif
@@ -189,10 +194,12 @@
189 %if inv.partner_id.parent_id:194 %if inv.partner_id.parent_id:
190 <tr><td class="name">${inv.partner_id.parent_id.name or ''}</td></tr>195 <tr><td class="name">${inv.partner_id.parent_id.name or ''}</td></tr>
191 <tr><td>${inv.partner_id.title and inv.partner_id.title.name or ''} ${inv.partner_id.name }</td></tr>196 <tr><td>${inv.partner_id.title and inv.partner_id.title.name or ''} ${inv.partner_id.name }</td></tr>
197 <% address_lines = inv.partner_id.contact_address.split("\n")[1:] %>
192 %else:198 %else:
193 <tr><td class="name">${inv.partner_id.title and inv.partner_id.title.name or ''} ${inv.partner_id.name }</td></tr>199 <tr><td class="name">${inv.partner_id.title and inv.partner_id.title.name or ''} ${inv.partner_id.name }</td></tr>
200 <% address_lines = inv.partner_id.contact_address.split("\n") %>
194 %endif201 %endif
195 %for part in inv.partner_id.contact_address.split("\n")[1:]:202 %for part in address_lines:
196 %if part:203 %if part:
197 <tr><td>${part}</td></tr>204 <tr><td>${part}</td></tr>
198 %endif205 %endif

Subscribers

People subscribed via source and target branches