From 9d82b201514bc4cbba41451a5cc2c20fcb0f60a7 Mon Sep 17 00:00:00 2001 From: Midgard Date: Wed, 22 Jul 2020 17:33:35 +0200 Subject: [PATCH] Best practices :3 --- users/templates/{ => users}/profile.html | 0 users/views.py | 10 +++++++--- 2 files changed, 7 insertions(+), 3 deletions(-) rename users/templates/{ => users}/profile.html (100%) diff --git a/users/templates/profile.html b/users/templates/users/profile.html similarity index 100% rename from users/templates/profile.html rename to users/templates/users/profile.html diff --git a/users/views.py b/users/views.py index c6ed891..6dd36c1 100644 --- a/users/views.py +++ b/users/views.py @@ -3,12 +3,16 @@ from pprint import pprint from django.contrib.auth import logout from django.http import HttpResponseRedirect from django.shortcuts import render, redirect +from django.urls import reverse from users.forms import UserMetaForm from users.models import CustomUser def profile(request): + if not request.user.is_authenticated: + return HttpResponseRedirect(reverse("events:index")) + # if this is a POST request we need to process the form data if request.method == 'POST': # create a form instance and populate it with data from the request: @@ -19,15 +23,15 @@ def profile(request): form.save() # redirect to a new URL: - return HttpResponseRedirect('/user/profile') + return HttpResponseRedirect(reverse("users:profile")) # if a GET (or any other method) we'll create a blank form else: form = UserMetaForm(instance=request.user) - return render(request, 'profile.html', {'form': form}) + return render(request, 'users/profile.html', {'form': form}) def logout_view(request): logout(request) - return redirect('/user/profile') + return redirect(reverse("users:profile"))