Hello Ravi Gohil,

Sorry for the way I express myself.

When I said "refactor code for getting default and delivery addresses by using 
the address_get() method", I thought:

Instead of:

addresses_default = [address.id for address in 
order.company_id.partner_id.address if address.type == 'default']
addresses_delivery = [address.id for address in 
order.company_id.partner_id.address if address.type == 'delivery']
reception_address_id = (addresses_delivery and addresses_delivery[0]) or 
(addresses_default and addresses_default[0]) or False

Something like this:

addresses = self.pool.get('res.partner').address_get(cr, uid, 
[order.company_id.partner_id.id], ['default', 'delivery'])
reception_address_id = addresses.get('delivery', False) or 
addresses.get('default', False)

What do you think about this ?

-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-381630-rgo/+merge/90042
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-381630-rgo.

_______________________________________________
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