From dde79105873eb908ca032b9c29d4e4a3d2614443 Mon Sep 17 00:00:00 2001
From: Lukas Burgey <ubedv@student.kit.edu>
Date: Tue, 3 Jul 2018 18:32:03 +0200
Subject: [PATCH] Change the update process of the userinfo

---
 django_backend/backend/models.py | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/django_backend/backend/models.py b/django_backend/backend/models.py
index 91e2aa9..9419b5c 100644
--- a/django_backend/backend/models.py
+++ b/django_backend/backend/models.py
@@ -293,10 +293,8 @@ class User(AbstractUser):
             return Exception('Two user instances with same subject from the same idp')
 
         user = query.first()
-        # update the users userinfo
-        user.userinfo = userinfo
+        user.update_userinfo(userinfo)
         user.save()
-
         return user
 
     @classmethod
@@ -309,6 +307,7 @@ class User(AbstractUser):
 
         if 'email' not in userinfo:
             username = sub
+            email = ''
         else:
             username = userinfo['email']
             email = userinfo['email']
@@ -319,15 +318,10 @@ class User(AbstractUser):
             sub=sub,
             email=email,
             idp=idp,
-            userinfo=userinfo,
         )
+        user.update_userinfo(userinfo)
         user.save()
 
-        for group in idp.get_user_groupinformation(
-                userinfo,
-        ).all():
-            group.users.add(user)
-
         return user
 
     @classmethod
-- 
GitLab