diff --git a/allianceauth/services/modules/discord/tasks.py b/allianceauth/services/modules/discord/tasks.py index 93ccc482..9fa1a5b1 100644 --- a/allianceauth/services/modules/discord/tasks.py +++ b/allianceauth/services/modules/discord/tasks.py @@ -62,12 +62,9 @@ class DiscordTasks: user = User.objects.get(pk=pk) logger.debug("Updating discord groups for user %s" % user) if DiscordTasks.has_account(user): - groups = [] + groups = [user.profile.state.name] for group in user.groups.all(): groups.append(str(group.name)) - if len(groups) == 0: - logger.debug("No syncgroups found for user. Adding empty group.") - groups.append('empty') logger.debug("Updating user %s discord groups to %s" % (user, groups)) try: DiscordOAuthManager.update_groups(user.discord.uid, groups) diff --git a/allianceauth/services/modules/discourse/manager.py b/allianceauth/services/modules/discourse/manager.py index ae2a1af6..1d98475d 100644 --- a/allianceauth/services/modules/discourse/manager.py +++ b/allianceauth/services/modules/discourse/manager.py @@ -345,7 +345,7 @@ class DiscourseManager: @staticmethod def update_groups(user): - groups = [] + groups = [DiscourseManager._sanitize_groupname(user.profile.state.name)] for g in user.groups.all(): groups.append(DiscourseManager._sanitize_groupname(str(g))) logger.debug("Updating discourse user %s groups to %s" % (user, groups)) diff --git a/allianceauth/services/modules/mumble/models.py b/allianceauth/services/modules/mumble/models.py index 73f8f1fb..221c63db 100644 --- a/allianceauth/services/modules/mumble/models.py +++ b/allianceauth/services/modules/mumble/models.py @@ -82,11 +82,9 @@ class MumbleUser(AbstractServiceModel): def update_groups(self, groups: Group=None): if groups is None: groups = self.user.groups.all() - groups_str = [] + groups_str = [self.user.profile.state.name] for group in groups: groups_str.append(str(group.name)) - if len(groups) == 0: - groups_str.append('empty') safe_groups = ','.join(set([g.replace(' ', '-') for g in groups_str])) logger.info("Updating mumble user {} groups to {}".format(self.user, safe_groups)) self.groups = safe_groups diff --git a/allianceauth/services/modules/openfire/tasks.py b/allianceauth/services/modules/openfire/tasks.py index fef2d072..4d6bcf8d 100644 --- a/allianceauth/services/modules/openfire/tasks.py +++ b/allianceauth/services/modules/openfire/tasks.py @@ -45,11 +45,9 @@ class OpenfireTasks: user = User.objects.get(pk=pk) logger.debug("Updating jabber groups for user %s" % user) if OpenfireTasks.has_account(user): - groups = [] + groups = [user.profile.state.name] for group in user.groups.all(): groups.append(str(group.name)) - if len(groups) == 0: - groups.append('empty') logger.debug("Updating user %s jabber groups to %s" % (user, groups)) try: OpenfireManager.update_user_groups(user.openfire.username, groups) diff --git a/allianceauth/services/modules/phpbb3/tasks.py b/allianceauth/services/modules/phpbb3/tasks.py index b6224ca8..107ead86 100644 --- a/allianceauth/services/modules/phpbb3/tasks.py +++ b/allianceauth/services/modules/phpbb3/tasks.py @@ -40,11 +40,9 @@ class Phpbb3Tasks: user = User.objects.get(pk=pk) logger.debug("Updating phpbb3 groups for user %s" % user) if Phpbb3Tasks.has_account(user): - groups = [] + groups = [user.profile.state.name] for group in user.groups.all(): groups.append(str(group.name)) - if len(groups) == 0: - groups.append('empty') logger.debug("Updating user %s phpbb3 groups to %s" % (user, groups)) try: Phpbb3Manager.update_groups(user.phpbb3.username, groups) diff --git a/allianceauth/services/modules/seat/tasks.py b/allianceauth/services/modules/seat/tasks.py index e0fffa57..0ed4e2a1 100644 --- a/allianceauth/services/modules/seat/tasks.py +++ b/allianceauth/services/modules/seat/tasks.py @@ -38,13 +38,10 @@ class SeatTasks: def update_roles(self, pk): user = User.objects.get(pk=pk) logger.debug("Updating SeAT roles for user %s" % user) - groups = [] if SeatTasks.has_account(user): + groups = [user.profile.state.name] for group in user.groups.all(): groups.append(str(group.name)) - if len(groups) == 0: - logger.debug("No syncgroups found for user. Adding empty group.") - groups.append('empty') logger.debug("Updating user %s SeAT roles to %s" % (user, groups)) try: SeatManager.update_roles(user.seat.username, groups) diff --git a/allianceauth/services/modules/smf/tasks.py b/allianceauth/services/modules/smf/tasks.py index e31ee944..91b45732 100644 --- a/allianceauth/services/modules/smf/tasks.py +++ b/allianceauth/services/modules/smf/tasks.py @@ -44,11 +44,9 @@ class SmfTasks: user = User.objects.get(pk=pk) logger.debug("Updating smf groups for user %s" % user) if SmfTasks.has_account(user): - groups = [] + groups = [user.profile.state.name] for group in user.groups.all(): groups.append(str(group.name)) - if len(groups) == 0: - groups.append('empty') logger.debug("Updating user %s smf groups to %s" % (user, groups)) try: SmfManager.update_groups(user.smf.username, groups)