diff --git a/src/memberdb/models.py b/src/memberdb/models.py index 87cb6ba81d0d527ed45f5d0cfa6697cc5a369537..8dd7e3851b07750ce64ce39c297c2ed575385a4d 100644 --- a/src/memberdb/models.py +++ b/src/memberdb/models.py @@ -69,8 +69,12 @@ def get_membership_choices(is_renew=None, get_prices=True): # so this option is only displayed on the renewal form continue else: - price = get_item_price(val['dispense']) - if (get_prices and price is not None): + if get_prices: + price = get_item_price(val['dispense']) + else: + price = None + + if price is not None: desc = "%s ($%1.2f)" % (val['desc'], price / 100.0) choices += [(key, desc)] else: