From 6a7fbd54cf207f88280f9d906a05e5a589f9bd12 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Thu, 18 Jun 2020 18:27:04 +0200
Subject: [PATCH] Fix registration error

---
 server/registration/tests/test_registration_proxy.py | 6 +++---
 server/registration/view.py                          | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/server/registration/tests/test_registration_proxy.py b/server/registration/tests/test_registration_proxy.py
index e051ccea..4de38221 100644
--- a/server/registration/tests/test_registration_proxy.py
+++ b/server/registration/tests/test_registration_proxy.py
@@ -42,6 +42,7 @@ RESPONSE = {
 }
 
 DATA = {
+    'accepted_terms': True,
     'customer': {
         'firstname': 'Pesche',
         'lastname': 'Zubrüti',
@@ -61,8 +62,7 @@ DATA = {
                 'default_billing': True,
             }
         ],
-        'password': '123454abasfd',
-        'accepted_terms': True
+        'password': '123454abasfd'
     }
 }
 
@@ -88,7 +88,7 @@ class ProxyTest(TestCase):
 
     def test_requires_accepted_terms(self):
 
-        del DATA['customer']['accepted_terms']
+        del DATA['accepted_terms']
 
         response = self.client.post(reverse('api:registration:proxy'), json.dumps(DATA), content_type="application/json")
         self.assertEquals(response.status_code, 400)
diff --git a/server/registration/view.py b/server/registration/view.py
index 36e55e52..88652e68 100644
--- a/server/registration/view.py
+++ b/server/registration/view.py
@@ -21,7 +21,7 @@ class RegistrationProxyView(View):
         hep_client = HepClient()
         data = json.loads(request.body)
 
-        if not self.terms_accepted(data['customer']):
+        if not self.terms_accepted(data):
             return JsonResponse(
                 {
                     'message': 'Sie müssen hier zustimmen, damit Sie sich registrieren können.'
@@ -40,9 +40,9 @@ class RegistrationProxyView(View):
 
         return JsonResponse(response_data)
 
-    def terms_accepted(self, customer):
-        if 'accepted_terms' in customer and customer['accepted_terms']:
-            del customer['accepted_terms']
+    def terms_accepted(self, data):
+        if 'accepted_terms' in data and data['accepted_terms']:
+            del data['accepted_terms']
             return True
 
         return False