Roberto, thanks for your changes.
I think using fiscal positions is a better choice.
Some remarks:
line 39: 'for partner_id in partner_obj.browse(cr, uid, [partner_id])' can just be 'partner = partner_obj.browse(cr, uid, partner_id, context=context)'
line 41: if 'property_account_position' was not set, partner.property_account_position.default_has_vat_on_payment would raise exception
line 89, I think the _defaults is unnecessary
Roberto, thanks for your changes.
I think using fiscal positions is a better choice.
Some remarks: obj.browse( cr, uid, [partner_id])' can just be 'partner = partner_ obj.browse( cr, uid, partner_id, context=context)' account_ position' was not set, partner. property_ account_ position. default_ has_vat_ on_payment would raise exception
line 39: 'for partner_id in partner_
line 41: if 'property_
line 89, I think the _defaults is unnecessary