nodebb-plugin-onekite-calendar 2.0.31 → 2.0.33

package/CHANGELOG.md

@@ -1,10 +1,5 @@
 # Changelog – calendar-onekite
 
-## 1.3.28
-- Perf (temps réel) : debounce global des `refetchEvents()` (calendrier + widget) et **skip** quand l’onglet est masqué (refetch unique au retour de visibilité).
-- Perf/Robustesse (multi-instance) : ajout d’un **lock distribué Redis** sur le tick scheduler (un seul runner exécute le cycle à un instant donné).
-- Perf (API events) : cache Redis très court (**TTL 2s**) par utilisateur/permissions sur l’endpoint events pour absorber les rafales.
-
 ## 1.3.27
 - Refactor : ajout d'un module `lib/utils.js` (format dates FR, lecture settings ACP, helpers listes/UIDs) pour supprimer les duplications.
 - Cleanup : suppression d'un doublon de fonctions dans `helloassoWebhook.js` (formatFR + getReservationIdFromPayload).

package/lib/api.js

@@ -9,8 +9,6 @@ const user = require.main.require('./src/user');
 const groups = require.main.require('./src/groups');
 const db = require.main.require('./src/database');
 const logger = require.main.require('./src/logger');
-let cache = null;
-try { cache = require.main.require('./src/cache'); } catch (e) { cache = null; }
 
 const dbLayer = require('./db');
 const { formatFR } = require('./utils');
@@ -479,42 +477,6 @@ function computeEtag(payload) {
   return `W/"${hash}"`;
 }
 
-async function cacheGet(key) {
-  try {
-    if (!cache) return null;
-    if (typeof cache.get === 'function') {
-      if (cache.get.length >= 2) {
-        return await new Promise((resolve) => {
-          cache.get(key, (err, data) => resolve(err ? null : data));
-        });
-      }
-      return await cache.get(key);
-    }
-  } catch (e) {}
-  return null;
-}
-
-async function cacheSet(key, value, ttlSeconds) {
-  try {
-    if (!cache) return;
-    const ttl = Math.max(1, parseInt(ttlSeconds, 10) || 2);
-    if (typeof cache.set === 'function') {
-      if (cache.set.length >= 4) {
-        await new Promise((resolve) => {
-          cache.set(key, value, ttl, (err) => resolve(!err));
-        });
-        return;
-      }
-      // Some NodeBB versions accept an options object.
-      try {
-        await cache.set(key, value, { ttl });
-        return;
-      } catch (e) {}
-      await cache.set(key, value, ttl);
-    }
-  } catch (e) {}
-}
-
 api.getEvents = async function (req, res) {
   const qStartRaw = (req && req.query && req.query.start !== undefined) ? String(req.query.start).trim() : '';
   const qEndRaw = (req && req.query && req.query.end !== undefined) ? String(req.query.end).trim() : '';
@@ -534,22 +496,6 @@ api.getEvents = async function (req, res) {
   const canSpecialCreate = req.uid ? await canCreateSpecial(req.uid, settings) : false;
   const canSpecialDelete = req.uid ? await canDeleteSpecial(req.uid, settings) : false;
 
-  // Ultra-short Redis-backed cache (2s) to absorb bursts and reduce redundant renders.
-  // Keyed per uid + permissions because payload can include private fields.
-  const uidKey = req.uid ? String(req.uid) : '0';
-  const cacheKey = `calendar-onekite:events:${uidKey}:${canMod ? 'm' : 'u'}:${widgetMode ? 'w' : 'p'}:${qStartYmd || startTs}:${qEndYmd || endTs}`;
-  try {
-    const cached = await cacheGet(cacheKey);
-    if (cached && cached.payload && cached.etag) {
-      res.setHeader('ETag', cached.etag);
-      res.setHeader('Cache-Control', 'private, max-age=0, must-revalidate');
-      if (String(req.headers['if-none-match'] || '') === String(cached.etag)) {
-        return res.status(304).end();
-      }
-      return res.json(cached.payload);
-    }
-  } catch (e) {}
-
   // Fetch a wider window because an event can start before the query range
   // and still overlap.
   const wideStart = Math.max(0, startTs - 366 * 24 * 3600 * 1000);
@@ -688,12 +634,6 @@ api.getEvents = async function (req, res) {
   });
 
   const etag = computeEtag(out);
-
-  // Store in cache for a very short period (burst absorption).
-  try {
-    await cacheSet(cacheKey, { etag, payload: out }, 2);
-  } catch (e) {}
-
   res.setHeader('ETag', etag);
   res.setHeader('Cache-Control', 'private, max-age=0, must-revalidate');
   if (String(req.headers['if-none-match'] || '') === etag) {

package/lib/scheduler.js

@@ -8,43 +8,9 @@ const realtime = require('./realtime');
 const nconf = require.main.require('nconf');
 const groups = require.main.require('./src/groups');
 const utils = require('./utils');
-let redis = null;
-try { redis = require.main.require('./src/redis'); } catch (e) { redis = null; }
 
 let timer = null;
 
-// Distributed lock (Redis) to avoid running the scheduler on multiple NodeBB instances.
-// Safe no-op if Redis is not available.
-async function acquireSchedulerLock(ttlMs) {
-  const key = 'calendar-onekite:scheduler:lock';
-  const value = `${process.pid}:${Date.now()}`;
-  const ttl = Math.max(1000, parseInt(ttlMs, 10) || 55000);
-
-  // Prefer Redis SET NX PX.
-  try {
-    const client = redis && (redis.client || redis);
-    if (client && typeof client.set === 'function') {
-      const res = await new Promise((resolve, reject) => {
-        // node_redis style
-        client.set(key, value, 'NX', 'PX', ttl, (err, out) => {
-          if (err) return reject(err);
-          resolve(out);
-        });
-      });
-      return String(res || '').toUpperCase() === 'OK';
-    }
-  } catch (e) {
-    // ignore and fall back
-  }
-
-  // Fallback: in-process lock only.
-  if (global.__onekiteSchedulerInProcessLockUntil && Date.now() < global.__onekiteSchedulerInProcessLockUntil) {
-    return false;
-  }
-  global.__onekiteSchedulerInProcessLockUntil = Date.now() + ttl;
-  return true;
-}
-
 // Some NodeBB database adapters don't expose setAdd/setRemove helpers.
 // Use a safe "add once" guard to avoid crashing the scheduler.
 async function addOnce(key, value) {
@@ -434,14 +400,13 @@ function start() {
   // eslint-disable-next-line no-console
   console.info('[calendar-onekite] Scheduler enabled');
   timer = setInterval(() => {
-    (async () => {
-      const locked = await acquireSchedulerLock(55 * 1000);
-      if (!locked) return;
-      await expirePending();
-      await processAwaitingPayment();
-    })().catch((err) => {
+    expirePending().catch((err) => {
+      // eslint-disable-next-line no-console
+      console.warn('[calendar-onekite] Scheduler error in expirePending', err && err.message ? err.message : err);
+    });
+    processAwaitingPayment().catch((err) => {
       // eslint-disable-next-line no-console
-      console.warn('[calendar-onekite] Scheduler tick error', err && err.message ? err.message : err);
+      console.warn('[calendar-onekite] Scheduler error in processAwaitingPayment', err && err.message ? err.message : err);
     });
   }, 60 * 1000);
 }

package/lib/widgets.js

@@ -376,20 +376,13 @@ dateClick: function() { window.location.href = calUrl; },
       let tRefetch = null;
       const refetch = function () {
         try {
-          // Skip work while tab is hidden; refetch once when visible again.
-          try {
-            if (typeof document !== 'undefined' && document && document.hidden) {
-              window.__oneKiteWidgetNeedsRefetch = true;
-              return;
-            }
-          } catch (e) {}
           if (tRefetch) return;
           tRefetch = setTimeout(() => {
             tRefetch = null;
             try {
               calendar.refetchEvents();
             } catch (e) {}
-          }, 450);
+          }, 200);
         } catch (e) {}
       };
 
@@ -409,21 +402,6 @@ dateClick: function() { window.location.href = calUrl; },
         };
         socket.on('event:calendar-onekite.calendarUpdated', triggerAll);
         socket.on('event:calendar-onekite.reservationUpdated', triggerAll);
-
-        // If updates occurred while hidden, refetch once when visible again.
-        try {
-          if (!window.__oneKiteWidgetVisibilityBound && typeof document !== 'undefined' && document && typeof document.addEventListener === 'function') {
-            window.__oneKiteWidgetVisibilityBound = true;
-            document.addEventListener('visibilitychange', () => {
-              try {
-                if (!document.hidden && window.__oneKiteWidgetNeedsRefetch) {
-                  window.__oneKiteWidgetNeedsRefetch = false;
-                  triggerAll();
-                }
-              } catch (e) {}
-            }, { passive: true });
-          }
-        } catch (e) {}
       }
     } catch (e) {}
 

package/package.json

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

package/public/client.js

@@ -543,40 +543,13 @@ define('forum/calendar-onekite', ['alerts', 'bootbox', 'hooks'], function (alert
   function scheduleRefetch(cal) {
     try {
       if (!cal || typeof cal.refetchEvents !== 'function') return;
-
-      // Avoid refetching while the tab is hidden; do it once when visible again.
-      try {
-        if (typeof document !== 'undefined' && document && document.hidden) {
-          window.__onekiteNeedsRefetch = true;
-          return;
-        }
-      } catch (e) {}
-
       clearTimeout(window.__onekiteRefetchTimer);
       window.__onekiteRefetchTimer = setTimeout(() => {
         try { cal.refetchEvents(); } catch (e) {}
-      }, 450);
+      }, 150);
     } catch (e) {}
   }
 
-  // If updates occurred while hidden, refetch once when visible again.
-  try {
-    if (!window.__onekiteVisibilityBound && typeof document !== 'undefined' && document && typeof document.addEventListener === 'function') {
-      window.__onekiteVisibilityBound = true;
-      document.addEventListener('visibilitychange', () => {
-        try {
-          if (!document.hidden && window.__onekiteNeedsRefetch) {
-            window.__onekiteNeedsRefetch = false;
-            const cal = window.oneKiteCalendar;
-            if (cal && typeof cal.refetchEvents === 'function') {
-              scheduleRefetch(cal);
-            }
-          }
-        } catch (e) {}
-      }, { passive: true });
-    }
-  } catch (e) {}
-
   async function fetchJsonCached(url, opts) {
     const cached = jsonCache.get(url);
     const headers = Object.assign({}, (opts && opts.headers) || {});

package/templates/emails/calendar-onekite_pending.tpl

@@ -13,3 +13,5 @@
 <p>{dateRange}</p>
 
 <p><strong>Total estimé :</strong> {total} €</p>
+
+<p><a href="{adminUrl}">Ouvrir l'ACP</a></p>

package/templates/emails/calendar-onekite_validator_expired.tpl

@@ -13,5 +13,3 @@
 </ul>
 
 <p>{dateRange}</p>
-
-<p><a href="{adminUrl}">Ouvrir l'ACP</a></p>

package/pkg/package/CHANGELOG.md

@@ -1,161 +0,0 @@
-# Changelog – calendar-onekite
-
-## 1.3.21
-- Comptabilisation : les **sorties gratuites annulées** ne sont plus comptées (elles sont exclues si `cancelledAt` est présent, même si le statut est resté `paid`).
-
-## 1.3.20
-- Temps réel : le **widget** (Calendrier 2 semaines) se met à jour dynamiquement comme la page calendrier (créations / changements de statut / annulations…), sans rechargement.
-
-## 1.3.19
-- Annulation : un validateur/gestionnaire peut désormais annuler **même sa propre** réservation après paiement (la restriction `cannot-cancel-paid` ne s'applique plus aux validateurs).
-
-## 1.3.18
-- Fix HelloAsso webhook : suppression d'une double déclaration `realtime` qui empêchait le plugin de se charger (SyntaxError).
-
-## 1.3.17
-- Temps réel : les **créations** (locations / évènements), **changements de statut** (validation, refus, annulation, paiement reçu) et **expirations** se répercutent automatiquement sur le calendrier pour **tous les utilisateurs connectés**, sans rechargement de page.
-
-## 1.3.16
-- Calendrier : le **jour actuel** n’est plus affiché comme indisponible (gris + panneau sens interdit). Seules les **dates passées** restent désactivées visuellement.
-
-## 1.3.15
-- Permissions : les **modérateurs globaux / gestionnaires** (groupe NodeBB « Global Moderators », selon le slug) sont désormais considérés comme validateurs pour les actions de modération (dont l’**annulation**).
-
-## 1.3.14
-- Mobile FAB : la modale de sélection de dates autorise désormais la réservation **le jour même** (seules les dates passées sont désactivées).
-
-## 1.3.13
-- Réservations : il est désormais possible de réserver pour le jour même (seules les dates passées sont refusées).
-- Annulation : les validateurs peuvent annuler une réservation déjà payée.
-
-## 1.3.12
-- Expiration automatique : envoi d’un email au demandeur avec la raison :
-  - « Demande non prise en charge dans le temps imparti » (demande en attente)
-  - « Paiement non reçu dans le temps imparti » (paiement en attente)
-- ACP : ajout de 2 paramètres pour envoyer un rappel email aux validateurs sur les demandes en attente / paiement en attente.
-- Emails validateurs : nouveaux templates (rappel + expiration) avec lien vers l’ACP.
-
-## 1.3.11
-- ACP (Demandes en attente) : affichage du pseudo du demandeur avec lien vers sa fiche utilisateur.
-
-## 1.3.10
-- Mobile FAB : surlignage de la sélection de dates rendu lisible en light mode (contraste renforcé, start/end en primary).
-
-## 1.3.9
-- Mobile FAB : correction de la sélection de plage (tous les jours sélectionnés sont désormais bien inclus dans la surbrillance).
-- Mobile FAB : amélioration du rendu en mode sombre (bordures/fonds adaptés, contraste renforcé).
-
-## 1.3.8
-- ACP Comptabilisation : ajout d’un **grand total** (montant total des locations payées sur la période) + compteurs (payées / sorties gratuites).
-
-## 1.3.7
-- Mobile FAB : remplacement des 2 champs date par un **sélecteur unique** (calendrier) permettant de choisir directement une plage (clic début puis fin, fin incluse), avec dates passées/aujourd'hui désactivées.
-
-## 1.3.6
-- ACP Comptabilisation : ajout d’un tableau séparé « Détails des sorties gratuites » (avec le nom du matériel).
-- Mobile FAB : la modale utilise maintenant des champs date avec calendrier (type="date") et empêche toute réservation pour le jour même ou dans le passé.
-
-## 1.3.5
-- ACP : ajout d’un paramètre « Location longue durée (jours) pour validateurs ». Si une réservation faite par un validateur dépasse ce nombre de jours, elle redevient payante et suit le workflow normal (demande → validation → paiement HelloAsso). Mets 0 pour conserver le comportement « toujours gratuit ».
-
-## 1.3.4
-- Réservations gratuites (validateurs) : ne sont plus comptées comme chiffre d’affaires dans la comptabilisation. Elles apparaissent désormais sur une ligne séparée « Sorties gratuites » (et sont marquées (gratuit) dans le détail).
-
-## 1.3.3
-- Comptabilité : les réservations « auto-checkées » (réservations faites par un validateur) recalculent désormais le total côté serveur (catalogue HelloAsso × nb jours calendaires) afin d’être comptabilisées correctement.
-
-## 1.3.2
-- Les membres validateurs (ceux qui peuvent valider/supprimer une demande) voient leurs propres réservations passer directement en statut payé/checked (pas de workflow paiement).
-- UI : message de succès adapté (réservation confirmée).
-
-## 1.3.1
-- ACP : ajout de 2 actions rapides dans l’onglet Maintenance : « Tout mettre en maintenance » et « Tout enlever de maintenance » (avec audit).
-
-## 1.3.0
-- Maintenance (sans dates) : blocage manuel ON/OFF par matériel (ACP + API). Les matériels en maintenance sont grisés et affichés comme : 🔧 Nom (en maintenance).
-- Audit : journal des actions (demande, validation, refus, annulation, maintenance) avec consultation et purge par année (ACP + API).
-
-## 1.2.18
-- API events + anti double booking : les tests de chevauchement utilisent désormais en priorité startDate/endDate (YYYY-MM-DD) quand disponibles (logique calendaire pure, endDate exclusive). Cela supprime définitivement les faux chevauchements liés aux timestamps/fuseaux/DST, notamment sur mobile et « Durée rapide ».
-
-## 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.
-
-## 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 ».
-
-## 1.2.13
-- ACP (mode sombre) : correction de la visibilité de la liste d’autocomplete d’adresse (couleurs via variables Bootstrap)
-
-## 1.2.12
-- Modales (calendrier + ACP) : autocomplete adresse rendu identique et compatible Bootstrap input-group (plus de wrapper qui casse l’affichage)
-
-## 1.2.11
-- ACP : ajout de la recherche automatique d’adresse (autocomplete Nominatim) dans la modale de validation (unitaire + batch), comme sur le calendrier
-
-## 1.2.10
-- HelloAsso : calcul des jours 100% fiable (différence en jours calendaires Y/M/J, sans dépendance aux heures/au fuseau/DST)
-- FullCalendar : endDate traitée comme exclusive partout (UI + checkout)
-- HelloAsso : montant du checkout recalculé côté serveur à partir du catalogue (prix/jour × nbJours)
-
-## 1.2.9
-- Modale réservation : retour du grisé des matériels indisponibles (API events expose à nouveau itemIds)
-
-## 1.2.8
-- Popup réservation : correction « Durée rapide » (la période envoyée correspond bien à la durée sélectionnée)
-
-## 1.2.7
-- UI : suppression du bouton flottant mobile « + Réserver »
-- Client : nettoyage du code associé (suppression du bloc FAB)
-
-## 1.2.6
-- ACP : anti double action (verrou UI + boutons désactivés/spinner) sur valider/refuser (unitaire + batch)
-
-## 1.2.5
-- Mobile : bouton flottant « + Réserver »
-- Création : anti double-tap/click (verrou actions) + invalidation cache events + refetch léger
-- Calendrier : jours passés / aujourd’hui affichés comme non-sélectionnables (règle visuelle)
-- Popup réservation : raccourcis de durée (1j/2j/3j/7j) + recalcul total + mise à jour des matériels bloqués
-- ACP : actions en batch (valider/refuser une sélection) + compteur de sélection
-- API : idempotence sur valider/refuser + audit enrichi (refusedBy, cancelledByUsername)
-
-## 1.0.3
-- Suppression du texte d’archivage dans le toast de purge (plus de « 0 archivés »)
-- Renommage du plugin : nodebb-plugin-onekite-calendar
-- Discord : notification « ❌ Réservation annulée » (annulation manuelle + annulation automatique) + option ACP
-
-## 1.0.2
-- Purge calendrier : suppression réelle des réservations (aucune logique d’archivage)
-- Compta conservée séparément (la purge n’y touche jamais)
-
-## 1.0.1
-- ACP : modales Valider / Refuser OK (plus de retour au premier onglet)
-- Scheduler : optimisation (batch DB, scans réduits)
-- API events : pagination interne + ETag optimisé
-- HelloAsso : token mis en cache + protection rate-limit (429 / Cloudflare 1015), sans logs supplémentaires
-- Discord : notifications en embeds + icônes ⏳ (demande) et 💳 (paiement)
-- Widget : suppression de “2 semaines” dans le titre affiché
-- Correctifs divers de stabilité/performance (dont crash au démarrage)
-
-## 1.0.0
-- Première version stable du plugin calendar-onekite
-- Gestion des réservations de matériel avec contrôle de disponibilité
-- Calendrier FullCalendar via CDN
-- Validation / refus des demandes depuis l’ACP
-- Notifications Discord
-- Intégration paiements HelloAsso
-
-## 1.2.19
-- Mobile: ajout d’un bouton flottant (FAB) sur la page calendrier uniquement.
-- Le FAB ouvre une mini-modale de sélection de dates (dd/mm/yyyy) puis ouvre la modale standard de réservation.
-- Le FAB est automatiquement retiré quand on navigue hors de la page calendrier.
-
-## 1.3.10
-- Widget : ne pas afficher le statut « Payée » pour les sorties gratuites (validateurs).
-- Modale réservation : affichage de la période corrigé (fin affichée inclusive, fin stockée exclusive).