fix: getTransactionsForInvoice returning invoiced transactions (#457)

Also changes the name to `getEligibleTransactions` since that describes the function better.
GEWIS · Feb 11, 2025 · c3c8bd0 · c3c8bd0
1 parent 7634f63
commit c3c8bd0
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 15 deletions.
diff --git a/src/controller/invoice-controller.ts b/src/controller/invoice-controller.ts
@@ -519,7 +519,7 @@ export default class InvoiceController extends BaseController {
     }
 
     try {
-      const transactions = await new InvoiceService().getTransactionsForInvoice({
+      const transactions = await new InvoiceService().getEligibleTransactions({
         forId,
         fromDate,
         tillDate,

diff --git a/src/service/invoice-service.ts b/src/service/invoice-service.ts
@@ -386,11 +386,10 @@ export default class InvoiceService extends WithManager {
   }
 
   /**
-   * Gets transactions for an invoice
-   * returns false if the transactions contain invoiced transactions
+   * Gets all uninvoiced transactions for a given user
    * @param params
    */
-  public async getTransactionsForInvoice(params: InvoiceTransactionsRequest): Promise<TransactionResponse[] | false> {
+  public async getEligibleTransactions(params: InvoiceTransactionsRequest): Promise<TransactionResponse[] | false> {
     const { forId, fromDate, tillDate } = params;
     const transactionService = new TransactionService(this.manager);
 
@@ -412,16 +411,10 @@ export default class InvoiceService extends WithManager {
       pointOfSale: true,
     };
 
-    const transactions = await this.manager.find(Transaction, { where: { id: In(tIds) },
-      relations });
-    transactions.forEach((t) => {
-      t.subTransactions.forEach((tSub) => {
-        tSub.subTransactionRows.forEach((tSubRow) => {
-          if (tSubRow.invoice) return false;
-        });
-      });
-    });
-
+    const transactions = await this.manager.find(Transaction, { where: { id: In(tIds),
+      subTransactions: { subTransactionRows: { invoice: false } } },
+    relations });
+
     const response: Promise<TransactionResponse>[] = [];
     transactions.forEach((t) => response.push(transactionService.asTransactionResponse(t)));
 

diff --git a/test/unit/service/invoice-service.ts b/test/unit/service/invoice-service.ts
@@ -249,7 +249,7 @@ describe('InvoiceService', () => {
           );
           const total = transactionsAfterDate.total;
 
-          const transactions = await new InvoiceService().getTransactionsForInvoice({
+          const transactions = await new InvoiceService().getEligibleTransactions({
             forId: debtor.id,
             fromDate,
           });