v5.0.15 — Absences partielles affichées comme rows (gris foncé)

background.js

@@ -298,8 +298,14 @@ async function extendSessionKeepAlive(origin, phpsessid) {
 
 /**
  * Détecte si on est sur le réseau interne (itsma.etat-de-vaud.ch accessible)
- * ou externe (seul itsma.vd.ch accessible). Fait un HEAD test avec timeout
- * court sur l'URL interne : si ça répond, on est interne ; sinon externe.
+ * ou externe (seul itsma.vd.ch accessible). Fait un fetch avec mode "no-cors"
+ * sur l'URL interne : si ça répond (même redirection SSO), on est interne.
+ * Si ça échoue (DNS unreachable, timeout), on est externe.
+ *
+ * v5.0.11 (fix) : mode "no-cors" pour éviter que l'erreur CORS de la
+ * redirection SSO (itsma.etat-de-vaud.ch → portail.etat-de-vaud.ch/sso/)
+ * soit interprétée comme un échec de connectivité. Au bureau, la redirection
+ * SSO passe → le fetch termine → on sait qu'on est interne.
  *
  * Le résultat est mis en cache dans chrome.storage.local pendant 1h pour
  * éviter de refaire le test à chaque démarrage.
@@ -308,7 +314,7 @@ async function extendSessionKeepAlive(origin, phpsessid) {
  * @returns {Promise<"internal"|"external">}
  */
 async function detectNetworkContext(force = false) {
-  const CACHE_KEY = "network_context";
+  const CACHE_KEY = "network_context_v2";  // v5.0.12 : nouvelle clé pour invalider le cache fautif v5.0.11
   const CACHE_MAX_AGE_MS = 60 * 60 * 1000;  // 1h
 
   if (!force) {
@@ -322,25 +328,28 @@ async function detectNetworkContext(force = false) {
     } catch (e) {}
   }
 
-  // Test HEAD sur l'URL interne avec timeout 2.5 sec
   const controller = new AbortController();
   const timer = setTimeout(() => controller.abort(), 2500);
   let context = "external";
   try {
-    console.log("[bg] detectNetworkContext : test de itsma.etat-de-vaud.ch...");
-    const r = await fetch("https://itsma.etat-de-vaud.ch/", {
-      method: "HEAD",
+    console.log("[bg] detectNetworkContext : test de itsma.etat-de-vaud.ch (mode no-cors)...");
+    // v5.0.11 (fix) : no-cors évite l'erreur CORS de la redirection SSO.
+    // Si le fetch termine sans throw, le serveur est joignable = interne.
+    // Si timeout ou DNS fail, throw → externe.
+    await fetch("https://itsma.etat-de-vaud.ch/", {
+      method: "GET",
+      mode: "no-cors",
       signal: controller.signal,
-      credentials: "omit"  // pas besoin des cookies pour ce test
+      credentials: "omit",
+      cache: "no-store"
     });
     clearTimeout(timer);
-    // Tout statut HTTP (même 302, 404, 403) indique que le serveur est joignable
-    console.log("[bg] detectNetworkContext : interne accessible (status=" + r.status + ")");
+    console.log("[bg] detectNetworkContext : interne accessible");
     context = "internal";
   } catch (err) {
     clearTimeout(timer);
-    // Timeout, DNS unreachable, erreur réseau = domaine interne inaccessible
-    console.log("[bg] detectNetworkContext : interne inaccessible, on est en externe (" + err.name + ")");
+    // AbortError (timeout) ou TypeError (DNS unreachable / réseau coupé)
+    console.log("[bg] detectNetworkContext : interne inaccessible → externe (" + err.name + ": " + err.message + ")");
     context = "external";
   }
 
@@ -635,33 +644,18 @@ async function submitDouchette(origin, phpsessid, opts) {
 async function deletePlanningItem(origin, phpsessid, actionId, kind) {
   if (!actionId) throw new Error("actionId manquant");
 
-  // v5.0.1 : plusieurs function_name à tester dans l'ordre (du plus probable
-  // au moins probable). Le premier qui renvoie 200 ET non-login est considéré OK.
-  const fnNames = kind === "reservation"
-    ? [
+  // v5.0.14 : confirmé par capture Network réelle — EasyVista utilise
+  // "Planning_delete_absence" pour TOUS les types d'entrée planning (absences,
+  // réservations, événements, etc.). Réponse XML : <Planning_delete_absence>true</...>
+  // On met donc ce nom en PREMIER pour tout, et on garde les autres en fallback.
+  const fnNames = [
+    "Planning_delete_absence",        // ← le seul qui marche vraiment côté EV
+    // Fallbacks historiques (au cas où EV change un jour) :
     "Planning_delete_reservation",
-        "delete_reservation",
-        "fc_delete_reservation",
-        "delete_act_reservation",
-        "delete_planning_reservation",
-        "remove_reservation",
-        // v5.0.2 : réservations sont parfois traitées comme absences côté API
-        "Planning_delete_absence",
     "delete_absence",
-        "fc_delete_absence"
-      ]
-    : [
-        // v5.0.2 : élargir la liste, on a essayé 3 sans succès. Les variantes
-        // plausibles vues dans les API EasyVista :
-        "Planning_delete_absence",      // le plus "officiel"
-        "delete_absence",               // le nom JS dans le onclick
-        "fc_delete_absence",            // pattern fc_*
-        "delete_act_absence",           // parfois "act_" dans les noms
-        "Planning_delete_holiday",      // en anglais
-        "delete_holiday",
-        "fc_delete_holiday",
-        "delete_planning_absence",      // variation complète
-        "remove_absence"
+    "delete_reservation",
+    "fc_delete_absence",
+    "fc_delete_reservation"
   ];
 
   let lastErr = null;
@@ -675,7 +669,11 @@ async function deletePlanningItem(origin, phpsessid, actionId, kind) {
     console.log(`[bg] deletePlanningItem → tente function_name=${fn}`, url.substring(0, 180));
 
     try {
-      const r = await fetch(url, { method: "GET", credentials: "include" });
+      // v5.0.13 : utiliser evFetch() au lieu de fetch() brut pour que les
+      // headers Referer + X-Requested-With soient envoyés — sinon EV renvoie
+      // un <script> de redirection CSRF qui ne ressemble pas à une erreur et
+      // notre heuristique le prenait à tort pour un succès.
+      const r = await evFetch(url, origin, { method: "GET" });
       const body = await r.text();
       console.log(`[bg]   status=${r.status} | body (200 premiers chars) : ${(body || "").substring(0, 200)}`);
 
@@ -690,24 +688,34 @@ async function deletePlanningItem(origin, phpsessid, actionId, kind) {
         throw new Error("session_expired");
       }
 
-      // v5.0.1 : heuristique pour détecter si la suppression a marché.
-      // EasyVista renvoie typiquement :
-      //   - une chaine vide ou "ok" ou "1" si succès
-      //   - un message d'erreur / html d'erreur si function_name inconnu
-      // On considère que tout ce qui n'est pas un message d'erreur évident
-      // est un succès. Si plusieurs fn renvoient 200, on prend le premier.
-      const trimmed = (body || "").trim().toLowerCase();
-      const looksLikeError = trimmed.includes("error")
-                          || trimmed.includes("erreur")
-                          || trimmed.includes("unknown function")
-                          || trimmed.includes("fonction inconnue")
-                          || trimmed.includes("<html");
-      if (!looksLikeError) {
-        console.log(`[bg]   → suppression OK avec function_name=${fn}`);
-        return { status: r.status, functionName: fn, body: body.substring(0, 200) };
+      // v5.0.14 : détection explicite du succès XML observé dans les captures
+      // réseau : <Planning_delete_absence>true</Planning_delete_absence>
+      const trimmed = (body || "").trim();
+      const lower = trimmed.toLowerCase();
+
+      // Succès explicite : réponse XML du type <X>true</X>
+      if (/^<\w+>true<\/\w+>\s*$/i.test(trimmed)) {
+        console.log(`[bg]   → SUCCÈS confirmé par XML <...>true</...> avec function_name=${fn}`);
+        return { status: r.status, functionName: fn, body: trimmed };
       }
+
+      // Détection d'échec : <X>false</X>, erreurs, html, redirect, etc.
+      const looksLikeError = /^<\w+>false<\/\w+>\s*$/i.test(trimmed)
+                          || lower.includes("error")
+                          || lower.includes("erreur")
+                          || lower.includes("unknown function")
+                          || lower.includes("fonction inconnue")
+                          || lower.includes("<html")
+                          || lower.includes("window.location.href");
+      if (looksLikeError) {
         console.log(`[bg]   → réponse ressemble à une erreur, on tente le prochain nom`);
         lastBody = body;
+        continue;
+      }
+      // Pas d'erreur évidente mais pas de succès explicite non plus
+      // (ex: réponse vide ou "1" ou "ok"). On considère comme succès.
+      console.log(`[bg]   → suppression probablement OK (body neutre) avec function_name=${fn}`);
+      return { status: r.status, functionName: fn, body: trimmed.substring(0, 200) };
     } catch (err) {
       if (err.message === "session_expired") throw err;
       console.warn(`[bg]   erreur avec ${fn}:`, err);

manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "Planification",
-  "version": "5.0.11",
+  "version": "5.0.15",
   "description": "Vue claire et rapide du planning des techniciens EasyVista. Regroupe interventions et réservations par tech, affiche horaires, contact, lieu, catégorie et statut en un coup d'œil.",
   "permissions": ["activeTab", "scripting", "storage", "tabs", "alarms"],
   "host_permissions": [

viewer.css

@@ -689,12 +689,9 @@ html, body {
 .timeline-slot.status-resolved  { background: var(--c-resolved); }
 
 .timeline-slot.kind-absence {
-  background: repeating-linear-gradient(
-    45deg,
-    var(--text-faint) 0 6px,
-    var(--bg-muted) 6px 12px
-  );
-  opacity: 0.6;
+  /* v5.0.15 : uni gris-noir au lieu de rayé, plus lisible */
+  background: #2a2f36;
+  border-right: 1px solid var(--bg-elevated);
 }
 
 .timeline-slot:hover,
@@ -1117,6 +1114,26 @@ html, body {
   color: var(--c-reservation);
   font-family: var(--font);
   letter-spacing: 0.02em;
+  /* v5.0.15 : étendre le titre sur toute la largeur de la carte pour le
+     vrai centrage (sinon il n'est centré que dans sa colonne grid) */
+  grid-column: 1 / -1;
+  text-align: center;
+  padding-left: 62px;   /* compense la colonne time (58px + gap) */
+  padding-right: 0;
+}
+
+/* v5.0.15 : absence partielle (demi-journée) affichée comme une row */
+.iv-ref-header.is-absence-title {
+  color: var(--c-absence, #a0a8b2);
+  font-family: var(--font);
+  letter-spacing: 0.02em;
+  grid-column: 1 / -1;
+  text-align: center;
+  padding-left: 62px;
+  padding-right: 0;
+}
+.intervention-v2.color-absence .intervention-dot {
+  background: var(--c-absence, #2a2f36);
 }
 .iv-reservation-par {
   font-size: 13px;

viewer.js

@@ -131,6 +131,7 @@ function deriveShortTitle(iv) {
 
 function deriveColorKey(iv) {
   if (iv.type === "AL-Reservation") return "reservation";
+  if (iv.type === "AL-Absence") return "absence";  // v5.0.15 : couleur noire/gris foncé
   if (iv.ref && /^I\d/.test(iv.ref)) return "incident";
   if (isRollOut(iv)) return "rollout";
   if (isRecupAction(iv)) return "recup";
@@ -4188,6 +4189,12 @@ function compareTechs(a, b, targetDate) {
   return aLast.localeCompare(bLast, "fr");
 }
 
+// v5.0.13 : un tech est considéré "absent toute la journée" uniquement si une
+// absence couvre RÉELLEMENT du matin au soir (ou quasi), pas juste s'il a des
+// absences (éventuellement partielles). Avant, une absence matin 08-12 seule
+// faisait passer le tech en "absent toute la journée" car il n'avait QUE des
+// absences. Maintenant on check explicitement que l'absence couvre ≥ 90% de
+// la plage 08:00-18:00.
 function isTechAbsent(tech, isoDate) {
   const recurring = RECURRING_ABSENCES[tech.id];
   if (recurring) {
@@ -4195,7 +4202,26 @@ function isTechAbsent(tech, isoDate) {
     if (recurring.includes(day)) return true;
   }
   if (tech.interventions.length === 0) return false;
-  return tech.interventions.every(iv => iv.type === "AL-Absence" && !iv.isPompier);
+  // Parmi les absences (hors pompier), est-ce qu'une seule couvre la journée ?
+  const fullDayAbsences = tech.interventions.filter(iv => {
+    if (iv.type !== "AL-Absence" || iv.isPompier) return false;
+    const startMin = timeToMinutes(iv.startTime);
+    const endMin = timeToMinutes(iv.endTime);
+    if (startMin == null || endMin == null) {
+      // Si on n'a pas d'horaires, on considère que c'est toute la journée
+      // (cas des absences multi-jours sans horaires précis)
+      return true;
+    }
+    // Absence couvre toute la journée si son créneau déborde largement
+    // la plage affichée (≥ 90%). Une demi-journée (4h) sur 10h = 40% → ne
+    // passera pas, donc on ne marquera pas le tech comme absent toute la journée.
+    const DAY_LEN_MIN = 10 * 60;  // 08:00 → 18:00 = 10h
+    const clampedStart = Math.max(startMin, 8 * 60);
+    const clampedEnd = Math.min(endMin, 18 * 60);
+    const coveredMin = Math.max(0, clampedEnd - clampedStart);
+    return coveredMin >= 0.9 * DAY_LEN_MIN;
+  });
+  return fullDayAbsences.length > 0;
 }
 
 // ============================================================================
@@ -4323,7 +4349,21 @@ function buildCard(tech, isoDate) {
     return card;
   }
 
-  if (realInterventions.length === 0 && !isPompier) {
+  // v5.0.14 : si le tech n'a aucune intervention mais a des absences
+  // partielles (demi-journée) ou pompier, on veut quand même afficher la
+  // timeline avec les blocs absence visibles. Sans ça, une absence 08-12
+  // seule n'apparaissait jamais sur la carte (affichait juste "Pas
+  // d'intervention planifiée").
+  const hasPartialAbsences = absenceBlocks.some(ab => {
+    if (ab.isPompier) return false;
+    const s = timeToMinutes(ab.startTime);
+    const e = timeToMinutes(ab.endTime);
+    if (s === null || e === null) return false;
+    // Absence qui couvre PAS toute la journée → c'est partiel
+    return !(s <= DAY_START && e >= DAY_END);
+  });
+
+  if (realInterventions.length === 0 && !isPompier && !hasPartialAbsences) {
     if (isPillonelFriday) {
       const note = document.createElement("div");
       note.className = "tech-absence-recurring";
@@ -4373,6 +4413,25 @@ function buildCard(tech, isoDate) {
     body.appendChild(buildInterventionRow(iv, card));
   }
 
+  // v5.0.15 : afficher aussi les absences partielles (demi-journée) comme
+  // des rows, avec le même style que les réservations mais en gris foncé.
+  // Les absences qui couvrent toute la journée sont déjà traitées plus haut
+  // (carte "Absent toute la journée") et ne doivent pas être dupliquées ici.
+  if (!isAbsent) {
+    const partialAbsences = absenceBlocks.filter(ab => {
+      if (ab.isPompier) return false;
+      const s = timeToMinutes(ab.startTime);
+      const e = timeToMinutes(ab.endTime);
+      if (s === null || e === null) return false;
+      return !(s <= DAY_START && e >= DAY_END);
+    });
+    // Trier par heure de début
+    partialAbsences.sort((a, b) => (a.startTime || "").localeCompare(b.startTime || ""));
+    for (const ab of partialAbsences) {
+      body.appendChild(buildInterventionRow(ab, card));
+    }
+  }
+
   card.appendChild(body);
   return card;
 }
@@ -4734,7 +4793,7 @@ function buildInterventionRow(iv, cardEl) {
   cardEl._rowIdxCounter = ivIdx + 1;
   row.dataset.ivIdx = ivIdx;
 
-  if (iv.formLink && !iv.ghost) {
+  if (iv.formLink && !iv.ghost && iv.type !== "AL-Absence") {
     row.classList.add("clickable");
     // v4.1.8 : plus de title au survol (info déjà dans le tooltip en bas)
 
@@ -4772,6 +4831,10 @@ function buildInterventionRow(iv, cardEl) {
   if (iv.type === "AL-Reservation") {
     refHeader.textContent = "Réservation";
     refHeader.classList.add("is-reservation-title");
+  } else if (iv.type === "AL-Absence") {
+    // v5.0.15 : absence partielle (demi-journée) affichée comme une row
+    refHeader.textContent = "Absence";
+    refHeader.classList.add("is-absence-title");
   } else if (iv.ref) {
     refHeader.textContent = iv.ref;
   } else {
@@ -4780,8 +4843,8 @@ function buildInterventionRow(iv, cardEl) {
   }
   row.appendChild(refHeader);
 
-  // Check ✓ + bouton copier à droite de la ref (pas pour réservation)
-  if (statusClass && iv.type !== "AL-Reservation") {
+  // Check ✓ + bouton copier à droite de la ref (pas pour réservation / absence)
+  if (statusClass && iv.type !== "AL-Reservation" && iv.type !== "AL-Absence") {
     const statusEl = document.createElement("div");
     statusEl.className = "iv-status-check";
     // v4.2.5 : ✓✓ double pour clôturé/résolu (statut officiel EasyVista)
@@ -4794,7 +4857,7 @@ function buildInterventionRow(iv, cardEl) {
     }
     row.appendChild(statusEl);
   }
-  if (iv.ref && iv.type !== "AL-Reservation") {
+  if (iv.ref && iv.type !== "AL-Reservation" && iv.type !== "AL-Absence") {
     const copyBtn = document.createElement("button");
     copyBtn.className = "intervention-copy";
     copyBtn.type = "button";
@@ -4874,6 +4937,40 @@ function buildInterventionRow(iv, cardEl) {
     return row;
   }
 
+  // v5.0.15 : absence partielle (demi-journée) affichée comme une row au
+  // même style que les réservations mais en gris foncé, avec le type d'absence
+  // (Congés, Maladie, Pompier) comme sujet.
+  if (iv.type === "AL-Absence") {
+    // Bloc "Par Nom, Prénom" si on a un créateur
+    if (iv.reservationCreator) {
+      const parEl = document.createElement("div");
+      parEl.className = "iv-reservation-par";
+      parEl.textContent = "Par " + iv.reservationCreator;
+      rightCol.appendChild(parEl);
+    }
+    // Type d'absence (Congés, Maladie, Pompier) si dispo dans label
+    const absenceTypeMatch = (iv.label || "").match(/^([^/]+?)\s*(?:\/|$)/);
+    const absenceType = absenceTypeMatch ? absenceTypeMatch[1].trim() : null;
+    if (absenceType) {
+      const sujetEl = document.createElement("div");
+      sujetEl.className = "iv-reservation-sujet";
+      sujetEl.textContent = "Type : " + absenceType;
+      rightCol.appendChild(sujetEl);
+    }
+    row.appendChild(rightCol);
+
+    // Tooltip au hover (avec bouton supprimer)
+    row.addEventListener("mouseenter", (e) => {
+      showTooltip(e, iv, row);
+      highlightIntervention(cardEl, ivIdx, true);
+    });
+    row.addEventListener("mouseleave", () => {
+      hideTooltip();
+      highlightIntervention(cardEl, ivIdx, false);
+    });
+    return row;
+  }
+
   // v4.1.2 : priorité à iv.infobulle (venant du xhr2 = données réelles vérifiées
   // par le tech sur place) puis fallback sur iv.bulleContact/iv.bulleLieu
   // (venant de attr1/attr2 = planification initiale, parfois incorrecte).