Fix broken behavior when removing all auxiliary groups from a user

(e.g. 'groups=' in the user module).
This commit is contained in:
Sean Chittenden 2014-02-10 13:08:53 -08:00
parent 7cf513d4ed
commit c892f51002

View file

@ -443,7 +443,7 @@ class User(object):
if self.groups is None:
return None
info = self.user_info()
groups = set(self.groups.split(','))
groups = set(filter(None, self.groups.split(',')))
for g in set(groups):
if not self.group_exists(g):
self.module.fail_json(msg="Group %s does not exist" % (g))