chore: bump version to v0.1.89

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

CLAUDE.md

@@ -44,12 +44,6 @@
 - Saisie d'heures (ou de jours de présence) autorisée sur un férié
 - **Crédit automatique des heures contractuelles** sur un férié Lun-Ven pour tout contrat hors Forfait, **uniquement en l'absence d'absence déclarée** : le total journalier = `max(saisie + credited_absence, référence_contractuelle)`. Référence : 35h→7h, 39h→8h Lun-Jeu/7h Ven, CUSTOM→weeklyHours/5, INTERIM→idem 35h/39h/custom selon weeklyHours. Aucune ligne BDD créée (crédit virtuel). Drivers : crédité en `dayHoursMinutes`. Impacte directement le total hebdo RTT (tranches 25%/50%). Dès qu'une absence est posée sur le férié, le crédit virtuel saute — c'est le `countAsWorkedHours` du type d'absence qui pilote. Services : `App\Service\WorkHours\HolidayVirtualHoursResolver` + `DailyReferenceMinutesResolver`. Doc complète : `doc/holiday-virtual-hours.md`.
 
-## Commentaires de semaine
-- Entité `EmployeeWeekComment` : commentaire libre par employé et semaine ISO (unique `(employee_id, week_start_date)`). `week_start_date` = lundi.
-- CRUD `/employee_week_comments` `ROLE_ADMIN`. Write processor audite via `AuditLogger`.
-- Picto bulle vue semaine (HoursWeekView + DriverHoursWeekView) : fond bleu/rouge. Intégré dans `WeeklySummaryRow.comment/commentId`.
-- Doc : `doc/week-comments.md`.
-
 ## Validation Rules
 - `isValid` (RH): locks line for everyone (admin can only untoggle validation)
 - `isSiteValid` (site manager): locks for non-admin, admin can still edit

config/version.yaml

@@ -1,2 +1,2 @@
 parameters:
-    app.version: '0.1.88'
+    app.version: '0.1.89'

doc/functional-rules.md

@@ -166,7 +166,7 @@ Documents complementaires:
 - Onglet congés: jours fériés affichés sur le calendrier avec fond `rgb(179, 229, 252)` et nom au survol
 - Écran Heures et Heures Conducteurs (vue jour): le nom du férié est affiché dans la colonne Absence sous forme de pill (fond `#b3e5fc`, icône `mdi:calendar-star`), distinct du pill absence
 - Règle courante:
-  - absences bloquées sur jour férié (création/édition) — bouton "Modifier" masqué comme pour les formations
+  - absences autorisées sur jour férié (création/édition depuis l'écran Heures et le Calendrier). Quand une absence est posée, le crédit virtuel férié est désactivé — c'est le `countAsWorkedHours` du type d'absence qui pilote
   - saisie d'heures ou de jours de présence autorisée — les heures saisies comptent normalement dans le total hebdo et le calcul RTT
   - la référence hebdomadaire n'est pas réduite par un férié: un salarié qui ne saisit rien sur un férié est en déficit de la journée correspondante
 

frontend/components/CalendarGrid.vue

@@ -45,9 +45,9 @@
                             <button
                                 type="button"
                                 class="relative flex h-8 w-full items-center justify-center overflow-hidden rounded-md border border-neutral-200 text-[11px] font-semibold text-neutral-800"
-                                :class="isHolidayDate(day.date) || getCellInfo(employee.id, day.date)?.hasFormation ? 'cursor-not-allowed opacity-80' : ''"
+                                :class="getCellInfo(employee.id, day.date)?.hasFormation ? 'cursor-not-allowed opacity-80' : ''"
                                 :style="getCellStyle(employee.id, day.date)"
-                                :disabled="isHolidayDate(day.date) || getCellInfo(employee.id, day.date)?.hasFormation"
+                                :disabled="getCellInfo(employee.id, day.date)?.hasFormation"
                                 @click="handleCellClick(employee, day.date)"
                             >
                                 <span v-if="!getCellInfo(employee.id, day.date)?.halfLabel">
@@ -80,9 +80,7 @@
                             <button
                                 type="button"
                                 class="relative flex h-8 w-full items-center justify-center rounded-md border border-neutral-200 text-[11px] font-semibold text-neutral-800 bg-white"
-                                :class="isHolidayDate(day.date) ? 'cursor-not-allowed opacity-80' : ''"
                                 :style="getCellStyle(employee.id, day.date)"
-                                :disabled="isHolidayDate(day.date)"
                                 @click="handleCellClick(employee, day.date)"
                             >
                                 <span></span>

frontend/components/driver-hours/DriverHoursWeekView.vue

@@ -33,11 +33,8 @@
             {{ row.firstName }} {{ row.lastName }}
             <span class="font-normal text-neutral-600">({{ row.contractName ?? '-' }})</span>
           </p>
-          <p class="text-[11px] text-neutral-500 truncate inline-flex items-center gap-2">
-            <span>{{ row.siteName ?? 'Sans site' }}<span v-if="row.contractNature"> — {{ contractNatureLabel(row.contractNature) }}</span></span>
-            <button v-if="isAdmin" type="button" class="inline-flex items-center justify-center rounded-md p-1 text-white transition-colors" :class="row.comment ? 'bg-red-500 hover:bg-red-600' : 'bg-primary-500 hover:bg-secondary-500'" :title="row.comment ?? 'Ajouter un commentaire'" @click="$emit('open-comment', row)">
-              <Icon name="mdi:comment-text-outline" size="12"/>
-            </button>
+          <p class="text-[11px] text-neutral-500 truncate">
+            {{ row.siteName ?? 'Sans site' }}<span v-if="row.contractNature"> — {{ contractNatureLabel(row.contractNature) }}</span>
           </p>
         </div>
 
@@ -106,12 +103,9 @@ const getDailyCellStyle = (daily: {
 
 defineProps<{
   isWeekLoading: boolean
-  isAdmin: boolean
   weekGridCols: string
   weeklySummary: WeeklyWorkHourSummary | null
   weekDayHeaders: Array<{ date: string; weekday: string; dayDate: string }>
   formatMinutes: (minutes: number) => string
 }>()
-
-defineEmits<{ (e: 'open-comment', row: WeeklyWorkHourSummary['rows'][number]): void }>()
 </script>

frontend/components/hours/HoursWeekView.vue

@@ -29,11 +29,8 @@
             {{ row.firstName }} {{ row.lastName }}
             <span class="font-normal text-neutral-600">({{ row.contractName ?? '-' }})</span>
           </p>
-          <p class="text-[11px] text-neutral-500 truncate inline-flex items-center gap-2">
-            <span>{{ row.siteName ?? 'Sans site' }}<span v-if="row.contractNature"> — {{ contractNatureLabel(row.contractNature) }}</span></span>
-            <button v-if="isAdmin" type="button" class="inline-flex items-center justify-center rounded-md p-1 text-white transition-colors" :class="row.comment ? 'bg-red-500 hover:bg-red-600' : 'bg-primary-500 hover:bg-secondary-500'" :title="row.comment ?? 'Ajouter un commentaire'" @click="$emit('open-comment', row)">
-              <Icon name="mdi:comment-text-outline" size="12"/>
-            </button>
+          <p class="text-[11px] text-neutral-500 truncate">
+            {{ row.siteName ?? 'Sans site' }}<span v-if="row.contractNature"> — {{ contractNatureLabel(row.contractNature) }}</span>
           </p>
         </div>
 
@@ -102,12 +99,9 @@ const getDailyCellStyle = (daily: {
 
 defineProps<{
   isWeekLoading: boolean
-  isAdmin: boolean
   weekGridCols: string
   weeklySummary: WeeklyWorkHourSummary | null
   weekDayHeaders: Array<{ date: string; label: string }>
   formatMinutes: (minutes: number) => string
 }>()
-
-defineEmits<{ (e: 'open-comment', row: WeeklyWorkHourSummary['rows'][number]): void }>()
 </script>

frontend/components/hours/WeekCommentDrawer.vue

@@ -1,67 +0,0 @@
-<template>
-  <AppDrawer v-model="drawerOpen" :title="`Commentaire — ${formatWeekRange}`">
-    <form class="space-y-4" @submit.prevent="onSave">
-      <div v-if="employeeLabel" class="text-md font-semibold text-neutral-700">{{ employeeLabel }}</div>
-      <div>
-        <label class="text-md font-semibold text-neutral-700" for="week-comment-content">Commentaire</label>
-        <textarea id="week-comment-content" v-model="content" rows="8" maxlength="5000" class="mt-2 w-full rounded-md border border-neutral-300 bg-white px-3 py-2 text-base text-neutral-900 focus:border-primary-500 focus:outline-none focus:ring-2 focus:ring-secondary-500/20" placeholder="Ex. Arrêt maladie lundi, reprise jeudi..." />
-        <p class="mt-1 text-xs text-neutral-400">{{ content.length }} / 5000</p>
-      </div>
-      <div class="sticky bottom-0 -mx-[20px] bg-white px-[20px] py-3 flex justify-between gap-3">
-        <button v-if="commentId" type="button" class="rounded-lg bg-red-500 px-4 py-2 text-md font-semibold text-white hover:bg-red-600 disabled:opacity-50" :disabled="isSubmitting" @click="onDelete">Supprimer</button>
-        <div class="flex gap-3 ml-auto">
-          <button type="button" class="rounded-lg border border-neutral-200 px-4 py-2 text-md font-semibold text-neutral-700 hover:bg-neutral-100" @click="drawerOpen = false">Annuler</button>
-          <button type="submit" class="rounded-lg bg-primary-500 px-4 py-2 text-md font-semibold text-white hover:bg-secondary-500 disabled:opacity-50" :disabled="isSubmitting || !canSubmit">Enregistrer</button>
-        </div>
-      </div>
-    </form>
-  </AppDrawer>
-</template>
-
-<script setup lang="ts">
-import { computed, ref, watch } from 'vue'
-import AppDrawer from '~/components/AppDrawer.vue'
-import { createWeekComment, deleteWeekComment, updateWeekComment } from '~/services/employee-week-comments'
-
-const props = defineProps<{
-  modelValue: boolean
-  employeeId: number | null
-  weekStart: string
-  weekEnd: string
-  initialContent: string
-  commentId: number | null
-  employeeLabel?: string
-}>()
-const emit = defineEmits<{ (e: 'update:modelValue', v: boolean): void; (e: 'saved'): void }>()
-
-const drawerOpen = computed({ get: () => props.modelValue, set: (v) => emit('update:modelValue', v) })
-const content = ref('')
-const isSubmitting = ref(false)
-
-watch(() => [props.modelValue, props.initialContent] as const, ([open, init]) => { if (open) content.value = init ?? '' }, { immediate: true })
-
-const formatWeekRange = computed(() => {
-  const fmt = (ymd: string) => { const p = ymd.split('-'); return p.length === 3 ? `${p[2]}/${p[1]}/${p[0]}` : ymd }
-  return `${fmt(props.weekStart)} → ${fmt(props.weekEnd)}`
-})
-
-const canSubmit = computed(() => content.value.trim().length > 0 || props.commentId !== null)
-
-const onSave = async () => {
-  if (!props.employeeId || isSubmitting.value) return
-  const trimmed = content.value.trim()
-  isSubmitting.value = true
-  try {
-    if (trimmed === '' && props.commentId) await deleteWeekComment(props.commentId)
-    else if (trimmed !== '' && props.commentId) await updateWeekComment(props.commentId, trimmed)
-    else if (trimmed !== '') await createWeekComment({ employeeId: props.employeeId, weekStartDate: props.weekStart, content: trimmed })
-    emit('saved'); drawerOpen.value = false
-  } finally { isSubmitting.value = false }
-}
-
-const onDelete = async () => {
-  if (!props.commentId || isSubmitting.value) return
-  isSubmitting.value = true
-  try { await deleteWeekComment(props.commentId); emit('saved'); drawerOpen.value = false } finally { isSubmitting.value = false }
-}
-</script>

frontend/composables/useDriverHoursPage.ts

@@ -922,15 +922,6 @@ export const useDriverHoursPage = () => {
     }
   }
 
-  const isWeekCommentDrawerOpen = ref(false)
-  const weekCommentContext = ref<{ employeeId: number; employeeLabel: string; weekStart: string; weekEnd: string; content: string; commentId: number | null } | null>(null)
-  const openWeekCommentDrawer = (row: { employeeId: number; firstName: string; lastName: string; comment?: string | null; commentId?: number | null }) => {
-    if (!weeklySummary.value) return
-    weekCommentContext.value = { employeeId: row.employeeId, employeeLabel: `${row.firstName} ${row.lastName}`.trim(), weekStart: weeklySummary.value.weekStart, weekEnd: weeklySummary.value.weekEnd, content: row.comment ?? '', commentId: row.commentId ?? null }
-    isWeekCommentDrawerOpen.value = true
-  }
-  const reloadWeeklySummary = async () => { await loadWeeklySummary() }
-
   return {
     isAdmin,
     isSelfUser,
@@ -997,10 +988,6 @@ export const useDriverHoursPage = () => {
     deleteAbsenceFromDrawer,
     closeAbsenceDrawer,
     formatMinutes,
-    handleSave,
-    isWeekCommentDrawerOpen,
-    weekCommentContext,
-    openWeekCommentDrawer,
-    reloadWeeklySummary
+    handleSave
   }
 }

frontend/composables/useHoursPage.ts

@@ -1108,15 +1108,6 @@ export const useHoursPage = () => {
     }
   }
 
-  const isWeekCommentDrawerOpen = ref(false)
-  const weekCommentContext = ref<{ employeeId: number; employeeLabel: string; weekStart: string; weekEnd: string; content: string; commentId: number | null } | null>(null)
-  const openWeekCommentDrawer = (row: { employeeId: number; firstName: string; lastName: string; comment?: string | null; commentId?: number | null }) => {
-    if (!weeklySummary.value) return
-    weekCommentContext.value = { employeeId: row.employeeId, employeeLabel: `${row.firstName} ${row.lastName}`.trim(), weekStart: weeklySummary.value.weekStart, weekEnd: weeklySummary.value.weekEnd, content: row.comment ?? '', commentId: row.commentId ?? null }
-    isWeekCommentDrawerOpen.value = true
-  }
-  const reloadWeeklySummary = async () => { await loadWeeklySummary() }
-
   return {
     isAdmin,
     isSelfUser,
@@ -1190,10 +1181,6 @@ export const useHoursPage = () => {
     deleteAbsenceFromDrawer,
     closeAbsenceDrawer,
     formatMinutes,
-    handleSave,
-    isWeekCommentDrawerOpen,
-    weekCommentContext,
-    openWeekCommentDrawer,
-    reloadWeeklySummary
+    handleSave
   }
 }

frontend/data/documentation-content.ts

@@ -79,16 +79,6 @@ export const documentationSections: DocSection[] = [
           { type: 'list', content: 'Jour : total des heures dans la plage 06:00–21:00\nNuit : total des heures dans les plages 00:00–06:00 et 21:00–24:00\nTotal : somme des heures de jour et de nuit' },
         ],
       },
-      {
-        id: 'commentaire-semaine',
-        title: 'Commentaires de semaine (admin)',
-        requiredLevel: 'admin',
-        blocks: [
-          { type: 'paragraph', content: 'Sur la vue semaine, un picto bulle permet d\'attacher un commentaire libre sur la semaine d\'un employé.' },
-          { type: 'list', content: 'Bulle bleue : pas de commentaire\nBulle rouge : un commentaire existe\nClic : ouvre le drawer avec textarea' },
-          { type: 'note', content: 'Les commentaires n\'affectent aucun calcul. Pour supprimer, videz la textarea puis Enregistrer, ou bouton Supprimer.' },
-        ],
-      },
     ],
   },
   {
@@ -397,7 +387,8 @@ export const documentationSections: DocSection[] = [
         requiredLevel: 'admin',
         blocks: [
           { type: 'paragraph', content: 'Le calendrier offre une vue d\'ensemble mensuelle des absences de tous les employés.' },
-          { type: 'list', content: 'Code couleur par type d\'absence\nDemi-journée : affichage en dégradé diagonal\nJournée complète : fond plein\nJours fériés : fond bleu clair\nFiltres par site et par employé\nNavigation par mois (précédent / suivant)' },
+          { type: 'list', content: 'Code couleur par type d\'absence\nDemi-journée : affichage en dégradé diagonal\nJournée complète : fond plein\nJours fériés : fond bleu clair (cliquable pour créer une absence)\nFiltres par site et par employé\nNavigation par mois (précédent / suivant)' },
+          { type: 'note', content: 'Les absences peuvent être créées sur les jours fériés. Quand une absence est posée sur un férié, elle remplace l\'affichage « Férié » dans la cellule.' },
         ],
       },
     ],

frontend/pages/calendar.vue

@@ -490,14 +490,15 @@ const hasFormationOn = (employeeId: number, date: string): boolean => {
     return cellFormationMap.value.has(`${employeeId}-${date}`)
 }
 
-// Jours fériés (interdit pour la création).
+// Jours fériés.
 const isHolidayDate = (date: string) => {
     return Boolean(publicHolidays.value[date])
 }
 
-// Renvoie l'absence effective pour une cellule (ou un "Férié").
+// Renvoie l'absence effective pour une cellule (ou un "Férié" si pas d'absence).
 const getCellAbsence = (employeeId: number, date: string) => {
-    if (isHolidayDate(date)) {
+    const absence = cellAbsenceMap.value.get(`${employeeId}-${date}`)
+    if (!absence && isHolidayDate(date)) {
         return {
             id: 0,
             code: 'Férié',
@@ -505,7 +506,6 @@ const getCellAbsence = (employeeId: number, date: string) => {
             textColor: '#0f172a'
         }
     }
-    const absence = cellAbsenceMap.value.get(`${employeeId}-${date}`)
     if (absence) return { ...absence, hasFormation: hasFormationOn(employeeId, date) }
     if (hasFormationOn(employeeId, date)) {
         return {
@@ -549,11 +549,6 @@ const getCellInfo = (employeeId: number, date: string) => {
 
 // Ouverture du drawer depuis une cellule.
 const openCreate = (employee: Employee, date: string) => {
-    if (isHolidayDate(date)) {
-        window.alert("Impossible de creer une absence un jour ferie.")
-        return
-    }
-
     const existing = absences.value.find((absence) => {
         const start = normalizeDate(absence.startDate)
         const end = normalizeDate(absence.endDate)
@@ -590,10 +585,6 @@ const openCreateFromToday = () => {
     form.typeId = ''
     const now = new Date()
     const today = toYmd(now.getFullYear(), now.getMonth(), now.getDate())
-    if (isHolidayDate(today)) {
-        window.alert("Impossible de creer une absence un jour ferie.")
-        return
-    }
     form.startDate = today
     form.endDate = today
     form.startHalf = 'AM'

frontend/pages/driver-hours.vue

@@ -73,13 +73,11 @@
                 <DriverHoursWeekView
                     v-else-if="isAdmin && viewMode === 'week'"
                     :is-week-loading="isWeekLoading"
-                    :is-admin="isAdmin"
                     :week-grid-cols="weekGridCols"
                     :weekly-summary="filteredWeeklySummary"
                     :week-day-headers="weekDayHeaders"
                     :format-minutes="formatMinutes"
                     class="max-h-[calc(100vh-300px)]"
-                    @open-comment="openWeekCommentDrawer"
                 />
             </div>
 
@@ -111,17 +109,6 @@
             @cancel="closeAbsenceDrawer"
         />
 
-        <HoursWeekCommentDrawer
-            v-if="weekCommentContext"
-            v-model="isWeekCommentDrawerOpen"
-            :employee-id="weekCommentContext.employeeId"
-            :employee-label="weekCommentContext.employeeLabel"
-            :week-start="weekCommentContext.weekStart"
-            :week-end="weekCommentContext.weekEnd"
-            :initial-content="weekCommentContext.content"
-            :comment-id="weekCommentContext.commentId"
-            @saved="reloadWeeklySummary"
-        />
     </div>
 </template>
 
@@ -190,11 +177,7 @@ const {
   formatMinutes,
   isSelectedDateHoliday,
   selectedHolidayLabel,
-  handleSave,
-  isWeekCommentDrawerOpen,
-  weekCommentContext,
-  openWeekCommentDrawer,
-  reloadWeeklySummary
+  handleSave
 } = useDriverHoursPage()
 
 useHead({

frontend/pages/hours.vue

@@ -80,13 +80,11 @@
                 <HoursWeekView
                     v-else-if="isAdmin && viewMode === 'week'"
                     :is-week-loading="isWeekLoading"
-                    :is-admin="isAdmin"
                     :week-grid-cols="weekGridCols"
                     :weekly-summary="filteredWeeklySummary"
                     :week-day-headers="weekDayHeaders"
                     :format-minutes="formatMinutes"
                     class="max-h-[calc(100vh-300px)]"
-                    @open-comment="openWeekCommentDrawer"
                 />
             </div>
 
@@ -118,17 +116,6 @@
             @cancel="closeAbsenceDrawer"
         />
 
-        <HoursWeekCommentDrawer
-            v-if="weekCommentContext"
-            v-model="isWeekCommentDrawerOpen"
-            :employee-id="weekCommentContext.employeeId"
-            :employee-label="weekCommentContext.employeeLabel"
-            :week-start="weekCommentContext.weekStart"
-            :week-end="weekCommentContext.weekEnd"
-            :initial-content="weekCommentContext.content"
-            :comment-id="weekCommentContext.commentId"
-            @saved="reloadWeeklySummary"
-        />
     </div>
 </template>
 
@@ -204,11 +191,7 @@ const {
   deleteAbsenceFromDrawer,
   closeAbsenceDrawer,
   formatMinutes,
-  handleSave,
-  isWeekCommentDrawerOpen,
-  weekCommentContext,
-  openWeekCommentDrawer,
-  reloadWeeklySummary
+  handleSave
 } = useHoursPage()
 
 useHead({

frontend/services/dto/work-hour.ts

@@ -88,8 +88,6 @@ export type WeeklyWorkHourRowSummary = {
   weeklyOvernightCount?: number
   hasContractForWeek?: boolean
   contractNature?: 'CDI' | 'CDD' | 'INTERIM' | null
-  comment?: string | null
-  commentId?: number | null
 }
 
 export type WeeklyWorkHourSummary = {

frontend/services/employee-week-comments.ts

@@ -1,24 +0,0 @@
-export type EmployeeWeekComment = {
-  id: number
-  weekStartDate: string
-  content: string
-}
-
-export const createWeekComment = async (payload: { employeeId: number; weekStartDate: string; content: string }) => {
-  const api = useApi()
-  return api.post<EmployeeWeekComment>('/employee_week_comments', {
-    employee: `/api/employees/${payload.employeeId}`,
-    weekStartDate: payload.weekStartDate,
-    content: payload.content
-  }, { toastSuccessKey: 'success.week-comment.save', toastErrorKey: 'errors.week-comment.save' })
-}
-
-export const updateWeekComment = async (id: number, content: string) => {
-  const api = useApi()
-  return api.patch<EmployeeWeekComment>(`/employee_week_comments/${id}`, { content }, { toastSuccessKey: 'success.week-comment.save', toastErrorKey: 'errors.week-comment.save' })
-}
-
-export const deleteWeekComment = async (id: number) => {
-  const api = useApi()
-  await api.delete(`/employee_week_comments/${id}`, {}, { toastSuccessKey: 'success.week-comment.delete', toastErrorKey: 'errors.week-comment.delete' })
-}

migrations/Version20260417100000.php

@@ -1,29 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace DoctrineMigrations;
-
-use Doctrine\DBAL\Schema\Schema;
-use Doctrine\Migrations\AbstractMigration;
-
-final class Version20260417100000 extends AbstractMigration
-{
-    public function getDescription(): string
-    {
-        return 'Create employee_week_comments table for per-week admin annotations on the hours weekly view';
-    }
-
-    public function up(Schema $schema): void
-    {
-        $this->addSql('CREATE TABLE employee_week_comments (id SERIAL NOT NULL, employee_id INT NOT NULL, week_start_date DATE NOT NULL, content TEXT NOT NULL, created_at TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, updated_at TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, PRIMARY KEY(id))');
-        $this->addSql('CREATE UNIQUE INDEX uniq_employee_week_comment ON employee_week_comments (employee_id, week_start_date)');
-        $this->addSql('CREATE INDEX idx_ewc_week_start ON employee_week_comments (week_start_date)');
-        $this->addSql('ALTER TABLE employee_week_comments ADD CONSTRAINT fk_ewc_employee FOREIGN KEY (employee_id) REFERENCES employees(id) ON DELETE CASCADE');
-    }
-
-    public function down(Schema $schema): void
-    {
-        $this->addSql('DROP TABLE employee_week_comments');
-    }
-}

src/Dto/WorkHours/WeeklySummaryRow.php

@@ -35,7 +35,5 @@ final class WeeklySummaryRow
         public int $weeklyOvernightCount = 0,
         public bool $hasContractForWeek = true,
         public ?string $contractNature = null,
-        public ?string $comment = null,
-        public ?int $commentId = null,
     ) {}
 }

src/Entity/EmployeeWeekComment.php

@@ -1,136 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace App\Entity;
-
-use ApiPlatform\Doctrine\Orm\Filter\DateFilter;
-use ApiPlatform\Doctrine\Orm\Filter\SearchFilter;
-use ApiPlatform\Metadata\ApiFilter;
-use ApiPlatform\Metadata\ApiResource;
-use ApiPlatform\Metadata\Delete;
-use ApiPlatform\Metadata\Get;
-use ApiPlatform\Metadata\GetCollection;
-use ApiPlatform\Metadata\Patch;
-use ApiPlatform\Metadata\Post;
-use App\Repository\EmployeeWeekCommentRepository;
-use App\State\EmployeeWeekCommentWriteProcessor;
-use DateTimeImmutable;
-use Doctrine\ORM\Mapping as ORM;
-use Symfony\Component\Serializer\Attribute\Groups;
-use Symfony\Component\Validator\Constraints as Assert;
-
-#[ApiResource(
-    operations: [
-        new Get(security: "is_granted('ROLE_ADMIN')"),
-        new GetCollection(security: "is_granted('ROLE_ADMIN')"),
-        new Post(security: "is_granted('ROLE_ADMIN')", processor: EmployeeWeekCommentWriteProcessor::class),
-        new Patch(security: "is_granted('ROLE_ADMIN')", processor: EmployeeWeekCommentWriteProcessor::class),
-        new Delete(security: "is_granted('ROLE_ADMIN')", processor: EmployeeWeekCommentWriteProcessor::class),
-    ],
-    normalizationContext: ['groups' => ['week_comment:read'], 'datetime_format' => 'Y-m-d'],
-    denormalizationContext: ['groups' => ['week_comment:write'], 'datetime_format' => 'Y-m-d'],
-    order: ['weekStartDate' => 'DESC'],
-    paginationEnabled: false,
-)]
-#[ApiFilter(DateFilter::class, properties: ['weekStartDate'])]
-#[ApiFilter(SearchFilter::class, properties: ['employee' => 'exact'])]
-#[ORM\Entity(repositoryClass: EmployeeWeekCommentRepository::class)]
-#[ORM\Table(name: 'employee_week_comments')]
-#[ORM\UniqueConstraint(name: 'uniq_employee_week_comment', columns: ['employee_id', 'week_start_date'])]
-class EmployeeWeekComment
-{
-    #[ORM\Id]
-    #[ORM\GeneratedValue]
-    #[ORM\Column(type: 'integer')]
-    #[Groups(['week_comment:read'])]
-    private ?int $id = null;
-
-    #[ORM\ManyToOne(targetEntity: Employee::class)]
-    #[ORM\JoinColumn(nullable: false, onDelete: 'CASCADE')]
-    #[Groups(['week_comment:read', 'week_comment:write'])]
-    #[Assert\NotNull]
-    private ?Employee $employee = null;
-
-    #[ORM\Column(type: 'date_immutable')]
-    #[Groups(['week_comment:read', 'week_comment:write'])]
-    #[Assert\NotNull]
-    private ?DateTimeImmutable $weekStartDate = null;
-
-    #[ORM\Column(type: 'text')]
-    #[Groups(['week_comment:read', 'week_comment:write'])]
-    #[Assert\NotBlank]
-    #[Assert\Length(max: 5000)]
-    private string $content = '';
-
-    #[ORM\Column(type: 'datetime_immutable')]
-    #[Groups(['week_comment:read'])]
-    private DateTimeImmutable $createdAt;
-
-    #[ORM\Column(type: 'datetime_immutable')]
-    #[Groups(['week_comment:read'])]
-    private DateTimeImmutable $updatedAt;
-
-    public function __construct()
-    {
-        $now             = new DateTimeImmutable();
-        $this->createdAt = $now;
-        $this->updatedAt = $now;
-    }
-
-    public function getId(): ?int
-    {
-        return $this->id;
-    }
-
-    public function getEmployee(): ?Employee
-    {
-        return $this->employee;
-    }
-
-    public function setEmployee(?Employee $employee): self
-    {
-        $this->employee = $employee;
-
-        return $this;
-    }
-
-    public function getWeekStartDate(): ?DateTimeImmutable
-    {
-        return $this->weekStartDate;
-    }
-
-    public function setWeekStartDate(?DateTimeImmutable $weekStartDate): self
-    {
-        $this->weekStartDate = $weekStartDate;
-
-        return $this;
-    }
-
-    public function getContent(): string
-    {
-        return $this->content;
-    }
-
-    public function setContent(string $content): self
-    {
-        $this->content = $content;
-
-        return $this;
-    }
-
-    public function getCreatedAt(): DateTimeImmutable
-    {
-        return $this->createdAt;
-    }
-
-    public function getUpdatedAt(): DateTimeImmutable
-    {
-        return $this->updatedAt;
-    }
-
-    public function touchUpdatedAt(): void
-    {
-        $this->updatedAt = new DateTimeImmutable();
-    }
-}

src/Repository/EmployeeWeekCommentRepository.php

@@ -1,58 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace App\Repository;
-
-use App\Entity\Employee;
-use App\Entity\EmployeeWeekComment;
-use DateTimeImmutable;
-use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\Persistence\ManagerRegistry;
-
-/**
- * @extends ServiceEntityRepository<EmployeeWeekComment>
- */
-class EmployeeWeekCommentRepository extends ServiceEntityRepository
-{
-    public function __construct(ManagerRegistry $registry)
-    {
-        parent::__construct($registry, EmployeeWeekComment::class);
-    }
-
-    public function findOneByEmployeeAndWeek(Employee $employee, DateTimeImmutable $weekStart): ?EmployeeWeekComment
-    {
-        return $this->findOneBy(['employee' => $employee, 'weekStartDate' => $weekStart]);
-    }
-
-    /**
-     * @param list<Employee> $employees
-     *
-     * @return array<int, EmployeeWeekComment> employee_id → comment
-     */
-    public function findByWeekAndEmployees(DateTimeImmutable $weekStart, array $employees): array
-    {
-        if ([] === $employees) {
-            return [];
-        }
-
-        $rows = $this->createQueryBuilder('c')
-            ->andWhere('c.weekStartDate = :weekStart')
-            ->andWhere('c.employee IN (:employees)')
-            ->setParameter('weekStart', $weekStart)
-            ->setParameter('employees', $employees)
-            ->innerJoin('c.employee', 'e')->addSelect('e')
-            ->getQuery()->getResult()
-        ;
-
-        $map = [];
-        foreach ($rows as $row) {
-            $eid = $row->getEmployee()?->getId();
-            if (null !== $eid) {
-                $map[$eid] = $row;
-            }
-        }
-
-        return $map;
-    }
-}

src/State/EmployeeWeekCommentWriteProcessor.php

@@ -1,80 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace App\State;
-
-use ApiPlatform\Metadata\DeleteOperationInterface;
-use ApiPlatform\Metadata\Operation;
-use ApiPlatform\State\ProcessorInterface;
-use App\Entity\Employee;
-use App\Entity\EmployeeWeekComment;
-use App\Service\AuditLogger;
-use Doctrine\ORM\EntityManagerInterface;
-use Symfony\Component\DependencyInjection\Attribute\Autowire;
-use Symfony\Component\HttpKernel\Exception\UnprocessableEntityHttpException;
-
-final readonly class EmployeeWeekCommentWriteProcessor implements ProcessorInterface
-{
-    public function __construct(
-        #[Autowire(service: 'api_platform.doctrine.orm.state.persist_processor')]
-        private ProcessorInterface $persistProcessor,
-        #[Autowire(service: 'api_platform.doctrine.orm.state.remove_processor')]
-        private ProcessorInterface $removeProcessor,
-        private EntityManagerInterface $entityManager,
-        private AuditLogger $auditLogger,
-    ) {}
-
-    public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): mixed
-    {
-        if (!$data instanceof EmployeeWeekComment) {
-            return $data;
-        }
-
-        $employee = $data->getEmployee();
-
-        if ($operation instanceof DeleteOperationInterface) {
-            $this->auditLogger->log(
-                $employee,
-                'delete',
-                'week_comment',
-                $data->getId(),
-                sprintf('Commentaire semaine supprimé pour %s (semaine du %s)', $this->label($employee), $data->getWeekStartDate()?->format('d/m/Y') ?? '?'),
-                ['old' => ['content' => $data->getContent()]],
-                $data->getWeekStartDate(),
-            );
-            $result = $this->removeProcessor->process($data, $operation, $uriVariables, $context);
-            $this->entityManager->flush();
-
-            return $result;
-        }
-
-        $weekStart = $data->getWeekStartDate();
-        if (null === $weekStart || '1' !== $weekStart->format('N')) {
-            throw new UnprocessableEntityHttpException('weekStartDate must be a Monday (ISO weekday 1).');
-        }
-
-        $prev = null;
-        if (null !== $data->getId()) {
-            $prev = $this->entityManager->getUnitOfWork()->getOriginalEntityData($data)['content'] ?? null;
-            $data->touchUpdatedAt();
-        }
-
-        $result = $this->persistProcessor->process($data, $operation, $uriVariables, $context);
-
-        if (null === $prev) {
-            $this->auditLogger->log($employee, 'create', 'week_comment', $data->getId(), sprintf('Commentaire semaine créé pour %s (semaine du %s)', $this->label($employee), $weekStart->format('d/m/Y')), ['new' => ['content' => $data->getContent()]], $weekStart);
-        } elseif ($prev !== $data->getContent()) {
-            $this->auditLogger->log($employee, 'update', 'week_comment', $data->getId(), sprintf('Commentaire semaine modifié pour %s (semaine du %s)', $this->label($employee), $weekStart->format('d/m/Y')), ['old' => ['content' => $prev], 'new' => ['content' => $data->getContent()]], $weekStart);
-        }
-
-        $this->entityManager->flush();
-
-        return $result;
-    }
-
-    private function label(mixed $e): string
-    {
-        return $e instanceof Employee ? trim(($e->getLastName() ?? '').' '.($e->getFirstName() ?? '')) : '?';
-    }
-}

src/State/WorkHourWeeklySummaryProvider.php

@@ -13,7 +13,6 @@ use App\Dto\WorkHours\WorkMetrics;
 use App\Entity\Absence;
 use App\Entity\Contract;
 use App\Entity\Employee;
-use App\Entity\EmployeeWeekComment;
 use App\Entity\User;
 use App\Entity\WorkHour;
 use App\Enum\ContractNature;
@@ -22,7 +21,6 @@ use App\Enum\TrackingMode;
 use App\Repository\Contract\AbsenceReadRepositoryInterface;
 use App\Repository\Contract\EmployeeScopedRepositoryInterface;
 use App\Repository\Contract\WorkHourReadRepositoryInterface;
-use App\Repository\EmployeeWeekCommentRepository;
 use App\Service\Contracts\EmployeeContractResolver;
 use App\Service\WorkHours\AbsenceSegmentsResolver;
 use App\Service\WorkHours\DailyReferenceMinutesResolver;
@@ -47,7 +45,6 @@ final readonly class WorkHourWeeklySummaryProvider implements ProviderInterface
         private EmployeeContractResolver $contractResolver,
         private DailyReferenceMinutesResolver $dailyReferenceResolver,
         private HolidayVirtualHoursResolver $holidayVirtualHoursResolver,
-        private EmployeeWeekCommentRepository $weekCommentRepository,
     ) {}
 
     public function provide(Operation $operation, array $uriVariables = [], array $context = []): WorkHourWeeklySummary
@@ -65,13 +62,11 @@ final readonly class WorkHourWeeklySummaryProvider implements ProviderInterface
         $workHours = $this->workHourRepository->findByDateRangeAndEmployees($weekStart, $weekEnd, $employees);
         $absences  = $this->absenceRepository->findForPrint($weekStart, $weekEnd, $employees);
 
-        $weekComments = $this->weekCommentRepository->findByWeekAndEmployees($weekStart, $employees);
-
         $summary            = new WorkHourWeeklySummary();
         $summary->weekStart = $weekStart->format('Y-m-d');
         $summary->weekEnd   = $weekEnd->format('Y-m-d');
         $summary->days      = $days;
-        $summary->rows      = $this->buildRows($employees, $workHours, $absences, $days, $anchorDate->format('Y-m-d'), $weekComments);
+        $summary->rows      = $this->buildRows($employees, $workHours, $absences, $days, $anchorDate->format('Y-m-d'));
 
         return $summary;
     }
@@ -114,15 +109,14 @@ final readonly class WorkHourWeeklySummaryProvider implements ProviderInterface
     }
 
     /**
-     * @param list<Employee>                  $employees
-     * @param list<WorkHour>                  $workHours
-     * @param list<Absence>                   $absences
-     * @param list<string>                    $days
-     * @param array<int, EmployeeWeekComment> $weekComments
+     * @param list<Employee> $employees
+     * @param list<WorkHour> $workHours
+     * @param list<Absence>  $absences
+     * @param list<string>   $days
      *
      * @return list<WeeklySummaryRow>
      */
-    private function buildRows(array $employees, array $workHours, array $absences, array $days, string $anchorDateYmd, array $weekComments = []): array
+    private function buildRows(array $employees, array $workHours, array $absences, array $days, string $anchorDateYmd): array
     {
         $contractsByEmployeeDate       = $this->contractResolver->resolveForEmployeesAndDays($employees, $days);
         $contractNaturesByEmployeeDate = $this->contractResolver->resolveNaturesForEmployeesAndDays($employees, $days);
@@ -376,8 +370,6 @@ final readonly class WorkHourWeeklySummaryProvider implements ProviderInterface
                 weeklyOvernightCount: $weeklyOvernightCount,
                 hasContractForWeek: $hasContractForWeek,
                 contractNature: $weekAnchorContractNature->value,
-                comment: ($weekComments[$employeeId] ?? null)?->getContent(),
-                commentId: ($weekComments[$employeeId] ?? null)?->getId(),
             );
         }
 

tests/State/EmployeeWeekCommentWriteProcessorTest.php

@@ -1,76 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace App\Tests\State;
-
-use ApiPlatform\Metadata\Delete;
-use ApiPlatform\Metadata\Post;
-use ApiPlatform\State\ProcessorInterface;
-use App\Entity\Employee;
-use App\Entity\EmployeeWeekComment;
-use App\Service\AuditLogger;
-use App\State\EmployeeWeekCommentWriteProcessor;
-use DateTimeImmutable;
-use Doctrine\ORM\EntityManagerInterface;
-use Doctrine\ORM\UnitOfWork;
-use PHPUnit\Framework\TestCase;
-use Symfony\Component\HttpKernel\Exception\UnprocessableEntityHttpException;
-
-/**
- * @internal
- */
-final class EmployeeWeekCommentWriteProcessorTest extends TestCase
-{
-    public function testRejectsNonMondayWeekStart(): void
-    {
-        $processor = new EmployeeWeekCommentWriteProcessor(
-            $this->createStub(ProcessorInterface::class),
-            $this->createStub(ProcessorInterface::class),
-            $this->createStub(EntityManagerInterface::class),
-            $this->createStub(AuditLogger::class),
-        );
-
-        $comment = new EmployeeWeekComment()
-            ->setEmployee(new Employee()->setFirstName('A')->setLastName('B'))
-            ->setWeekStartDate(new DateTimeImmutable('2026-04-14'))
-            ->setContent('test')
-        ;
-
-        $this->expectException(UnprocessableEntityHttpException::class);
-        $processor->process($comment, new Post());
-    }
-
-    public function testAcceptsMondayAndAuditsCreate(): void
-    {
-        $persist = $this->createMock(ProcessorInterface::class);
-        $persist->expects(self::once())->method('process');
-        $em = $this->createMock(EntityManagerInterface::class);
-        $em->method('getUnitOfWork')->willReturn($this->createStub(UnitOfWork::class));
-        $em->expects(self::once())->method('flush');
-        $auditor = $this->createMock(AuditLogger::class);
-        $auditor->expects(self::once())->method('log')->with(self::anything(), 'create', 'week_comment');
-
-        $processor = new EmployeeWeekCommentWriteProcessor($persist, $this->createStub(ProcessorInterface::class), $em, $auditor);
-        $processor->process(
-            new EmployeeWeekComment()->setEmployee(new Employee()->setFirstName('A')->setLastName('B'))->setWeekStartDate(new DateTimeImmutable('2026-04-13'))->setContent('x'),
-            new Post()
-        );
-    }
-
-    public function testDeleteAudits(): void
-    {
-        $remove = $this->createMock(ProcessorInterface::class);
-        $remove->expects(self::once())->method('process');
-        $em = $this->createMock(EntityManagerInterface::class);
-        $em->expects(self::once())->method('flush');
-        $auditor = $this->createMock(AuditLogger::class);
-        $auditor->expects(self::once())->method('log')->with(self::anything(), 'delete', 'week_comment');
-
-        $processor = new EmployeeWeekCommentWriteProcessor($this->createStub(ProcessorInterface::class), $remove, $em, $auditor);
-        $processor->process(
-            new EmployeeWeekComment()->setEmployee(new Employee()->setFirstName('A')->setLastName('B'))->setWeekStartDate(new DateTimeImmutable('2026-04-13'))->setContent('x'),
-            new Delete()
-        );
-    }
-}

tests/State/WorkHourWeeklySummaryProviderTest.php

@@ -15,7 +15,6 @@ use App\Enum\HalfDay;
 use App\Repository\Contract\AbsenceReadRepositoryInterface;
 use App\Repository\Contract\EmployeeScopedRepositoryInterface;
 use App\Repository\Contract\WorkHourReadRepositoryInterface;
-use App\Repository\EmployeeWeekCommentRepository;
 use App\Service\Contracts\EmployeeContractResolver;
 use App\Service\PublicHolidayServiceInterface;
 use App\Service\WorkHours\AbsenceSegmentsResolver;
@@ -67,7 +66,6 @@ final class WorkHourWeeklySummaryProviderTest extends TestCase
             $this->buildResolverStub(),
             new DailyReferenceMinutesResolver(),
             $this->buildHolidayResolver(),
-            $this->buildWeekCommentRepoStub(),
         );
 
         $this->expectException(AccessDeniedHttpException::class);
@@ -130,7 +128,6 @@ final class WorkHourWeeklySummaryProviderTest extends TestCase
             $this->buildWeeklyResolverStub($employees),
             new DailyReferenceMinutesResolver(),
             $this->buildHolidayResolver(),
-            $this->buildWeekCommentRepoStub(),
         );
 
         $result = $provider->provide(new Get());
@@ -181,14 +178,6 @@ final class WorkHourWeeklySummaryProviderTest extends TestCase
         $property->setValue($entity, $id);
     }
 
-    private function buildWeekCommentRepoStub(): EmployeeWeekCommentRepository
-    {
-        $r = $this->createStub(EmployeeWeekCommentRepository::class);
-        $r->method('findByWeekAndEmployees')->willReturn([]);
-
-        return $r;
-    }
-
     private function buildHolidayResolver(array $holidayMap = []): HolidayVirtualHoursResolver
     {
         $service = $this->createStub(PublicHolidayServiceInterface::class);