How to return from another function in Django View function

Question

I have a function in views.py

def login(request):
   actor = LoginActor(request)
   actor.authenticated_user() # Cannot use return here, this is problematic, we need to redirect here without using a return statement

   ctx = actor.get()

   if request.method == 'POST':
      ctx = actor.post()

      return render(request, 'user/forms/auth.jinja', ctx)
   return render(request, 'user/login.jinja', ctx)

and there is a redirect in authenticated_user() function which is defined as:

def authenticated_user(self):
    if self.request.user and self.request.user.is_authenticated:
        return redirect('home')

How do i return from the initial view without calling a return, basically i want to return the callee function where there is a return in called function I am using Django 2.1 with Python 3.7

blhsing · Accepted Answer · 2018-09-09 06:04:28Z

2

You should leave both rendering and redirection to the view function, and have your utility function authenticated_user return just a Boolean value instead:

def authenticated_user(self):
    return self.request.user and self.request.user.is_authenticated

def login(request):
   actor = LoginActor(request)
   if actor.authenticated_user():
       return redirect('home')

   ctx = actor.get()

   if request.method == 'POST':
      ctx = actor.post()

      return render(request, 'user/forms/auth.jinja', ctx)
   return render(request, 'user/login.jinja', ctx)

answered Sep 9, 2018 at 6:04

blhsing

109k9 gold badges88 silver badges132 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

abdelwaheb moalla · Accepted Answer · 2018-09-09 10:08:50Z

Yo should use classes instead of functions to handle requests in fact in your case you can do this

class LoginView(View):

    login_url = "your login url"
    path = "your path to login template"

    def get(self, request):
        if request.user.is_authenticated():
            return redirect('home')
        else:
            return render(request, self.path)

    def post(self,request):
        #treatment here
        actor = LoginActor(request)
        #I am not sure how you defined LoginActor but should return None if user is not logged in otherwise it returns a logged in user I can help you more if you provide how you defined ActorLogin()
        if actor:
             redirect('home')
        return render(request, self.path,{"error_message":"couldn't login in"})`

Now you add LoginRequiredMixin in your other views so you don't need to check if the user is logged in or use use UserPassesTest if you need other requirments than user login in. for more details check the official documentation about login

https://docs.djangoproject.com/en/1.11/topics/auth/default/#django.contrib.auth.mixins.UserPassesTestMixin

gurpreet singh chahal · Accepted Answer · 2018-09-09 06:05:14Z

0

Django has decorators for this purpose. For example To allow only logged-in users to view the home page,

from django.contrib.auth.decorators import login_required

@login_required(login_url='/accounts/login/')
def home(request):
    ## How home page for only registered users

You can create a custom decorator to validate guest(non-logged in) users. Refer to https://medium.com/@MicroPyramid/custom-decorators-to-check-user-roles-and-permissions-in-django-ece6b8a98d9d for help on creating a custom decorator.

answered Sep 9, 2018 at 6:05

gurpreet singh chahal

3701 silver badge8 bronze badges

Collectives™ on Stack Overflow

How to return from another function in Django View function

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related