nodebb-plugin-equipment-calendar 8.9.0 → 8.9.2

package/library.js

@@ -1324,31 +1324,23 @@ async function handleCreateReservation(req, res) {
       return helpers.notAllowed(req, res);
     }
 
-    const startIsoInput = String(req.body.start || req.body.startDate || req.body.startIso || '').trim();
-    const endIsoInput = String(req.body.end || req.body.endDate || req.body.endIso || '').trim();
-
     const itemIdsRaw = (req.body.itemIds !== undefined ? req.body.itemIds : req.body.itemId);
     const itemIds = normalizeItemIds(itemIdsRaw);
     if (!itemIds.length) return res.status(400).send('itemIds required');
 
-    // Dates come from modal as YYYY-MM-DD
-    const startVal = req.body.start || req.body.startDate || req.body.startIso || req.body.startMs || req.body.startTime || '';
-    const endVal = req.body.end || req.body.endDate || req.body.endIso || req.body.endMs || req.body.endTime || '';
-    const start = parseDateInput(startVal);
-    const end = parseDateInput(endVal);
-    if (!start || !end) return res.status(400).send('dates required');
-
-    const startIso = (/^\d{4}-\d{2}-\d{2}$/.test(startIsoInput) ? startIsoInput : `${start.getUTCFullYear()}-${String(start.getUTCMonth()+1).padStart(2,'0')}-${String(start.getUTCDate()).padStart(2,'0')}`);
-    const endIso = (/^\d{4}-\d{2}-\d{2}$/.test(endIsoInput) ? endIsoInput : `${end.getUTCFullYear()}-${String(end.getUTCMonth()+1).padStart(2,'0')}-${String(end.getUTCDate()).padStart(2,'0')}`);
-
+    // Dates expected as YYYY-MM-DD (end inclusive in UI)
+    const startIso = String(req.body.start || req.body.startDate || req.body.startIso || '').trim();
+    const endIso = String(req.body.end || req.body.endDate || req.body.endIso || '').trim();
+    if (!/^\d{4}-\d{2}-\d{2}$/.test(startIso) || !/^\d{4}-\d{2}-\d{2}$/.test(endIso)) {
+      return res.status(400).send('dates required');
+    }
 
-    const startMs = Date.UTC(start.getUTCFullYear(), start.getUTCMonth(), start.getUTCDate());
-    let endMs = Date.UTC(end.getUTCFullYear(), end.getUTCMonth(), end.getUTCDate());
-    if (endMs <= startMs) endMs = startMs + 24 * 60 * 60 * 1000;
+    const startMs = Date.parse(startIso + 'T00:00:00Z');
+    const endMs = Date.parse(addDaysIso(endIso, 1) + 'T00:00:00Z'); // exclusive end
+    if (!startMs || !endMs || endMs <= startMs) {
+      return res.status(400).send('dates required');
+    }
 
-    const endMsInclusive = endMs;
-    const endMsExclusive = endMsInclusive + 24 * 60 * 60 * 1000;
-    endMs = endMsExclusive;
     const days = Math.max(1, Math.round((endMs - startMs) / (24 * 60 * 60 * 1000)));
 
     const items = await getActiveItems(settings);
@@ -1372,14 +1364,14 @@ async function handleCreateReservation(req, res) {
         itemId,
         startMs,
         endMs,
-        startIso: startIso,
-        endIso: endIso,
+        startIso,
+        endIso,
         days,
         unitPrice,
         total,
         notesUser,
         status: 'pending',
-        createdAt: formatDateTimeFR(r.createdAt || 0),
+        createdAt: Date.now(),
       };
       // eslint-disable-next-line no-await-in-loop
       await saveReservation(r);
@@ -1388,10 +1380,11 @@ async function handleCreateReservation(req, res) {
     return res.redirect(nconf.get('relative_path') + '/calendar?created=1');
   } catch (err) {
     winston.error('[equipment-calendar] create error: ' + (err && err.message ? err.message : ''), err);
-    return res.status(500).send(err && err.message ? err.message : 'error');
+    return res.status(500).send('error');
   }
 }
 
+
 async function handleApproveReservation(req, res) {
   try {
     const settings = await getSettings();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-equipment-calendar",
-  "version": "8.9.0",
+  "version": "8.9.2",
   "description": "Equipment reservation calendar for NodeBB (FullCalendar, approvals, HelloAsso payments)",
   "main": "library.js",
   "scripts": {

package/plugin.json

@@ -25,6 +25,6 @@
   "scripts": [
     "public/js/client.js"
   ],
-  "version": "3.0.0-stable4k-syntaxfix",
+  "version": "3.0.0-stable4l-dateslice-noemoji",
   "minver": "4.7.1"
 }

package/public/js/client.js

@@ -144,16 +144,18 @@ function updateTotalPrice() {
           try {
             var status = (arg.event.extendedProps && arg.event.extendedProps.status) ? arg.event.extendedProps.status : '';
             var iconClass = '';
-            if (status === 'pending') iconClass = 'fa-hourglass-half text-warning';
-            else if (status === 'paid') iconClass = 'fa-check-circle text-success';
-            else if (status === 'approved') iconClass = 'fa-check text-primary';
-            else if (status === 'rejected' || status === 'cancelled') iconClass = 'fa-times-circle text-danger';
+            if (status === 'pending') iconClass = 'fa fa-hourglass-half text-warning';
+            else if (status === 'paid') iconClass = 'fa fa-check-circle text-success';
+            else if (status === 'approved') iconClass = 'fa fa-check text-success';
+            else if (status === 'rejected' || status === 'cancelled') iconClass = 'fa fa-times text-danger';
+
             var wrap = document.createElement('div');
             wrap.className = 'ec-event';
             if (iconClass) {
-              var i = document.createElement('i');
-              i.className = 'fa ' + iconClass + ' ec-icon';
-              wrap.appendChild(i);
+              var icon = document.createElement('span');
+              icon.className = 'ec-icon';
+              icon.innerHTML = '<i class="' + iconClass + '"></i>';
+              wrap.appendChild(icon);
             }
             var t = document.createElement('span');
             t.className = 'ec-title';
@@ -168,8 +170,9 @@ function updateTotalPrice() {
       events: events,
       select: function (info) {
         if (!window.EC_CAN_CREATE) return;
-        const startIso = info.startStr; // YYYY-MM-DD
-        const endIsoInclusive = addDaysIsoLocal(info.endStr, -1); // endStr is exclusive
+        const startIso = String(info.startStr || '').slice(0,10); // YYYY-MM-DD
+        const endEnd = String(info.endStr || '').slice(0,10);
+        const endIsoInclusive = addDaysIsoLocal(endEnd, -1); // endStr is exclusive
         openCreateModal(startIso, endIsoInclusive);
       },
       dateClick: function (info) {