Большое обновление календаря. Объединили модель событий, добавили полноценную повторяемость (hourly/daily/weekly/monthly/annual), материализацию вхождений в списке, навели порядок с правами отделов и обновили фронтовой шаблон. Покрыли API pytest-тестами и починили мелкие баги.
### Ключевые изменения
**Модель**
* Заменили `CompanyEvent` и `DepartmentEvent` на единую **`CalendarEvent`**:
* `department` (nullable FK) — если задан, событие отдела; если `NULL`, событие компании.
* Даты/время: `start_date`, `end_date`, `start_time`, `end_time`, `all_day`.
* Повторяемость: `recurrence` ∈ {`one_time`, `hourly`, `daily`, `weekly`, `monthly`, `annual`}.
* Параметры повторов: `recurrence_interval` (int>0), `recurrence_until` (date), `recurrence_count` (int), `weekdays_mask` (bitmask для weekly).
* Индексы под выборки и сортировку; check-constraint `cal_event_date_time_range_valid` (валидность диапазонов даты/времени; для одного дня `start_time <= end_time` при наличии обоих).
* Поля оформления: `title`, `description`, `color`, `location`, `created_by`, `created_at`.
* Правка по умолчаниям: `weekdays_mask` теперь с безопасным default=0 (исправляет `NOT NULL` в SQLite).
**Сервисы**
* Добавлен сервис `expand_event_occurrences(event, range_start, range_end, max_instances=1000)`:
* Материализует все вхождения события в окне `[range_start, range_end)`.
* Корректно обрабатывает: leap day (29 Feb), месяцы с «дырками» (31-е), еженедельные маски, ежечасные интервалы.
* Выдаёт ровные границы дат/датавремени для FullCalendar.
**Сериализаторы**
* `CalendarEventWriteSerializer` — строгая валидация:
* Для hourly требуется `start_time` и `end_time`.
* Для weekly — пустой/непустой `weekdays` валидируется и мапится в `weekdays_mask`.
* `recurrence_interval` > 0; взаимоисключение `recurrence_until` и `recurrence_count`.
* `CalendarEventSerializer` — read-вариант с плоскими полями для фронта.
* Доп. сериализатор вьюхи для списка не нужен — материализуем в контроллере (см. ниже).
**ViewSet / API**
* Новый единый роут: **`/api/v1/calendar/events/`** (`ModelViewSet`):
* `list`: возвращает **материализованные** вхождения в окне `[start, end)` (query-параметры `YYYY-MM-DD`). Если передан `department_id` — фильтрует события отдела, иначе — компании.
* `retrieve/create/update/partial_update/destroy`: операции над самим `CalendarEvent`.
* `get_queryset`: для `list` фильтрует по department/company, для detail — полный QS.
* При `create`: подставляет `department_id` либо из контекста/параметров, либо из body, проставляет `created_by`, чистит кеш.
* **Права**:
* Используется комбинированный пермишен **`ManageCalendarPerm(AdminOrDeptAllowed)`**.
* Для компании делегирует на `AdminOrActionOrModelPerms`.
* Для отдела требует **строковый код**: `"manage_department_events"`.
* Обновляет инстанс пользователя из БД (сбрасывает кэши), чтобы в одном тесте «выдали право → сразу работает».
* `get_permissions()` → `[IsAuthenticated(), ManageCalendarPerm()]`.
* Починки:
* Подключены `parse_date`/`status` где нужно.
* Исправлен конфликт дат/датавремени в материализации.
* Возврат полей в `list`: `start`, `end`, `allDay`, `color`, `recurrence`, `department_id` (ID родителя можно оставить как `event.id`).
**Signals**
* Добавлен сигнал на изменение `date_of_birth` сотрудника:
* Создаёт/обновляет/удаляет соответствующее ежегодное событие в **глобальном** календаре.
* Идентификация через `source="employee_birthday"` + `source_ref=<employee_id>`.
**Админка**
* `CalendarEventAdmin`:
* Поиск по названию/описанию/локации.
* Фильтры по отделу, типу повторения, all\_day, наличию времени, «company vs department».
* Инлайнов нет; list-представление упрощено (исправлены ошибки `admin.E116` — фильтры теперь валидные `SimpleListFilter`/поля).
**Фронт (шаблон правой колонки)**
* Переписан `<script>` под единый эндпоинт:
* Список: GET `/api/v1/calendar/events/?[department_id=...]&start=YYYY-MM-DD&end=YYYY-MM-DD`.
* Создание:
* Компания: POST `/api/v1/calendar/events/` с `{title, start_date, [end_date], all_day}`.
* Отдел: POST туда же, но с `department_id` (в query или в body).
* Если указано время — **оба** (`start_time`, `end_time`) обязательны; иначе сохраняем как all-day.
* Нормализация дат в локали, аккуратные конвертации для FullCalendar.
* Заголовки авторизации: `Authorization: Bearer <token>` из `<meta name="api-access">`/`localStorage.api.access`. Добавлен fallback на CSRF-куку при отсутствии токена (для SessionAuth).
* Улучшен UI выбора отдела (`/api/v1/departments/my-departments/`), подписи, недельная лента.
**Тесты**
* Добавлены/актуализированы pytest-тесты API:
* Роуты и методы (CRUD).
* Позитив/негатив создания (валидация дат/времени/повторов).
* Изменение повторяемости (weekly → daily и пр.).
* Материализация (hourly alignment, weekly mask, monthly edge-cases, leap day).
* Права: отдел без/с `manage_department_events` (фикс за счёт строкового кода).
* Сигналы: апсерт/удаление дней рождения.
* Исправлены прежние падения: `NOT NULL weekdays_mask`, `NameError(parse_date/status)`, сравнения date/datetime, несоответствие `all_day` vs `allDay`, и др.
### Совместимость и заметки по деплою
* **BREAKING**: единый эндпоинт `/api/v1/calendar/events/` заменяет старые `/company-events/` и `/department-events/`. Если есть сторонние клиенты — переориентировать на новый.
* Миграции обязательны (новая таблица, индексы, дефолты).
* Если где-то ещё использовались константы прав — убедиться, что везде строка `"manage_department_events"` (тесты ориентируются на неё).
* Для фронта: убедиться, что реально прокидывается access-токен в `<meta name="api-access">` **или** есть валидная CSRF-кука при SessionAuth. Иначе POST вернёт 401.
### Почему так
* Единая модель/роут упрощают код и устраняют разъезд схем компании/отделов.
* Материализация в контроллере избавляет фронт от логики повторов и держит поведение единообразным.
* Явная строка кода права синхронизирована с тестовой выдачей пермишена и реальным хранилищем прав отделов.
* Валидация сериализатора закрывает большую часть негативных кейсов на уровне API.
227 lines
8.4 KiB
Python
227 lines
8.4 KiB
Python
# backend/calendar_app/admin.py
|
||
from __future__ import annotations
|
||
|
||
from typing import Iterable
|
||
|
||
from django.contrib import admin, messages
|
||
from django.db.models import QuerySet
|
||
from django.http import HttpRequest
|
||
from django.utils.html import format_html
|
||
from django.utils.translation import gettext_lazy as _
|
||
|
||
from .models import CalendarEvent, Recurrence
|
||
|
||
|
||
class ScopeListFilter(admin.SimpleListFilter):
|
||
"""Фильтр области события: компания или отдел.
|
||
|
||
Values:
|
||
- 'company' → department IS NULL
|
||
- 'dept' → department IS NOT NULL
|
||
"""
|
||
title = _("Область")
|
||
parameter_name = "scope"
|
||
|
||
def lookups(self, request: HttpRequest, model_admin: admin.ModelAdmin) -> Iterable[tuple[str, str]]:
|
||
"""Возвращает варианты выбора фильтра.
|
||
|
||
Args:
|
||
request: Текущий запрос.
|
||
model_admin: Админ-класс.
|
||
|
||
Returns:
|
||
Iterable[tuple[str, str]]: (value, label)
|
||
"""
|
||
return (
|
||
("company", _("Компания")),
|
||
("dept", _("Отдел")),
|
||
)
|
||
|
||
def queryset(self, request: HttpRequest, queryset):
|
||
"""Применяет фильтр к queryset.
|
||
|
||
Args:
|
||
request: Текущий запрос.
|
||
queryset: Исходный queryset.
|
||
|
||
Returns:
|
||
QuerySet: Отфильтрованный queryset.
|
||
"""
|
||
val = self.value()
|
||
if val == "company":
|
||
return queryset.filter(department__isnull=True)
|
||
if val == "dept":
|
||
return queryset.filter(department__isnull=False)
|
||
return queryset
|
||
|
||
|
||
class HasTimeListFilter(admin.SimpleListFilter):
|
||
"""Фильтр наличия времени у события (есть оба time или нет ни одного)."""
|
||
title = _("Есть время")
|
||
parameter_name = "has_time"
|
||
|
||
def lookups(self, request: HttpRequest, model_admin: admin.ModelAdmin) -> Iterable[tuple[str, str]]:
|
||
"""Опции фильтра (Да/Нет)."""
|
||
return (
|
||
("yes", _("Да")),
|
||
("no", _("Нет")),
|
||
)
|
||
|
||
def queryset(self, request: HttpRequest, queryset):
|
||
"""Применяет фильтр к queryset."""
|
||
val = self.value()
|
||
if val == "yes":
|
||
return queryset.filter(start_time__isnull=False, end_time__isnull=False)
|
||
if val == "no":
|
||
return queryset.filter(start_time__isnull=True, end_time__isnull=True)
|
||
return queryset
|
||
|
||
|
||
@admin.register(CalendarEvent)
|
||
class CalendarEventAdmin(admin.ModelAdmin):
|
||
"""Админка для модели CalendarEvent (события календаря компании и отделов)."""
|
||
|
||
# ===== Список =====
|
||
list_display = (
|
||
"id",
|
||
"title",
|
||
"scope_display",
|
||
"start_date",
|
||
"end_date",
|
||
"start_time",
|
||
"end_time",
|
||
"all_day",
|
||
"recurrence",
|
||
"color_swatch",
|
||
"created_by",
|
||
"created_at",
|
||
)
|
||
list_select_related = ("department", "created_by")
|
||
list_per_page = 50
|
||
ordering = ("department_id", "start_date", "start_time")
|
||
|
||
# ===== Поиск/Фильтры/Иерархия =====
|
||
search_fields = (
|
||
"title",
|
||
"description",
|
||
"location",
|
||
"department__name",
|
||
"department__title",
|
||
"department__code",
|
||
)
|
||
list_filter = (
|
||
"department",
|
||
"recurrence",
|
||
"all_day",
|
||
("start_date", admin.DateFieldListFilter),
|
||
("created_at", admin.DateFieldListFilter),
|
||
HasTimeListFilter, # <— ПЕРЕДАЁМ КЛАСС, НЕ СТРОКУ
|
||
ScopeListFilter, # <— ПЕРЕДАЁМ КЛАСС, НЕ СТРОКУ
|
||
)
|
||
date_hierarchy = "start_date"
|
||
|
||
# ===== Поля формы =====
|
||
readonly_fields = ("created_at",)
|
||
raw_id_fields = ("department", "created_by")
|
||
fieldsets = (
|
||
(_("Основное"), {
|
||
"fields": ("title", "description", "department", "location"),
|
||
}),
|
||
(_("Даты и время"), {
|
||
"fields": (
|
||
("start_date", "end_date"),
|
||
("start_time", "end_time"),
|
||
"all_day",
|
||
),
|
||
}),
|
||
(_("Отображение"), {
|
||
"fields": ("recurrence", "color"),
|
||
}),
|
||
(_("Служебное"), {
|
||
"fields": ("created_by", "created_at"),
|
||
}),
|
||
)
|
||
|
||
# ===== Экшены =====
|
||
actions = ("make_all_day", "clear_time", "set_annual", "set_one_time")
|
||
|
||
@admin.display(description=_("Область"), ordering="department_id")
|
||
def scope_display(self, obj: CalendarEvent) -> str:
|
||
"""Возвращает 'Компания' или название отдела для строки списка.
|
||
|
||
Args:
|
||
obj: Объект CalendarEvent.
|
||
|
||
Returns:
|
||
str: Человекочитаемое имя области.
|
||
"""
|
||
return _("Компания") if obj.department_id is None else str(obj.department)
|
||
|
||
@admin.display(description=_("Цвет"))
|
||
def color_swatch(self, obj: CalendarEvent) -> str:
|
||
"""Отрисовывает mini-swatch цвета + hex-код.
|
||
|
||
Args:
|
||
obj: Объект CalendarEvent.
|
||
|
||
Returns:
|
||
str: HTML.
|
||
"""
|
||
if not obj.color:
|
||
return "—"
|
||
return format_html(
|
||
'<span style="display:inline-block;width:12px;height:12px;'
|
||
'border-radius:2px;border:1px solid #ccc;vertical-align:-2px;'
|
||
'background:{};margin-right:6px;"></span><code>{}</code>',
|
||
obj.color, obj.color
|
||
)
|
||
|
||
# ===== Экшены логика =====
|
||
@admin.action(description=_("Сделать целодневными (очистить время)"))
|
||
def make_all_day(self, request: HttpRequest, queryset: QuerySet[CalendarEvent]) -> None:
|
||
"""Делает выбранные события целодневными.
|
||
|
||
Args:
|
||
request: Запрос администратора.
|
||
queryset: Выбранные события.
|
||
"""
|
||
updated = queryset.update(start_time=None, end_time=None, all_day=True)
|
||
self.message_user(request, _(f"Обновлено событий: {updated}"), level=messages.SUCCESS)
|
||
|
||
@admin.action(description=_("Очистить время (оставить даты как есть)"))
|
||
def clear_time(self, request: HttpRequest, queryset: QuerySet[CalendarEvent]) -> None:
|
||
"""Очищает время начала/окончания у выбранных событий.
|
||
|
||
Args:
|
||
request: Запрос администратора.
|
||
queryset: Выбранные события.
|
||
"""
|
||
updated = queryset.update(start_time=None, end_time=None)
|
||
self.message_user(request, _(f"Очищено время у событий: {updated}"), level=messages.SUCCESS)
|
||
|
||
@admin.action(description=_("Пометить как ежегодные"))
|
||
def set_annual(self, request: HttpRequest, queryset: QuerySet[CalendarEvent]) -> None:
|
||
"""Ставит повторяемость 'Ежегодно' у выбранных событий."""
|
||
updated = queryset.update(recurrence=Recurrence.ANNUAL)
|
||
self.message_user(request, _(f"Обновлено: {updated}"), level=messages.SUCCESS)
|
||
|
||
@admin.action(description=_("Пометить как одноразовые"))
|
||
def set_one_time(self, request: HttpRequest, queryset: QuerySet[CalendarEvent]) -> None:
|
||
"""Ставит повторяемость 'Одноразовое' у выбранных событий."""
|
||
updated = queryset.update(recurrence=Recurrence.ONE_TIME)
|
||
self.message_user(request, _(f"Обновлено: {updated}"), level=messages.SUCCESS)
|
||
|
||
# ===== Сохранение =====
|
||
def save_model(self, request: HttpRequest, obj: CalendarEvent, form, change: bool) -> None:
|
||
"""Проставляет автора при первом сохранении.
|
||
|
||
Args:
|
||
request: Текущий запрос.
|
||
obj: Объект события.
|
||
form: Форма админки.
|
||
change: True, если редактирование; False, если создание.
|
||
"""
|
||
if not change and obj.created_by_id is None:
|
||
obj.created_by = request.user
|
||
super().save_model(request, obj, form, change)
|