From 8646e668c1181851fd0624ba34185c21e5baf19e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=A1=D1=82=D0=B5=D0=BF=D0=B0=D0=BD=D0=B5=D0=BD=D0=BA?= =?UTF-8?q?=D0=BE=20=D0=9E=D0=BB=D1=8C=D0=B3=D0=B0?= Date: Thu, 6 May 2021 17:36:38 +0300 Subject: [PATCH] Pylint for extra_func --- .pylintrc | 2 +- main/extra_func.py | 20 +++++++++++--------- main/views.py | 16 ++++++++-------- 3 files changed, 20 insertions(+), 18 deletions(-) diff --git a/.pylintrc b/.pylintrc index 83b9512..07ab0f1 100644 --- a/.pylintrc +++ b/.pylintrc @@ -525,7 +525,7 @@ spelling-store-unknown-words=no max-args=5 # Maximum number of attributes for a class (see R0902). -max-attributes=7 +max-attributes=10 # Maximum number of boolean expressions in an if statement (see R0916). max-bool-expr=5 diff --git a/main/extra_func.py b/main/extra_func.py index fbbe3f7..af1555d 100644 --- a/main/extra_func.py +++ b/main/extra_func.py @@ -39,17 +39,17 @@ def update_role(user_profile: UserProfile, role: int, who_changes: get_user_mode zendesk.admin.users.update(user) -def make_engineer(user_profile: UserProfile) -> None: +def make_engineer(user_profile: UserProfile, who_changes: get_user_model()) -> None: """ Функция устанавливает пользователю роль инженера. :param user_profile: Профиль пользователя :return: Вызов функции **update_role** с параметрами: профиль пользователя, роль "engineer" """ - update_role(user_profile, ROLES['engineer']) + update_role(user_profile, ROLES['engineer'], who_changes) -def make_light_agent(user_profile: UserProfile) -> None: +def make_light_agent(user_profile: UserProfile, who_changes: get_user_model()) -> None: """ Функция устанавливает пользователю роль легкого агента. @@ -76,7 +76,7 @@ def make_light_agent(user_profile: UserProfile) -> None: attempts, success = 5, False while not success and attempts != 0: try: - update_role(user_profile, ROLES['light_agent'], get_user_model()) + update_role(user_profile, ROLES['light_agent'], who_changes) success = True except APIException as e: attempts -= 1 @@ -214,17 +214,17 @@ def daterange(start_date: timedelta, end_date: timedelta) -> list: return dates -def get_timedelta(log: RoleChangeLogs, time: timedelta = None) -> timedelta: +def get_timedelta(current_log: RoleChangeLogs, time: timedelta = None) -> timedelta: """ Функция возвращает объект класса timedelta, который хранит промежуток времени от начала суток до момента, который находится в log (объект класса RoleChangeLogs) или в time(datetime.time), если введён. - :param log: Лог + :param current_log: Лог :param time: Время :return: Сколько времени прошло от начала суток до события """ if time is None: - time = log.change_time.time() + time = current_log.change_time.time() time = timedelta(hours=time.hour, minutes=time.minute, seconds=time.second) return time @@ -285,7 +285,8 @@ class StatisticData: """ Функция возвращает статистику работы пользователя. - :return: Словарь statistic с применением формата отображения и интервала работы(если они есть). None, если были ошибки при создании. + :return: Словарь statistic с применением формата отображения и интервала работы(если они есть). None, если были + ошибки при создании. """ if self.is_valid_statistic(): stat = self.statistic @@ -399,7 +400,8 @@ class StatisticData: """ Функция возвращает логи в диапазоне дат start_date - end_date для пользователя с указанным email. - :return: Данные о смене статусов пользователя. Если пользователь не найден или интервал времени некорректен - ошибку. + :return: Данные о смене статусов пользователя. Если пользователь не найден или интервал времени + некорректен - ошибку. """ if not self.check_time(): self.errors += ['Конец диапазона должен быть позже начала диапазона и раньше текущего времени'] diff --git a/main/views.py b/main/views.py index 72fc682..ffc2c68 100644 --- a/main/views.py +++ b/main/views.py @@ -228,7 +228,7 @@ def work_hand_over(request: WSGIRequest) -> HttpResponseRedirect: :param request: данные текущего пользователя (login_required) :return: перезагрузка текущей страницы после выполнения смены роли """ - make_light_agent(request.user.userprofile) + make_light_agent(request.user.userprofile, request.user) return set_session_params_for_work_page(request) @@ -241,7 +241,7 @@ def work_become_engineer(request: WSGIRequest) -> HttpResponseRedirect: :param request: данные текущего пользователя (login_required) :return: перезагрузка текущей страницы после выполнения смены роли """ - make_engineer(request.user.userprofile) + make_engineer(request.user.userprofile, request.user) return set_session_params_for_work_page(request) @@ -307,8 +307,8 @@ class AdminPageView(LoginRequiredMixin, PermissionRequiredMixin, SuccessMessageM self.make_light_agents(users) return super().form_valid(form) - @staticmethod - def make_engineers(users: list) -> None: + + def make_engineers(self, users: list) -> None: """ Функция проходит по списку пользователей, проставляя статус "engineer". @@ -316,9 +316,9 @@ class AdminPageView(LoginRequiredMixin, PermissionRequiredMixin, SuccessMessageM :return: Обновленный список пользователей """ for user in users: - make_engineer(user) - @staticmethod - def make_light_agents(users: list) -> None: + make_engineer(user, self.request.user) + + def make_light_agents(self, users: list) -> None: """ Функция проходит по списку пользователей, проставляя статус "light agent". @@ -326,7 +326,7 @@ class AdminPageView(LoginRequiredMixin, PermissionRequiredMixin, SuccessMessageM :return: Обновленный список пользователей """ for user in users: - make_light_agent(user) + make_light_agent(user, self.request.user) class CustomLoginView(LoginView):