Review: Needs Fixing

In parent class, on account/invoice.py file, onchange_partner_id method 
signature is like this:

def onchange_partner_id(self, cr, uid, ids, type, partner_id,\
            date_invoice=False, payment_term=False, partner_bank_id=False, 
company_id=False):

I think it is better to not forget the company_id parameter in the super() 
function, and wrote:

result =  super(account_invoice, self).onchange_partner_id(cr, uid, ids, type, 
partner_id,
            date_invoice=date_invoice, payment_term=payment_term, 
partner_bank_id=partner_bank_id, company_id=company_id)

What do you think about this ?
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-381849-ado/+merge/90088
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-381849-ado.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to