From 06b53ddeaa41c96e3d6e62df6fe01daae2b214fa Mon Sep 17 00:00:00 2001
From: harid <harid_pr61@live.rmutl.com>
Date: Thu, 29 Jan 2026 15:09:14 +0700
Subject: [PATCH] Fix Bug Leave Report Issue #2233

---
 .../Controllers/LeaveReportController.cs      | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/BMA.EHR.Leave/Controllers/LeaveReportController.cs b/BMA.EHR.Leave/Controllers/LeaveReportController.cs
index e71b4780..b89f22a4 100644
--- a/BMA.EHR.Leave/Controllers/LeaveReportController.cs
+++ b/BMA.EHR.Leave/Controllers/LeaveReportController.cs
@@ -160,7 +160,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false 
                 ? profile.Oc.ToThaiNumber() 
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
 
             return new
             {
@@ -251,7 +251,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false
                 ? profile.Oc.ToThaiNumber()
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             return new
             {
                 template = "leave10",
@@ -341,7 +341,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false 
                 ? profile.Oc.ToThaiNumber() 
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             return new
             {
                 template = "leave11",
@@ -434,7 +434,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false
                ? profile.Oc.ToThaiNumber()
                : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             if (isHajj == true)
             {
                 return new
@@ -577,7 +577,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false
                 ? profile.Oc.ToThaiNumber()
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             return new
             {
                 template = "leave14",
@@ -658,7 +658,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false
                 ? profile.Oc.ToThaiNumber()
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             return new
             {
                 template = "leave15",
@@ -761,7 +761,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false
                 ? profile.Oc.ToThaiNumber()
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             return new
             {
                 template = "leave16",
@@ -880,7 +880,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false
                 ? profile.Oc.ToThaiNumber()
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             return new
             {
                 template = "leave17",
@@ -976,7 +976,7 @@ namespace BMA.EHR.Leave.Service.Controllers
             var Oc = profile.isCommission == false
                 ? profile.Oc.ToThaiNumber()
                 : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-            var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+            var approveResult = await GetApproverData(data.Approvers);
             return new
             {
                 template = "leave18",
@@ -1202,7 +1202,7 @@ namespace BMA.EHR.Leave.Service.Controllers
                 var Oc = profile.isCommission == false
                     ? profile.Oc.ToThaiNumber()
                     : profile.Oc.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber();
-                var approveResult = await GetApproverData(data.Approvers, profile.isCommission);
+                var approveResult = await GetApproverData(data.Approvers);
                 var result = new
                 {
                     template = "แบบใบขอยกเลิกวันลา",
@@ -2888,7 +2888,7 @@ namespace BMA.EHR.Leave.Service.Controllers
         }
         #endregion
 
-        private async Task<object> GetApproverData(List<LeaveRequestApprover> list, bool? isCommission = false)
+        private async Task<object> GetApproverData(List<LeaveRequestApprover> list)
         {
             var _default = new
             {
@@ -2919,10 +2919,10 @@ namespace BMA.EHR.Leave.Service.Controllers
                                        : $"{x.PositionName}{x.PositionLevelName}").ToThaiNumber(),
                     posExOrg = !string.IsNullOrEmpty(x.PositionLevelName) &&
                             (x.PositionLevelName.Contains("อำนวยการ") || x.PositionLevelName.Contains("บริหาร"))
-                               ? new[] { (isCommission == false ? x!.OrganizationName : x!.OrganizationName.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.")).ToThaiNumber() }
+                               ? new[] { (x!.OrganizationName.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.")).ToThaiNumber() }
                                : !string.IsNullOrEmpty(x.PosExecutiveName)
-                                   ? new[] { (x.PosExecutiveName.ToThaiNumber()), (isCommission == false ? x!.OrganizationName : x!.OrganizationName.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.")).ToThaiNumber() }
-                                   : new[] { (isCommission == false ? x!.OrganizationName : x!.OrganizationName.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.")).ToThaiNumber() },
+                                   ? new[] { x.PosExecutiveName.ToThaiNumber(), x!.OrganizationName.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber() }
+                                   : new[] { x!.OrganizationName.Replace("สำนักงานคณะกรรมการข้าราชการกรุงเทพมหานคร", "สำนักงาน ก.ก.").ToThaiNumber() },
                     positionSign = !string.IsNullOrEmpty(x.PositionSign)
                         ? x.PositionSign.Replace("\r", "").Replace("\n", " ").ToThaiNumber()
                         : "............................................",