nodebb-plugin-onekite-calendar 1.0.31 → 1.0.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-onekite-calendar",
-  "version": "1.0.31",
+  "version": "1.0.33",
   "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": "1.0.31"
+  "version": "1.0.33"
 }

package/public/client.js

@@ -36,6 +36,10 @@ define('forum/calendar-onekite', ['alerts', 'bootbox', 'hooks'], function (alert
   // interactions or quick re-renders.
   let isDialogOpen = false;
 
+  // Avoid duplicate toasts when the same validation fails both client-side and
+  // server-side (e.g. "today or past" date rule).
+  let lastDateRuleToastAt = 0;
+
   function escapeHtml(str) {
     return String(str)
       .replace(/&/g, '&')
@@ -1193,6 +1197,7 @@ function toDatetimeLocalValue(date) {
             const startDateCheck = toLocalYmd(info.start);
             const todayCheck = toLocalYmd(new Date());
             if (startDateCheck <= todayCheck) {
+              lastDateRuleToastAt = Date.now();
               showAlert('error', "Impossible de réserver pour aujourd’hui ou une date passée.");
               calendar.unselect();
               isDialogOpen = false;
@@ -1243,7 +1248,10 @@ function toDatetimeLocalValue(date) {
           } else if (code === '409') {
             showAlert('error', 'Impossible : au moins un matériel est déjà réservé ou en attente sur cette période.');
           } else if (code === '400' && payload && (payload.error === 'date-too-soon' || payload.code === 'date-too-soon')) {
-            showAlert('error', String(payload.message || "Impossible de réserver pour aujourd’hui ou une date passée."));
+            // If we already showed the client-side toast a moment ago, avoid a duplicate.
+            if (!lastDateRuleToastAt || (Date.now() - lastDateRuleToastAt) > 1500) {
+              showAlert('error', String(payload.message || "Impossible de réserver pour aujourd’hui ou une date passée."));
+            }
           } else {
             const msgRaw = payload && (payload.message || payload.error || payload.msg)
               ? String(payload.message || payload.error || payload.msg)
@@ -1299,8 +1307,12 @@ function toDatetimeLocalValue(date) {
           return;
         }
 
-        const end = new Date(start.getTime() + 24 * 60 * 60 * 1000);
-        calendar.select(start, end);
+        // IMPORTANT: Do not call `calendar.select()` here.
+        // On mobile (and depending on FullCalendar behavior), a simple day click can already
+        // trigger the `select` callback. Calling `calendar.select()` would fire `select` a second
+        // time, causing duplicate toasts and duplicate dialogs.
+        // We rely on FullCalendar's built-in selection on click.
+        return;
       },
 
       eventClick: async function (info) {