nodebb-plugin-onekite-calendar 2.0.75 → 2.0.77

package/lib/api.js

@@ -1321,6 +1321,9 @@ api.createReservation = async function (req, res) {
 
   const isForSelf = (targetUid === uid);
 
+  // Explicit regularization flag: admin records a payment directly (any date, any target)
+  const isRegularization = !!isValidator && !!(req.body && req.body.isRegularization);
+
   // Free only when validator creates for themselves with a future date (within configured maxDays)
   const validatorFreeMaxDays = (() => {
     try {
@@ -1328,10 +1331,17 @@ api.createReservation = async function (req, res) {
       return Number.isFinite(v) ? v : 0;
     } catch (e) { return 0; }
   })();
-  const isValidatorFree = !!isValidator && isForSelf && !isPastDate && (validatorFreeMaxDays <= 0 || nbDays <= validatorFreeMaxDays);
+  const isValidatorFree = !!isValidator && !isRegularization && isForSelf && !isPastDate && (validatorFreeMaxDays <= 0 || nbDays <= validatorFreeMaxDays);
+
+  // Past-dated reservation for self → paid immediately (no checkbox needed)
+  const isRegularizationSelf = !!isValidator && !isRegularization && isForSelf && isPastDate;
+  // Past-dated reservation for a third party without explicit regularization → awaiting_payment
+  const isRegularizationOther = !!isValidator && !isRegularization && !isForSelf && isPastDate;
 
-  // Regularization: past-dated reservation created by a validator → paid immediately with amount
-  const isRegularization = !!isValidator && isPastDate;
+  const isAutoPaid = isValidatorFree || isRegularizationSelf || isRegularization;
+  const resolvedStatus = isAutoPaid ? 'paid'
+    : isRegularizationOther ? 'awaiting_payment'
+    : 'pending';
 
   const resv = {
     rid,
@@ -1348,14 +1358,15 @@ api.createReservation = async function (req, res) {
     end,
     startDate,
     endDate,
-    status: (isValidatorFree || isRegularization) ? 'paid' : 'pending',
+    status: resolvedStatus,
     createdAt: now,
-    paidAt: (isValidatorFree || isRegularization) ? now : 0,
-    approvedBy: (isValidatorFree || isRegularization) ? uid : 0,
-    approvedByUsername: (isValidatorFree || isRegularization) ? (creatorUsername || '') : '',
+    paidAt: isAutoPaid ? now : 0,
+    approvedAt: (isAutoPaid || isRegularizationOther) ? now : 0,
+    approvedBy: (isAutoPaid || isRegularizationOther) ? uid : 0,
+    approvedByUsername: (isAutoPaid || isRegularizationOther) ? (creatorUsername || '') : '',
     isFree: !!isValidatorFree,
     total: isValidatorFree ? 0 : (isNaN(total) ? 0 : total),
-    manuallyPaid: isRegularization ? true : undefined,
+    manuallyPaid: (isRegularizationSelf || isRegularization) ? true : undefined,
   };
 
   // Validator self-reservations are FREE (no payment required) and tracked separately in accounting.
@@ -1368,7 +1379,8 @@ api.createReservation = async function (req, res) {
 
   // Audit
   const auditAction = isValidatorFree ? 'reservation_self_checked'
-    : isRegularization ? 'reservation_manually_paid'
+    : (isRegularizationSelf || isRegularization) ? 'reservation_manually_paid'
+    : isRegularizationOther ? 'reservation_regularization_other'
     : 'reservation_requested';
   await auditLog(auditAction, uid, {
     targetType: 'reservation',
@@ -1385,7 +1397,7 @@ api.createReservation = async function (req, res) {
     status: resv.status,
   });
 
-  if (!isValidatorFree && !isRegularization) {
+  if (!isValidatorFree && !isRegularizationSelf && !isRegularization) {
 
 
   // Notify groups by email (NodeBB emailer config)
@@ -1459,7 +1471,7 @@ api.createReservation = async function (req, res) {
 
   }
 
-  res.json({ ok: true, rid, status: resv.status, autoPaid: !!(isValidatorFree || isRegularization) });
+  res.json({ ok: true, rid, status: resv.status, autoPaid: isAutoPaid });
 };
 
 api.searchUsers = async function (req, res) {

package/package.json

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

package/public/client.js

@@ -1229,6 +1229,12 @@ function toDatetimeLocalValue(date) {
         <label style="font-size:13px; font-weight:600; margin-bottom:3px; display:block;">Pour (pseudo, vide = moi-même)</label>
         <input type="text" class="form-control form-control-sm" id="onekite-target-username" placeholder="Pseudo de l'adhérent" autocomplete="off">
       </div>
+      <div class="mb-2">
+        <div class="form-check">
+          <input class="form-check-input" type="checkbox" id="onekite-regularization">
+          <label class="form-check-label" for="onekite-regularization" style="font-size:13px;">Régularisation <span class="text-muted" style="font-size:12px;">(passe directement en payé)</span></label>
+        </div>
+      </div>
     ` : '';
 
     const messageHtml = `
@@ -1280,7 +1286,8 @@ function toDatetimeLocalValue(date) {
               // Return the effective end date (exclusive) because duration shortcuts can
               // change the range without updating the original FullCalendar selection.
               const targetUsername = isValidatorMode ? ((document.getElementById('onekite-target-username') || {}).value || '').trim() : '';
-              resolve({ itemIds, itemNames, total, days, endDate: toLocalYmd(end), targetUsername });
+              const isRegularization = isValidatorMode ? !!((document.getElementById('onekite-regularization') || {}).checked) : false;
+              resolve({ itemIds, itemNames, total, days, endDate: toLocalYmd(end), targetUsername, isRegularization });
             },
           },
         },
@@ -1568,6 +1575,7 @@ function toDatetimeLocalValue(date) {
               total: chosen.total,
             };
             if (chosen.targetUsername) reqPayload.targetUsername = chosen.targetUsername;
+            if (chosen.isRegularization) reqPayload.isRegularization = true;
             const resp = await requestReservation(reqPayload);
             if (resp && (resp.autoPaid || String(resp.status) === 'paid')) {
               showAlert('success', chosen.targetUsername ? `Réservation confirmée pour ${chosen.targetUsername}.` : 'Réservation confirmée.');
@@ -1659,6 +1667,7 @@ function toDatetimeLocalValue(date) {
               total: chosen.total,
             };
             if (chosen.targetUsername) reqPayload.targetUsername = chosen.targetUsername;
+            if (chosen.isRegularization) reqPayload.isRegularization = true;
             const resp = await requestReservation(reqPayload);
             if (resp && (resp.autoPaid || String(resp.status) === 'paid')) {
               showAlert('success', chosen.targetUsername ? `Réservation confirmée pour ${chosen.targetUsername}.` : 'Réservation confirmée.');