nodebb-plugin-onekite-calendar 2.0.28 → 2.0.30

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog – calendar-onekite
 
+## 1.3.26
+- Scheduler : garde "send-once" rendue **atomique** en multi-instance (priorité à `db.incrObjectField(key, rid)`), pour garantir qu'au moins une instance envoie les emails/Discord avant suppression.
+- Scheduler : en cas d'erreur DB sur la garde, on **privilégie l'envoi** plutôt que "expire sans email" (dernier recours : autoriser l'envoi même si la garde échoue).
+
+## 1.3.25
+- Scheduler : correction robuste du mécanisme "send-once" en environnement multi-process (utilise `db.incrObjectField` quand disponible).
+- Fix : évite le cas "réservation expirée/supprimée sans email/Discord" lorsque la DB ne supporte pas correctement les sets/objets utilisés comme garde.
+
+## 1.3.24
+- Multi-process : le scheduler ne démarre pas si NodeBB a `runJobs=false` (évite les exécutions multiples de jobs en cluster).
+
 ## 1.3.23
 - Scheduler (paiement en attente) : correction de la garde "send-once" pour compatibilité avec tous les adaptateurs DB NodeBB (fallback si `db.setAdd` n'est pas disponible).
 - Scheduler : les erreurs ne sont plus avalées silencieusement (logs `Scheduler error ...`).

package/lib/scheduler.js

@@ -16,6 +16,18 @@ async function addOnce(key, value) {
   const v = String(value);
   if (!v) return false;
 
+  // Best-effort atomic guard across multi-process / multi-instance.
+  // incrObjectField is implemented by most NodeBB DB adapters (Redis/Mongo).
+  // If it exists, it gives us a reliable "first winner" (value === 1).
+  if (typeof db.incrObjectField === 'function') {
+    try {
+      const n = await db.incrObjectField(key, v);
+      return Number(n) === 1;
+    } catch (e) {
+      // fall through
+    }
+  }
+
   // Preferred atomic helper (Redis + some adapters)
   if (typeof db.setAdd === 'function') {
     try {
@@ -33,7 +45,9 @@ async function addOnce(key, value) {
     await db.setObjectField(key, v, 1);
     return true;
   } catch (e) {
-    return false;
+    // Last resort: allow sending rather than silently doing nothing.
+    // This may duplicate emails in edge cases, but avoids "expires with no email".
+    return true;
   }
 }
 
@@ -419,7 +433,15 @@ async function processAwaitingPayment() {
 }
 
 function start() {
+  const runJobs = nconf.get('runJobs');
+  if (runJobs === false || runJobs === 'false' || runJobs === 0 || runJobs === '0') {
+    // eslint-disable-next-line no-console
+    console.info('[calendar-onekite] Scheduler disabled (runJobs=false)');
+    return;
+  }
   if (timer) return;
+  // eslint-disable-next-line no-console
+  console.info('[calendar-onekite] Scheduler enabled');
   timer = setInterval(() => {
     expirePending().catch((err) => {
       // eslint-disable-next-line no-console

package/package.json

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