Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Некорректная работа валидатора даты рождения игроков #560

Closed
Toksi86 opened this issue Oct 2, 2024 · 3 comments · Fixed by #571
Assignees
Labels
bug Something isn't working

Comments

@Toksi86
Copy link
Contributor

Toksi86 commented Oct 2, 2024

Где был обнаружен баг ?

В базе данных существуют игроки, чей возраст меньше, чем нижнее ограничение валидатора.

Скриншоты

Например игрок Аверкиев имеет дату рождения 10.05.2006, а минимальная дата валидатора 02.10.2006
1

Предложения и предположения как исправить

Изменить нижнее ограничение валидатора возраста игроков. Понизить лет на 10-15

@Toksi86 Toksi86 added the bug Something isn't working label Oct 2, 2024
@webn1nja
Copy link
Contributor

webn1nja commented Oct 3, 2024

Добрый вечер. Обнаружил странное.
class AgeLimits(IntEnum):
"""Возрастные лимиты."""

MIN_AGE_PLAYER = 6
MAX_AGE_PLAYER = 18

Вы пишете: "Изменить нижнее ограничение валидатора возраста игроков. Понизить лет на 10-15Image

"
До какого значения вы бы хотели понизить нижнюю границу?

@KonstantinRaikhert
Copy link
Member

Так, у нас проблема, что БД имеет сейчас игроков, которым на данный момент уже больше 18 лет. Видимо нужно до 25. А это точно тот валидатор, который используется в данной форме?

@webn1nja
Copy link
Contributor

webn1nja commented Oct 7, 2024

в конечном итоге валидаторы формы хотят класс AgeLimits из core/constants.py, в котором определены MIN_AGE_PLAYER и MAX_AGE_PLAYER

@webn1nja webn1nja linked a pull request Oct 7, 2024 that will close this issue
@KonstantinRaikhert KonstantinRaikhert linked a pull request Oct 8, 2024 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants