From 75a5931891e49e2304e981275c6a1b66f466e1df Mon Sep 17 00:00:00 2001
From: sistason <c3infra@sistason.de>
Date: Sun, 14 Feb 2021 00:25:20 +0000
Subject: [PATCH] fixes #40 by recalculating groups _before_ to_ldap(), which
 sets the groups in ldap

---
 uffd/user/views_user.py | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/uffd/user/views_user.py b/uffd/user/views_user.py
index 14fcb690..77488a22 100644
--- a/uffd/user/views_user.py
+++ b/uffd/user/views_user.py
@@ -155,16 +155,11 @@ def csvimport():
 						continue
 					role.add_member(newuser)
 
+			recalculate_user_groups(newuser)
+
 			result = newuser.to_ldap(new=True)
-			print(result)
 			if result:
 				send_passwordreset(newuser.loginname, new=True)
-
-				usergroups = set()
-				for role in Role.get_for_user(newuser).all():
-					usergroups.update(role.group_dns())
-				newuser.replace_group_dns(usergroups)
-
 				session.commit()
 				usersadded += 1
 			else:
-- 
GitLab