nodebb-plugin-calendar-onekite 11.1.46 → 11.1.47

package/lib/admin.js

@@ -25,7 +25,8 @@ async function sendEmail(template, toEmail, subject, data) {
         return;
       }
       if (emailer.send.length === 3) {
-        await emailer.send(template, toEmail, data);
+        const dataWithSubject = Object.assign({}, data || {}, subject ? { subject } : {});
+        await emailer.send(template, toEmail, dataWithSubject);
         return;
       }
       await emailer.send(template, toEmail, subject, data);
@@ -35,6 +36,19 @@ async function sendEmail(template, toEmail, subject, data) {
   }
 }
 
+function normalizeCallbackUrl(configured, meta) {
+  const base = (meta && meta.config && (meta.config.url || meta.config['url'])) ? (meta.config.url || meta.config['url']) : '';
+  let url = (configured || '').trim();
+  if (!url) {
+    url = base ? `${base}/helloasso` : '';
+  }
+  if (url && url.startsWith('/') && base) {
+    url = `${base}${url}`;
+  }
+  return url;
+}
+
+
 const dbLayer = require('./db');
 const helloasso = require('./helloasso');
 
@@ -89,6 +103,9 @@ admin.approveReservation = async function (req, res) {
     clientId: settings.helloassoClientId,
     clientSecret: settings.helloassoClientSecret,
   });
+        if (!token) {
+          console.warn('[calendar-onekite] HelloAsso access token not obtained (approve ACP)', { rid: r && r.rid });
+        }
 
   let paymentUrl = null;
   if (token) {

package/lib/api.js

@@ -27,8 +27,10 @@ async function sendEmail(template, toEmail, subject, data) {
         return;
       }
       // Some builds: (template, email, data)
+      // In that case, subject is expected inside `data.subject`.
       if (emailer.send.length === 3) {
-        await emailer.send(template, toEmail, data);
+        const dataWithSubject = Object.assign({}, data || {}, subject ? { subject } : {});
+        await emailer.send(template, toEmail, dataWithSubject);
         return;
       }
       // Fallback: try 4-args anyway
@@ -41,6 +43,20 @@ async function sendEmail(template, toEmail, subject, data) {
   }
 }
 
+function normalizeCallbackUrl(configured, meta) {
+  const base = (meta && meta.config && (meta.config.url || meta.config['url'])) ? (meta.config.url || meta.config['url']) : '';
+  let url = (configured || '').trim();
+  if (!url) {
+    url = base ? `${base}/helloasso` : '';
+  }
+  // If relative, prefix with forum base url
+  if (url && url.startsWith('/') && base) {
+    url = `${base}${url}`;
+  }
+  return url;
+}
+
+
 function overlap(aStart, aEnd, bStart, bEnd) {
   return aStart < bEnd && bStart < aEnd;
 }
@@ -168,6 +184,9 @@ api.getItems = async function (req, res) {
     clientId: settings.helloassoClientId,
     clientSecret: settings.helloassoClientSecret,
   });
+    if (!token) {
+      console.warn('[calendar-onekite] HelloAsso access token not obtained (approve API)', { rid });
+    }
 
   if (!token) {
     return res.json([]);
@@ -321,11 +340,17 @@ api.approveReservation = async function (req, res) {
       formType: settings2.helloassoFormType,
       formSlug: settings2.helloassoFormSlug,
       // r.total is stored as an estimated total in euros; HelloAsso expects cents.
-      totalAmount: Math.max(0, Math.round((Number(r.total) || 0) * 100)),
+      totalAmount: (() => {
+        const cents = Math.max(0, Math.round((Number(r.total) || 0) * 100));
+        if (!cents) {
+          console.warn('[calendar-onekite] HelloAsso totalAmount is 0 (approve API)', { rid, total: r.total });
+        }
+        return cents;
+      })(),
       payerEmail: payer && payer.email ? payer.email : '',
       // By default, point to the forum base url so the webhook hits this NodeBB instance.
       // Can be overridden via ACP setting `helloassoCallbackUrl`.
-      callbackUrl: (settings2.helloassoCallbackUrl || (meta.config && meta.config.url ? `${meta.config.url}/helloasso` : '')),
+      callbackUrl: normalizeCallbackUrl(settings2.helloassoCallbackUrl, meta),
       itemName: 'Réservation matériel OneKite',
       containsDonation: false,
       metadata: { reservationId: String(rid) },

package/lib/helloasso.js

@@ -194,9 +194,15 @@ async function createCheckoutIntent({ env, token, organizationSlug, formType, fo
   if (status >= 200 && status < 300 && json) {
     return json.redirectUrl || json.checkoutUrl || json.url || null;
   }
+  // Log the error payload to help diagnose configuration issues (slug, env, urls, amount, etc.)
+  try {
+    // eslint-disable-next-line no-console
+    console.warn('[calendar-onekite] HelloAsso checkout-intent failed', { status, json });
+  } catch (e) { /* ignore */ }
   return null;
 }
 
+
 module.exports = {
   getAccessToken,
   listItems,

package/package.json

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

package/public/admin.js

@@ -77,7 +77,7 @@ define('admin/plugins/calendar-onekite', ['alerts', 'bootbox'], function (alerts
     for (const r of list) {
       const created = r.createdAt ? fmtFR(r.createdAt) : '';
       const itemNames = Array.isArray(r.itemNames) && r.itemNames.length ? r.itemNames : [r.itemName || r.itemId].filter(Boolean);
-      const itemsHtml = itemNames.map(n => `<div>• ${escapeHtml(String(n))}</div>`).join('');
+      const itemsHtml = `<ul style="margin: 0 0 10px 18px;">${itemNames.map(n => `<li>${escapeHtml(String(n))}</li>`).join('')}</ul>`;
       const div = document.createElement('div');
       div.className = 'list-group-item';
       div.innerHTML = `
@@ -107,9 +107,12 @@ define('admin/plugins/calendar-onekite', ['alerts', 'bootbox'], function (alerts
       });
 
       approveBtn.addEventListener('click', async () => {
+        const itemNamesModal = Array.isArray(r.itemNames) && r.itemNames.length ? r.itemNames : [r.itemName || r.itemId].filter(Boolean);
+        const itemsHtmlModal = `<ul style="margin: 0 0 10px 18px;">${itemNamesModal.map(n => `<li>${escapeHtml(String(n))}</li>`).join('')}</ul>`;
+
         const opts = (() => {
           const out = [];
-          for (let h = 0; h < 24; h++) {
+          for (let h = 7; h < 24; h++) {
             for (let m = 0; m < 60; m += 5) {
               out.push(`${String(h).padStart(2, '0')}:${String(m).padStart(2, '0')}`);
             }
@@ -162,7 +165,7 @@ define('admin/plugins/calendar-onekite', ['alerts', 'bootbox'], function (alerts
   function timeOptions(stepMinutes) {
     const step = stepMinutes || 5;
     const out = [];
-    for (let h = 0; h < 24; h++) {
+    for (let h = 7; h < 24; h++) {
       for (let m = 0; m < 60; m += step) {
         const hh = String(h).padStart(2, '0');
         const mm = String(m).padStart(2, '0');

package/public/client.js

@@ -339,9 +339,11 @@ define('forum/calendar-onekite', ['alerts', 'bootbox', 'hooks'], function (alert
             label: 'Valider',
             className: 'btn-success',
             callback: async () => {
+              const itemNames = Array.isArray(p.itemNames) && p.itemNames.length ? p.itemNames : String(itemsLabel || '').split(',').map(s => s.trim()).filter(Boolean);
+              const itemsHtml = itemNames.map(n => `<li>${String(n).replace(/</g, '&lt;').replace(/>/g, '&gt;')}</li>`).join('');
               const opts = (() => {
                 const out = [];
-                for (let h = 0; h < 24; h++) {
+                for (let h = 7; h < 24; h++) {
                   for (let m = 0; m < 60; m += 5) {
                     out.push(`${String(h).padStart(2, '0')}:${String(m).padStart(2, '0')}`);
                   }
@@ -416,4 +418,4 @@ define('forum/calendar-onekite', ['alerts', 'bootbox', 'hooks'], function (alert
   return {
     init,
   };
-});
+});

package/templates/emails/calendar-onekite_approved.tpl

@@ -19,5 +19,10 @@
 <!-- ENDIF pickupTime -->
 
 <!-- IF paymentUrl -->
-<p><strong>Lien de paiement :</strong> <a href="{paymentUrl}">{paymentUrl}</a></p>
+<p>
+  <a href="{paymentUrl}" style="display:inline-block;padding:12px 18px;border-radius:6px;text-decoration:none;border:1px solid #333;">
+    Payer maintenant
+  </a>
+</p>
+<p style="font-size: 12px; color: #666;">Si le bouton ne fonctionne pas, utilisez ce lien : <a href="{paymentUrl}">{paymentUrl}</a></p>
 <!-- ENDIF paymentUrl -->