Marco.Pms.Services/Controllers/EmployeeController.cs

@@ -148,14 +148,13 @@ namespace MarcoBMS.Services.Controllers
                     .Distinct()
                     .ToListAsync();
 
-                var employees = await _context.Employees
+                result = await _context.Employees
                     .Include(fp => fp.JobRole)
-                        .Where(e => employeeIds.Contains(e.Id) && e.JobRole != null && e.IsActive && e.TenantId == tenantId)
+                    .Where(e => employeeIds.Contains(e.Id) && e.IsActive && e.TenantId == tenantId)
+                    .Select(e => e.ToEmployeeVMFromEmployee())
                     .Distinct()
                     .ToListAsync();
 
-                result = employees.Select(e => e.ToEmployeeVMFromEmployee()).ToList();
-
                 _logger.LogInfo("Employee list fetched using limited access (active only).");
             }
             else

Marco.Pms.Services/Helpers/EmployeeHelper.cs

@@ -85,12 +85,12 @@ namespace MarcoBMS.Services.Helpers
                     .Distinct()
                     .ToListAsync();
 
-                    var employees = await _context.Employees
+                    result = await _context.Employees
                         .Include(fp => fp.JobRole)
-                        .Where(e => employeeIds.Contains(e.Id) && e.JobRole != null && e.IsActive && e.TenantId == tenantId)
+                        .Where(e => employeeIds.Contains(e.Id) && e.IsActive && e.TenantId == tenantId)
+                        .Select(e => e.ToEmployeeVMFromEmployee())
                         .Distinct()
                         .ToListAsync();
-                    result = employees.Select(e => e.ToEmployeeVMFromEmployee()).ToList();
 
                 }
                 else if (ShowInActive)