openerp-community-reviewer team mailing list archive

["Vincent Renaville@camptocamp" <vincent.renaville@xxxxxxxxxxxxxx>]

Vincent Renaville@camptocamp has proposed merging lp:~camptocamp/project-reporting/blank_invoice_vre into lp:project-reporting.

Requested reviews:
  Project Core Editors (project-core-editors)

For more details, see:
https://code.launchpad.net/~camptocamp/project-reporting/blank_invoice_vre/+merge/213842

This fix prevent blank invoice wizard to crash because contact_id is not present on V7 project.
-- 
https://code.launchpad.net/~camptocamp/project-reporting/blank_invoice_vre/+merge/213842
Your team Project Core Editors is requested to review the proposed merge of lp:~camptocamp/project-reporting/blank_invoice_vre into lp:project-reporting.

=== modified file 'project_billing_utils/wizard/blank_invoice.py'
--- project_billing_utils/wizard/blank_invoice.py	2013-06-20 16:14:51 +0000
+++ project_billing_utils/wizard/blank_invoice.py	2014-04-02 13:16:22 +0000
@@ -62,24 +62,11 @@
                 pterm_list.sort()
                 date_due = pterm_list[-1]
 
-        if project.contact_id:
-            inv_contact = project.contact_id.id
-        else:
-            inv_contact = partner_obj.address_get(
-                    cr, uid,
-                    [partner.id],
-                    adr_pref=['invoice'])['invoice']
-
         return {
             'name': '%s - %s' % (time.strftime('%D'), project.name),
             'type':'out_invoice',
             'date_due': date_due,
             'partner_id': partner.id,
-            'address_contact_id': partner_obj.address_get(
-                    cr, uid,
-                    [partner.id],
-                    adr_pref=['contact'])['contact'],
-            'address_invoice_id': inv_contact,
             'payment_term': partner.property_payment_term.id or False,
             'account_id': partner.property_account_receivable.id,
             'currency_id': project.pricelist_id.currency_id.id,