From 80c7c1c9fc31f3381b1540e4724a09ed7ae2e108 Mon Sep 17 00:00:00 2001 From: Jared Kipe Date: Sat, 13 Jul 2019 08:04:34 -0700 Subject: [PATCH] FIX `sale_planner` accidental recursion when creating a FakePartner from a dictionary withou 'date_localization' --- sale_planner/wizard/order_planner.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/sale_planner/wizard/order_planner.py b/sale_planner/wizard/order_planner.py index 75c81fca..54b654fa 100644 --- a/sale_planner/wizard/order_planner.py +++ b/sale_planner/wizard/order_planner.py @@ -50,13 +50,14 @@ class FakePartner(): self.partner_latitude = 0.0 self.partner_longitude = 0.0 self.is_company = False + self._date_localization = kwargs.pop('date_localization', False) for attr, value in kwargs.items(): setattr(self, attr, value) @property def date_localization(self): - if not hasattr(self, 'date_localization') and self.date_localization: - self.date_localization = 'TODAY!' + if not self._date_localization: + self._date_localization = 'TODAY!' # The fast way. if SearchEngine and self.zip: with SearchEngine() as search: @@ -64,7 +65,7 @@ class FakePartner(): if zipcode and zipcode.lat: self.partner_latitude = zipcode.lat self.partner_longitude = zipcode.lng - return self.date_localization + return self._date_localization # The slow way. result = geo_find(geo_query_address( @@ -76,7 +77,7 @@ class FakePartner(): self.partner_latitude = result[0] self.partner_longitude = result[1] - return self.date_localization + return self._date_localization class FakeOrderLine():