143 lines
7.1 KiB
Markdown
143 lines
7.1 KiB
Markdown
# Notification de fin de contrat (veille du dernier jour) — Design
|
|
|
|
**Date :** 2026-06-24
|
|
**Branche :** feature/SIRH-43-ajouter-une-notif-la-veille-d-un-contrat-qui-se-te
|
|
**Statut :** Validé (brainstorming)
|
|
|
|
## Objectif
|
|
|
|
Prévenir automatiquement **les administrateurs**, sur le **dernier jour ouvré précédant la fin
|
|
d'un contrat**, qu'un salarié arrive au terme de son emploi — afin qu'ils puissent anticiper
|
|
(solde de tout compte, désactivation des accès, etc.).
|
|
|
|
La notification réutilise le **système de notification existant** (entité `Notification`, cloche
|
|
admin dans `AppTopNav.vue`). Aucune migration de base de données.
|
|
|
|
## Décisions de cadrage
|
|
|
|
| Sujet | Décision |
|
|
|---|---|
|
|
| **Déclencheur** | Vraie fin d'emploi uniquement : la période qui se termine est la **dernière** période de contrat du salarié (aucune période ne lui succède). Un changement de contrat enchaîné (ex. CDD 35h → CDI 39h) ne notifie pas. |
|
|
| **Timing** | Le **dernier jour ouvré strictement avant** `endDate`, en sautant **week-ends ET jours fériés**. |
|
|
| **Message** | « Fin de {nature} de {Prénom Nom} le {dd/mm/yyyy} » — nature = libellé FR (CDI / CDD / Intérim). |
|
|
| **Catégorie** | `Contrat` |
|
|
| **Cible du clic** | `/employees/{id}` (fiche employé). |
|
|
| **Destinataires** | Tous les `ROLE_ADMIN` (`UserRepository::findAllAdmins()`). |
|
|
| **Acteur** | `null` (notif générée par un job automatique, pas par un utilisateur). |
|
|
| **Déclenchement** | Commande console quotidienne via crontab prod (~6h du matin). |
|
|
|
|
## Rappels sur l'existant
|
|
|
|
- `Notification` (table `notifications`) : `recipient` (NOT NULL), `actor` (nullable),
|
|
`message`, `category`, `target`, `isRead`, `createdAt`. Exposé via `getActorName()`.
|
|
- `endDate` d'une `EmployeeContractPeriod` est **inclusif** : c'est le dernier jour couvert
|
|
par le contrat (`findOneCoveringDate` : `endDate >= :date`).
|
|
- Pattern de notif existant : `WorkHourSiteValidationProcessor` crée une `Notification` par
|
|
admin (`findAllAdmins`) avec `new Notification()` + persist + flush. Pas de service factory.
|
|
- Pattern cron existant : `RttRolloverCommand` / `LeaveRolloverCommand` (`#[AsCommand]`,
|
|
logger `monolog.logger.cron`, options `--force`/`--recompute`). Déclenchées par le crontab
|
|
système (pas de Symfony Scheduler dans le projet).
|
|
- `PublicHolidayService` : source des fériés (cache 30j), déjà en place.
|
|
|
|
## Approche retenue
|
|
|
|
**Commande cron quotidienne + service métier dédié et testable.**
|
|
|
|
Alternatives écartées :
|
|
- **Symfony Scheduler (Messenger)** : brique non utilisée dans le projet, inutile ici.
|
|
- **Calcul à la volée dans le provider** : casse `isRead`/historique, recalcul à chaque
|
|
ouverture de la cloche, mélange notifs persistées et virtuelles.
|
|
|
|
## Conception détaillée
|
|
|
|
### 1. Détection (cœur métier)
|
|
|
|
Nouveau service `App\Service\Notification\ContractEndNotificationService`.
|
|
|
|
Algorithme (date du jour `T` injectable pour les tests) :
|
|
|
|
1. Si `T` est un week-end ou un férié → **sortie** (aucun jour chômé ne génère de notif).
|
|
2. Calculer `N` = **prochain jour ouvré strictement après `T`** (saute week-ends + fériés via
|
|
`PublicHolidayService`).
|
|
3. Charger en **une seule requête** la dernière période de chaque employé
|
|
(`EmployeeContractPeriodRepository::findLatestPeriodsForAllEmployees()`).
|
|
4. Candidat = dernière période dont `endDate` est non nul et vérifie `T < endDate <= N`.
|
|
- Le test « dernière période » assure nativement la règle « vraie fin d'emploi » : un
|
|
changement de contrat enchaîné a une période suivante, donc n'est jamais la dernière.
|
|
- `endDate = null` (CDI ouvert) → jamais candidat.
|
|
|
|
**Exemples :**
|
|
- Mardi (`T`), `N` = mercredi → notifie les contrats finissant mercredi (J-1 classique).
|
|
- Vendredi (`T`), `N` = lundi → notifie les contrats finissant samedi, dimanche **ou** lundi.
|
|
- Vendredi (`T`), lundi férié → `N` = mardi → notifie samedi…mardi.
|
|
- Week-end (`T`) → rien.
|
|
|
|
### 2. Création des notifications & idempotence
|
|
|
|
Pour chaque candidat :
|
|
- Message : `Fin de {nature} de {Prénom Nom} le {endDate->format('d/m/Y')}`.
|
|
- Une `Notification` par admin : `recipient=admin`, `actor=null`, `category='Contrat'`,
|
|
`target='/employees/{id}'`. Persist groupé, un seul `flush()` final.
|
|
|
|
**Idempotence** (le job peut être relancé le même jour) : avant création, vérifier qu'il
|
|
n'existe pas déjà une notif identique pour ce destinataire via
|
|
`(recipient, category='Contrat', target='/employees/{id}', message)` **exact**. Le message
|
|
étant unique par (employé + date + nature), cela empêche tout doublon — y compris après que
|
|
l'admin a lu la notif (`isRead=true`). Nouvelle méthode
|
|
`NotificationRepository::existsForRecipientTargetMessage(...)` (ou `findOneBy`). Pas de
|
|
migration : on ne stocke pas de FK « période » sur `Notification`.
|
|
|
|
### 3. Affichage front (cloche)
|
|
|
|
`AppTopNav.vue` rend aujourd'hui `**{actorName}** {message}`. Pour `actorName` vide :
|
|
- N'afficher que `{message}` (pas de span gras vide ni `capitalize` orphelin).
|
|
- Le reste est inchangé : avatar/pastille, `formatTimeAgo` + catégorie « Contrat », point
|
|
non-lu, lien `target`.
|
|
|
|
Aucune route ni service front nouveau : `category` et `target` passent par le DTO existant.
|
|
Cloche déjà admin-only → rien d'autre côté visibilité.
|
|
|
|
### 4. Commande console
|
|
|
|
`App\Command\ContractEndNotificationCommand` — `app:contract:end-notifications`.
|
|
- Délègue tout au service.
|
|
- Option `--date=YYYY-MM-DD` : forcer la date du jour (tests / rattrapage manuel).
|
|
- Logger `monolog.logger.cron`. Sortie `SymfonyStyle` (nb de notifs créées / employés concernés).
|
|
- **Idempotente** par construction (cf. §2) → relançable sans risque.
|
|
- Crontab prod (infra) : `0 6 * * *` (tous les jours, 6h). Pas de restriction jour de semaine
|
|
(la commande s'auto-neutralise week-ends/fériés).
|
|
|
|
## Fichiers
|
|
|
|
**Backend — nouveaux**
|
|
- `src/Service/Notification/ContractEndNotificationService.php`
|
|
- `src/Command/ContractEndNotificationCommand.php`
|
|
- `tests/...` (tests du service)
|
|
|
|
**Backend — modifiés**
|
|
- `src/Repository/EmployeeContractPeriodRepository.php` — `findLatestPeriodsForAllEmployees()`
|
|
- `src/Repository/NotificationRepository.php` — existence anti-doublon
|
|
- Helper « jour ouvré » (week-end + férié) — dans le service ou petit util réutilisable
|
|
|
|
**Frontend — modifié**
|
|
- `frontend/components/AppTopNav.vue` — gérer `actorName` vide
|
|
|
|
**Docs**
|
|
- `doc/functional-rules.md` — compléter la section 15) Notifications
|
|
- `doc/contract-end-notifications.md` — nouveau (règle complète)
|
|
- `frontend/data/documentation-content.ts` — entrée admin
|
|
- `CLAUDE.md` — note du nouveau pattern (commande cron de notification)
|
|
|
|
Pas de migration DB.
|
|
|
|
## Tests (PHPUnit)
|
|
|
|
Cœur isolé dans `ContractEndNotificationService` (date `T` fixe + `PublicHolidayService` mocké) :
|
|
- Fin mercredi, `T`=mardi → 1 notif/admin.
|
|
- Fin lundi, `T`=vendredi → notifié vendredi ; rien samedi/dimanche.
|
|
- Lundi férié + fin mardi, `T`=vendredi → notifié vendredi (`N` saute le lundi férié).
|
|
- Période suivante existante (changement enchaîné) → pas de notif.
|
|
- `endDate=null` → pas de notif.
|
|
- Idempotence : 2ᵉ exécution même jour → aucun doublon.
|
|
- `T` = week-end → aucune création.
|