Доступ к различным частям сайта пользователям из разных групп?

Question

[Андрей Шубин]

Хочу дать пользователям из разных групп доступ только в их часть сайта. Авторизацию делаю таким образом:

def signin(request):
    if request.method == 'GET':
        return render(request, 'site/login.html')
    elif request.method == 'POST':
        username = request.POST['username']
        password = request.POST['password']
        user = authenticate(username=username, password=password)
        if user is not None:
            if user.is_active:
                login(request, user)
                if user.groups.filter(name__in=['Admins']) or user.is_superuser:
                    return HttpResponseRedirect('/cp/')
                elif user.groups.filter(name__in=['Counteragents']):
                    return HttpResponseRedirect('/counteragent/')
                elif user.groups.filter(name__in=['Clients']):
                    return HttpResponseRedirect('/client/')
            else:
                pass
        else:
            pass

Затем на вьюшках проверяю пренадлежность пользователя к нужной группе:

def admin_check(user):
    if user.groups.filter(name__in=['Admins']) or user.is_superuser:
        return True
    else:
        return False

@user_passes_test(admin_check)
def admin_home(request):
    return render(request, 'admin/home.html')

Есть ли лучшее решение данного вопроса? Может такой подход вообще никуда не годится?

Answer 1

[Roman K]

Permissions

Comments

[Андрей Шубин]

Группы и содержат набор Permissions. Может вы что-то конкретное имели в виду?

[Roman K]

Андрей Шубин: Да. Проверять permission'ы, а не группы странными конструкциями. У юзера для этого даже метод есть https://docs.djangoproject.com/en/1.9/ref/contrib/...

[Андрей Шубин]

О! Это даже лучше!
Иногда требуется к одной вьюхе дать доступ нескольким группам и я городил такое:

def counteragent_admin_check(user):
    return user.groups.filter(name__in=['Counteragents', 'Admins']) or user.is_superuser

[Андрей Шубин]

Roman Kitaev: Спасибо!