make anonymous field not required, bugfix

MayOneUS · Jun 17, 2014 · fe17c2d · fe17c2d
1 parent 74f0b78
commit fe17c2d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/backend/handlers.py b/backend/handlers.py
@@ -81,7 +81,7 @@ class PledgeHandler(webapp2.RequestHandler):
       employer=_STR,
       target=_STR,
       subscribe=dict(type='boolean'),
-      anonymous=dict(type='boolean'),
+      anonymous=dict(type='boolean', required=False),
       amountCents=dict(type='integer', minimum=100),
       pledgeType=dict(enum=model.Pledge.TYPE_VALUES, required=False),
       team=dict(type='string', blank=True),
@@ -162,7 +162,7 @@ def post(self):
                                'pledgeType', model.Pledge.TYPE_CONDITIONAL),
                              team=data['team'],
                              mail_list_optin=data['subscribe'],
-                             anonymous=data['anonymous'])
+                             anonymous=data.get('anonymous', False))
 
     if data['subscribe']:
       env.mailing_list_subscriber.Subscribe(

diff --git a/backend/model.py b/backend/model.py
@@ -227,7 +227,8 @@ def create(email, stripe_customer_id, stripe_charge_id,
                     url_nonce=os.urandom(32).encode("hex"),
                     anonymous=anonymous)
     pledge.put()
-    TeamTotal.add(team, amount_cents)
+    if team:
+      TeamTotal.add(team, amount_cents)
     return pledge