Skip to content
Snippets Groups Projects
Commit 9404cf4f authored by Christian Dresen's avatar Christian Dresen
Browse files

[WarpMain] Fixed refactor errors

parent 1a52bbc4
No related branches found
No related tags found
No related merge requests found
...@@ -8,7 +8,7 @@ ...@@ -8,7 +8,7 @@
<div class="alert alert-success"> <div class="alert alert-success">
{% trans "password_change_successful" %} {% trans "redirect_within_5_sec_if_not_click" %} <!-- Your password was sucessfully changed. You will be redirected within 5 seconds. If not please click --> {% trans "password_change_successful" %} {% trans "redirect_within_5_sec_if_not_click" %} <!-- Your password was sucessfully changed. You will be redirected within 5 seconds. If not please click -->
<a href="/login">{% trans "here" %}</a> <a href="/login">{% trans "here" %}</a>
<meta http-equiv="refresh" content="5; url=/login"> <meta http-equiv="refresh" content="5; url=/account/login">
</div> </div>
{% else %} {% else %}
<form class="form" style="max-width: 330px; margin: 0 auto; padding: 40px"> <form class="form" style="max-width: 330px; margin: 0 auto; padding: 40px">
......
...@@ -16,7 +16,7 @@ from django.conf import settings ...@@ -16,7 +16,7 @@ from django.conf import settings
## ##
def setUserObject(request): def set_user_object(request):
page_context['ldap_groups'] = request.user.ldap_user.group_names page_context['ldap_groups'] = request.user.ldap_user.group_names
page_context['ldap_user_form'] = LdapUserForm(instance=LdapUser.objects.get(uid=str(request.user))) page_context['ldap_user_form'] = LdapUserForm(instance=LdapUser.objects.get(uid=str(request.user)))
...@@ -26,7 +26,8 @@ def clear_error_messages(): ...@@ -26,7 +26,8 @@ def clear_error_messages():
page_context["success_info"] = False page_context["success_info"] = False
page_context["error_passwd"] = "" page_context["error_passwd"] = ""
page_context["success_passwd"] = False page_context["success_passwd"] = False
page_context["selected_tab"]="" page_context["selected_tab"] = ""
@login_required(login_url=settings.LOGIN_URL, redirect_field_name=None) @login_required(login_url=settings.LOGIN_URL, redirect_field_name=None)
def index(request): def index(request):
...@@ -39,7 +40,7 @@ def index(request): ...@@ -39,7 +40,7 @@ def index(request):
@login_required(login_url=settings.LOGIN_URL, redirect_field_name=None) @login_required(login_url=settings.LOGIN_URL, redirect_field_name=None)
def change_information(request): def change_information(request):
clear_error_messages() clear_error_messages()
setUserObject(request) set_user_object(request)
if request.method != 'POST': if request.method != 'POST':
return redirect("profile_index") return redirect("profile_index")
...@@ -71,7 +72,7 @@ def change_information(request): ...@@ -71,7 +72,7 @@ def change_information(request):
user.save() user.save()
page_context["success_info"] = True page_context["success_info"] = True
setUserObject(request) set_user_object(request)
return HttpResponse(render(request, 'warpauth/profile.html', page_context)) return HttpResponse(render(request, 'warpauth/profile.html', page_context))
...@@ -97,4 +98,3 @@ def change_password(request): ...@@ -97,4 +98,3 @@ def change_password(request):
page_context['ldap_groups'] = request.user.ldap_user.group_names page_context['ldap_groups'] = request.user.ldap_user.group_names
page_context['ldap_user_form'] = LdapUserForm(instance=LdapUser.objects.get(uid=str(request.user))) page_context['ldap_user_form'] = LdapUserForm(instance=LdapUser.objects.get(uid=str(request.user)))
return HttpResponse(render(request, 'warpauth/profile.html', page_context)) return HttpResponse(render(request, 'warpauth/profile.html', page_context))
...@@ -19,7 +19,7 @@ def send_token(token): ...@@ -19,7 +19,7 @@ def send_token(token):
def register(request): def register(request):
pages['success'] = False page_context['success'] = False
if request.method == "POST": if request.method == "POST":
username = request.POST['username'] username = request.POST['username']
...@@ -31,15 +31,15 @@ def register(request): ...@@ -31,15 +31,15 @@ def register(request):
try: try:
f.clean(email) f.clean(email)
except ValidationError as e: except ValidationError as e:
pages["error"] = _("err_invalid_email_format") page_context["error"] = _("err_invalid_email_format")
if username == "" or email == "" or password == "" or password2 == "": if username == "" or email == "" or password == "" or password2 == "":
pages['error'] = _("err_fill_all_fields") page_context['error'] = _("err_fill_all_fields")
elif not re.search(r'^\w+$', username): elif not re.search(r'^\w+$', username):
pages['error'] = _("err_invalid_username") page_context['error'] = _("err_invalid_username")
elif password != password2: elif password != password2:
pages['error'] = _("err_passwords_not_matching") page_context['error'] = _("err_passwords_not_matching")
elif not pages["error"]: elif not page_context["error"]:
u = LdapUser.objects.filter(uid=username) u = LdapUser.objects.filter(uid=username)
if u.count() == 0: if u.count() == 0:
user = LdapUser() user = LdapUser()
...@@ -61,13 +61,13 @@ def register(request): ...@@ -61,13 +61,13 @@ def register(request):
token.save() token.save()
ret = send_token(token) ret = send_token(token)
if not ret: if not ret:
pages["error"] = _("err_sending_mail") #Error while sending the email. Please contact the administrator. page_context["error"] = _("err_sending_mail") #Error while sending the email. Please contact the administrator.
else: else:
pages['success'] = True page_context['success'] = True
else: else:
pages['error'] = _("err_username_not_available") page_context['error'] = _("err_username_not_available")
return HttpResponse(render(request, 'warpauth/register/register.html', pages)) return HttpResponse(render(request, 'warpauth/register/register.html', page_context))
def activate(request, token=None): def activate(request, token=None):
...@@ -78,9 +78,9 @@ def activate(request, token=None): ...@@ -78,9 +78,9 @@ def activate(request, token=None):
group.save() group.save()
activation_token.delete() activation_token.delete()
except ObjectDoesNotExist: except ObjectDoesNotExist:
pages["error"] = _("err_token_invalid") page_context["error"] = _("err_token_invalid")
return HttpResponse(render(request, 'warpauth/register/activate.html', pages)) return HttpResponse(render(request, 'warpauth/register/activate.html', page_context))
def resend_token(request): def resend_token(request):
if request.method == "POST": if request.method == "POST":
...@@ -89,9 +89,9 @@ def resend_token(request): ...@@ -89,9 +89,9 @@ def resend_token(request):
if token.email == request.POST["email"]: if token.email == request.POST["email"]:
ret = send_token(token) ret = send_token(token)
if not ret: if not ret:
pages["error"] = _("err_processing_request") page_context["error"] = _("err_processing_request")
pages["success"] = True page_context["success"] = True
except ObjectDoesNotExist: except ObjectDoesNotExist:
pages["error"] = _("err_processing_request") page_context["error"] = _("err_processing_request")
return HttpResponse(render(request, 'warpauth/register/resend_token.html', pages)) return HttpResponse(render(request, 'warpauth/register/resend_token.html', page_context))
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment