nodebb-plugin-onekite-calendar 2.0.8 → 2.0.10

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog – calendar-onekite
 
+## 1.2.17
+- Modale réservation : la requête de disponibilité initiale utilise aussi des dates calendaires (YYYY-MM-DD) au lieu de startStr/endStr/toISOString(), ce qui corrige le grisé erroné (mobile + durée rapide).
+
+## 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.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-onekite-calendar",
-  "version": "2.0.8",
+  "version": "2.0.10",
   "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.8"
+  "version": "2.0.10"
 }

package/public/client.js

@@ -837,6 +837,13 @@ function toDatetimeLocalValue(date) {
     const start = selectionInfo.start;
     let end = selectionInfo.end;
 
+    // In some FullCalendar flows (notably on mobile), selectionInfo.end can be undefined.
+    // For all-day bookings, end is always treated as EXCLUSIVE.
+    if (!end) {
+      end = new Date(start);
+      end.setDate(end.getDate() + 1);
+    }
+
     // Days (end is exclusive in FullCalendar) — compute in calendar days only
     // (no dependency on hours, timezone or DST).
     let days = calendarDaysExclusive(start, end);
@@ -844,9 +851,13 @@ function toDatetimeLocalValue(date) {
     // Fetch existing events overlapping the selection to disable already reserved items.
     let blocked = new Set();
     try {
+      // IMPORTANT: never use toISOString() for availability checks.
+      // Local midnight in Europe/Paris becomes 23:00Z/22:00Z depending on DST, which makes
+      // consecutive all-day ranges appear to overlap and wrongly greys out items.
+      // Always query using calendar dates (YYYY-MM-DD) with end exclusive.
       const qs = new URLSearchParams({
-        start: (selectionInfo.startStr || (start instanceof Date ? start.toISOString() : String(start))),
-        end: (selectionInfo.endStr || (end instanceof Date ? end.toISOString() : String(end))),
+        start: toLocalYmd(start),
+        end: toLocalYmd(end),
       });
       const evs = await fetchJson(`/api/v3/plugins/calendar-onekite/events?${qs.toString()}`);
       (evs || []).forEach((ev) => {
@@ -960,9 +971,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 '';
           }