From f21eb2dcd41b024f23f539622169c8985fd81ddb Mon Sep 17 00:00:00 2001 From: Iurii Tatishchev Date: Thu, 15 Apr 2021 08:43:10 -0700 Subject: [PATCH] minor fixes --- access_controller/settings.py | 2 +- access_controller/urls.py | 26 -------------------------- main/views.py | 28 ++-------------------------- 3 files changed, 3 insertions(+), 53 deletions(-) diff --git a/access_controller/settings.py b/access_controller/settings.py index ddcd372..f9b5119 100644 --- a/access_controller/settings.py +++ b/access_controller/settings.py @@ -58,8 +58,8 @@ EMAIL_PORT = 587 EMAIL_USE_TLS = True EMAIL_HOST_USER = 'group02django@gmail.com' EMAIL_HOST_PASSWORD = 'djangogroup02' -SERVER_EMAIL = EMAIL_HOST_USER DEFAULT_FROM_EMAIL = EMAIL_HOST_USER +SERVER_EMAIL = EMAIL_HOST_USER TEMPLATES = [ { diff --git a/access_controller/urls.py b/access_controller/urls.py index 57d1d01..f6a6754 100644 --- a/access_controller/urls.py +++ b/access_controller/urls.py @@ -14,7 +14,6 @@ Including another URLconf 2. Add a URL to urlpatterns: path('blog/', include('blog.urls')) """ from django.contrib import admin -from django.contrib.auth import views as auth_views from django.urls import path, include from main.views import main_page, profile_page, CustomRegistrationView, CustomLoginView, registration_error @@ -30,41 +29,16 @@ urlpatterns = [ path('accounts/register/error/', registration_error, name='registration_email_error'), path('accounts/login/', CustomLoginView.as_view(), name='login'), path('accounts/', include('django.contrib.auth.urls')), - path('accounts/', include('django_registration.backends.one_step.urls')), path('work/', work_page, name="work"), path('work/hand_over/', work_hand_over, name="work_hand_over"), path('work/become_engineer/', work_become_engineer, name="work_become_engineer"), path('work/get_tickets', work_get_tickets, name='work_get_tickets'), path('accounts/', include('django_registration.backends.activation.urls')), - path('accounts/login/', include('django.contrib.auth.urls')), path('control/', AdminPageView.as_view(), name='control'), path('statistic/', statistic_page, name='statistic'), ] -urlpatterns += [ - path( - 'password_reset/', - auth_views.PasswordResetView.as_view(), - name='password_reset' - ), - path( - 'password_reset/done/', - auth_views.PasswordResetDoneView.as_view(), - name='password_reset_done' - ), - path( - 'reset///', - auth_views.PasswordResetConfirmView.as_view(), - name='password_reset_confirm' - ), - path( - 'reset/done/', - auth_views.PasswordResetCompleteView.as_view(), - name='password_reset_complete' - ), -] - # Django REST urlpatterns += [ path('api/', include(router.urls)) diff --git a/main/views.py b/main/views.py index 9971c58..0129b3a 100644 --- a/main/views.py +++ b/main/views.py @@ -1,7 +1,3 @@ -import logging -import os -from datetime import datetime - from smtplib import SMTPException from django.contrib import messages @@ -13,7 +9,6 @@ from django.contrib.auth.forms import PasswordResetForm from django.contrib.auth.views import LoginView from django.contrib.contenttypes.models import ContentType from django.contrib.messages.views import SuccessMessageMixin -from django.core.exceptions import PermissionDenied from django.core.handlers.wsgi import WSGIRequest from django.http import HttpResponseRedirect, HttpResponse from django.shortcuts import render, redirect, get_list_or_404 @@ -25,7 +20,7 @@ from rest_framework import viewsets from rest_framework.response import Response from zenpy.lib.api_objects import User as ZenpyUser -from access_controller.settings import EMAIL_HOST_USER, ZENDESK_ROLES, ZENDESK_MAX_AGENTS, ZENDESK_GROUPS +from access_controller.settings import DEFAULT_FROM_EMAIL, ZENDESK_ROLES, ZENDESK_MAX_AGENTS from main.extra_func import check_user_exist, update_profile, get_user_organization, \ make_engineer, make_light_agent, get_users_list, update_users_in_model, count_users, \ StatisticData, log, ZendeskAdmin @@ -36,7 +31,6 @@ from .models import UserProfile def setup_context(profile_lit: bool = False, control_lit: bool = False, work_lit: bool = False, registration_lit: bool = False, login_lit: bool = False): - print(profile_lit, control_lit, work_lit, registration_lit, login_lit) context = { 'profile_lit': profile_lit, 'control_lit': control_lit, @@ -88,7 +82,7 @@ class CustomRegistrationView(RegistrationView): opts = { 'use_https': self.request.is_secure(), 'token_generator': default_token_generator, - 'from_email': EMAIL_HOST_USER, + 'from_email': DEFAULT_FROM_EMAIL, 'email_template_name': 'registration/password_reset_email.html', 'subject_template_name': 'registration/password_reset_subject.txt', 'request': self.request, @@ -205,23 +199,6 @@ def work_page(request: WSGIRequest, id: int) -> HttpResponse: return redirect("login") -def user_update(zenpy_user: User, admin: User, request: WSGIRequest) -> UserProfile: - """ - Функция устанавливает пользователю роль "agent" (изменяет профиль). - - :param zenpy_user: Пользователь Zendesk - :param admin: Пользователь - :param request: Запрос установки роли "agent" в Userprofile - :return: Обновленный профиль пользователя - """ - - admin.users.update(zenpy_user) - request.user.userprofile.role = "agent" - request.user.userprofile.custom_role_id = zenpy_user.custom_role_id - request.user.userprofile.save() - messages.success(request, "Права были изменены") - - @login_required() def work_hand_over(request: WSGIRequest) -> HttpResponseRedirect: """ @@ -348,7 +325,6 @@ class AdminPageView(LoginRequiredMixin, PermissionRequiredMixin, SuccessMessageM return context # TODO: need to get profile page url - class CustomLoginView(LoginView): """ Отображение страницы авторизации пользователя