use .get for has_msrp
This commit is contained in:
1 parent
1f5224a92c
commit
6c309b1e62
1 file changed
+6
-2
@@ -169,7 +169,11 @@ def build_product_details(
|
|||||||
original_price = product["originalPrice"]
|
original_price = product["originalPrice"]
|
||||||
|
|
||||||
percentage = 0
|
percentage = 0
|
||||||
has_msrp = product["hasMsrp"]
|
# Defaulted, not indexed: documents written before the field existed have no
|
||||||
|
# hasMsrp at all, and stale ones that drop out of a monitor's feed are never
|
||||||
|
# re-upserted, so they would KeyError here forever. True keeps them on the
|
||||||
|
# pre-existing threshold path rather than silently routing them as MSRP-less.
|
||||||
|
has_msrp = product.get("hasMsrp", True)
|
||||||
if has_msrp:
|
if has_msrp:
|
||||||
if not original_price:
|
if not original_price:
|
||||||
LOGGER.warning(
|
LOGGER.warning(
|
||||||
@@ -218,7 +222,7 @@ def build_product_details(
|
|||||||
original_price=original_price,
|
original_price=original_price,
|
||||||
price_before_drop=product["priceBeforeDrop"],
|
price_before_drop=product["priceBeforeDrop"],
|
||||||
edu_price=product["eduPrice"],
|
edu_price=product["eduPrice"],
|
||||||
past_prices=product["pastPrices"],
|
past_prices=product.get("pastPrices"),
|
||||||
has_msrp=has_msrp,
|
has_msrp=has_msrp,
|
||||||
),
|
),
|
||||||
availability=AvailabilityInfo(
|
availability=AvailabilityInfo(
|
||||||
|
|||||||
Reference in new issue
Block a user