From a2dfa2b13942d775d5f1436086a30b2c3076babf Mon Sep 17 00:00:00 2001
From: frekk <frekk@ucc.asn.au>
Date: Mon, 28 Jan 2019 14:08:10 +0800
Subject: [PATCH] fix get_membership_choices

---
 src/memberdb/models.py | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/memberdb/models.py b/src/memberdb/models.py
index 87cb6ba..8dd7e38 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:
-- 
GitLab