diff --git a/addons/account/models/partner.py b/addons/account/models/partner.py index 8699dc0614a..75a9b3acdb1 100644 --- a/addons/account/models/partner.py +++ b/addons/account/models/partner.py @@ -183,10 +183,11 @@ class AccountFiscalPosition(models.Model): # This can be easily overridden to apply more complex fiscal rules PartnerObj = self.env['res.partner'] partner = PartnerObj.browse(partner_id) - delivery = PartnerObj.browse(delivery_id) - # If partner and delivery have the same vat prefix, use invoicing - if not delivery or (delivery.vat and partner.vat and delivery.vat[:2] == partner.vat[:2]): + # if no delivery use invoicing + if delivery_id: + delivery = PartnerObj.browse(delivery_id) + else: delivery = partner # partner manually set fiscal position always win diff --git a/addons/account/tests/test_fiscal_position.py b/addons/account/tests/test_fiscal_position.py index 7ff2401a6d5..5bcb87d1aba 100644 --- a/addons/account/tests/test_fiscal_position.py +++ b/addons/account/tests/test_fiscal_position.py @@ -154,15 +154,3 @@ class TestFiscalPosition(common.TransactionCase): mapped_taxes = self.fp2m.map_tax(self.src_tax) self.assertEqual(mapped_taxes, self.dst1_tax | self.dst2_tax) - - def test_30_fp_country_delivery(self): - """ - Customer is in Belgium - Delivery is in France - Check if fiscal position is France - """ - self.george.vat = False - self.assertEqual( - self.fp.get_fiscal_position(self.ben.id, self.george.id).id, - self.fr_b2c.id, - "FR B2C should be set")