tango-app-api-payment-subscription 3.5.29 → 3.5.31

package/src/routes/invoice.routes.js

@@ -1,5 +1,5 @@
 import express from 'express';
-import { createInvoice, invoiceDownload, invoiceDownloadBulk, clientInvoiceList, creditTransactionlist, pendingInvoices, applyDiscount, migrateInvoice, PaymentStatusChange, checkPaymentStatus, getInvoice, invoiceAnnexure, invoiceBankDetails, updateInvoice, getClientBasePricing, deleteInvoice, approveInvoiceCsm, approveInvoiceFinance, approveInvoiceApproval, recordPayment } from '../controllers/invoice.controller.js';
+import { createInvoice, invoiceDownload, invoiceDownloadBulk, invoiceBulkEmailPreview, invoiceBulkEmailSend, clientInvoiceList, creditTransactionlist, pendingInvoices, applyDiscount, migrateInvoice, PaymentStatusChange, checkPaymentStatus, getInvoice, invoiceAnnexure, invoiceBankDetails, updateInvoice, getClientBasePricing, deleteInvoice, approveInvoiceCsm, approveInvoiceFinance, approveInvoiceApproval, recordPayment } from '../controllers/invoice.controller.js';
 import { isAllowedSessionHandler, accessVerification, validate } from 'tango-app-api-middleware';
 import { getInvoiceHeads, updateInvoiceHeads } from '../controllers/applicationDefault.controllers.js';
 import { uploadBankStatement, bankTransactionList, resolveOptions, resolveBankTransaction } from '../controllers/bankTransaction.controller.js';
@@ -24,6 +24,8 @@ function superadminBypass( accessConfig ) {
 invoiceRouter.post( '/createInvoice', isAllowedSessionHandler, superadminBypass( { userType: [ 'tango' ], access: [ { featureName: 'Global', name: 'Billing', permissions: [ 'isEdit' ] } ] } ), createInvoice );
 invoiceRouter.post( '/regerateInvoice', isAllowedSessionHandler, superadminBypass( { userType: [ 'tango' ], access: [ { featureName: 'Global', name: 'Billing', permissions: [ 'isEdit' ] } ] } ), createInvoice );
 invoiceRouter.post( '/invoiceDownload/bulk', isAllowedSessionHandler, accessVerification( { userType: [ 'tango' ], access: [ { featureName: 'Global', name: 'Billing', permissions: [] } ] } ), invoiceDownloadBulk );
+invoiceRouter.post( '/invoiceEmail/bulk/preview', isAllowedSessionHandler, accessVerification( { userType: [ 'tango' ], access: [ { featureName: 'Global', name: 'Billing', permissions: [] } ] } ), invoiceBulkEmailPreview );
+invoiceRouter.post( '/invoiceEmail/bulk/send', isAllowedSessionHandler, accessVerification( { userType: [ 'tango' ], access: [ { featureName: 'Global', name: 'Billing', permissions: [] } ] } ), invoiceBulkEmailSend );
 invoiceRouter.post( '/invoiceDownload/:invoiceId', isAllowedSessionHandler, invoiceDownload );
 invoiceRouter.post( '/clientInvoiceList', isAllowedSessionHandler, accessVerification( { userType: [ 'tango', 'client' ], access: [ { featureName: 'Global', name: 'Billing', permissions: [] } ] } ), clientInvoiceList );
 invoiceRouter.post( '/creditTransactionlist', isAllowedSessionHandler, accessVerification( { userType: [ 'tango' ], access: [ { featureName: 'Global', name: 'Billing', permissions: [] } ] } ), creditTransactionlist );

package/src/services/applicationDefault.service.js

@@ -11,3 +11,32 @@ export async function upsertApplicationDefault( query, payload ) {
       { new: true, upsert: true },
   );
 }
+
+// Atomically reserve the next value for a named counter and return it.
+// Uses a single findOneAndUpdate($inc) — MongoDB serialises concurrent
+// increments on the same document, so simultaneous callers (e.g. bulk invoice
+// finance-approval) each get a distinct number, eliminating the read-max-then-
+// write race that produced duplicate invoice numbers.
+//
+// `subType` scopes the counter (e.g. 'INV-26-27'). `seedFrom` is the current
+// real-world max (e.g. the highest existing invoiceIndex for that series); it
+// initialises the counter ONLY on first creation via $setOnInsert, so existing
+// numbers are never re-used. Returns the reserved integer.
+export async function nextCounterValue( type, subType, seedFrom = 0 ) {
+  // Two-step, both atomic:
+  // 1) Ensure the counter doc exists, seeded to the current max WITHOUT
+  //    incrementing (upsert + $setOnInsert). If it already exists this is a
+  //    no-op and never lowers a counter that has moved past the seed.
+  await model.applicationDefaultModel.updateOne(
+      { type, subType },
+      { $setOnInsert: { type, subType, data: { seq: Number( seedFrom ) || 0 } } },
+      { upsert: true },
+  );
+  // 2) Atomically claim the next value.
+  const doc = await model.applicationDefaultModel.findOneAndUpdate(
+      { type, subType },
+      { $inc: { 'data.seq': 1 } },
+      { new: true },
+  );
+  return doc?.data?.seq;
+}