Purchase info can change, so do not depend on it being 3 items (#47)

django-oscar · Oct 21, 2024 · 699687a · 699687a
1 parent 17f9b78
commit 699687a
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/oscar_odin/mappings/catalogue.py b/oscar_odin/mappings/catalogue.py
@@ -236,14 +236,14 @@ def map_stock_price(self) -> Tuple[Decimal, str, int, bool]:
         stock_strategy: DefaultStrategy = self.context["stock_strategy"]
 
         if self.source.is_parent:
-            price, availability, _ = stock_strategy.fetch_for_parent(self.source)
+            price_info = stock_strategy.fetch_for_parent(self.source)
         else:
-            price, availability, _ = stock_strategy.fetch_for_product(self.source)
+            price_info = stock_strategy.fetch_for_product(self.source)
         return (
-            getattr(price, "excl_tax", Decimal(0)),
-            getattr(price, "currency", ""),
-            getattr(availability, "num_available", 0),
-            availability.is_available_to_buy,
+            getattr(price_info.price, "excl_tax", Decimal(0)),
+            getattr(price_info.price, "currency", ""),
+            getattr(price_info.availability, "num_available", 0),
+            price_info.availability.is_available_to_buy,
         )