Commit a69982ab by Oláh István Gergely

dashboard: cleanup and fix adding users to group

parent e8c93525
...@@ -600,7 +600,15 @@ class GroupDetailView(CheckedDetailView): ...@@ -600,7 +600,15 @@ class GroupDetailView(CheckedDetailView):
def __add_user(self, request): def __add_user(self, request):
self.object = self.get_object() self.object = self.get_object()
self.add_levels(request) name = request.POST['list-new-name']
if not name:
return
try:
entity = User.objects.get(username=name)
except User.DoesNotExist:
warning(request, _('User "%s" not found.') % name)
return
self.object.user_set.add(entity)
return redirect(reverse_lazy("dashboard.views.group-detail", return redirect(reverse_lazy("dashboard.views.group-detail",
kwargs={'pk': self.object.pk})) kwargs={'pk': self.object.pk}))
...@@ -626,18 +634,6 @@ class GroupDetailView(CheckedDetailView): ...@@ -626,18 +634,6 @@ class GroupDetailView(CheckedDetailView):
return redirect(reverse_lazy("dashboard.views.group-detail", return redirect(reverse_lazy("dashboard.views.group-detail",
kwargs={'pk': self.object.pk})) kwargs={'pk': self.object.pk}))
def add_levels(self, request):
name = request.POST['list-new-name']
if not name:
return
try:
entity = User.objects.get(username=name)
except User.DoesNotExist:
warning(request, _('User "%s" not found.') % name)
return
self.object.user_set.add(entity)
class AclUpdateView(LoginRequiredMixin, View, SingleObjectMixin): class AclUpdateView(LoginRequiredMixin, View, SingleObjectMixin):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment