Skip to content

Исправить ошибку с проверкой времени на лимит комментов на лектора #173

Closed

Description

@petrCher

У нас есть проверка, если пользователь оставил слишком много комментариев этому преподавателю после этой даты, то новый комментарий не разрешён. В сеттингс стоит дефолтное значение COMMENT_LECTURER_FREQUENCE_IN_MONTH: int = 6
Проблема в том, что если месяц 6 или 12 (июнь и декабрь) то формула (6-6)%12 дает 0, а такого месяца не существует. Вылетает ошибка ValueError: month must be in 1..12
код по ссылке:
https://github.com/profcomff/rating-api/blob/main/rating_api/routes/comment.py#L82

Также надо скорректировать год, чтобы вычислялся правильно

пример от иишки, мб даже правильно реализовано:
total_months = now.year * 12 + now.month - 1 - settings.COMMENT_LECTURER_FREQUENCE_IN_MONTH
year = total_months // 12
month = total_months % 12 + 1
cutoff_date_lecturer = datetime(year, month, 1)

Activity

  1. moved this from Todo to In Progress in Viribus Teamon Jun 28, 2026
  2. linked a pull request that will close this issueBug fix ready #176on Jul 4, 2026
  3. moved this from In Progress to Done in Viribus Teamon Jul 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

bug 🐞Что-то не работает

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions