NotBigGhostandClaude Opus 5
8ac2757cd6
Проход /simplify по backend/app: - Три источника «текущего времени» (_utcnow в models.py и match_service.py при живом timeutil.utcnow) сведены к одному — это прямо инвариант из CLAUDE.md. - Бамп версии партии из двух независимых мест собран в match_service.touch: следующая точка мутации, не трогающая строку matches, теперь имеет очевидный способ сделать правильно. - membership_service переиспользует group_service.get_membership вместо трёх копий одного запроса; защита последнего владельца — один хелпер на удаление и смену роли вместо двух похожих блоков. - Убраны N+1: участники страницы партий и ники пригласивших берутся одним запросом вместо запроса на строку (20 партий = 20 лишних запросов с двумя join каждый). - Счётчики партий считает СУБД (COUNT/MAX) вместо выгрузки всех строк ради len() и max() в Python. #8 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BoiJK9ux8peeyjLb8TYjFf
94 lines
3.7 KiB
Python
94 lines
3.7 KiB
Python
"""Членство в группе: список, добавление по нику, удаление, смена роли."""
|
|
from __future__ import annotations
|
|
|
|
from sqlmodel import Session, select
|
|
|
|
from app.core.errors import ConflictError, ForbiddenError, NotFoundError, ValidationError
|
|
from app.models import Group, GroupMember, User
|
|
from app.services import group_service
|
|
|
|
MAX_GROUP_SIZE = 10
|
|
|
|
|
|
def list_members(session: Session, group_id: int) -> list[tuple[GroupMember, User]]:
|
|
rows = session.exec(
|
|
select(GroupMember, User)
|
|
.join(User, User.id == GroupMember.user_id)
|
|
.where(GroupMember.group_id == group_id)
|
|
.order_by(GroupMember.role.desc(), User.nickname)
|
|
).all()
|
|
return [(m, u) for m, u in rows]
|
|
|
|
|
|
def add_member_by_nickname(session: Session, group: Group, nickname: str) -> tuple[GroupMember, User]:
|
|
nickname = (nickname or "").strip()
|
|
if not nickname:
|
|
raise ValidationError("Укажите никнейм игрока.")
|
|
user = session.exec(select(User).where(User.nickname == nickname)).first()
|
|
if user is None:
|
|
raise NotFoundError("Игрок с таким ником не найден.")
|
|
|
|
member_count = len(
|
|
session.exec(select(GroupMember.id).where(GroupMember.group_id == group.id)).all()
|
|
)
|
|
if member_count >= MAX_GROUP_SIZE:
|
|
raise ConflictError(f"В группе уже максимум участников ({MAX_GROUP_SIZE}).")
|
|
|
|
existing = group_service.get_membership(session, group.id, user.id)
|
|
if existing is not None:
|
|
raise ConflictError("Игрок уже в группе.")
|
|
|
|
member = GroupMember(group_id=group.id, user_id=user.id, role="member") # type: ignore[arg-type]
|
|
session.add(member)
|
|
session.commit()
|
|
session.refresh(member)
|
|
return member, user
|
|
|
|
|
|
def _assert_not_last_owner(session: Session, group: Group, member: GroupMember, message: str) -> None:
|
|
"""Группа без владельца неисправима: назначить нового становится некому."""
|
|
if member.role != "owner":
|
|
return
|
|
owners = session.exec(
|
|
select(GroupMember.id).where(
|
|
GroupMember.group_id == group.id, GroupMember.role == "owner"
|
|
)
|
|
).all()
|
|
if len(owners) <= 1:
|
|
raise ForbiddenError(message)
|
|
|
|
|
|
def remove_member(session: Session, group: Group, user_id: int) -> None:
|
|
member = group_service.get_membership(session, group.id, user_id)
|
|
if member is None:
|
|
raise NotFoundError("Игрок не состоит в группе.")
|
|
_assert_not_last_owner(
|
|
session, group, member, "Нельзя удалить последнего владельца группы."
|
|
)
|
|
|
|
# Сбросить активную группу у тех, для кого она была активной.
|
|
user = session.get(User, user_id)
|
|
if user is not None and user.active_group_id == group.id:
|
|
user.active_group_id = None
|
|
session.add(user)
|
|
|
|
session.delete(member)
|
|
session.commit()
|
|
|
|
|
|
def change_role(session: Session, group: Group, user_id: int, role: str) -> GroupMember:
|
|
if role not in ("owner", "member"):
|
|
raise ValidationError("Недопустимая роль.")
|
|
member = group_service.get_membership(session, group.id, user_id)
|
|
if member is None:
|
|
raise NotFoundError("Игрок не состоит в группе.")
|
|
if role != "owner":
|
|
_assert_not_last_owner(
|
|
session, group, member, "Нельзя снять роль с последнего владельца группы."
|
|
)
|
|
member.role = role
|
|
session.add(member)
|
|
session.commit()
|
|
session.refresh(member)
|
|
return member
|