nodebb-plugin-calendar-onekite 11.2.26 → 11.2.27

package/CHANGELOG.md

@@ -1,8 +1,13 @@
+# Changelog
 
-## 1.0.1
-- Correctif : après validation d'une réservation depuis l’ACP, le lien HelloAsso (paymentUrl) est maintenant bien enregistré et transmis au calendrier (bouton **Payer maintenant** visible dans la modale).
+## 1.0.2
+- Calendrier : compatibilité thèmes/legacy — expose `window.$` / `window.jQuery` si absents (dependency AMD `jquery`) pour éviter l’erreur **`$ is not defined`**.
+- Calendrier : compatibilité NodeBB — shim minimal pour `app.alert(...)` (redirige vers le module `alerts`) afin d’éviter l’erreur **`app.alert is not a function`**.
+- Calendrier : évite la double initialisation (hook `action:ajaxify.end` + tick initial) qui pouvait provoquer un clignotement au chargement.
+- ACP : suppression du `require(...)` inline dans le template ACP (l’ACP script est déjà chargé via `acpScripts`) et auto-init via `ajaxify`.
 
-# Changelog
+## 1.0.1
+- Correctif : après validation d’une réservation depuis l’ACP, le lien HelloAsso (`paymentUrl`) est maintenant bien enregistré et transmis au calendrier (bouton **Payer maintenant** visible dans la modale).
 
 ## 1.0.0
 
@@ -15,7 +20,7 @@
 - Évènements : alignement de l’horloge identique aux icônes des réservations (rendu sans colonne de temps FullCalendar).
 - Création d’évènement : clic sur un jour = valeur par défaut **07:00 → 07:00** (gestion robuste de l’`end` exclusif FullCalendar).
 - Modales : correction d’un cas où, après annulation (ESC/clic extérieur), les modales ne réapparaissaient plus.
-- **Nouveau** : conservation du dernier mode sélectionné (**Location** / **Évènement**) même après création/annulation/refetch (persisté par utilisateur via `localStorage`).
+- Nouveau : conservation du dernier mode sélectionné (**Location** / **Évènement**) même après création/annulation/refetch (persisté par utilisateur via `localStorage`).
 
 ### Emails
 - Template relance : ajout du même bloc de paiement que dans pending (bouton + lien de secours).

package/package.json

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

package/plugin.json

@@ -31,5 +31,5 @@
   "acpScripts": [
     "public/admin.js"
   ],
-  "version": "1.0.1"
+  "version": "1.0.2"
 }

package/public/admin.js

@@ -1,5 +1,5 @@
 
-define('admin/plugins/calendar-onekite', ['alerts', 'bootbox'], function (alerts, bootbox) {
+define('admin/plugins/calendar-onekite', ['alerts', 'bootbox', 'hooks'], function (alerts, bootbox, hooks) {
   'use strict';
 
   // Cache of pending reservations keyed by rid so delegated click handlers
@@ -782,5 +782,23 @@ define('admin/plugins/calendar-onekite', ['alerts', 'bootbox'], function (alerts
     }
   }
 
+  // Auto-init when the ACP page is rendered (ajaxify navigation).
+  function autoInit(data) {
+    try {
+      const tpl = data && data.template ? data.template.name : (window.ajaxify && window.ajaxify.data && window.ajaxify.data.template ? window.ajaxify.data.template.name : '');
+      if (tpl === 'admin/plugins/calendar-onekite') {
+        init();
+      }
+    } catch (e) {}
+  }
+
+  try {
+    if (hooks && typeof hooks.on === 'function') {
+      hooks.on('action:ajaxify.end', autoInit);
+    }
+    // Initial load fallback
+    setTimeout(() => autoInit({ template: (window.ajaxify && window.ajaxify.data && window.ajaxify.data.template) || { name: '' } }), 0);
+  } catch (e) {}
+
   return { init };
 });

package/public/client.js

@@ -1,8 +1,34 @@
 /* global FullCalendar, ajaxify */
 
-define('forum/calendar-onekite', ['alerts', 'bootbox', 'hooks'], function (alerts, bootbox, hooks) {
+define('forum/calendar-onekite', ['jquery', 'alerts', 'bootbox', 'hooks'], function ($, alerts, bootbox, hooks) {
   'use strict';
 
+  // Some themes or legacy scripts still expect a global jQuery ($).
+  // Provide a safe global alias when missing.
+  try {
+    if (typeof window.$ === 'undefined' && $) window.$ = $;
+    if (typeof window.jQuery === 'undefined' && $) window.jQuery = $;
+  } catch (e) {}
+
+  // NodeBB core evolved from app.alert(...) to the alerts module.
+  // Provide a tiny compatibility shim to prevent console errors from legacy code.
+  try {
+    if (typeof window.app !== 'undefined' && window.app && typeof window.app.alert !== 'function') {
+      window.app.alert = function (data) {
+        try {
+          const type = (data && data.type) ? String(data.type) : 'info';
+          const msg = (data && (data.message || data.text)) ? String(data.message || data.text) : '';
+          if (!msg) return;
+          if (alerts && typeof alerts[type] === 'function') {
+            alerts[type](msg);
+          } else if (alerts && typeof alerts.alert === 'function') {
+            alerts.alert({ type, message: msg });
+          }
+        } catch (e) {}
+      };
+    }
+  } catch (e) {}
+
   // Ensure small UI tweaks are applied even when themes override bootstrap defaults.
   (function ensureOneKiteStyles() {
     try {
@@ -828,6 +854,17 @@ function toDatetimeLocalValue(date) {
       return;
     }
 
+    // Prevent double initialisation (ajaxify.end + initial-load tick) which can
+    // cause a visible "double refresh" flicker.
+    try {
+      if (el.dataset && el.dataset.onekiteInit === '1') {
+        return;
+      }
+      if (el.dataset) {
+        el.dataset.onekiteInit = '1';
+      }
+    } catch (e) {}
+
     if (typeof FullCalendar === 'undefined') {
       showAlert('error', 'FullCalendar non chargé');
       return;
@@ -1633,10 +1670,19 @@ function toDatetimeLocalValue(date) {
   }
 
   // Auto-init on /calendar when ajaxify finishes rendering.
+  // Prevent double initialisation (ajaxify.end + initial tick) which causes a visible flicker.
+  let didInit = false;
+
   function autoInit(data) {
     try {
       const tpl = data && data.template ? data.template.name : (ajaxify && ajaxify.data && ajaxify.data.template ? ajaxify.data.template.name : '');
       if (tpl === 'calendar-onekite') {
+        const el = document.querySelector('#onekite-calendar');
+        if (!el) return;
+        if (el.dataset && el.dataset.onekiteInit === '1') return;
+        if (didInit) return;
+        didInit = true;
+        try { if (el.dataset) el.dataset.onekiteInit = '1'; } catch (e) {}
         init('#onekite-calendar');
       }
     } catch (e) {}

package/templates/admin/plugins/calendar-onekite.tpl

@@ -179,12 +179,10 @@
   </div>
 </div>
 
-<script>
-  require(['admin/plugins/calendar-onekite'], function (mod) {
-    if (mod && mod.init) {
-      mod.init();
-    }
-  });
-</script>
+<!--
+  No inline require() here.
+  NodeBB loads this plugin's ACP script via plugin.json (acpScripts).
+  The admin module auto-initialises on ajaxify.
+-->
 
 <!-- IMPORT admin/partials/settings/footer.tpl -->