From 0a3d9c615fe223217e82d5cfe21035a98d651c95 Mon Sep 17 00:00:00 2001
From: TaprootFreak <142087526+TaprootFreak@users.noreply.github.com>
Date: Tue, 6 Jan 2026 22:04:54 +0100
Subject: [PATCH] fix: remove broken user status check from BankTxReturn
 auto-refund cronjob (#2861)

The chargebackTx() cronjob was checking transaction.user.status, but
BankTxReturn transactions never have a userId set. This caused the
query to never find any entries, preventing automatic refund approval.

Remove the user status check since userData validation is sufficient.
---
 .../bank-tx/bank-tx-return/bank-tx-return.service.ts        | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/subdomains/supporting/bank-tx/bank-tx-return/bank-tx-return.service.ts b/src/subdomains/supporting/bank-tx/bank-tx-return/bank-tx-return.service.ts
index b47c598542..1d2b236162 100644
--- a/src/subdomains/supporting/bank-tx/bank-tx-return/bank-tx-return.service.ts
+++ b/src/subdomains/supporting/bank-tx/bank-tx-return/bank-tx-return.service.ts
@@ -8,7 +8,6 @@ import { Util } from 'src/shared/utils/util';
 import { BankTxRefund, RefundInternalDto } from 'src/subdomains/core/history/dto/refund-internal.dto';
 import { TransactionUtilService } from 'src/subdomains/core/transaction/transaction-util.service';
 import { KycStatus, RiskStatus, UserDataStatus } from 'src/subdomains/generic/user/models/user-data/user-data.enum';
-import { UserStatus } from 'src/subdomains/generic/user/models/user/user.enum';
 import { In, IsNull, Not } from 'typeorm';
 import { FiatOutputType } from '../../fiat-output/fiat-output.entity';
 import { FiatOutputService } from '../../fiat-output/fiat-output.service';
@@ -50,16 +49,13 @@ export class BankTxReturnService {
         chargebackAmount: Not(IsNull()),
         chargebackIban: Not(IsNull()),
         chargebackOutput: IsNull(),
-        transaction: {
-          user: { status: In([UserStatus.NA, UserStatus.ACTIVE]) },
-        },
         userData: {
           kycStatus: In([KycStatus.NA, KycStatus.COMPLETED]),
           status: Not(UserDataStatus.BLOCKED),
           riskStatus: In([RiskStatus.NA, RiskStatus.RELEASED]),
         },
       },
-      relations: { bankTx: true, userData: true, transaction: { user: true } },
+      relations: { bankTx: true, userData: true },
     });
 
     for (const entity of entities) {