nodebb-plugin-onekite-calendar 2.0.7 → 2.0.9

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog – calendar-onekite
 
+## 1.2.16
+- Modale réservation (Durée rapide) : correction du grisé erroné des matériels réservés la veille. Les requêtes de disponibilité utilisent désormais des dates calendaires (YYYY-MM-DD) au lieu de toISOString() (UTC), pour éviter tout faux chevauchement lié au fuseau/DST.
+
+## 1.2.15
+- Réservations (Durée rapide) : génération des évènements all-day basée sur startDate/endDate (YYYY-MM-DD) quand disponibles, pour éviter tout décalage lié à toISOString() (UTC) et empêcher un grisé « non disponible » le jour suivant.
+
 ## 1.2.14
 - Réservations : correction d’un faux chevauchement (problème 1h) quand FullCalendar envoie des bornes all-day à minuit UTC (Z / +00:00) — le lendemain n’est plus marqué « non disponible ».
 

package/lib/api.js

@@ -356,8 +356,22 @@ function eventsFor(resv) {
   const status = resv.status;
   const icons = { pending: '⏳', awaiting_payment: '💳', paid: '✅' };
   const colors = { pending: '#f39c12', awaiting_payment: '#d35400', paid: '#27ae60' };
-  const startIsoDate = new Date(parseInt(resv.start, 10)).toISOString().slice(0, 10);
-  const endIsoDate = new Date(parseInt(resv.end, 10)).toISOString().slice(0, 10);
+  // IMPORTANT:
+  // Prefer stored date-only strings when available.
+  //
+  // Rationale: when reservations are stored as local-midnight timestamps (because
+  // the client sends YYYY-MM-DD), converting those timestamps with toISOString()
+  // shifts the date in Europe/Paris (UTC+1/+2) and can make ranges appear to
+  // overlap the following day in some UI flows (notably "Durée rapide").
+  //
+  // FullCalendar expects `end` to be EXCLUSIVE for all-day ranges, so we keep
+  // using the stored endDate as-is.
+  const startIsoDate = (resv.startDate && /^\d{4}-\d{2}-\d{2}$/.test(String(resv.startDate)))
+    ? String(resv.startDate)
+    : new Date(parseInt(resv.start, 10)).toISOString().slice(0, 10);
+  const endIsoDate = (resv.endDate && /^\d{4}-\d{2}-\d{2}$/.test(String(resv.endDate)))
+    ? String(resv.endDate)
+    : new Date(parseInt(resv.end, 10)).toISOString().slice(0, 10);
 
   const itemIds = Array.isArray(resv.itemIds) ? resv.itemIds : (resv.itemId ? [resv.itemId] : []);
   const itemNames = Array.isArray(resv.itemNames) ? resv.itemNames : (resv.itemName ? [resv.itemName] : []);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-onekite-calendar",
-  "version": "2.0.7",
+  "version": "2.0.9",
   "description": "FullCalendar-based equipment reservation workflow with admin approval & HelloAsso payment for NodeBB",
   "main": "library.js",
   "license": "MIT",

package/plugin.json

@@ -39,5 +39,5 @@
   "acpScripts": [
     "public/admin.js"
   ],
-  "version": "2.0.7"
+  "version": "2.0.9"
 }

package/public/client.js

@@ -960,9 +960,16 @@ function toDatetimeLocalValue(date) {
 
         function rangeQuery(s, e) {
           try {
-            const sIso = (s instanceof Date) ? s.toISOString() : new Date(s).toISOString();
-            const eIso = (e instanceof Date) ? e.toISOString() : new Date(e).toISOString();
-            return new URLSearchParams({ start: sIso, end: eIso }).toString();
+            // IMPORTANT: For availability checks we must avoid ISO strings with timezone
+            // (e.g. toISOString()) because local midnight in Europe/Paris becomes
+            // 23:00Z or 22:00Z depending on DST, which makes consecutive all-day ranges
+            // appear to overlap and wrongly greys out items.
+            //
+            // We always query using *calendar dates* (YYYY-MM-DD) with end exclusive,
+            // matching FullCalendar's all-day rule.
+            const sYmd = toLocalYmd(s);
+            const eYmd = toLocalYmd(e);
+            return new URLSearchParams({ start: sYmd, end: eYmd }).toString();
           } catch (err) {
             return '';
           }