From 18d63d4717657f4b43a61df2fda016f0a6316a65 Mon Sep 17 00:00:00 2001 From: Nicolas Martinelli Date: Wed, 6 Sep 2017 17:31:02 +0200 Subject: [PATCH] [FIX] delivery: zip code comparison Some countries, such as Canada, are using letters in their zip code. When we check if the zip code is within the accepted range, we should make sure to normalize it by making it case insensitive. opw-767866 --- addons/delivery/models/delivery_carrier.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/delivery/models/delivery_carrier.py b/addons/delivery/models/delivery_carrier.py index 6dbaa4410f3..67d352b77f3 100644 --- a/addons/delivery/models/delivery_carrier.py +++ b/addons/delivery/models/delivery_carrier.py @@ -201,9 +201,9 @@ class DeliveryCarrier(models.Model): return False if self.state_ids and contact.state_id not in self.state_ids: return False - if self.zip_from and (contact.zip or '') < self.zip_from: + if self.zip_from and (contact.zip or '').upper() < self.zip_from.upper(): return False - if self.zip_to and (contact.zip or '') > self.zip_to: + if self.zip_to and (contact.zip or '').upper() > self.zip_to.upper(): return False return self