mirror of
https://github.com/freedombox/FreedomBox.git
synced 2026-04-29 10:10:19 +00:00
groups: Don't duplicate checking of group existence
Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
This commit is contained in:
parent
0dea420ffe
commit
0609f6a107
@ -109,7 +109,6 @@ class CreateUserForm(ValidNewUsernameCheckMixin, UserCreationForm):
|
|||||||
|
|
||||||
for group in self.cleaned_data['groups']:
|
for group in self.cleaned_data['groups']:
|
||||||
try:
|
try:
|
||||||
users.create_group(group)
|
|
||||||
actions.superuser_run('users', [
|
actions.superuser_run('users', [
|
||||||
'add-user-to-group',
|
'add-user-to-group',
|
||||||
user.get_username(), group
|
user.get_username(), group
|
||||||
@ -206,7 +205,6 @@ class UserUpdateForm(ValidNewUsernameCheckMixin, forms.ModelForm):
|
|||||||
for new_group in new_groups:
|
for new_group in new_groups:
|
||||||
if new_group not in old_groups:
|
if new_group not in old_groups:
|
||||||
try:
|
try:
|
||||||
users.create_group(new_group)
|
|
||||||
actions.superuser_run('users', [
|
actions.superuser_run('users', [
|
||||||
'add-user-to-group',
|
'add-user-to-group',
|
||||||
user.get_username(), new_group
|
user.get_username(), new_group
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user