From a5e338e83135754a15c1304ac3653d32dcf94838 Mon Sep 17 00:00:00 2001
From: Gelo <chrisgelosulit@gmail.com>
Date: Mon, 9 Oct 2023 15:02:04 +0800
Subject: [PATCH] Remove filter for expense_type_id in
 getAllReimbursementRequest.dto.ts and reimbursement.get-all.service.ts

---
 .../dto/get-all-reimbursement-request.dto.ts  | 29 ++-----------------
 .../services/reimbursement.get-all.service.ts |  8 -----
 2 files changed, 2 insertions(+), 35 deletions(-)

diff --git a/src/finance/common/dto/get-all-reimbursement-request.dto.ts b/src/finance/common/dto/get-all-reimbursement-request.dto.ts
index 3769c8c..0285a4a 100644
--- a/src/finance/common/dto/get-all-reimbursement-request.dto.ts
+++ b/src/finance/common/dto/get-all-reimbursement-request.dto.ts
@@ -11,22 +11,6 @@ const GetAllReimbursementRequestSchema = z
       .uuid({
         message: 'reimbursement_type_id must be a valid uuid',
       }),
-    expense_type_id: z
-      .string({
-        description: 'expense_type_id',
-        invalid_type_error: 'expense_type_id must be a string',
-      })
-      .uuid({
-        message: 'expense_type_id must be a valid uuid',
-      }),
-    requestor_id: z
-      .string({
-        description: 'requestor_id',
-        invalid_type_error: 'requestor_id must be a string',
-      })
-      .uuid({
-        message: 'requestor_id must be a valid uuid',
-      }),
     request_status_id: z.array(
       z
         .string({
@@ -61,14 +45,6 @@ const GetAllReimbursementRequestSchema = z
       .positive({
         message: 'amount_max must be a positive number',
       }),
-    date_filed: z
-      .string({
-        description: 'date_filed',
-        invalid_type_error: 'date_filed must be a string',
-      })
-      .datetime({
-        message: 'date_filed must be a valid datetime',
-      }),
     text_search: z
       .string({
         description: 'text_search',
@@ -116,18 +92,17 @@ const GetAllReimbursementRequestSchema = z
       }
 
       if (input?.amount_min && !input?.amount_max) {
-        return false;
+        input.amount_max = 999_999_999;
       }
 
       if (!input?.amount_min && input?.amount_max) {
-        return false;
+        input.amount_min = 1;
       }
 
       return true;
     },
     {
       path: ['amount_min', 'amount_max'],
-      message: 'amount_min must be less than or equal to amount_max',
     },
   );
 
diff --git a/src/finance/services/reimbursement.get-all.service.ts b/src/finance/services/reimbursement.get-all.service.ts
index 03e23df..cd0e1fb 100644
--- a/src/finance/services/reimbursement.get-all.service.ts
+++ b/src/finance/services/reimbursement.get-all.service.ts
@@ -203,14 +203,6 @@ export class ReimbursementGetAllService {
       );
     }
 
-    if (filter?.expense_type_id) {
-      query = query.where(
-        'finance_reimbursement_requests.expense_type_id',
-        '=',
-        filter.expense_type_id,
-      );
-    }
-
     if (filter?.request_status_id) {
       query = query.where(
         'finance_reimbursement_requests.request_status_id',