Pylint for extra_func

.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

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 += ['Конец диапазона должен быть позже начала диапазона и раньше текущего времени']

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):