Почему редирект работает в «чистом» виде, но не работает из функции?

Question

[Р.]

Возможно это очень простая, но пока не очевидная для меня вещь. Почему этот код работает...

@login_required
def mem_area(request):
    """ Returns Member Area HTML with data """
    user = request.user

    if user.is_staff:
        return redirect('/')

    user_info = UserInfo.objects.get(user_id=user.id)
    page_title = 'Dashboard'
    return render(request, 'user_resume/member_area/member_area.html', locals())

а этот нет? Под "не работает" имею ввиду, что не происходит редиректа, а админ попадает туда, куда не следует.

def admin_redirect(user):
    """ Checks if user is admin and don't let him open member area  """
    if user.is_staff:
        return redirect('/')

@login_required
def mem_area(request):
    """ Returns Member Area HTML with data """
    user = request.user

    admin_redirect(user)

    user_info = UserInfo.objects.get(user_id=user.id)
    page_title = 'Dashboard'
    return render(request, 'user_resume/member_area/member_area.html', locals())

Разница лишь в том, что я вынес проверку "на админа" и редирект в отдельную функцию.
Если я меняю return на raise Http404, то все ок, но мне нужен редирект

Answer 1

[Артём Белоусов]

Потому что объект redirect возвращается в поле видимости вашего view, но сам view его не возвращает.
Исключение же прерывает все функции и возвращает 404

Comments

[Р.]

Артём Белоусов как лучше поступить в данное ситуации? просто копипастить этот код 5 раз в каждое вью?

Answer 2

[Astrohas]

def admin_redirect(user):
    """ Checks if user is admin and don't let him open member area  """
    if user.is_staff:
        return redirect('/')

@login_required
def mem_area(request):
    """ Returns Member Area HTML with data """
    user = request.user

    redirect = admin_redirect(user)
    if redirect:
      return redirect
    user_info = UserInfo.objects.get(user_id=user.id)
    page_title = 'Dashboard'
    return render(request, 'user_resume/member_area/member_area.html', locals())

Comments

[Артём Белоусов]

Поздравляю, вы сэкономили ему целых 0 строк

[Astrohas]

Артём Белоусов: знаю быдлокод, но что просили то я и сделал