refactor(qodana): Рефакторинг Qodana, удаление неиспользуемых аргументов и функций

Signed-off-by: Roman Chursanov <[email protected]>

Author: AuthFailed

infrastructure/database/repo/STP/group_member.py

@@ -255,7 +255,7 @@ async def is_member_muted(self, group_id: int, member_id: int) -> bool:
                 return True
 
             # Если заглушен временно, проверяем время
-            current_time = datetime.utcnow()
+            current_time = datetime.now()
             if current_time >= unmute_at:
                 # Время мьюта истекло, автоматически размучиваем
                 await self.unmute_member(group_id, member_id)

infrastructure/database/repo/STP/transactions.py

@@ -47,6 +47,7 @@ async def add_transaction(
             source_id: Идентификатор достижения или предмета (опционально)
             comment: Комментарий (опционально)
             created_by: ID администратора, создавшего транзакцию (опционально)
+            kpi_extracted_at: Дата выгрузки KPI
 
         Returns:
             Объект Transaction или None в случае ошибки

tgbot/handlers/deep/group/settings.py

@@ -488,9 +488,6 @@ async def handle_member_detail(
         await callback.answer("Не удалось найти группу в базе :(")
         return
 
-    member_name = ""
-    member_info = ""
-
     if callback_data.member_type == "employee":
         # Get employee data
         employee = await stp_repo.employee.get_user(user_id=callback_data.member_id)

tgbot/handlers/group/management.py

@@ -779,9 +779,6 @@ async def handle_member_detail(
         await callback.answer("❌ Группа не найдена в базе данных")
         return
 
-    member_name = ""
-    member_info = ""
-
     if callback_data.member_type == "employee":
         employee = await stp_repo.employee.get_user(user_id=callback_data.member_id)
         if employee:

tgbot/handlers/head/main.py

@@ -1,7 +1,6 @@
 from aiogram import F, Router
 from aiogram.filters import CommandStart
 from aiogram.types import CallbackQuery, Message
-from aiogram.utils.deep_linking import create_startgroup_link
 
 from infrastructure.database.models import Employee
 from tgbot.filters.role import HeadFilter
@@ -15,23 +14,21 @@
 
 @head_router.message(CommandStart())
 async def admin_start_cmd(message: Message, user: Employee):
-    group_link = await create_startgroup_link(message.bot, payload="start")
     await message.answer(
         f"""👋 Привет, <b>{user.fullname}</b>!
 
 Я - бот-помощник СТП
 Здесь ты можешь найти графики, показатели своей группы и многое другое""",
-        reply_markup=main_kb(group_link),
+        reply_markup=main_kb(),
     )
 
 
 @head_router.callback_query(MainMenu.filter(F.menu == "main"))
 async def user_start_cb(callback: CallbackQuery, user: Employee):
-    group_link = await create_startgroup_link(callback.bot, payload="start")
     await callback.message.edit_text(
         f"""👋 Привет, <b>{user.fullname}</b>!
 
 Я - бот-помощник СТП
 Здесь ты можешь найти графики, показатели своей группы и многое другое""",
-        reply_markup=main_kb(group_link),
+        reply_markup=main_kb(),
     )

tgbot/handlers/inline/inline.py

@@ -148,8 +148,6 @@ async def search_users_with_filters(
     ) -> List[Employee]:
         """Поиск пользователей с применением фильтров"""
         try:
-            users = []
-
             # Приоритетный поиск по explicit фильтрам
             if filters["user_id"] is not None:
                 user = await stp_repo.employee.get_user(user_id=filters["user_id"])

tgbot/handlers/search/main.py

@@ -404,14 +404,9 @@ async def show_user_details(
             await callback.answer("❌ Пользователь не найден", show_alert=True)
             return
 
-        # Получаем статистику пользователя (только для роли 2 и выше)
-        stats = None
-        if viewer_role >= 2:
-            stats = await SearchService.get_user_statistics(user_id, stp_repo)
-
         # Формирование информации о пользователе в зависимости от роли смотрящего
         user_info = SearchService.format_user_info_role_based(
-            target_user, user_head, stats, viewer_role
+            target_user, user_head, viewer_role
         )
 
         # Дополнительная информация для руководителей (только для роли 2 и выше)
@@ -1598,14 +1593,9 @@ async def search_head_group_member_detail(
             else None
         )
 
-        # Получаем статистику пользователя (только для роли 2 и выше)
-        stats = None
-        if viewer_role >= 2:
-            stats = await SearchService.get_user_statistics(member_id, stp_repo)
-
         # Формирование информации о пользователе в зависимости от роли смотрящего
         user_info = SearchService.format_user_info_role_based(
-            member, user_head, stats, viewer_role
+            member, user_head, viewer_role
         )
 
         # Дополнительная информация для руководителей (только для роли 2 и выше)

tgbot/handlers/user/search/main.py

@@ -284,7 +284,6 @@ async def show_user_search_details(
         return
 
     user_id = callback_data.user_id
-    return_to = callback_data.return_to
 
     try:
         target_user = await stp_repo.employee.get_user(user_id=user_id)
@@ -300,12 +299,12 @@ async def show_user_search_details(
 
         # Базовая информация о пользователе (без статистики для обычных пользователей)
         user_info = SearchService.format_user_info_role_based(
-            target_user, user_head, None, user.role
+            target_user, user_head, user.role
         )
 
         await callback.message.edit_text(
             user_info,
-            reply_markup=user_user_detail_kb(target_user, return_to),
+            reply_markup=user_user_detail_kb(),
         )
 
     except Exception as e:

tgbot/keyboards/group/main.py

@@ -388,7 +388,7 @@ def group_members_kb(
     current_page: int = 1,
     list_page: int = 1,
     members_per_page: int = 8,
-) -> InlineKeyboardMarkup | None:
+) -> InlineKeyboardMarkup:
     """Group members management keyboard."""
     buttons = []
 
@@ -436,6 +436,110 @@ def group_members_kb(
         )
         return InlineKeyboardMarkup(inline_keyboard=buttons)
 
+    # Implement pagination
+    total_members = len(all_members)
+    total_pages = max(1, (total_members + members_per_page - 1) // members_per_page)
+    start_index = (current_page - 1) * members_per_page
+    end_index = min(start_index + members_per_page, total_members)
+
+    page_members = all_members[start_index:end_index]
+
+    # Add member buttons in pairs
+    for i in range(0, len(page_members), 2):
+        row = []
+
+        member = page_members[i]
+        role_emoji = (
+            roles.get(member["role"], {}).get("emoji", "👤") if member["role"] else "👤"
+        )
+        button_text = f"{role_emoji} {member['name']}"
+
+        row.append(
+            InlineKeyboardButton(
+                text=button_text,
+                callback_data=GroupMemberDetailMenu(
+                    group_id=group_id,
+                    member_id=member["id"],
+                    member_type=member["type"],
+                    page=current_page,
+                    list_page=list_page,
+                ).pack(),
+            )
+        )
+
+        if i + 1 < len(page_members):
+            member = page_members[i + 1]
+            role_emoji = (
+                roles.get(member["role"], {}).get("emoji", "👤")
+                if member["role"]
+                else "👤"
+            )
+            button_text = f"{role_emoji} {member['name']}"
+
+            row.append(
+                InlineKeyboardButton(
+                    text=button_text,
+                    callback_data=GroupMemberDetailMenu(
+                        group_id=group_id,
+                        member_id=member["id"],
+                        member_type=member["type"],
+                        page=current_page,
+                        list_page=list_page,
+                    ).pack(),
+                )
+            )
+
+        buttons.append(row)
+
+    # Add pagination controls if needed
+    if total_pages > 1:
+        pagination_row = []
+        if current_page > 1:
+            pagination_row.append(
+                InlineKeyboardButton(
+                    text="◀️",
+                    callback_data=GroupMembersMenu(
+                        group_id=group_id,
+                        page=current_page - 1,
+                        list_page=list_page,
+                    ).pack(),
+                )
+            )
+
+        pagination_row.append(
+            InlineKeyboardButton(
+                text=f"{current_page}/{total_pages}", callback_data="noop"
+            )
+        )
+
+        if current_page < total_pages:
+            pagination_row.append(
+                InlineKeyboardButton(
+                    text="▶️",
+                    callback_data=GroupMembersMenu(
+                        group_id=group_id,
+                        page=current_page + 1,
+                        list_page=list_page,
+                    ).pack(),
+                )
+            )
+
+        buttons.append(pagination_row)
+
+    # Back button
+    buttons.append(
+        [
+            InlineKeyboardButton(
+                text="↩️ Назад",
+                callback_data=GroupSettingsMenu(
+                    group_id=group_id, menu="back", page=list_page
+                ).pack(),
+            )
+        ]
+    )
+
+    return InlineKeyboardMarkup(inline_keyboard=buttons)
+
 
 def group_member_detail_kb(
     group_id: int,
@@ -449,7 +553,7 @@ def group_member_detail_kb(
     buttons = [
         [
             InlineKeyboardButton(
-                text="🚫 Забанить",
+                text=f"🚫 Забанить {member_name}",
                 callback_data=GroupMemberActionMenu(
                     group_id=group_id,
                     member_id=member_id,
@@ -460,6 +564,19 @@ def group_member_detail_kb(
                 ).pack(),
             )
         ],
+        [
+            InlineKeyboardButton(
+                text=f"👢 Исключить {member_name}",
+                callback_data=GroupMemberActionMenu(
+                    group_id=group_id,
+                    member_id=member_id,
+                    action="kick",
+                    member_type=member_type,
+                    page=page,
+                    list_page=list_page,
+                ).pack(),
+            )
+        ],
         [
             InlineKeyboardButton(
                 text="↩️ Назад",

tgbot/keyboards/group/settings.py

@@ -389,36 +389,6 @@ def group_members_keyboard(
     return InlineKeyboardMarkup(inline_keyboard=buttons)
 
 
-def group_member_detail_keyboard(
-    group_id: int, member_id: int, member_type: str, member_name: str, page: int = 1
-) -> InlineKeyboardMarkup:
-    """
-    Keyboard for member detail view with ban option.
-    """
-    buttons = [
-        [
-            InlineKeyboardButton(
-                text="🚫 Забанить участника",
-                callback_data=GroupMemberActionMenu(
-                    group_id=group_id,
-                    member_id=member_id,
-                    action="ban",
-                    member_type=member_type,
-                    page=page,
-                ).pack(),
-            )
-        ],
-        [
-            InlineKeyboardButton(
-                text="↩️ Назад",
-                callback_data=GroupMembersMenu(group_id=group_id, page=page).pack(),
-            )
-        ],
-    ]
-
-    return InlineKeyboardMarkup(inline_keyboard=buttons)
-
-
 def group_service_messages_keyboard(
     group: Group, pending_categories: list = None
 ) -> InlineKeyboardMarkup: