search

Vauxoo / odoo

Fork of Odoo (formerly OpenERP). [This project is not publically mantained just born for internal usage with some little patches] go to official repository on github.com/odoo/odoo
https://www.odoo.com
Other
9 stars 9 forks source link

[10.0][base_address_extended] FIX fields in address_format #250

Closed alan196 closed 6 years ago

alan196 commented 6 years ago

This PR fix an issue when you try to use the new fields 'street_name', 'street_number' 'street_number2' in address_format from res.country you get error.

alan196 commented 6 years ago

@nhomar @moylop260 @luistorresm Could you check this PR.

Thanks in advance

luistorresm commented 6 years ago

@alan196

Which is the error?

And how I could reproduce?

Regards

alan196 commented 6 years ago

@luistorresm Here is a video to reproduce this issue. http://youtu.be/p65Orsg6s7A

I also use address widget for custom reports and I get incomplete address

luistorresm commented 6 years ago

@nhomar @moylop260

What do you think about this?

I test in the odoo runbot to v11, and have the same issue.

This module is a copy from Odoo Module in v11, must We report the issue in v11? and apply this fix here?

Regards

nhomar commented 6 years ago

@luistorresm

Yes please... the fix should be applied also on v11 and the patch here.

luistorresm commented 6 years ago

@moylop260

Could you merge this fix, please?

Regards

luistorresm commented 6 years ago

@alan196

Question,

If street is the same that '%s %s %s' % ('street_name', 'street_number' 'street_number2'), why not add only street?