From cb2db678babb7e84d18e9206c706b898d67984c0 Mon Sep 17 00:00:00 2001
From: tristan <tristan@yuno.malio.fr>
Date: Tue, 26 May 2026 10:05:25 +0200
Subject: [PATCH] =?UTF-8?q?fix=20:=20s=C3=A9lecteur=20d'exercice=20vide=20?=
 =?UTF-8?q?par=20d=C3=A9faut=20sur=20l'onglet=20Cong=C3=A9s?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Le watch sur le changement de phase (useEmployeeLeave/useEmployeeRtt)
remettait selectedYear à null pendant l'await du chargement eager du
récap congés, à la résolution initiale de la phase (undefined → phase
courante). Comme le chargement eager marquait ensuite leaveDataLoaded=true
et que le watch de useEmployeeDetailPage ignore la résolution initiale
(oldId === undefined), l'année restait à null et le <select> s'affichait
vide. L'onglet RTT y échappait car chargé en lazy.

Ajoute le garde oldId === undefined dans les watchs des deux composables,
même convention que useEmployeeDetailPage : la résolution initiale est
ignorée, seul un vrai changement de phase réinitialise l'année.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 frontend/composables/useEmployeeLeave.ts | 10 ++++++++--
 frontend/composables/useEmployeeRtt.ts   |  8 ++++++--
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/frontend/composables/useEmployeeLeave.ts b/frontend/composables/useEmployeeLeave.ts
index d19ea5d..1712873 100644
--- a/frontend/composables/useEmployeeLeave.ts
+++ b/frontend/composables/useEmployeeLeave.ts
@@ -124,8 +124,14 @@ export const useEmployeeLeave = (
     selectedLeaveYear.value = null
   }
 
-  watch(() => selectedPhase.value?.id, () => {
-    // Reset l'année car la plage a peut-être changé.
+  watch(() => selectedPhase.value?.id, (newId, oldId) => {
+    // Ignore la résolution initiale (undefined → phase courante au montage) :
+    // le chargement eager du récap initialise déjà l'année sélectionnée. Sans ce
+    // garde, ce watch (asynchrone) s'exécute PENDANT l'await du chargement eager et
+    // remet selectedLeaveYear à null, ce qui laisse le sélecteur d'exercice vide.
+    // Même convention que le watch de useEmployeeDetailPage.
+    if (oldId === undefined || newId === oldId) return
+    // Changement de phase réel : reset l'année car la plage a peut-être changé.
     selectedLeaveYear.value = null
     leaveDataLoaded.value = false
     // Le rechargement effectif est piloté par useEmployeeDetailPage.
diff --git a/frontend/composables/useEmployeeRtt.ts b/frontend/composables/useEmployeeRtt.ts
index 0a4ed6a..cdb92ee 100644
--- a/frontend/composables/useEmployeeRtt.ts
+++ b/frontend/composables/useEmployeeRtt.ts
@@ -95,8 +95,12 @@ export const useEmployeeRtt = (
     selectedRttYear.value = null
   }
 
-  watch(() => selectedPhase.value?.id, () => {
-    // Reset l'année car la plage a peut-être changé.
+  watch(() => selectedPhase.value?.id, (newId, oldId) => {
+    // Ignore la résolution initiale (undefined → phase courante au montage) :
+    // l'initialisation de l'année est pilotée par loadRttData. Même convention que
+    // le watch de useEmployeeDetailPage (évite un reset concurrent du sélecteur).
+    if (oldId === undefined || newId === oldId) return
+    // Changement de phase réel : reset l'année car la plage a peut-être changé.
     selectedRttYear.value = null
     rttDataLoaded.value = false
     // Le rechargement effectif est piloté par useEmployeeDetailPage.