From a26411f746e56841c7d519342881e8e7d3a0940d Mon Sep 17 00:00:00 2001
From: "Pavel \"LEdoian\" Turinsky" <ledoian@matfyz.cz>
Date: Wed, 28 Apr 2021 21:54:51 +0200
Subject: [PATCH] =?UTF-8?q?Login:=20Spousta=20vylep=C5=A1en=C3=AD?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Přesměrování na stránku, kam jsme chtěli
- Varování, pokud nemáme dost práv
- Zjednodušení LoginView pomocí standardních Django featur
---
 mamweb/settings_common.py                   | 1 +
 seminar/templates/seminar/profil/login.html | 4 ++++
 seminar/utils.py                            | 4 ++--
 seminar/views/views_all.py                  | 6 ------
 4 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/mamweb/settings_common.py b/mamweb/settings_common.py
index 925d5961..569d5e13 100644
--- a/mamweb/settings_common.py
+++ b/mamweb/settings_common.py
@@ -44,6 +44,7 @@ STATICFILES_FINDERS = (
 
 # Where redirect for login required services
 LOGIN_URL = 'login'
+LOGIN_REDIRECT_URL = 'profil'
 
 # Modules configuration
 
diff --git a/seminar/templates/seminar/profil/login.html b/seminar/templates/seminar/profil/login.html
index 63d16eed..75926519 100644
--- a/seminar/templates/seminar/profil/login.html
+++ b/seminar/templates/seminar/profil/login.html
@@ -8,6 +8,10 @@
     Přihlášení
   {% endblock %}{% endblock %}
 </h1>
+{# Obšlehnuto z Admina :-) #}
+{% if user.is_authenticated %}
+<p>K této stránce nejspíš nemáte přístup. Můžete se zkusit přihlásit jako uživatel, který přístup má.</p>
+{% endif %}
 <form action="{% url 'login' %}" method="post">
   {% csrf_token %}
   <table class="form">
diff --git a/seminar/utils.py b/seminar/utils.py
index 9390f1a3..b96d499e 100644
--- a/seminar/utils.py
+++ b/seminar/utils.py
@@ -17,8 +17,8 @@ from enum import auto
 import seminar.models as m
 import seminar.treelib as t
 
-org_required = permission_required('auth.org', raise_exception=True)
-resitel_required = permission_required('auth.resitel', raise_exception=True)
+org_required = permission_required('auth.org')
+resitel_required = permission_required('auth.resitel')
 User = get_user_model()
 # Není to úplně hezké, ale budeme doufat, že to je funkční...
 User.je_org = property(lambda self: self.has_perm('auth.org'))
diff --git a/seminar/views/views_all.py b/seminar/views/views_all.py
index 750a4718..461eca80 100644
--- a/seminar/views/views_all.py
+++ b/seminar/views/views_all.py
@@ -1230,12 +1230,6 @@ class LoginView(auth_views.LoginView):
 	# Jen vezmeme vestavěný a dáme mu vhodný template a přesměrovací URL
 	template_name = 'seminar/profil/login.html'
 
-	# Přesměrovací URL má být v kontextu:
-	def get_context_data(self, **kwargs):
-		ctx = super().get_context_data(**kwargs)
-		ctx['next'] = reverse('profil')
-		return ctx
-
 class LogoutView(auth_views.LogoutView):
 	# Jen vezmeme vestavěný a dáme mu vhodný template a přesměrovací URL
 	template_name = 'seminar/profil/logout.html'