Marco.Pms.DataAccess/Data/ApplicationDbContext.cs

@@ -993,6 +993,11 @@ namespace Marco.Pms.DataAccess.Data
                 );
 
             modelBuilder.Entity<OrgTypeMaster>().HasData(
+               new OrgTypeMaster
+               {
+                   Id = Guid.Parse("743806fe-d991-4079-b223-e4e2da44f435"),
+                   Name = "Tenant"
+               },
                new OrgTypeMaster
                {
                    Id = Guid.Parse("5ee49bcd-b6d3-482f-9aaf-484afe04abec"),
@@ -1002,6 +1007,11 @@ namespace Marco.Pms.DataAccess.Data
                {
                    Id = Guid.Parse("a283356a-9b02-4029-afb7-e65c703efdd4"),
                    Name = "Sub-Contractor"
+               },
+               new OrgTypeMaster
+               {
+                   Id = Guid.Parse("b1877a3b-8832-47b1-bbe3-dc7e98672f49"),
+                   Name = "PMC"
                }
            );
 

Marco.Pms.DataAccess/Migrations/20250924103942_Added_Assigned_By_In_Mapping_Tables.cs

@@ -1,84 +0,0 @@
-using Microsoft.EntityFrameworkCore.Migrations;
-
-#nullable disable
-
-namespace Marco.Pms.DataAccess.Migrations
-{
-    /// <inheritdoc />
-    public partial class Added_Assigned_By_In_Mapping_Tables : Migration
-    {
-        /// <inheritdoc />
-        protected override void Up(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.AddColumn<Guid>(
-                name: "AssignedById",
-                table: "TenantOrgMappings",
-                type: "char(36)",
-                nullable: false,
-                defaultValue: new Guid("08dd8b35-d98b-44f1-896d-12aec3f035aa"),
-                collation: "ascii_general_ci");
-
-            migrationBuilder.AddColumn<Guid>(
-                name: "AssignedById",
-                table: "ProjectOrgMappings",
-                type: "char(36)",
-                nullable: false,
-                defaultValue: new Guid("08dd8b35-d98b-44f1-896d-12aec3f035aa"),
-                collation: "ascii_general_ci");
-
-            migrationBuilder.CreateIndex(
-                name: "IX_TenantOrgMappings_AssignedById",
-                table: "TenantOrgMappings",
-                column: "AssignedById");
-
-            migrationBuilder.CreateIndex(
-                name: "IX_ProjectOrgMappings_AssignedById",
-                table: "ProjectOrgMappings",
-                column: "AssignedById");
-
-            migrationBuilder.AddForeignKey(
-                name: "FK_ProjectOrgMappings_Employees_AssignedById",
-                table: "ProjectOrgMappings",
-                column: "AssignedById",
-                principalTable: "Employees",
-                principalColumn: "Id",
-                onDelete: ReferentialAction.Cascade);
-
-            migrationBuilder.AddForeignKey(
-                name: "FK_TenantOrgMappings_Employees_AssignedById",
-                table: "TenantOrgMappings",
-                column: "AssignedById",
-                principalTable: "Employees",
-                principalColumn: "Id",
-                onDelete: ReferentialAction.Cascade);
-        }
-
-        /// <inheritdoc />
-        protected override void Down(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DropForeignKey(
-                name: "FK_ProjectOrgMappings_Employees_AssignedById",
-                table: "ProjectOrgMappings");
-
-            migrationBuilder.DropForeignKey(
-                name: "FK_TenantOrgMappings_Employees_AssignedById",
-                table: "TenantOrgMappings");
-
-            migrationBuilder.DropIndex(
-                name: "IX_TenantOrgMappings_AssignedById",
-                table: "TenantOrgMappings");
-
-            migrationBuilder.DropIndex(
-                name: "IX_ProjectOrgMappings_AssignedById",
-                table: "ProjectOrgMappings");
-
-            migrationBuilder.DropColumn(
-                name: "AssignedById",
-                table: "TenantOrgMappings");
-
-            migrationBuilder.DropColumn(
-                name: "AssignedById",
-                table: "ProjectOrgMappings");
-        }
-    }
-}

Marco.Pms.DataAccess/Migrations/20250924113835_Deleted_Organization_Types.cs

@@ -1,40 +0,0 @@
-using System;
-using Microsoft.EntityFrameworkCore.Migrations;
-
-#nullable disable
-
-#pragma warning disable CA1814 // Prefer jagged arrays over multidimensional
-
-namespace Marco.Pms.DataAccess.Migrations
-{
-    /// <inheritdoc />
-    public partial class Deleted_Organization_Types : Migration
-    {
-        /// <inheritdoc />
-        protected override void Up(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DeleteData(
-                table: "OrgTypeMasters",
-                keyColumn: "Id",
-                keyValue: new Guid("743806fe-d991-4079-b223-e4e2da44f435"));
-
-            migrationBuilder.DeleteData(
-                table: "OrgTypeMasters",
-                keyColumn: "Id",
-                keyValue: new Guid("b1877a3b-8832-47b1-bbe3-dc7e98672f49"));
-        }
-
-        /// <inheritdoc />
-        protected override void Down(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.InsertData(
-                table: "OrgTypeMasters",
-                columns: new[] { "Id", "Name" },
-                values: new object[,]
-                {
-                    { new Guid("743806fe-d991-4079-b223-e4e2da44f435"), "Tenant" },
-                    { new Guid("b1877a3b-8832-47b1-bbe3-dc7e98672f49"), "PMC" }
-                });
-        }
-    }
-}

Marco.Pms.DataAccess/Migrations/20250927091428_Added_Forgin_Key_For_Approver.cs

@@ -1,92 +0,0 @@
-using Microsoft.EntityFrameworkCore.Migrations;
-
-#nullable disable
-
-namespace Marco.Pms.DataAccess.Migrations
-{
-    /// <inheritdoc />
-    public partial class Added_Forgin_Key_For_Approver : Migration
-    {
-        /// <inheritdoc />
-        protected override void Up(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DropForeignKey(
-                name: "FK_Attendes_Employees_EmployeeID",
-                table: "Attendes");
-
-            migrationBuilder.RenameColumn(
-                name: "EmployeeID",
-                table: "Attendes",
-                newName: "EmployeeId");
-
-            migrationBuilder.RenameColumn(
-                name: "ApprovedBy",
-                table: "Attendes",
-                newName: "ApprovedById");
-
-            migrationBuilder.RenameIndex(
-                name: "IX_Attendes_EmployeeID",
-                table: "Attendes",
-                newName: "IX_Attendes_EmployeeId");
-
-            migrationBuilder.CreateIndex(
-                name: "IX_Attendes_ApprovedById",
-                table: "Attendes",
-                column: "ApprovedById");
-
-            migrationBuilder.AddForeignKey(
-                name: "FK_Attendes_Employees_ApprovedById",
-                table: "Attendes",
-                column: "ApprovedById",
-                principalTable: "Employees",
-                principalColumn: "Id");
-
-            migrationBuilder.AddForeignKey(
-                name: "FK_Attendes_Employees_EmployeeId",
-                table: "Attendes",
-                column: "EmployeeId",
-                principalTable: "Employees",
-                principalColumn: "Id",
-                onDelete: ReferentialAction.Cascade);
-        }
-
-        /// <inheritdoc />
-        protected override void Down(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DropForeignKey(
-                name: "FK_Attendes_Employees_ApprovedById",
-                table: "Attendes");
-
-            migrationBuilder.DropForeignKey(
-                name: "FK_Attendes_Employees_EmployeeId",
-                table: "Attendes");
-
-            migrationBuilder.DropIndex(
-                name: "IX_Attendes_ApprovedById",
-                table: "Attendes");
-
-            migrationBuilder.RenameColumn(
-                name: "EmployeeId",
-                table: "Attendes",
-                newName: "EmployeeID");
-
-            migrationBuilder.RenameColumn(
-                name: "ApprovedById",
-                table: "Attendes",
-                newName: "ApprovedBy");
-
-            migrationBuilder.RenameIndex(
-                name: "IX_Attendes_EmployeeId",
-                table: "Attendes",
-                newName: "IX_Attendes_EmployeeID");
-
-            migrationBuilder.AddForeignKey(
-                name: "FK_Attendes_Employees_EmployeeID",
-                table: "Attendes",
-                column: "EmployeeID",
-                principalTable: "Employees",
-                principalColumn: "Id",
-                onDelete: ReferentialAction.Cascade);
-        }
-    }
-}

Marco.Pms.DataAccess/Migrations/20251004112239_Added_ExpenceUID_In_Expense_Table.cs

@@ -1,29 +0,0 @@
-using Microsoft.EntityFrameworkCore.Migrations;
-
-#nullable disable
-
-namespace Marco.Pms.DataAccess.Migrations
-{
-    /// <inheritdoc />
-    public partial class Added_ExpenceUID_In_Expense_Table : Migration
-    {
-        /// <inheritdoc />
-        protected override void Up(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.AddColumn<string>(
-                name: "ExpenseUId",
-                table: "Expenses",
-                type: "longtext",
-                nullable: false)
-                .Annotation("MySql:CharSet", "utf8mb4");
-        }
-
-        /// <inheritdoc />
-        protected override void Down(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DropColumn(
-                name: "ExpenseUId",
-                table: "Expenses");
-        }
-    }
-}

Marco.Pms.DataAccess/Migrations/20251010062100_Added_Requested_In_Attendance_Table.cs

@@ -1,70 +0,0 @@
-using System;
-using Microsoft.EntityFrameworkCore.Migrations;
-
-#nullable disable
-
-namespace Marco.Pms.DataAccess.Migrations
-{
-    /// <inheritdoc />
-    public partial class Added_Requested_In_Attendance_Table : Migration
-    {
-        /// <inheritdoc />
-        protected override void Up(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.AddColumn<DateTime>(
-                name: "ApprovedAt",
-                table: "Attendes",
-                type: "datetime(6)",
-                nullable: true);
-
-            migrationBuilder.AddColumn<DateTime>(
-                name: "RequestedAt",
-                table: "Attendes",
-                type: "datetime(6)",
-                nullable: true);
-
-            migrationBuilder.AddColumn<Guid>(
-                name: "RequestedById",
-                table: "Attendes",
-                type: "char(36)",
-                nullable: true,
-                collation: "ascii_general_ci");
-
-            migrationBuilder.CreateIndex(
-                name: "IX_Attendes_RequestedById",
-                table: "Attendes",
-                column: "RequestedById");
-
-            migrationBuilder.AddForeignKey(
-                name: "FK_Attendes_Employees_RequestedById",
-                table: "Attendes",
-                column: "RequestedById",
-                principalTable: "Employees",
-                principalColumn: "Id");
-        }
-
-        /// <inheritdoc />
-        protected override void Down(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DropForeignKey(
-                name: "FK_Attendes_Employees_RequestedById",
-                table: "Attendes");
-
-            migrationBuilder.DropIndex(
-                name: "IX_Attendes_RequestedById",
-                table: "Attendes");
-
-            migrationBuilder.DropColumn(
-                name: "ApprovedAt",
-                table: "Attendes");
-
-            migrationBuilder.DropColumn(
-                name: "RequestedAt",
-                table: "Attendes");
-
-            migrationBuilder.DropColumn(
-                name: "RequestedById",
-                table: "Attendes");
-        }
-    }
-}

Marco.Pms.DataAccess/Migrations/ApplicationDbContextModelSnapshot.cs

@@ -172,10 +172,7 @@ namespace Marco.Pms.DataAccess.Migrations
                     b.Property<int>("Activity")
                         .HasColumnType("int");
 
-                    b.Property<DateTime?>("ApprovedAt")
+                    b.Property<Guid?>("ApprovedBy")
-                        .HasColumnType("datetime(6)");
-
-                    b.Property<Guid?>("ApprovedById")
                         .HasColumnType("char(36)");
 
                     b.Property<DateTime>("AttendanceDate")
@@ -188,7 +185,7 @@ namespace Marco.Pms.DataAccess.Migrations
                     b.Property<DateTime>("Date")
                         .HasColumnType("datetime(6)");
 
-                    b.Property<Guid>("EmployeeId")
+                    b.Property<Guid>("EmployeeID")
                         .HasColumnType("char(36)");
 
                     b.Property<DateTime?>("InTime")
@@ -203,22 +200,12 @@ namespace Marco.Pms.DataAccess.Migrations
                     b.Property<Guid>("ProjectID")
                         .HasColumnType("char(36)");
 
-                    b.Property<DateTime?>("RequestedAt")
-                        .HasColumnType("datetime(6)");
-
-                    b.Property<Guid?>("RequestedById")
-                        .HasColumnType("char(36)");
-
                     b.Property<Guid>("TenantId")
                         .HasColumnType("char(36)");
 
                     b.HasKey("Id");
 
-                    b.HasIndex("ApprovedById");
+                    b.HasIndex("EmployeeID");
-
-                    b.HasIndex("EmployeeId");
-
-                    b.HasIndex("RequestedById");
 
                     b.HasIndex("TenantId");
 
@@ -1843,10 +1830,6 @@ namespace Marco.Pms.DataAccess.Migrations
                         .IsRequired()
                         .HasColumnType("longtext");
 
-                    b.Property<string>("ExpenseUId")
-                        .IsRequired()
-                        .HasColumnType("longtext");
-
                     b.Property<Guid>("ExpensesTypeId")
                         .HasColumnType("char(36)");
 
@@ -3605,6 +3588,11 @@ namespace Marco.Pms.DataAccess.Migrations
                     b.ToTable("OrgTypeMasters");
 
                     b.HasData(
+                        new
+                        {
+                            Id = new Guid("743806fe-d991-4079-b223-e4e2da44f435"),
+                            Name = "Tenant"
+                        },
                         new
                         {
                             Id = new Guid("5ee49bcd-b6d3-482f-9aaf-484afe04abec"),
@@ -3614,6 +3602,11 @@ namespace Marco.Pms.DataAccess.Migrations
                         {
                             Id = new Guid("a283356a-9b02-4029-afb7-e65c703efdd4"),
                             Name = "Sub-Contractor"
+                        },
+                        new
+                        {
+                            Id = new Guid("b1877a3b-8832-47b1-bbe3-dc7e98672f49"),
+                            Name = "PMC"
                         });
                 });
 
@@ -3689,9 +3682,6 @@ namespace Marco.Pms.DataAccess.Migrations
                         .ValueGeneratedOnAdd()
                         .HasColumnType("char(36)");
 
-                    b.Property<Guid>("AssignedById")
-                        .HasColumnType("char(36)");
-
                     b.Property<DateTime>("AssignedDate")
                         .HasColumnType("datetime(6)");
 
@@ -3715,8 +3705,6 @@ namespace Marco.Pms.DataAccess.Migrations
 
                     b.HasKey("Id");
 
-                    b.HasIndex("AssignedById");
-
                     b.HasIndex("OrganizationId");
 
                     b.HasIndex("OrganizationTypeId");
@@ -3777,9 +3765,6 @@ namespace Marco.Pms.DataAccess.Migrations
                         .ValueGeneratedOnAdd()
                         .HasColumnType("char(36)");
 
-                    b.Property<Guid>("AssignedById")
-                        .HasColumnType("char(36)");
-
                     b.Property<DateTime>("AssignedDate")
                         .HasColumnType("datetime(6)");
 
@@ -3800,8 +3785,6 @@ namespace Marco.Pms.DataAccess.Migrations
 
                     b.HasKey("Id");
 
-                    b.HasIndex("AssignedById");
-
                     b.HasIndex("OrganizationId");
 
                     b.HasIndex("TenantId");
@@ -4718,18 +4701,10 @@ namespace Marco.Pms.DataAccess.Migrations
                 {
                     b.HasOne("Marco.Pms.Model.Employees.Employee", "Approver")
                         .WithMany()
-                        .HasForeignKey("ApprovedById");
+                        .HasForeignKey("EmployeeID")
-
-                    b.HasOne("Marco.Pms.Model.Employees.Employee", "Employee")
-                        .WithMany()
-                        .HasForeignKey("EmployeeId")
                         .OnDelete(DeleteBehavior.Cascade)
                         .IsRequired();
 
-                    b.HasOne("Marco.Pms.Model.Employees.Employee", "RequestedBy")
-                        .WithMany()
-                        .HasForeignKey("RequestedById");
-
                     b.HasOne("Marco.Pms.Model.TenantModels.Tenant", "Tenant")
                         .WithMany()
                         .HasForeignKey("TenantId")
@@ -4738,10 +4713,6 @@ namespace Marco.Pms.DataAccess.Migrations
 
                     b.Navigation("Approver");
 
-                    b.Navigation("Employee");
-
-                    b.Navigation("RequestedBy");
-
                     b.Navigation("Tenant");
                 });
 
@@ -5766,12 +5737,6 @@ namespace Marco.Pms.DataAccess.Migrations
 
             modelBuilder.Entity("Marco.Pms.Model.OrganizationModel.ProjectOrgMapping", b =>
                 {
-                    b.HasOne("Marco.Pms.Model.Employees.Employee", "AssignedBy")
-                        .WithMany()
-                        .HasForeignKey("AssignedById")
-                        .OnDelete(DeleteBehavior.Cascade)
-                        .IsRequired();
-
                     b.HasOne("Marco.Pms.Model.OrganizationModel.Organization", "Organization")
                         .WithMany()
                         .HasForeignKey("OrganizationId")
@@ -5802,8 +5767,6 @@ namespace Marco.Pms.DataAccess.Migrations
                         .OnDelete(DeleteBehavior.Cascade)
                         .IsRequired();
 
-                    b.Navigation("AssignedBy");
-
                     b.Navigation("Organization");
 
                     b.Navigation("OrganizationType");
@@ -5844,12 +5807,6 @@ namespace Marco.Pms.DataAccess.Migrations
 
             modelBuilder.Entity("Marco.Pms.Model.OrganizationModel.TenantOrgMapping", b =>
                 {
-                    b.HasOne("Marco.Pms.Model.Employees.Employee", "AssignedBy")
-                        .WithMany()
-                        .HasForeignKey("AssignedById")
-                        .OnDelete(DeleteBehavior.Cascade)
-                        .IsRequired();
-
                     b.HasOne("Marco.Pms.Model.OrganizationModel.Organization", "Organization")
                         .WithMany()
                         .HasForeignKey("OrganizationId")
@@ -5862,8 +5819,6 @@ namespace Marco.Pms.DataAccess.Migrations
                         .OnDelete(DeleteBehavior.Cascade)
                         .IsRequired();
 
-                    b.Navigation("AssignedBy");
-
                     b.Navigation("Organization");
 
                     b.Navigation("Tenant");

Marco.Pms.Helpers/CacheHelper/EmployeeCache.cs

@@ -212,48 +212,6 @@ namespace Marco.Pms.Helpers.CacheHelper
 
             return true;
         }
-        public async Task<bool> ClearAllEmployeesFromCacheByOnlyEmployeeId(Guid employeeId)
-        {
-            var employeeIdString = employeeId.ToString();
-
-            try
-            {
-                var filter = Builders<EmployeePermissionMongoDB>.Filter.Eq(e => e.Id, employeeIdString);
-
-                var result = await _collection.DeleteManyAsync(filter);
-
-                if (result.DeletedCount == 0)
-                    return false;
-
-                return true;
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "Error occured while deleting employee profile");
-                return false;
-            }
-        }
-        public async Task<bool> ClearAllEmployeesFromCacheByTenantId(Guid tenantId)
-        {
-            var tenantIdString = tenantId.ToString();
-
-            try
-            {
-                var filter = Builders<EmployeePermissionMongoDB>.Filter.Eq(e => e.TenantId, tenantIdString);
-
-                var result = await _collection.DeleteManyAsync(filter);
-
-                if (result.DeletedCount == 0)
-                    return false;
-
-                return true;
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "Error occured while deleting employee profile");
-                return false;
-            }
-        }
         public async Task<bool> ClearAllEmployeesFromCacheByEmployeeIds(List<string> employeeIds, Guid tenantId)
         {
             var tenantIdString = tenantId.ToString();

Marco.Pms.Helpers/CacheHelper/ProjectCache.cs

@@ -1,9 +1,7 @@
 using Marco.Pms.DataAccess.Data;
 using Marco.Pms.Model.Master;
-using Marco.Pms.Model.MongoDBModels;
 using Marco.Pms.Model.MongoDBModels.Masters;
 using Marco.Pms.Model.MongoDBModels.Project;
-using Marco.Pms.Model.OrganizationModel;
 using Marco.Pms.Model.Projects;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.Configuration;
@@ -57,7 +55,7 @@ namespace Marco.Pms.Helpers
             var indexModel = new CreateIndexModel<ProjectMongoDB>(indexKeys, indexOptions);
             await _projectCollection.Indexes.CreateOneAsync(indexModel);
         }
-        public async Task<bool> UpdateProjectDetailsOnlyToCache(Project project, StatusMaster projectStatus, Organization promotor, Organization pmc)
+        public async Task<bool> UpdateProjectDetailsOnlyToCache(Project project, StatusMaster projectStatus)
         {
             // Build the update definition
             var updates = Builders<ProjectMongoDB>.Update.Combine(
@@ -69,26 +67,6 @@ namespace Marco.Pms.Helpers
                     Id = projectStatus.Id.ToString(),
                     Status = projectStatus.Status
                 }),
-                Builders<ProjectMongoDB>.Update.Set(r => r.Promoter, new OrganizationMongoDB
-                {
-                    Id = promotor.Id.ToString(),
-                    Name = promotor.Name,
-                    ContactPerson = promotor.ContactPerson,
-                    Email = promotor.Email,
-                    Address = promotor.Address,
-                    ContactNumber = promotor.ContactNumber,
-                    SPRID = promotor.SPRID
-                }),
-                Builders<ProjectMongoDB>.Update.Set(r => r.PMC, new OrganizationMongoDB
-                {
-                    Id = pmc.Id.ToString(),
-                    Name = pmc.Name,
-                    ContactPerson = pmc.ContactPerson,
-                    Email = pmc.Email,
-                    Address = promotor.Address,
-                    ContactNumber = promotor.ContactNumber,
-                    SPRID = promotor.SPRID
-                }),
                 Builders<ProjectMongoDB>.Update.Set(r => r.StartDate, project.StartDate),
                 Builders<ProjectMongoDB>.Update.Set(r => r.EndDate, project.EndDate),
                 Builders<ProjectMongoDB>.Update.Set(r => r.ContactPerson, project.ContactPerson)

Marco.Pms.Model/AttendanceModule/Attendance.cs

@@ -1,8 +1,8 @@
-using Marco.Pms.Model.Dtos.Attendance;
+using System.ComponentModel.DataAnnotations.Schema;
+using Marco.Pms.Model.Dtos.Attendance;
 using Marco.Pms.Model.Employees;
 using Marco.Pms.Model.Utilities;
 using Microsoft.AspNetCore.Mvc.ModelBinding.Validation;
-using System.ComponentModel.DataAnnotations.Schema;
 
 namespace Marco.Pms.Model.AttendanceModule
 {
@@ -10,11 +10,9 @@ namespace Marco.Pms.Model.AttendanceModule
     {
         public Guid Id { get; set; }
         public string Comment { get; set; } = string.Empty;
-        public Guid EmployeeId { get; set; }
+        public Guid EmployeeID { get; set; }
+
 
-        [ForeignKey("EmployeeId")]
-        [ValidateNever]
-        public Employee? Employee { get; set; }
         public DateTime Date { get; set; }
         public Guid ProjectID { get; set; }
 
@@ -24,17 +22,9 @@ namespace Marco.Pms.Model.AttendanceModule
         public bool IsApproved { get; set; }
         public ATTENDANCE_MARK_TYPE Activity { get; set; }
 
-        public Guid? ApprovedById { get; set; }
+        public Guid? ApprovedBy { get; set; }
-
+        [ForeignKey("EmployeeID")]
-        [ForeignKey("ApprovedById")]
         [ValidateNever]
         public Employee? Approver { get; set; }
-        public DateTime? RequestedAt { get; set; }
-        public DateTime? ApprovedAt { get; set; }
-        public Guid? RequestedById { get; set; }
-
-        [ForeignKey("RequestedById")]
-        [ValidateNever]
-        public Employee? RequestedBy { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Activities/CreateActivityMasterDto.cs

@@ -2,7 +2,7 @@
 {
     public class CreateActivityMasterDto
     {
-        public required Guid ActivityGroupId { get; set; }
+        public required Guid ActitvityGroupId { get; set; }
         public required string ActivityName { get; set; }
         public required string UnitOfMeasurement { get; set; }
         public List<CreateCheckListDto>? CheckList { get; set; }

Marco.Pms.Model/Dtos/Activities/CreateWorkStatusMasterDto.cs

@@ -2,7 +2,7 @@
 {
     public class CreateWorkStatusMasterDto
     {
-        public required string Name { get; set; }
+        public string? Name { get; set; }
-        public required string Description { get; set; }
+        public string? Description { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Activities/UpdateWorkStatusMasterDto.cs

@@ -3,7 +3,7 @@
     public class UpdateWorkStatusMasterDto
     {
         public Guid Id { get; set; }
-        public required string Name { get; set; }
+        public string? Name { get; set; }
-        public required string Description { get; set; }
+        public string? Description { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Authentication/VerifyMPINDto.cs

@@ -5,5 +5,6 @@
         public required Guid EmployeeId { get; set; }
         public required string MPIN { get; set; }
         public required string MPINToken { get; set; }
+        public required string FcmToken { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Employees/CreateUserDto.cs

@@ -9,8 +9,8 @@
         public string? Email { get; set; }
 
         public required string Gender { get; set; }
-        public required DateTime BirthDate { get; set; }
+        public required string BirthDate { get; set; }
-        public required DateTime JoiningDate { get; set; }
+        public required string JoiningDate { get; set; }
 
         public required string PermanentAddress { get; set; }
         public required string CurrentAddress { get; set; }
@@ -19,8 +19,6 @@
         public string? EmergencyPhoneNumber { get; set; }
         public string? EmergencyContactPerson { get; set; }
         public Guid JobRoleId { get; set; }
-        public required Guid OrganizationId { get; set; }
-        public required bool HasApplicationAccess { get; set; }
     }
     public class MobileUserManageDto
     {
@@ -28,13 +26,10 @@
         public required string FirstName { get; set; }
         public required string LastName { get; set; }
         public required string PhoneNumber { get; set; }
-        public string? Email { get; set; }
         public required DateTime JoiningDate { get; set; }
         public required string Gender { get; set; }
         public Guid JobRoleId { get; set; }
         public string? ProfileImage { get; set; }
-        public required Guid OrganizationId { get; set; }
-        public required bool HasApplicationAccess { get; set; }
     }
 
 }

Marco.Pms.Model/Dtos/Master/CreateContactCategoryDto.cs

@@ -2,7 +2,7 @@
 {
     public class CreateContactCategoryDto
     {
-        public required string Name { get; set; }
+        public string? Name { get; set; }
-        public required string Description { get; set; }
+        public string? Description { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Master/CreateContactTagDto.cs

@@ -2,7 +2,7 @@
 {
     public class CreateContactTagDto
     {
-        public required string Name { get; set; }
+        public string? Name { get; set; }
-        public required string Description { get; set; }
+        public string? Description { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Master/UpdateContactCategoryDto.cs

@@ -2,8 +2,8 @@
 {
     public class UpdateContactCategoryDto
     {
-        public required Guid Id { get; set; }
+        public Guid Id { get; set; }
-        public required string Name { get; set; }
+        public string? Name { get; set; }
-        public required string Description { get; set; }
+        public string? Description { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Master/UpdateContactTagDto.cs

@@ -3,7 +3,7 @@
     public class UpdateContactTagDto
     {
         public Guid Id { get; set; }
-        public required string Name { get; set; }
+        public string? Name { get; set; }
-        public required string Description { get; set; }
+        public string? Description { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Organization/CreateOrganizationDto.cs

@@ -8,6 +8,6 @@
         public required string Address { get; set; }
         public required string ContactNumber { get; set; }
         public string? logoImage { get; set; }
-        public List<Guid>? ServiceIds { get; set; }
+        public required List<Guid> ServiceIds { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Organization/UpdateOrganizationDto.cs

@@ -7,6 +7,6 @@
         public required string ContactPerson { get; set; }
         public required string Address { get; set; }
         public required string ContactNumber { get; set; }
-        public List<Guid>? ServiceIds { get; set; }
+        public required List<Guid> ServiceIds { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Projects/CreateProjectDto.cs

@@ -7,18 +7,17 @@ namespace Marco.Pms.Model.Dtos.Project
     {
         [Required(ErrorMessage = "Project Name is required!")]
         [DisplayName("Project Name")]
-        public required string Name { get; set; }
+        public string? Name { get; set; }
-
         [DisplayName("Short Name")]
         public string? ShortName { get; set; }
 
         [DisplayName("Project Address")]
         [Required(ErrorMessage = "Project Address is required!")]
-        public required string ProjectAddress { get; set; }
+        public string? ProjectAddress { get; set; }
 
 
         [DisplayName("Contact Person")]
-        public required string ContactPerson { get; set; }
+        public string? ContactPerson { get; set; }
 
 
         public DateTime? StartDate { get; set; }
@@ -26,8 +25,6 @@ namespace Marco.Pms.Model.Dtos.Project
 
         [DisplayName("Project Status")]
         [Required(ErrorMessage = "Project Status is required!")]
-        public required Guid ProjectStatusId { get; set; }
+        public Guid ProjectStatusId { get; set; }
-        public required Guid PromoterId { get; set; }
-        public required Guid PMCId { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Projects/ProjectAllocationDot.cs

@@ -5,7 +5,7 @@
         public Guid EmployeeId { get; set; }
         public Guid JobRoleId { get; set; }
         public Guid ProjectId { get; set; }
-        public Guid? ServiceId { get; set; }
+        public Guid ServiceId { get; set; }
         public bool Status { get; set; }
     }
 
@@ -14,7 +14,7 @@
     {
         public Guid ProjectId { get; set; }
         public Guid JobRoleId { get; set; }
-        public Guid? ServiceId { get; set; }
+        public Guid ServiceId { get; set; }
         public bool Status { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Projects/UpdateProjectDto.cs

@@ -5,20 +5,20 @@ namespace Marco.Pms.Model.Dtos.Project
 {
     public class UpdateProjectDto
     {
-        public required Guid Id { get; set; }
+        public Guid Id { get; set; }
         [Required(ErrorMessage = "Project Name is required!")]
         [DisplayName("Project Name")]
-        public required string Name { get; set; }
+        public string? Name { get; set; }
         [DisplayName("Short Name")]
         public string? ShortName { get; set; }
 
         [DisplayName("Project Address")]
         [Required(ErrorMessage = "Project Address is required!")]
-        public required string ProjectAddress { get; set; }
+        public string? ProjectAddress { get; set; }
 
 
         [DisplayName("Contact Person")]
-        public required string ContactPerson { get; set; }
+        public string? ContactPerson { get; set; }
 
 
         public DateTime? StartDate { get; set; }
@@ -26,8 +26,6 @@ namespace Marco.Pms.Model.Dtos.Project
 
         [DisplayName("Project Status")]
         [Required(ErrorMessage = "Project Status is required!")]
-        public required Guid ProjectStatusId { get; set; }
+        public Guid ProjectStatusId { get; set; }
-        public required Guid PromoterId { get; set; }
-        public required Guid PMCId { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Tenant/CreateTenantDto.cs

@@ -17,6 +17,5 @@
         public required string OrganizationSize { get; set; }
         public required Guid IndustryId { get; set; }
         public required string Reference { get; set; }
-        public List<Guid>? ServiceIds { get; set; }
     }
 }

Marco.Pms.Model/Entitlements/MenuStaticMaster.cs

@@ -1,224 +0,0 @@
-using Marco.Pms.Model.AppMenu;
-
-namespace Marco.Pms.Model.Entitlements
-{
-    public static class MenuStaticMaster
-    {
-        public static readonly MenuSection menu = new MenuSection
-        {
-            Header = "Main Navigation",
-            Title = "Main Menu",
-            Items = new List<MenuItem>
-            {
-                new MenuItem
-                {
-                    Text = "Dashboard",
-                    Icon = "bx bx-home",
-                    Available = true,
-                    Link = "/dashboard",
-                    PermissionIds = new List<string>(),
-                    Submenu = new List<SubMenuItem>()
-                },
-                new MenuItem
-                {
-                    Text = "Projects",
-                    Icon = "bx bx-building-house",
-                    Available = true,
-                    Link = "",
-                    PermissionIds = new List<string>
-                    {
-                        "6ea44136-987e-44ba-9e5d-1cf8f5837ebc",
-                        "172fc9b6-755b-4f62-ab26-55c34a330614",
-                        "b94802ce-0689-4643-9e1d-11c86950c35b",
-                        "8d7cc6e3-9147-41f7-aaa7-fa507e450bd4",
-                        "cf2825ad-453b-46aa-91d9-27c124d63373",
-                        "9fcc5f87-25e3-4846-90ac-67a71ab92e3c",
-                        "08752f33-3b29-4816-b76b-ea8a968ed3c5",
-                        "6a32379b-8b3f-49a6-8c48-4b7ac1b55dc2",
-                        "db4e40c5-2ba9-4b6d-b8a6-a16a250ff99c"
-                    },
-                    Submenu = new List<SubMenuItem>
-                    {
-                        new SubMenuItem
-                        {
-                            Text = "Project List",
-                            Available = true,
-                            Link = "/projects",
-                            PermissionIds = new List<string>
-                            {
-                                "6ea44136-987e-44ba-9e5d-1cf8f5837ebc",
-                                "172fc9b6-755b-4f62-ab26-55c34a330614",
-                                "b94802ce-0689-4643-9e1d-11c86950c35b",
-                                "8d7cc6e3-9147-41f7-aaa7-fa507e450bd4",
-                                "cf2825ad-453b-46aa-91d9-27c124d63373",
-                                "9fcc5f87-25e3-4846-90ac-67a71ab92e3c",
-                                "08752f33-3b29-4816-b76b-ea8a968ed3c5"
-                            }
-                        },
-                        new SubMenuItem
-                        {
-                            Text = "Daily Task Planning",
-                            Available = true,
-                            Link = "/activities/task",
-                            PermissionIds = new List<string>
-                            {
-                                "8d7cc6e3-9147-41f7-aaa7-fa507e450bd4",
-                                "9fcc5f87-25e3-4846-90ac-67a71ab92e3c",
-                                "6a32379b-8b3f-49a6-8c48-4b7ac1b55dc2"
-                            }
-                        },
-                        new SubMenuItem
-                        {
-                            Text = "Daily Progress Report",
-                            Available = true,
-                            Link = "/activities/records",
-                            PermissionIds = new List<string>
-                            {
-                                "9fcc5f87-25e3-4846-90ac-67a71ab92e3c",
-                                "6a32379b-8b3f-49a6-8c48-4b7ac1b55dc2",
-                                "db4e40c5-2ba9-4b6d-b8a6-a16a250ff99c"
-                            }
-                        },
-                        new SubMenuItem
-                        {
-                            Text = "Image Gallary",
-                            Available = true,
-                            Link = "/gallary",
-                            PermissionIds = new List<string>()
-                        },
-                        new SubMenuItem
-                        {
-                            Text = "Organizations",
-                            Available = true,
-                            Link = "/organizations",
-                            PermissionIds = new List<string>
-                            {
-                                "068cb3c1-49c5-4746-9f29-1fce16e820ac",
-                                "c1ae1363-ab8a-4bd9-a9d1-8c2c6083873a",
-                                "7a6cf830-0008-4e03-b31d-0d050cb634f4"
-                            }
-                        },
-                        new SubMenuItem
-                        {
-                            Text = "Project Report",
-                            Available = true,
-                            Link = "/activities/reports",
-                            PermissionIds = new List<string>()
-                        },
-                    }
-                },
-                new MenuItem
-                {
-                    Text = "Employees",
-                    Icon = "bx bx-user",
-                    Available = true,
-                    Link = "/employees",
-                    PermissionIds = new List<string>
-                    {
-                        "60611762-7f8a-4fb5-b53f-b1139918796b",
-                        "b82d2b7e-0d52-45f3-997b-c008ea460e7f",
-                        "a97d366a-c2bb-448d-be93-402bd2324566",
-                        "fbd213e0-0250-46f1-9f5f-4b2a1e6e76a3"
-                    },
-                    Submenu = new List<SubMenuItem>()
-                },
-                new MenuItem
-                {
-                    Text = "Attendance",
-                    Icon = "bx bx-list-ul",
-                    Available = true,
-                    Link = "/activities/Attendance",
-                    PermissionIds = new List<string>
-                    {
-                        "915e6bff-65f6-4e3f-aea8-3fd217d3ea9e",
-                        "57802c4a-00aa-4a1f-a048-fd2f70dd44b6",
-                        "ccb0589f-712b-43de-92ed-5b6088e7dc4e"
-                    },
-                    Submenu = new List<SubMenuItem>()
-                },
-                new MenuItem
-                {
-                    Text = "Directory",
-                    Icon = "bx bx-group",
-                    Available = true,
-                    Link = "/directory",
-                    PermissionIds = new List<string>
-                    {
-                        "4286a13b-bb40-4879-8c6d-18e9e393beda",
-                        "62668630-13ce-4f52-a0f0-db38af2230c5",
-                        "0f919170-92d4-4337-abd3-49b66fc871bb"
-                    },
-                    Submenu = new List<SubMenuItem>()
-                },
-                new MenuItem
-                {
-                    Text = "Expense",
-                    Icon = "bx bx-receipt",
-                    Available = true,
-                    Link = "/expenses",
-                    PermissionIds = new List<string>
-                    {
-                        "385be49f-8fde-440e-bdbc-3dffeb8dd116",
-                        "01e06444-9ca7-4df4-b900-8c3fa051b92f",
-                        "0f57885d-bcb2-4711-ac95-d841ace6d5a7",
-                        "1f4bda08-1873-449a-bb66-3e8222bd871b",
-                        "eaafdd76-8aac-45f9-a530-315589c6deca",
-                        "ea5a1529-4ee8-4828-80ea-0e23c9d4dd11",
-                        "ea5a1529-4ee8-4828-80ea-0e23c9d4dd11"
-                    },
-                    Submenu = new List<SubMenuItem>()
-                },
-                new MenuItem
-                {
-                    Text = "Administration",
-                    Icon = "bx bx-box",
-                    Available = true,
-                    Link = "",
-                    PermissionIds = new List<string>
-                    {
-                        "5ffbafe0-7ab0-48b1-bb50-c1bf76b65f9d",
-                        "588a8824-f924-4955-82d8-fc51956cf323",
-                        "d032cb1a-3f30-462c-bef0-7ace73a71c0b",
-                        "00e20637-ce8d-4417-bec4-9b31b5e65092",
-                        "647145c6-2108-4c98-aab4-178602236e55"
-                    },
-                    Submenu = new List<SubMenuItem>
-                    {
-                        new SubMenuItem
-                        {
-                            Text = "Tenant",
-                            Available = true,
-                            Link = "/tenants",
-                            PermissionIds = new List<string>
-                            {
-                                "d032cb1a-3f30-462c-bef0-7ace73a71c0b",
-                                "00e20637-ce8d-4417-bec4-9b31b5e65092",
-                                "647145c6-2108-4c98-aab4-178602236e55"
-                            }
-                        },
-                        new SubMenuItem
-                        {
-                            Text = "Masters",
-                            Available = true,
-                            Link = "/masters",
-                            PermissionIds = new List<string>
-                            {
-                                "5ffbafe0-7ab0-48b1-bb50-c1bf76b65f9d",
-                                "588a8824-f924-4955-82d8-fc51956cf323"
-                            }
-                        },
-                    }
-                },
-                new MenuItem
-                {
-                    Text = "Inventory",
-                    Icon = "bx bx-store",
-                    Available = true,
-                    Link = "/inventory",
-                    PermissionIds = new List<string>(),
-                    Submenu = new List<SubMenuItem>()
-                },
-            }
-        };
-    }
-}

Marco.Pms.Model/Expenses/Expenses.cs

@@ -54,7 +54,6 @@ namespace Marco.Pms.Model.Expenses
         public DateTime CreatedAt { get; set; }
         public string? TransactionId { get; set; }
         public string Description { get; set; } = string.Empty;
-        public string ExpenseUId { get; set; } = string.Empty;
         public string? Location { get; set; }
         public string? GSTNumber { get; set; }
         public string SupplerName { get; set; } = string.Empty;

Marco.Pms.Model/Filters/ImageFilter.cs

@@ -8,7 +8,6 @@
         public List<Guid>? WorkCategoryIds { get; set; }
         public List<Guid>? ActivityIds { get; set; }
         public List<Guid>? UploadedByIds { get; set; }
-        public List<Guid>? ServiceIds { get; set; }
         public DateTime? StartDate { get; set; }
         public DateTime? EndDate { get; set; }
     }

Marco.Pms.Model/Filters/TaskFilter.cs

@@ -6,7 +6,5 @@
         public List<Guid>? FloorIds { get; set; }
         public List<Guid>? ActivityIds { get; set; }
         public List<Guid>? ServiceIds { get; set; }
-        public DateTime? dateFrom { get; set; }
-        public DateTime? dateTo { get; set; }
     }
 }

Marco.Pms.Model/Mapper/EmployeeMapper.cs

@@ -36,9 +36,7 @@ namespace Marco.Pms.Model.Mapper
                 IsRootUser = model.ApplicationUser?.IsRootUser ?? false,
                 IsSystem = model.IsSystem,
                 JoiningDate = model.JoiningDate,
-                TenantId = model.TenantId ?? Guid.Empty,
+                TenantId = model.TenantId ?? Guid.Empty
-                HasApplicationAccess = model.HasApplicationAccess,
-                OrganizationId = model.OrganizationId
             };
         }
         public static BasicEmployeeVM ToBasicEmployeeVMFromEmployee(this Employee employee)

Marco.Pms.Model/MongoDBModels/Expenses/ExpenseDetailsMongoDB.cs

@@ -19,7 +19,6 @@ namespace Marco.Pms.Model.MongoDBModels.Expenses
         public DateTime CreatedAt { get; set; }
         public DateTime ExpireAt { get; set; } = DateTime.UtcNow.Date.AddDays(1);
         public string SupplerName { get; set; } = string.Empty;
-        public string? ExpenseUId { get; set; }
         public double Amount { get; set; }
         public ExpensesStatusMasterMongoDB Status { get; set; } = new ExpensesStatusMasterMongoDB();
         public List<ExpensesStatusMasterMongoDB> NextStatus { get; set; } = new List<ExpensesStatusMasterMongoDB>();

Marco.Pms.Model/MongoDBModels/OrganizationMongoDB.cs

@@ -1,13 +0,0 @@
-namespace Marco.Pms.Model.MongoDBModels
-{
-    public class OrganizationMongoDB
-    {
-        public string Id { get; set; } = string.Empty;
-        public string? Name { get; set; }
-        public string? Email { get; set; }
-        public string? ContactPerson { get; set; }
-        public string? Address { get; set; }
-        public string? ContactNumber { get; set; }
-        public double SPRID { get; set; }
-    }
-}

Marco.Pms.Model/MongoDBModels/Project/ProjectMongoDB.cs

@@ -13,8 +13,6 @@ namespace Marco.Pms.Model.MongoDBModels.Project
         public DateTime? StartDate { get; set; }
         public DateTime? EndDate { get; set; }
         public StatusMasterMongoDB? ProjectStatus { get; set; }
-        public OrganizationMongoDB? Promoter { get; set; }
-        public OrganizationMongoDB? PMC { get; set; }
         public int TeamSize { get; set; }
         public double CompletedWork { get; set; }
         public double PlannedWork { get; set; }

Marco.Pms.Model/OrganizationModel/ProjectOrgMapping.cs

@@ -1,5 +1,4 @@
-using Marco.Pms.Model.Employees;
+using Marco.Pms.Model.Utilities;
-using Marco.Pms.Model.Utilities;
 using Microsoft.AspNetCore.Mvc.ModelBinding.Validation;
 using System.ComponentModel.DataAnnotations.Schema;
 
@@ -28,12 +27,6 @@ namespace Marco.Pms.Model.OrganizationModel
         [ValidateNever]
         [ForeignKey("OrganizationTypeId")]
         public OrgTypeMaster? OrganizationType { get; set; }
-
-        public Guid AssignedById { get; set; }
-
-        [ValidateNever]
-        [ForeignKey("AssignedById")]
-        public Employee? AssignedBy { get; set; }
         public DateTime AssignedDate { get; set; }
         public DateTime? CompletionDate { get; set; }
     }

Marco.Pms.Model/OrganizationModel/TenantOrgMapping.cs

@@ -1,5 +1,4 @@
-using Marco.Pms.Model.Employees;
+using Marco.Pms.Model.Utilities;
-using Marco.Pms.Model.Utilities;
 using Microsoft.AspNetCore.Mvc.ModelBinding.Validation;
 using System.ComponentModel.DataAnnotations.Schema;
 
@@ -15,11 +14,6 @@ namespace Marco.Pms.Model.OrganizationModel
         public Organization? Organization { get; set; }
         public double SPRID { get; set; }
         public bool IsActive { get; set; } = true;
-        public Guid AssignedById { get; set; }
-
-        [ValidateNever]
-        [ForeignKey("AssignedById")]
-        public Employee? AssignedBy { get; set; }
         public DateTime AssignedDate { get; set; }
         public DateTime? ReassignedDate { get; set; }
     }

Marco.Pms.Model/ViewModels/AttendanceVM/EmployeeAttendanceVM.cs

@@ -1,5 +1,4 @@
 using Marco.Pms.Model.Dtos.Attendance;
-using Marco.Pms.Model.ViewModels.Activities;
 
 namespace Marco.Pms.Model.ViewModels.AttendanceVM
 {
@@ -7,20 +6,15 @@ namespace Marco.Pms.Model.ViewModels.AttendanceVM
     {
         public Guid Id { get; set; }
         public Guid EmployeeId { get; set; }
-        public Guid ProjectId { get; set; }
         public string? FirstName { get; set; }
         public string? LastName { get; set; }
         public string? EmployeeAvatar { get; set; }
         public string? OrganizationName { get; set; }
-        public string? ProjectName { get; set; }
         public DateTime? CheckInTime { get; set; }
         public DateTime? CheckOutTime { get; set; }
-        public DateTime? RequestedAt { get; set; }
-        public DateTime? ApprovedAt { get; set; }
         public string? JobRoleName { get; set; }
         public ATTENDANCE_MARK_TYPE Activity { get; set; }
-        public BasicEmployeeVM? Approver { get; set; }
+
-        public BasicEmployeeVM? RequestedBy { get; set; }
         public Guid? DocumentId { get; set; }
         public string? ThumbPreSignedUrl { get; set; }
         public string? PreSignedUrl { get; set; }

Marco.Pms.Model/ViewModels/Employee/EmployeeVM.cs

@@ -25,8 +25,6 @@
         public Guid TenantId { get; set; }
         public bool IsSystem { get; set; }
         public string? JobRole { get; set; }
-        public bool HasApplicationAccess { get; set; }
-        public Guid OrganizationId { get; set; }
 
     }
 }

Marco.Pms.Model/ViewModels/Expenses/ExpenseDetailsVM.cs

@@ -19,7 +19,6 @@ namespace Marco.Pms.Model.ViewModels.Expenses
         public DateTime TransactionDate { get; set; }
         public DateTime CreatedAt { get; set; }
         public string SupplerName { get; set; } = string.Empty;
-        public string? ExpenseUId { get; set; }
         public double Amount { get; set; }
         public ExpensesStatusMasterVM? Status { get; set; }
         public List<ExpensesStatusMasterVM>? NextStatus { get; set; }

Marco.Pms.Model/ViewModels/Expenses/ExpenseList.cs

@@ -18,7 +18,6 @@ namespace Marco.Pms.Model.ViewModels.Expanses
         public DateTime TransactionDate { get; set; }
         public DateTime CreatedAt { get; set; }
         public string SupplerName { get; set; } = string.Empty;
-        public string? ExpenseUId { get; set; }
         public string Description { get; set; } = string.Empty;
         public string TransactionId { get; set; } = string.Empty;
         public double Amount { get; set; }

Marco.Pms.Model/ViewModels/Organization/OrganizationDetailsVM.cs

@@ -1,26 +0,0 @@
-using Marco.Pms.Model.Master;
-using Marco.Pms.Model.ViewModels.Activities;
-
-namespace Marco.Pms.Model.ViewModels.Organization
-{
-    public class OrganizationDetailsVM
-    {
-        public Guid Id { get; set; }
-        public string? Name { get; set; }
-        public string? Email { get; set; }
-        public string? ContactPerson { get; set; }
-        public string? Address { get; set; }
-        public string? ContactNumber { get; set; }
-        public double SPRID { get; set; }
-        public int ActiveEmployeeCount { get; set; }
-        public int ActiveApplicationUserCount { get; set; }
-        public DateTime CreatedAt { get; set; }
-        public BasicEmployeeVM? CreatedBy { get; set; }
-        public BasicEmployeeVM? UpdatedBy { get; set; }
-        public DateTime? UpdatedAt { get; set; }
-        public bool IsActive { get; set; }
-        public List<ProjectServiceMappingVM>? Projects { get; set; }
-        public List<GlobalServiceMaster>? Services { get; set; }
-        public string? logoImage { get; set; }
-    }
-}

Marco.Pms.Model/ViewModels/Organization/ProjectOrganizationVM.cs

@@ -1,20 +0,0 @@
-using Marco.Pms.Model.ViewModels.Activities;
-using Marco.Pms.Model.ViewModels.Master;
-
-namespace Marco.Pms.Model.ViewModels.Organization
-{
-    public class ProjectOrganizationVM
-    {
-        public Guid Id { get; set; }
-        public string? Name { get; set; }
-        public string? Email { get; set; }
-        public string? ContactPerson { get; set; }
-        public double SPRID { get; set; }
-        public string? logoImage { get; set; }
-        public string? OrganizationType { get; set; }
-        public DateTime AssignedDate { get; set; }
-        public BasicEmployeeVM? AssignedBy { get; set; }
-        public ServiceMasterVM? Service { get; set; }
-        public DateTime? CompletionDate { get; set; }
-    }
-}

Marco.Pms.Model/ViewModels/Organization/ProjectServiceMappingVM.cs

@@ -1,16 +0,0 @@
-using Marco.Pms.Model.ViewModels.Master;
-using Marco.Pms.Model.ViewModels.Projects;
-
-namespace Marco.Pms.Model.ViewModels.Organization
-{
-    public class ProjectServiceMappingVM
-    {
-        public BasicProjectVM? Project { get; set; }
-        public ServiceMasterVM? Service { get; set; }
-        public DateTime PlannedStartDate { get; set; }
-        public DateTime PlannedEndDate { get; set; }
-        public DateTime ActualStartDate { get; set; }
-        public DateTime? ActualEndDate { get; set; }
-        public bool IsActive { get; set; }
-    }
-}

Marco.Pms.Model/ViewModels/Projects/ProjectVM.cs

@@ -1,5 +1,4 @@
 using Marco.Pms.Model.Master;
-using Marco.Pms.Model.ViewModels.Organization;
 
 namespace Marco.Pms.Model.ViewModels.Projects
 {
@@ -13,8 +12,6 @@ namespace Marco.Pms.Model.ViewModels.Projects
         public DateTime? StartDate { get; set; }
         public DateTime? EndDate { get; set; }
         public StatusMaster? ProjectStatus { get; set; }
-        public BasicOrganizationVm? Promoter { get; set; }
-        public BasicOrganizationVm? PMC { get; set; }
 
     }
 }

Marco.Pms.Model/ViewModels/Report/ProjectStatisticReport.cs

@@ -7,18 +7,15 @@
         public required string TimeStamp { get; set; }
         public int TodaysAttendances { get; set; }
         public int TotalEmployees { get; set; }
-        public double AttendancePercentage { get; set; }
         public int RegularizationPending { get; set; }
         public int CheckoutPending { get; set; }
         public double TotalPlannedWork { get; set; }
         public double TotalCompletedWork { get; set; }
-        public double CompletionStatus { get; set; }
         public double TotalPlannedTask { get; set; }
         public double TotalCompletedTask { get; set; }
-        public double TaskPercentage { get; set; }
+        public double CompletionStatus { get; set; }
         public int ReportPending { get; set; }
         public int TodaysAssignTasks { get; set; }
-        public int TodaysCompletedTasks { get; set; }
         public List<TeamOnSite> TeamOnSite { get; set; } = new List<TeamOnSite>();
         public List<PerformedTask> PerformedTasks { get; set; } = new List<PerformedTask>();
         public List<PerformedAttendance> PerformedAttendance { get; set; } = new List<PerformedAttendance>();

Marco.Pms.Model/ViewModels/Tenant/TenantListVM.cs

@@ -12,7 +12,6 @@ namespace Marco.Pms.Model.ViewModels.Tenant
         public string ContactNumber { get; set; } = string.Empty;
         public string? logoImage { get; set; }   // Base64
         public string? OrganizationSize { get; set; }
-        public Guid OrganizationId { get; set; }
         public Industry? Industry { get; set; }
         public TenantStatus? TenantStatus { get; set; }
     }

Marco.Pms.Services/Controllers/AppMenuController.cs

@@ -456,14 +456,6 @@ namespace Marco.Pms.Services.Controllers
                 // Step 2: Fetch all menu sections for the tenant
                 var menus = await _sideBarMenuHelper.GetAllMenuSectionsAsync(tenantId);
 
-                if (!(menus?.Any() ?? false))
-                {
-                    menus = new List<MenuSection>
-                    {
-                        MenuStaticMaster.menu
-                    };
-                }
-
                 foreach (var menu in menus)
                 {
                     var allowedItems = new List<MenuItem>();
@@ -575,23 +567,24 @@ namespace Marco.Pms.Services.Controllers
                     {
                         ProjectManagement, new List<MasterMenuVM>
                         {
-                            new MasterMenuVM { Id = 3, Name = "Work Category" },
+                            new MasterMenuVM { Id = 3, Name = "Activity" },
-                            new MasterMenuVM { Id = 8, Name = "Services" }
+                            new MasterMenuVM { Id = 4, Name = "Work Category" },
+                            new MasterMenuVM { Id = 9, Name = "Services" }
                             //new MasterMenuVM { Id = 10, Name = "Payment Mode" }
                         }
                     },
                     {
                         DirectoryManagement, new List<MasterMenuVM>
                         {
-                            new MasterMenuVM { Id = 4, Name = "Contact Category" },
+                            new MasterMenuVM { Id = 5, Name = "Contact Category" },
-                            new MasterMenuVM { Id = 5, Name = "Contact Tag" }
+                            new MasterMenuVM { Id = 6, Name = "Contact Tag" }
                         }
                     },
                     {
                         ExpenseManagement, new List<MasterMenuVM>
                         {
-                            new MasterMenuVM { Id = 6, Name = "Expense Type" },
+                            new MasterMenuVM { Id = 7, Name = "Expense Type" },
-                            new MasterMenuVM { Id = 7, Name = "Payment Mode" }
+                            new MasterMenuVM { Id = 8, Name = "Payment Mode" }
                         }
                     }
                 };
@@ -640,13 +633,6 @@ namespace Marco.Pms.Services.Controllers
             {
                 // Step 2: Fetch all menu sections for the tenant
                 var menus = await _sideBarMenuHelper.GetAllMenuSectionsAsync(tenantId);
-                if (!(menus?.Any() ?? false))
-                {
-                    menus = new List<MenuSection>
-                    {
-                        MenuStaticMaster.menu
-                    };
-                }
                 List<MenuSectionApplicationVM> response = new List<MenuSectionApplicationVM>();
 
                 foreach (var menu in menus)
@@ -733,42 +719,7 @@ namespace Marco.Pms.Services.Controllers
                     menu.Items = allowedItems;
                 }
 
-                var viewDocumentTask = Task.Run(async () =>
+                if (await _permissions.HasPermission(PermissionsMaster.ViewDocument, employeeId))
-                {
-                    using var taskScope = _serviceScopeFactory.CreateScope();
-                    var permissions = taskScope.ServiceProvider.GetRequiredService<PermissionServices>();
-                    return await permissions.HasPermission(PermissionsMaster.ViewDocument, employeeId);
-                });
-
-                var uploadDocumentTask = Task.Run(async () =>
-                {
-                    using var taskScope = _serviceScopeFactory.CreateScope();
-                    var permissions = taskScope.ServiceProvider.GetRequiredService<PermissionServices>();
-                    return await permissions.HasPermission(PermissionsMaster.UploadDocument, employeeId);
-                });
-
-                var verifyDocumentTask = Task.Run(async () =>
-                {
-                    using var taskScope = _serviceScopeFactory.CreateScope();
-                    var permissions = taskScope.ServiceProvider.GetRequiredService<PermissionServices>();
-                    return await permissions.HasPermission(PermissionsMaster.VerifyDocument, employeeId);
-                });
-
-                var downloadDocumentTask = Task.Run(async () =>
-                {
-                    using var taskScope = _serviceScopeFactory.CreateScope();
-                    var permissions = taskScope.ServiceProvider.GetRequiredService<PermissionServices>();
-                    return await permissions.HasPermission(PermissionsMaster.DownloadDocument, employeeId);
-                });
-
-                await Task.WhenAll(viewDocumentTask, uploadDocumentTask, verifyDocumentTask, downloadDocumentTask);
-
-                var viewDocument = viewDocumentTask.Result;
-                var uploadDocument = uploadDocumentTask.Result;
-                var verifyDocument = verifyDocumentTask.Result;
-                var downloadDocument = downloadDocumentTask.Result;
-
-                if (viewDocument || uploadDocument || verifyDocument || downloadDocument)
                 {
                     response.Add(new MenuSectionApplicationVM
                     {

Marco.Pms.Services/Controllers/AttendanceController.cs

@@ -1,5 +1,4 @@
-using AutoMapper;
+using Marco.Pms.DataAccess.Data;
-using Marco.Pms.DataAccess.Data;
 using Marco.Pms.Model.AttendanceModule;
 using Marco.Pms.Model.Dtos.Attendance;
 using Marco.Pms.Model.Employees;
@@ -7,7 +6,6 @@ using Marco.Pms.Model.Entitlements;
 using Marco.Pms.Model.Mapper;
 using Marco.Pms.Model.Projects;
 using Marco.Pms.Model.Utilities;
-using Marco.Pms.Model.ViewModels.Activities;
 using Marco.Pms.Model.ViewModels.AttendanceVM;
 using Marco.Pms.Services.Hubs;
 using Marco.Pms.Services.Service;
@@ -30,41 +28,48 @@ namespace MarcoBMS.Services.Controllers
     public class AttendanceController : ControllerBase
     {
         private readonly ApplicationDbContext _context;
-        private readonly IServiceScopeFactory _serviceScopeFactory;
+        private readonly EmployeeHelper _employeeHelper;
+        private readonly IProjectServices _projectServices;
         private readonly UserHelper _userHelper;
+        private readonly S3UploadService _s3Service;
         private readonly PermissionServices _permission;
         private readonly ILoggingService _logger;
-        private readonly Guid tenantId;
+        private readonly IHubContext<MarcoHub> _signalR;
-        private readonly IMapper _mapper;
+        private readonly IFirebaseService _firebase;
 
         public AttendanceController(
-             ApplicationDbContext context,
+             ApplicationDbContext context, EmployeeHelper employeeHelper, IProjectServices projectServices, UserHelper userHelper,
-             UserHelper userHelper,
+             S3UploadService s3Service, ILoggingService logger, PermissionServices permission, IHubContext<MarcoHub> signalR, IFirebaseService firebase)
-             IServiceScopeFactory serviceScopeFactory,
-             ILoggingService logger,
-             PermissionServices permission,
-             IMapper mapper)
         {
             _context = context;
-            _serviceScopeFactory = serviceScopeFactory;
+            _employeeHelper = employeeHelper;
+            _projectServices = projectServices;
             _userHelper = userHelper;
+            _s3Service = s3Service;
             _logger = logger;
             _permission = permission;
-            _mapper = mapper;
+            _signalR = signalR;
-            tenantId = userHelper.GetTenantId();
+            _firebase = firebase;
+        }
+
+        private Guid GetTenantId()
+        {
+            return _userHelper.GetTenantId();
+            //var tenant = User.FindFirst("TenantId")?.Value;
+            //return (tenant != null ? Convert.ToInt32(tenant) : 1);
         }
 
         [HttpGet("log/attendance/{attendanceid}")]
+
         public async Task<IActionResult> GetAttendanceLogById(Guid attendanceid)
         {
-            using var scope = _serviceScopeFactory.CreateScope();
+            Guid TenantId = GetTenantId();
-            var _s3Service = scope.ServiceProvider.GetRequiredService<S3UploadService>();
 
             List<AttendanceLog> lstAttendance = await _context.AttendanceLogs
                 .Include(a => a.Document)
                 .Include(a => a.Employee)
                 .Include(a => a.UpdatedByEmployee)
-                .Where(c => c.AttendanceId == attendanceid && c.TenantId == tenantId)
+                .Where(c => c.AttendanceId == attendanceid && c.TenantId == TenantId)
                 .ToListAsync();
 
             List<AttendanceLogVM> attendanceLogVMs = new List<AttendanceLogVM>();
@@ -80,42 +85,30 @@ namespace MarcoBMS.Services.Controllers
         }
 
         [HttpGet("log/employee/{employeeId}")]
-        public async Task<IActionResult> GetAttendanceLogByEmployeeId(Guid employeeId, [FromQuery] DateTime? dateFrom = null, [FromQuery] DateTime? dateTo = null)
+        public async Task<IActionResult> GetAttendanceLogByEmployeeId(Guid employeeId, [FromQuery] string? dateFrom = null, [FromQuery] string? dateTo = null)
         {
+            Guid TenantId = GetTenantId();
+            DateTime fromDate = new DateTime();
+            DateTime toDate = new DateTime();
+
+            if (dateFrom != null && DateTime.TryParse(dateFrom, out fromDate) == false)
+            {
+                _logger.LogWarning("User sent Invalid from Date while featching attendance logs");
+                return BadRequest(ApiResponse<object>.ErrorResponse("Invalid Date", "Invalid Date", 400));
+            }
+            if (dateTo != null && DateTime.TryParse(dateTo, out toDate) == false)
+            {
+                _logger.LogWarning("User sent Invalid to Date while featching attendance logs");
+                return BadRequest(ApiResponse<object>.ErrorResponse("Invalid Date", "Invalid Date", 400));
+            }
+
             if (employeeId == Guid.Empty)
             {
                 _logger.LogWarning("The employee Id sent by user is empty");
                 return BadRequest(ApiResponse<object>.ErrorResponse("Employee ID is required and must not be Empty.", "Employee ID is required and must not be empty.", 400));
             }
-
+            List<Attendance> attendances = await _context.Attendes.Where(c => c.EmployeeID == employeeId && c.TenantId == TenantId && c.AttendanceDate.Date >= fromDate && c.AttendanceDate.Date <= toDate).ToListAsync();
-            Employee? employee = await _context.Employees.Include(e => e.JobRole).FirstOrDefaultAsync(e => e.Id == employeeId && e.TenantId == tenantId);
+            Employee? employee = await _context.Employees.Include(e => e.JobRole).FirstOrDefaultAsync(e => e.Id == employeeId && e.TenantId == TenantId && e.IsActive);
-            if (employee == null)
-            {
-                _logger.LogWarning("Employee {EmployeeId} not found", employeeId);
-                return NotFound(ApiResponse<object>.ErrorResponse("Employee not found", "Employee not found in database", 404));
-            }
-
-            if (!dateFrom.HasValue)
-            {
-                dateFrom = DateTime.UtcNow;
-            }
-            if (!dateTo.HasValue)
-            {
-                var days = 0 - 7;
-                dateTo = dateFrom.Value.AddDays(days);
-            }
-
-            List<Attendance> attendances = await _context.Attendes
-                .Include(a => a.RequestedBy)
-                    .ThenInclude(e => e!.JobRole)
-                .Include(a => a.RequestedBy)
-                    .ThenInclude(e => e!.JobRole)
-                .Where(c => c.EmployeeId == employeeId && c.TenantId == tenantId && c.AttendanceDate.Date >= dateFrom && c.AttendanceDate.Date <= dateTo).ToListAsync();
-
-            var projectIds = attendances.Select(a => a.ProjectID).Distinct().ToList();
-
-            var projects = await _context.Projects.Where(p => projectIds.Contains(p.Id) && p.TenantId == tenantId).ToListAsync();
-
             List<EmployeeAttendanceVM> results = new List<EmployeeAttendanceVM>();
 
             if (employee != null)
@@ -128,17 +121,11 @@ namespace MarcoBMS.Services.Controllers
                         EmployeeId = employee.Id,
                         FirstName = employee.FirstName,
                         LastName = employee.LastName,
-                        ProjectId = attendance.ProjectID,
-                        ProjectName = projects.Where(p => p.Id == attendance.ProjectID).Select(p => p.Name).FirstOrDefault(),
                         CheckInTime = attendance.InTime,
                         CheckOutTime = attendance.OutTime,
                         JobRoleName = employee.JobRole != null ? employee.JobRole.Name : "",
                         Activity = attendance.Activity,
-                        EmployeeAvatar = null,
+                        EmployeeAvatar = null
-                        RequestedAt = attendance.RequestedAt,
-                        RequestedBy = _mapper.Map<BasicEmployeeVM>(attendance.RequestedBy),
-                        ApprovedAt = attendance.ApprovedAt,
-                        Approver = _mapper.Map<BasicEmployeeVM>(attendance.Approver)
                     };
                     results.Add(result);
                 }
@@ -158,12 +145,11 @@ namespace MarcoBMS.Services.Controllers
         /// <returns></returns>
 
         [HttpGet("project/log")]
+
         public async Task<IActionResult> EmployeeAttendanceByDateRange([FromQuery] Guid projectId, [FromQuery] Guid? organizationId, [FromQuery] string? dateFrom = null, [FromQuery] string? dateTo = null)
         {
-            using var scope = _serviceScopeFactory.CreateScope();
+            Guid tenantId = GetTenantId();
-            var _projectServices = scope.ServiceProvider.GetRequiredService<IProjectServices>();
+            var LoggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-
-            var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
 
             var project = await _context.Projects.AsNoTracking().FirstOrDefaultAsync(p => p.Id == projectId && p.TenantId == tenantId);
             if (project == null)
@@ -172,13 +158,13 @@ namespace MarcoBMS.Services.Controllers
                 return NotFound(ApiResponse<object>.ErrorResponse("Project not found."));
             }
 
-            var hasTeamAttendancePermission = await _permission.HasPermission(PermissionsMaster.TeamAttendance, loggedInEmployee.Id);
+            var hasTeamAttendancePermission = await _permission.HasPermission(PermissionsMaster.TeamAttendance, LoggedInEmployee.Id);
-            var hasSelfAttendancePermission = await _permission.HasPermission(PermissionsMaster.SelfAttendance, loggedInEmployee.Id);
+            var hasSelfAttendancePermission = await _permission.HasPermission(PermissionsMaster.SelfAttendance, LoggedInEmployee.Id);
-            var hasProjectPermission = await _permission.HasProjectPermission(loggedInEmployee, projectId);
+            var hasProjectPermission = await _permission.HasProjectPermission(LoggedInEmployee, projectId);
 
             if (!hasProjectPermission)
             {
-                _logger.LogWarning("Employee {EmployeeId} tries to access attendance of project {ProjectId}, but don't have access", loggedInEmployee.Id, projectId);
+                _logger.LogWarning("Employee {EmployeeId} tries to access attendance of project {ProjectId}, but don't have access", LoggedInEmployee.Id, projectId);
                 return Unauthorized(ApiResponse<object>.ErrorResponse("Unauthorized access", "Unauthorized access", 404));
             }
 
@@ -211,13 +197,7 @@ namespace MarcoBMS.Services.Controllers
 
             if (hasTeamAttendancePermission)
             {
-                List<Attendance> lstAttendance = await _context.Attendes
+                List<Attendance> lstAttendance = await _context.Attendes.Where(c => c.ProjectID == projectId && c.AttendanceDate.Date >= fromDate.Date && c.AttendanceDate.Date <= toDate.Date && c.TenantId == tenantId).ToListAsync();
-                    .Include(a => a.RequestedBy)
-                        .ThenInclude(e => e!.JobRole)
-                    .Include(a => a.Approver)
-                        .ThenInclude(e => e!.JobRole)
-                    .Where(c => c.ProjectID == projectId && c.AttendanceDate.Date >= fromDate.Date && c.AttendanceDate.Date <= toDate.Date && c.TenantId == tenantId)
-                    .ToListAsync();
 
 
                 List<ProjectAllocation> projectteam = await _projectServices.GetTeamByProject(tenantId, projectId, organizationId, true);
@@ -229,13 +209,9 @@ namespace MarcoBMS.Services.Controllers
                         Id = attendance.Id,
                         CheckInTime = attendance.InTime,
                         CheckOutTime = attendance.OutTime,
-                        Activity = attendance.Activity,
+                        Activity = attendance.Activity
-                        ApprovedAt = attendance.ApprovedAt,
-                        Approver = _mapper.Map<BasicEmployeeVM>(attendance.Approver),
-                        RequestedAt = attendance.RequestedAt,
-                        RequestedBy = _mapper.Map<BasicEmployeeVM>(attendance.RequestedBy)
                     };
-                    teamMember = projectteam.Find(x => x.EmployeeId == attendance.EmployeeId);
+                    teamMember = projectteam.Find(x => x.EmployeeId == attendance.EmployeeID);
                     if (teamMember != null)
                     {
                         result1.EmployeeAvatar = null;
@@ -246,8 +222,6 @@ namespace MarcoBMS.Services.Controllers
                             result1.LastName = teamMember.Employee.LastName;
                             result1.JobRoleName = teamMember.Employee.JobRole != null ? teamMember.Employee.JobRole.Name : null;
                             result1.OrganizationName = teamMember.Employee.Organization?.Name;
-                            result1.ProjectId = projectId;
-                            result1.ProjectName = teamMember.Project?.Name;
                         }
                         else
                         {
@@ -265,23 +239,13 @@ namespace MarcoBMS.Services.Controllers
             else if (hasSelfAttendancePermission)
             {
                 List<Attendance> lstAttendances = await _context.Attendes
-                    .Include(a => a.RequestedBy)
+                    .Where(c => c.ProjectID == projectId && c.EmployeeID == LoggedInEmployee.Id && c.AttendanceDate.Date >= fromDate.Date && c.AttendanceDate.Date <= toDate.Date && c.TenantId == tenantId)
-                        .ThenInclude(e => e!.JobRole)
-                    .Include(a => a.Approver)
-                        .ThenInclude(e => e!.JobRole)
-                    .Where(c => c.ProjectID == projectId && c.EmployeeId == loggedInEmployee.Id && c.AttendanceDate.Date >= fromDate.Date &&
-                        c.AttendanceDate.Date <= toDate.Date && c.TenantId == tenantId)
                     .ToListAsync();
 
                 var projectAllocationQuery = _context.ProjectAllocations
-                    .Include(pa => pa.Project)
                     .Include(pa => pa.Employee)
                         .ThenInclude(e => e!.Organization)
-                    .Include(pa => pa.Employee)
+                    .Where(pa => pa.ProjectId == projectId && pa.EmployeeId == LoggedInEmployee.Id && pa.TenantId == tenantId && pa.IsActive);
-                        .ThenInclude(e => e!.JobRole)
-                    .Where(pa => pa.EmployeeId == loggedInEmployee.Id && pa.TenantId == tenantId && pa.IsActive &&
-                        pa.ProjectId == projectId && pa.Project != null &&
-                        pa.Employee != null && pa.Employee.Organization != null && pa.Employee.JobRole != null);
 
                 if (organizationId.HasValue)
                 {
@@ -303,15 +267,9 @@ namespace MarcoBMS.Services.Controllers
                             LastName = projectAllocation.Employee?.LastName,
                             JobRoleName = projectAllocation.Employee?.JobRole?.Name,
                             OrganizationName = projectAllocation.Employee?.Organization?.Name,
-                            ProjectId = attendance.ProjectID,
-                            ProjectName = projectAllocation.Project?.Name,
                             CheckInTime = attendance.InTime,
                             CheckOutTime = attendance.OutTime,
-                            Activity = attendance.Activity,
+                            Activity = attendance.Activity
-                            ApprovedAt = attendance.ApprovedAt,
-                            Approver = _mapper.Map<BasicEmployeeVM>(attendance.Approver),
-                            RequestedAt = attendance.RequestedAt,
-                            RequestedBy = _mapper.Map<BasicEmployeeVM>(attendance.RequestedBy)
                         };
                         result.Add(result1);
                     }
@@ -333,6 +291,7 @@ namespace MarcoBMS.Services.Controllers
         /// <returns>An IActionResult containing a list of employee attendance records or an error response.</returns>
         public async Task<IActionResult> EmployeeAttendanceByProjectAsync([FromQuery] Guid projectId, [FromQuery] Guid? organizationId, [FromQuery] bool includeInactive, [FromQuery] string? date = null)
         {
+            var tenantId = GetTenantId();
             var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
 
             // --- 1. Initial Validation and Permission Checks ---
@@ -394,9 +353,7 @@ namespace MarcoBMS.Services.Controllers
         [HttpGet("regularize")]
         public async Task<IActionResult> GetRequestRegularizeAttendance([FromQuery] Guid projectId, [FromQuery] Guid? organizationId, [FromQuery] bool IncludeInActive)
         {
-            using var scope = _serviceScopeFactory.CreateScope();
+            Guid TenantId = GetTenantId();
-            var _projectServices = scope.ServiceProvider.GetRequiredService<IProjectServices>();
-
             Employee LoggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
             var result = new List<EmployeeAttendanceVM>();
             var hasProjectPermission = await _permission.HasProjectPermission(LoggedInEmployee, projectId);
@@ -407,45 +364,35 @@ namespace MarcoBMS.Services.Controllers
                 return Unauthorized(ApiResponse<object>.ErrorResponse("Unauthorized access", "Unauthorized access", 404));
             }
 
-            List<Attendance> lstAttendance = await _context.Attendes
+            List<Attendance> lstAttendance = await _context.Attendes.Where(c => c.ProjectID == projectId && c.Activity == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE && c.TenantId == TenantId).ToListAsync();
-                .Include(a => a.RequestedBy)
-                    .ThenInclude(e => e!.JobRole)
-                .Include(a => a.Approver)
-                    .ThenInclude(e => e!.JobRole)
-                .Where(c => c.ProjectID == projectId && c.Activity == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE && c.TenantId == tenantId)
-                .ToListAsync();
 
-            List<ProjectAllocation> projectteam = await _projectServices.GetTeamByProject(tenantId, projectId, organizationId, true);
+            List<ProjectAllocation> projectteam = await _projectServices.GetTeamByProject(TenantId, projectId, organizationId, true);
             var idList = projectteam.Select(p => p.EmployeeId).ToList();
+            var jobRole = await _context.JobRoles.ToListAsync();
 
             foreach (Attendance attende in lstAttendance)
             {
-                var teamMember = projectteam.Find(m => m.EmployeeId == attende.EmployeeId);
+                var result1 = new EmployeeAttendanceVM()
+                {
+                    Id = attende.Id,
+                    CheckInTime = attende.InTime,
+                    CheckOutTime = attende.OutTime,
+                    Activity = attende.Activity,
+                    EmployeeAvatar = null,
+                    EmployeeId = attende.EmployeeID,
+
+                };
+
+                var teamMember = projectteam.Find(m => m.EmployeeId == attende.EmployeeID);
                 if (teamMember != null && teamMember.Employee != null && teamMember.Employee.JobRole != null)
                 {
-                    var result1 = new EmployeeAttendanceVM()
+                    result1.FirstName = teamMember.Employee.FirstName;
-                    {
+                    result1.LastName = teamMember.Employee.LastName;
-                        Id = attende.Id,
+                    result1.JobRoleName = teamMember.Employee.JobRole.Name;
-                        CheckInTime = attende.InTime,
+                    result1.OrganizationName = teamMember.Employee.Organization?.Name;
-                        CheckOutTime = attende.OutTime,
-                        Activity = attende.Activity,
-                        EmployeeAvatar = null,
-                        EmployeeId = attende.EmployeeId,
-                        FirstName = teamMember.Employee.FirstName,
-                        LastName = teamMember.Employee.LastName,
-                        JobRoleName = teamMember.Employee.JobRole.Name,
-                        OrganizationName = teamMember.Employee.Organization?.Name,
-                        ProjectId = projectId,
-                        ProjectName = teamMember.Project?.Name,
-                        ApprovedAt = attende.ApprovedAt,
-                        Approver = _mapper.Map<BasicEmployeeVM>(attende.Approver),
-                        RequestedAt = attende.RequestedAt,
-                        RequestedBy = _mapper.Map<BasicEmployeeVM>(attende.RequestedBy)
-                    };
-                    result.Add(result1);
                 }
 
-
+                result.Add(result1);
             }
 
             result.Sort(delegate (EmployeeAttendanceVM x, EmployeeAttendanceVM y)
@@ -470,17 +417,13 @@ namespace MarcoBMS.Services.Controllers
                 return BadRequest(ApiResponse<object>.ErrorResponse("Invalid data", errors, 400));
             }
 
-            using var scope = _serviceScopeFactory.CreateScope();
+            Guid TenantId = GetTenantId();
-            var _signalR = scope.ServiceProvider.GetRequiredService<IHubContext<MarcoHub>>();
-            var _employeeHelper = scope.ServiceProvider.GetRequiredService<EmployeeHelper>();
-            var _firebase = scope.ServiceProvider.GetRequiredService<IFirebaseService>();
-
             var currentEmployee = await _userHelper.GetCurrentEmployeeAsync();
-            using var transaction = await _context.Database.BeginTransactionAsync();
 
+            using var transaction = await _context.Database.BeginTransactionAsync();
             try
             {
-                Attendance? attendance = await _context.Attendes.FirstOrDefaultAsync(a => a.Id == recordAttendanceDot.Id && a.TenantId == tenantId); ;
+                Attendance? attendance = await _context.Attendes.FirstOrDefaultAsync(a => a.Id == recordAttendanceDot.Id && a.TenantId == TenantId); ;
 
                 if (recordAttendanceDot.MarkTime == null)
                 {
@@ -508,6 +451,10 @@ namespace MarcoBMS.Services.Controllers
                     {
                         attendance.IsApproved = true;
                         attendance.Activity = ATTENDANCE_MARK_TYPE.REGULARIZE;
+
+
+                        //string timeString = "10:30 PM"; // Format: "hh:mm tt"
+
                         attendance.OutTime = finalDateTime;
                     }
                     else if (recordAttendanceDot.Action == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE)
@@ -518,8 +465,6 @@ namespace MarcoBMS.Services.Controllers
                         {
                             attendance.OutTime = finalDateTime;
                             attendance.Activity = ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE;
-                            attendance.RequestedById = currentEmployee.Id;
-                            attendance.RequestedAt = DateTime.UtcNow;
                         }
                         else
                         {
@@ -532,16 +477,13 @@ namespace MarcoBMS.Services.Controllers
                     {
                         attendance.IsApproved = true;
                         attendance.Activity = ATTENDANCE_MARK_TYPE.REGULARIZE;
-                        attendance.ApprovedById = currentEmployee.Id;
+                        attendance.ApprovedBy = currentEmployee.Id;
-                        attendance.ApprovedAt = DateTime.UtcNow;
                         // do nothing
                     }
                     else if (recordAttendanceDot.Action == ATTENDANCE_MARK_TYPE.REGULARIZE_REJECT)
                     {
                         attendance.IsApproved = false;
                         attendance.Activity = ATTENDANCE_MARK_TYPE.REGULARIZE_REJECT;
-                        attendance.ApprovedById = currentEmployee.Id;
-                        attendance.ApprovedAt = DateTime.UtcNow;
                         // do nothing
                     }
                     attendance.Date = DateTime.UtcNow;
@@ -552,11 +494,11 @@ namespace MarcoBMS.Services.Controllers
                 else
                 {
                     attendance = new Attendance();
-                    attendance.TenantId = tenantId;
+                    attendance.TenantId = TenantId;
                     attendance.AttendanceDate = recordAttendanceDot.Date;
                     // attendance.Activity = recordAttendanceDot.Action;
                     attendance.Comment = recordAttendanceDot.Comment;
-                    attendance.EmployeeId = recordAttendanceDot.EmployeeID;
+                    attendance.EmployeeID = recordAttendanceDot.EmployeeID;
                     attendance.ProjectID = recordAttendanceDot.ProjectID;
                     attendance.Date = DateTime.UtcNow;
 
@@ -584,7 +526,7 @@ namespace MarcoBMS.Services.Controllers
                     Latitude = recordAttendanceDot.Latitude,
                     Longitude = recordAttendanceDot.Longitude,
 
-                    TenantId = tenantId,
+                    TenantId = TenantId,
                     UpdatedBy = currentEmployee.Id,
                     UpdatedOn = recordAttendanceDot.Date
                 };
@@ -631,7 +573,7 @@ namespace MarcoBMS.Services.Controllers
 
                         var name = $"{vm.FirstName} {vm.LastName}";
 
-                        await _firebase.SendAttendanceMessageAsync(attendance.ProjectID, name, recordAttendanceDot.Action, attendance.EmployeeId, tenantId);
+                        await _firebase.SendAttendanceMessageAsync(attendance.ProjectID, name, recordAttendanceDot.Action, attendance.EmployeeID, TenantId);
 
                     });
 
@@ -667,12 +609,7 @@ namespace MarcoBMS.Services.Controllers
                 return BadRequest(ApiResponse<object>.ErrorResponse("Invalid data", errors, 400));
             }
 
-            using var scope = _serviceScopeFactory.CreateScope();
+            Guid tenantId = GetTenantId();
-            var _s3Service = scope.ServiceProvider.GetRequiredService<S3UploadService>();
-            var _signalR = scope.ServiceProvider.GetRequiredService<IHubContext<MarcoHub>>();
-            var _employeeHelper = scope.ServiceProvider.GetRequiredService<EmployeeHelper>();
-            var _firebase = scope.ServiceProvider.GetRequiredService<IFirebaseService>();
-
             var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
             var batchId = Guid.NewGuid();
 
@@ -704,7 +641,7 @@ namespace MarcoBMS.Services.Controllers
                         TenantId = tenantId,
                         AttendanceDate = recordAttendanceDot.Date,
                         Comment = recordAttendanceDot.Comment,
-                        EmployeeId = recordAttendanceDot.EmployeeID,
+                        EmployeeID = recordAttendanceDot.EmployeeID,
                         ProjectID = recordAttendanceDot.ProjectID,
                         Date = DateTime.UtcNow,
                         InTime = finalDateTime,
@@ -736,8 +673,6 @@ namespace MarcoBMS.Services.Controllers
                             {
                                 attendance.OutTime = finalDateTime;
                                 attendance.Activity = ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE;
-                                attendance.RequestedById = loggedInEmployee.Id;
-                                attendance.RequestedAt = DateTime.UtcNow;
                             }
                             else
                             {
@@ -748,14 +683,10 @@ namespace MarcoBMS.Services.Controllers
                         case ATTENDANCE_MARK_TYPE.REGULARIZE:
                             attendance.IsApproved = true;
                             attendance.Activity = ATTENDANCE_MARK_TYPE.REGULARIZE;
-                            attendance.ApprovedById = loggedInEmployee.Id;
-                            attendance.ApprovedAt = DateTime.UtcNow;
                             break;
                         case ATTENDANCE_MARK_TYPE.REGULARIZE_REJECT:
                             attendance.IsApproved = false;
                             attendance.Activity = ATTENDANCE_MARK_TYPE.REGULARIZE_REJECT;
-                            attendance.ApprovedById = loggedInEmployee.Id;
-                            attendance.ApprovedAt = DateTime.UtcNow;
                             break;
                     }
 
@@ -851,7 +782,7 @@ namespace MarcoBMS.Services.Controllers
 
                     var name = $"{vm.FirstName} {vm.LastName}";
 
-                    await _firebase.SendAttendanceMessageAsync(attendance.ProjectID, name, recordAttendanceDot.Action, attendance.EmployeeId, tenantId);
+                    await _firebase.SendAttendanceMessageAsync(attendance.ProjectID, name, recordAttendanceDot.Action, attendance.EmployeeID, tenantId);
 
                 });
 
@@ -888,7 +819,6 @@ namespace MarcoBMS.Services.Controllers
             // This single query joins ProjectAllocations with Employees and performs a LEFT JOIN with Attendances.
             // This is far more efficient than fetching collections and joining them in memory.
             var query = _context.ProjectAllocations
-                .Include(pa => pa.Project)
                 .Include(pa => pa.Employee)
                     .ThenInclude(e => e!.Organization)
                 .Include(pa => pa.Employee)
@@ -905,12 +835,7 @@ namespace MarcoBMS.Services.Controllers
                 query = query.Where(pa => pa.Employee != null && pa.Employee.OrganizationId == organizationId);
             }
 
-            List<Attendance> lstAttendance = await _context.Attendes
+            List<Attendance> lstAttendance = await _context.Attendes.Where(c => c.ProjectID == projectId && c.AttendanceDate.Date == forDate && c.TenantId == tenantId).ToListAsync();
-                .Include(a => a.RequestedBy)
-                    .ThenInclude(e => e!.JobRole)
-                .Include(a => a.Approver)
-                    .ThenInclude(e => e!.JobRole)
-                .Where(c => c.ProjectID == projectId && c.AttendanceDate.Date == forDate && c.TenantId == tenantId).ToListAsync();
 
             var teamAttendance = await query
                 .AsNoTracking()
@@ -927,21 +852,18 @@ namespace MarcoBMS.Services.Controllers
                         LastName = teamMember.Employee?.LastName,
                         OrganizationName = teamMember.Employee?.Organization?.Name,
                         JobRoleName = teamMember.Employee?.JobRole?.Name,
-                        ProjectId = projectId,
-                        ProjectName = teamMember.Project?.Name
                     };
 
-                    var attendance = lstAttendance.Find(x => x.EmployeeId == teamMember.EmployeeId) ?? new Attendance();
+                    //var member = emp.Where(e => e.Id == teamMember.EmployeeId);
+
+
+                    var attendance = lstAttendance.Find(x => x.EmployeeID == teamMember.EmployeeId) ?? new Attendance();
                     if (attendance != null)
                     {
                         result1.Id = attendance.Id;
                         result1.CheckInTime = attendance.InTime;
                         result1.CheckOutTime = attendance.OutTime;
                         result1.Activity = attendance.Activity;
-                        result1.ApprovedAt = attendance.ApprovedAt;
-                        result1.Approver = _mapper.Map<BasicEmployeeVM>(attendance.Approver);
-                        result1.RequestedAt = attendance.RequestedAt;
-                        result1.RequestedBy = _mapper.Map<BasicEmployeeVM>(attendance.RequestedBy);
                     }
                     return result1;
                 })
@@ -960,21 +882,12 @@ namespace MarcoBMS.Services.Controllers
 
             // This query fetches the employee's project allocation and their attendance in a single trip.
             Attendance lstAttendance = await _context.Attendes
-                .Include(a => a.RequestedBy)
+                    .FirstOrDefaultAsync(c => c.ProjectID == projectId && c.EmployeeID == employeeId && c.AttendanceDate.Date == forDate && c.TenantId == tenantId) ?? new Attendance();
-                    .ThenInclude(e => e!.JobRole)
-                .Include(a => a.Approver)
-                    .ThenInclude(e => e!.JobRole)
-                    .FirstOrDefaultAsync(c => c.ProjectID == projectId && c.EmployeeId == employeeId && c.AttendanceDate.Date == forDate && c.TenantId == tenantId) ?? new Attendance();
 
             var projectAllocationQuery = _context.ProjectAllocations
-                .Include(pa => pa.Project)
                 .Include(pa => pa.Employee)
                     .ThenInclude(e => e!.Organization)
-                .Include(pa => pa.Employee)
+                .Where(pa => pa.ProjectId == projectId && pa.EmployeeId == employeeId && pa.TenantId == tenantId && pa.IsActive);
-                    .ThenInclude(e => e!.JobRole)
-                .Where(pa => pa.EmployeeId == employeeId && pa.TenantId == tenantId && pa.IsActive &&
-                    pa.ProjectId == projectId && pa.Project != null &&
-                    pa.Employee != null && pa.Employee.Organization != null && pa.Employee.JobRole != null);
 
             if (organizationId.HasValue)
             {
@@ -994,15 +907,9 @@ namespace MarcoBMS.Services.Controllers
                     OrganizationName = projectAllocation.Employee?.Organization?.Name,
                     LastName = projectAllocation.Employee?.LastName,
                     JobRoleName = projectAllocation.Employee?.JobRole?.Name,
-                    ProjectId = projectId,
-                    ProjectName = projectAllocation.Project?.Name,
                     CheckInTime = lstAttendance.InTime,
                     CheckOutTime = lstAttendance.OutTime,
-                    Activity = lstAttendance.Activity,
+                    Activity = lstAttendance.Activity
-                    ApprovedAt = lstAttendance.ApprovedAt,
-                    Approver = _mapper.Map<BasicEmployeeVM>(lstAttendance.Approver),
-                    RequestedAt = lstAttendance.RequestedAt,
-                    RequestedBy = _mapper.Map<BasicEmployeeVM>(lstAttendance.RequestedBy)
                 };
                 result.Add(result1);
             }

Marco.Pms.Services/Controllers/AuthController.cs

@@ -7,7 +7,6 @@ using Marco.Pms.Model.Employees;
 using Marco.Pms.Model.Entitlements;
 using Marco.Pms.Model.Utilities;
 using Marco.Pms.Model.ViewModels.Tenant;
-using Marco.Pms.Services.Helpers;
 using MarcoBMS.Services.Helpers;
 using MarcoBMS.Services.Service;
 using Microsoft.AspNetCore.Authorization;
@@ -320,6 +319,36 @@ namespace MarcoBMS.Services.Controllers
                     return Unauthorized(ApiResponse<object>.ErrorResponse("MPIN mismatch", "MPIN did not match", 401));
                 }
 
+                if (!string.IsNullOrWhiteSpace(verifyMPIN.FcmToken))
+                {
+                    var existingFCMTokenMapping = await _context.FCMTokenMappings.Where(ft => ft.FcmToken == verifyMPIN.FcmToken).ToListAsync();
+
+                    if (existingFCMTokenMapping.Any())
+                    {
+                        _context.FCMTokenMappings.RemoveRange(existingFCMTokenMapping);
+                    }
+
+                    var fcmTokenMapping = new FCMTokenMapping
+                    {
+                        EmployeeId = requestEmployee.Id,
+                        FcmToken = verifyMPIN.FcmToken,
+                        ExpiredAt = DateTime.UtcNow.AddDays(6),
+                        TenantId = tenantId
+                    };
+                    _context.FCMTokenMappings.Add(fcmTokenMapping);
+                    _logger.LogInfo("New FCM Token registering for employee {EmployeeId}", requestEmployee.Id);
+                    try
+                    {
+                        await _context.SaveChangesAsync();
+                    }
+                    catch (Exception ex)
+                    {
+                        _logger.LogError(ex, "Exception occured while saving FCM Token for employee {EmployeeId}", requestEmployee.Id);
+                        return StatusCode(500, ApiResponse<object>.ErrorResponse("Internal Error", ex.Message, 500));
+                    }
+
+                }
+
                 // Generate new tokens
                 var jwtToken = _refreshTokenService.GenerateJwtToken(requestEmployee.Email, tenantId, requestEmployee.OrganizationId, _jwtSettings);
                 var refreshToken = await _refreshTokenService.CreateRefreshToken(requestEmployee.ApplicationUserId, tenantId.ToString(), requestEmployee.OrganizationId, _jwtSettings);
@@ -339,100 +368,6 @@ namespace MarcoBMS.Services.Controllers
             }
         }
 
-        [HttpPost("login-otp/v1")]
-        public async Task<IActionResult> LoginWithOTP([FromBody] VerifyOTPDto verifyOTP)
-        {
-            await using var _context = await _dbContextFactory.CreateDbContextAsync();
-            using var scope = _serviceScopeFactory.CreateScope();
-            var _refreshTokenService = scope.ServiceProvider.GetRequiredService<RefreshTokenService>();
-
-            try
-            {
-                // Validate input
-                if (string.IsNullOrWhiteSpace(verifyOTP.Email) ||
-                    string.IsNullOrWhiteSpace(verifyOTP.OTP) ||
-                    verifyOTP.OTP.Length != 4 ||
-                    !verifyOTP.OTP.All(char.IsDigit))
-                {
-                    _logger.LogWarning("OTP login failed - invalid input provided");
-                    return BadRequest(ApiResponse<object>.ErrorResponse("Invalid input", "Please provide a valid 4-digit OTP and Email", 400));
-                }
-
-                // Fetch employee by email
-                var requestEmployee = await _context.Employees
-                    .Include(e => e.ApplicationUser)
-                    .FirstOrDefaultAsync(e => e.Email == verifyOTP.Email && e.IsActive);
-
-                if (requestEmployee == null || string.IsNullOrWhiteSpace(requestEmployee.ApplicationUserId))
-                {
-                    _logger.LogWarning("OTP login failed - user not found for email {Email}", verifyOTP.Email);
-                    return NotFound(ApiResponse<object>.ErrorResponse("User not found", "User not found", 404));
-                }
-
-                Guid userId = Guid.Parse(requestEmployee.ApplicationUserId);
-
-                // Fetch most recent OTP
-                var otpDetails = await _context.OTPDetails
-                    .Where(o => o.UserId == userId)
-                    .OrderByDescending(o => o.TimeStamp)
-                    .FirstOrDefaultAsync();
-
-                if (otpDetails == null)
-                {
-                    _logger.LogWarning("OTP login failed - no OTP found for user {UserId}", userId);
-                    return NotFound(ApiResponse<object>.ErrorResponse("OTP not found", "No OTP was generated for this user", 404));
-                }
-
-                // Validate OTP expiration
-                var validUntil = otpDetails.TimeStamp.AddSeconds(otpDetails.ExpriesInSec);
-                if (DateTime.UtcNow > validUntil || otpDetails.IsUsed)
-                {
-                    _logger.LogWarning("OTP login failed - OTP expired for user {UserId}", userId);
-                    return BadRequest(ApiResponse<object>.ErrorResponse("OTP expired", "The OTP has expired, please request a new one", 400));
-                }
-
-                // Match OTP
-                if (otpDetails.OTP != verifyOTP.OTP)
-                {
-                    _logger.LogWarning("OTP login failed - incorrect OTP entered for user {UserId}", userId);
-                    return Unauthorized(ApiResponse<object>.ErrorResponse("Invalid OTP", "OTP did not match", 401));
-                }
-
-                // Generate access and refresh tokens
-                //var accessToken = _refreshTokenService.GenerateJwtTokenWithOrganization(requestEmployee.ApplicationUser?.UserName, requestEmployee.OrganizationId, _jwtSettings);
-                //var refreshToken = await _refreshTokenService.CreateRefreshTokenWithOrganization(requestEmployee.ApplicationUserId, requestEmployee.OrganizationId, _jwtSettings);
-
-                var tenant = await _context.Tenants.FirstOrDefaultAsync(t => t.OrganizationId == requestEmployee.OrganizationId);
-
-                var accessToken = _refreshTokenService.GenerateJwtToken(requestEmployee.ApplicationUser?.UserName,
-                    tenant?.Id ?? Guid.Empty, requestEmployee.OrganizationId, _jwtSettings);
-
-                var refreshToken = await _refreshTokenService.CreateRefreshToken(requestEmployee.ApplicationUserId,
-                    tenant?.Id.ToString(), requestEmployee.OrganizationId, _jwtSettings);
-
-                // Fetch MPIN token if exists
-                var mpinDetails = await _context.MPINDetails
-                    .FirstOrDefaultAsync(p => p.UserId == userId);
-
-                // Build and return response
-                var response = new
-                {
-                    token = accessToken,
-                    refreshToken,
-                    mpinToken = mpinDetails?.MPINToken
-                };
-                otpDetails.IsUsed = true;
-                await _context.SaveChangesAsync();
-                _logger.LogInfo("OTP login successful for employee {EmployeeId}", requestEmployee.Id);
-                return Ok(ApiResponse<object>.SuccessResponse(response, "User logged in successfully.", 200));
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "An unexpected error occurred during OTP login for email {Email}", verifyOTP.Email ?? string.Empty);
-                return StatusCode(500, ApiResponse<object>.ErrorResponse("Unexpected error", ex.Message, 500));
-            }
-        }
-
         // new login APIs
 
         [HttpPost("login")]
@@ -1009,7 +944,7 @@ namespace MarcoBMS.Services.Controllers
         }
 
         [HttpPost("login-otp")]
-        public async Task<IActionResult> LoginWithOTPAsync([FromBody] VerifyOTPDto verifyOTP)
+        public async Task<IActionResult> LoginWithOTP([FromBody] VerifyOTPDto verifyOTP)
         {
             await using var _context = await _dbContextFactory.CreateDbContextAsync();
             using var scope = _serviceScopeFactory.CreateScope();
@@ -1359,37 +1294,33 @@ namespace MarcoBMS.Services.Controllers
             _logger.LogInfo("Fetching TenantOrgMappings for OrganizationId: {OrganizationId}", organizationId);
 
             // Retrieve all TenantOrgMappings that match the organizationId and have a related Tenant
-            var tenantOrgMappingTask = Task.Run(async () =>
+            var tenantOrganizationMapping = await _context.TenantOrgMappings
-            {
+                .Include(to => to.Tenant)
-                await using var context = await _dbContextFactory.CreateDbContextAsync();
+                    .ThenInclude(t => t!.TenantStatus)
-                return await context.TenantOrgMappings.Where(to => to.OrganizationId == organizationId && to.Tenant != null).Select(to => to.TenantId).ToListAsync();
+                .Include(to => to.Tenant)
-            });
+                    .ThenInclude(t => t!.Industry)
-            var projectTask = Task.Run(async () =>
+                .Where(to => to.OrganizationId == organizationId && to.Tenant != null)
-            {
+                .ToListAsync();
-                await using var context = await _dbContextFactory.CreateDbContextAsync();
-                return await context.Projects.Where(to => to.PromoterId == organizationId || to.PMCId == organizationId).Select(to => to.TenantId).ToListAsync();
-            });
 
-            await Task.WhenAll(tenantOrgMappingTask, projectTask);
+            var tenantList = tenantOrganizationMapping.Select(to => to.Tenant!).ToList();
-
-            var tenantIds = tenantOrgMappingTask.Result;
-
-            tenantIds.AddRange(projectTask.Result);
-
-            tenantIds = tenantIds.Distinct().ToList();
 
             // Additionally fetch the Tenant record associated directly with this OrganizationId if any
-            var tenants = await _context.Tenants
+            var tenant = await _context.Tenants
                 .Include(t => t.Industry)
                 .Include(t => t.TenantStatus)
-                .Where(t => t.OrganizationId == organizationId || tenantIds.Contains(t.Id)).ToListAsync();
+                .FirstOrDefaultAsync(t => t.OrganizationId == organizationId);
+            if (tenant != null)
+            {
+                tenantList.Add(tenant);
+            }
 
-            tenants = tenants.Distinct().ToList();
+
+            tenantList = tenantList.Distinct().ToList();
 
             // Map the tenant entities to TenantListVM view models
-            var response = _mapper.Map<List<TenantListVM>>(tenants);
+            var response = _mapper.Map<List<TenantListVM>>(tenantList);
 
-            _logger.LogInfo("Fetched {Count} tenants for OrganizationId: {OrganizationId}", tenants.Count, organizationId);
+            _logger.LogInfo("Fetched {Count} tenants for OrganizationId: {OrganizationId}", tenantList.Count, organizationId);
             _logger.LogDebug("GetTenantAsync method completed successfully.");
 
             return Ok(ApiResponse<object>.SuccessResponse(response, "Successfully fetched the list of tenant", 200));
@@ -1470,9 +1401,6 @@ namespace MarcoBMS.Services.Controllers
             // Generate and store refresh token
             var refreshToken = await _refreshTokenService.CreateRefreshToken(loggedInEmployee.ApplicationUserId, tenantId.ToString(), loggedInEmployee.OrganizationId, _jwtSettings);
 
-            var _cache = scope.ServiceProvider.GetRequiredService<CacheUpdateHelper>();
-            await _cache.ClearAllEmployeesFromCacheByOnlyEmployeeId(loggedInEmployee.Id);
-
             _logger.LogInfo("Tenant selected and tokens generated for TenantId: {TenantId} and Employee: {EmployeeEmail}", tenantId, loggedInEmployee.Email ?? string.Empty);
 
             // Return success response including tokens

Marco.Pms.Services/Controllers/DashboardController.cs

@@ -235,7 +235,7 @@ namespace Marco.Pms.Services.Controllers
                 int inTodays = await _context.Attendes
                     .Where(a => a.InTime >= today && a.InTime < tomorrow &&
                                  finalProjectIds.Contains(a.ProjectID))
-                    .Select(a => a.EmployeeId)
+                    .Select(a => a.EmployeeID)
                     .Distinct()
                     .CountAsync();
 
@@ -354,7 +354,7 @@ namespace Marco.Pms.Services.Controllers
             Guid tenantId = _userHelper.GetTenantId();
             var LoggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
 
-            var attendance = await _context.Attendes.Where(a => a.EmployeeId == LoggedInEmployee.Id && a.TenantId == tenantId).ToListAsync();
+            var attendance = await _context.Attendes.Where(a => a.EmployeeID == LoggedInEmployee.Id && a.TenantId == tenantId).ToListAsync();
             if (attendance.Any())
             {
                 var pendingRegularization = attendance.Where(a => a.Activity == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE).ToList().Count;
@@ -395,12 +395,12 @@ namespace Marco.Pms.Services.Controllers
             var employeeIds = projectAllocation.Select(p => p.EmployeeId).Distinct().ToList();
             List<Employee>? employees = await _context.Employees.Where(e => employeeIds.Contains(e.Id)).ToListAsync();
 
-            var attendances = await _context.Attendes.Where(a => employeeIds.Contains(a.EmployeeId) && a.ProjectID == projectId && a.InTime.HasValue && a.InTime.Value.Date == currentDate.Date).ToListAsync();
+            var attendances = await _context.Attendes.Where(a => employeeIds.Contains(a.EmployeeID) && a.ProjectID == projectId && a.InTime.HasValue && a.InTime.Value.Date == currentDate.Date).ToListAsync();
             List<EmployeeAttendanceVM> employeeAttendanceVMs = new List<EmployeeAttendanceVM>();
             foreach (var attendance in attendances)
             {
 
-                Employee? employee = employees.FirstOrDefault(e => e.Id == attendance.EmployeeId);
+                Employee? employee = employees.FirstOrDefault(e => e.Id == attendance.EmployeeID);
                 if (employee != null)
                 {
                     EmployeeAttendanceVM employeeAttendanceVM = new EmployeeAttendanceVM
@@ -579,7 +579,7 @@ namespace Marco.Pms.Services.Controllers
                         .ToList();
 
                     int presentCount = attendances
-                        .Count(a => employeeIds.Contains(a.EmployeeId) && a.InTime!.Value.Date == date);
+                        .Count(a => employeeIds.Contains(a.EmployeeID) && a.InTime!.Value.Date == date);
 
                     overviewList.Add(new AttendanceOverviewVM
                     {

Marco.Pms.Services/Controllers/DirectoryController.cs

@@ -33,11 +33,11 @@ namespace Marco.Pms.Services.Controllers
         #region =================================================================== Contact Get APIs ===================================================================
 
         [HttpGet("list")]
-        public async Task<IActionResult> GetContactList([FromQuery] string? searchString, [FromQuery] string? filter, [FromQuery] Guid? projectId, [FromQuery] bool active = true,
+        public async Task<IActionResult> GetContactList([FromQuery] string? search, [FromQuery] string? filter, [FromQuery] Guid? projectId, [FromQuery] bool active = true,
             [FromQuery] int pageNumber = 1, [FromQuery] int pageSize = 20)
         {
             var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-            var response = await _directoryService.GetListOfContactsAsync(search: searchString, filter: filter, projectId: projectId, active: active, pageSize: pageSize, pageNumber: pageNumber, tenantId, loggedInEmployee);
+            var response = await _directoryService.GetListOfContactsAsync(search: search, filter: filter, projectId: projectId, active: active, pageSize: pageSize, pageNumber: pageNumber, tenantId, loggedInEmployee);
 
             return StatusCode(response.StatusCode, response);
 

Marco.Pms.Services/Controllers/EmployeeController.cs

@@ -19,7 +19,6 @@ using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Identity;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.SignalR;
-using Microsoft.CodeAnalysis;
 using Microsoft.EntityFrameworkCore;
 using System.Data;
 using System.Net;
@@ -32,9 +31,9 @@ namespace MarcoBMS.Services.Controllers
 
     public class EmployeeController : ControllerBase
     {
-        private readonly IDbContextFactory<ApplicationDbContext> _dbContextFactory;
+
         private readonly ApplicationDbContext _context;
-        private readonly IServiceScopeFactory _serviceScopeFactory;
+        private readonly IServiceScopeFactory _serviceScope;
         private readonly UserManager<ApplicationUser> _userManager;
         private readonly IEmailSender _emailSender;
         private readonly EmployeeHelper _employeeHelper;
@@ -47,11 +46,9 @@ namespace MarcoBMS.Services.Controllers
         private readonly IMapper _mapper;
         private readonly IProjectServices _projectServices;
         private readonly Guid tenantId;
-        private readonly Guid organizationId;
 
 
-        public EmployeeController(IDbContextFactory<ApplicationDbContext> dbContextFactory,
+        public EmployeeController(IServiceScopeFactory serviceScope,
-            IServiceScopeFactory serviceScopeFactory,
             UserManager<ApplicationUser> userManager,
             IEmailSender emailSender,
             ApplicationDbContext context,
@@ -65,8 +62,7 @@ namespace MarcoBMS.Services.Controllers
             IMapper mapper,
             GeneralHelper generalHelper)
         {
-            _dbContextFactory = dbContextFactory ?? throw new ArgumentNullException(nameof(dbContextFactory));
+            _serviceScope = serviceScope;
-            _serviceScopeFactory = serviceScopeFactory;
             _context = context;
             _userManager = userManager;
             _emailSender = emailSender;
@@ -80,7 +76,6 @@ namespace MarcoBMS.Services.Controllers
             _projectServices = projectServices;
             _mapper = mapper;
             tenantId = _userHelper.GetTenantId();
-            organizationId = _userHelper.GetCurrentOrganizationId();
         }
 
         [HttpGet]
@@ -122,93 +117,9 @@ namespace MarcoBMS.Services.Controllers
             }
         }
 
-        [HttpGet("list/organizations/{projectId}")]
+        [HttpGet]
-        public async Task<IActionResult> GetEmployeesByProjectAsync(Guid projectId, [FromQuery] string searchString, [FromQuery] Guid? organizationId)
+        [Route("list/{projectid?}")]
-        {
+        public async Task<IActionResult> GetEmployeesByProject(Guid? projectid, [FromQuery] bool ShowInactive)
-            try
-            {
-                // Get the currently logged-in employee information
-                var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-
-                var projectTask = Task.Run(async () =>
-                {
-                    await using var context = await _dbContextFactory.CreateDbContextAsync();
-                    return await context.Projects.FirstOrDefaultAsync(p => p.Id == projectId && p.TenantId == tenantId);
-                });
-
-                var tenantTask = Task.Run(async () =>
-                {
-                    await using var context = await _dbContextFactory.CreateDbContextAsync();
-                    return await context.Tenants.FirstOrDefaultAsync(t => t.Id == tenantId);
-                });
-
-                await Task.WhenAll(projectTask, tenantTask);
-
-                var project = projectTask.Result;
-                var tenant = tenantTask.Result;
-
-                if (project == null || tenant == null)
-                {
-                    _logger.LogWarning("Project {ProjectId} not found in database for tenant {TenantId}", projectId, tenantId);
-                    return NotFound(ApiResponse<object>.ErrorResponse("Project not found", "Project not found", 404));
-                }
-                // Check if the logged-in employee has permission for the requested project
-                var hasProjectPermission = await _permission.HasProjectPermission(loggedInEmployee, projectId);
-                if (!hasProjectPermission)
-                {
-                    _logger.LogWarning("User {EmployeeId} attempts to get employees for project {ProjectId} without permission", loggedInEmployee.Id, projectId);
-                    return StatusCode(403, ApiResponse<object>.ErrorResponse("Access denied", "User does not have access to view the employees for this project", 403));
-                }
-
-                var organizationQuery = _context.ProjectOrgMappings
-                    .Include(po => po.ProjectService)
-                    .Where(po => po.ProjectService != null && po.ProjectService.ProjectId == projectId);
-
-                if (loggedInEmployee.OrganizationId != project.PMCId && loggedInEmployee.OrganizationId != project.PromoterId && loggedInEmployee.OrganizationId != tenant.OrganizationId)
-                {
-                    organizationQuery = organizationQuery.Where(po => po.ParentOrganizationId == loggedInEmployee.OrganizationId || po.OrganizationId == loggedInEmployee.OrganizationId);
-                }
-
-                var organizationIds = await organizationQuery.Select(po => po.OrganizationId).ToListAsync();
-
-                if (loggedInEmployee.OrganizationId == project.PMCId || loggedInEmployee.OrganizationId == project.PromoterId || loggedInEmployee.OrganizationId == tenant.OrganizationId)
-                {
-                    organizationIds.Add(project.PMCId);
-                    organizationIds.Add(project.PromoterId);
-                    organizationIds.Add(tenant.OrganizationId);
-                }
-
-                // Fetch employees allocated to the project matching the search criteria
-                var employeesQuery = _context.Employees
-                    .AsNoTracking()  // Improves performance by disabling change tracking for read-only query
-                    .Include(e => e.JobRole)
-                    .Where(e => (e.FirstName + " " + e.LastName).Contains(searchString) && organizationIds.Contains(e.OrganizationId));
-
-                if (organizationId.HasValue)
-                {
-                    employeesQuery = employeesQuery.Where(e => e.OrganizationId == organizationId);
-                }
-
-                var employees = await employeesQuery
-                    .ToListAsync();
-
-                var result = employees.Select(e => _mapper.Map<EmployeeVM>(e)).Distinct().ToList();
-                _logger.LogInfo("Employees fetched for project {ProjectId} by user {EmployeeId}. Count: {Count}", projectId, loggedInEmployee.Id, employees.Count);
-
-                // Return the employee list wrapped in a successful API response
-                return Ok(ApiResponse<object>.SuccessResponse(result, "Employee list fetched successfully", 200));
-            }
-            catch (Exception ex)
-            {
-                // Log the exception and return a 500 status code with error message
-                _logger.LogError(ex, "Error occurred while fetching employees for project {ProjectId}", projectId);
-                return StatusCode(500, ApiResponse<object>.ErrorResponse("Internal server error", "An unexpected error occurred", 500));
-            }
-        }
-
-
-        [HttpGet("list/{projectId?}")]
-        public async Task<IActionResult> GetEmployeesByProjectAsync(Guid? projectId, [FromQuery] bool showInactive = false)
         {
             // Step 1: Validate incoming request model state
             if (!ModelState.IsValid)
@@ -222,117 +133,61 @@ namespace MarcoBMS.Services.Controllers
                 return BadRequest(ApiResponse<object>.ErrorResponse("Invalid data", errors, 400));
             }
 
-            List<EmployeeVM> result = new List<EmployeeVM>();
+            // Step 2: Get logged-in employee
-            try
+            var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
+            _logger.LogInfo("GetEmployeesByProject called by EmployeeId: {EmployeeId}, ProjectId: {ProjectId}, ShowInactive: {ShowInactive}",
+                loggedInEmployee.Id, projectid ?? Guid.Empty, ShowInactive);
+
+            // Step 3: Fetch project access and permissions
+            var projectIds = await _projectServices.GetMyProjectIdsAsync(tenantId, loggedInEmployee);
+
+            var hasViewAllEmployeesPermission = await _permission.HasPermission(PermissionsMaster.ViewAllEmployees, loggedInEmployee.Id);
+            var hasViewTeamMembersPermission = await _permission.HasPermission(PermissionsMaster.ViewTeamMembers, loggedInEmployee.Id);
+
+            List<EmployeeVM> result = new();
+
+            // Step 4: Determine access level and fetch employees accordingly
+            if (hasViewAllEmployeesPermission || projectid != null)
             {
-                // Dependency injection scope for services
+                result = await _employeeHelper.GetEmployeeByProjectId(tenantId, projectid, ShowInactive);
-                using var scope = _serviceScopeFactory.CreateScope();
+                _logger.LogInfo("Employee list fetched using full access or specific project.");
+            }
+            else if (hasViewTeamMembersPermission && !ShowInactive)
+            {
+                var employeeIds = await _context.ProjectAllocations
+                    .Where(pa => projectIds.Contains(pa.ProjectId) && pa.IsActive && pa.TenantId == tenantId)
+                    .Select(pa => pa.EmployeeId)
+                    .Distinct()
+                    .ToListAsync();
 
-                // Step 2: Get logged-in employee details
+                var employees = await _context.Employees
-                var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
+                        .Include(fp => fp.JobRole)
-                _logger.LogInfo("GetEmployeesByProject called. EmployeeId: {EmployeeId}, ProjectId: {ProjectId}, showInactive: {ShowInactive}",
+                        .Where(e => employeeIds.Contains(e.Id) && e.JobRole != null && e.IsActive && e.TenantId == tenantId)
-                    loggedInEmployee.Id, projectId ?? Guid.Empty, showInactive);
-
-                // Step 3: Fetch permissions concurrently
-                var viewAllTask = Task.Run(async () =>
-                {
-                    var _permission = scope.ServiceProvider.GetRequiredService<PermissionServices>();
-                    return await _permission.HasPermission(PermissionsMaster.ViewAllEmployees, loggedInEmployee.Id);
-                });
-                var viewTeamTask = Task.Run(async () =>
-                {
-                    var _permission = scope.ServiceProvider.GetRequiredService<PermissionServices>();
-                    return await _permission.HasPermission(PermissionsMaster.ViewTeamMembers, loggedInEmployee.Id);
-                });
-
-                await Task.WhenAll(viewAllTask, viewTeamTask);
-
-                var hasViewAllEmployeesPermission = viewAllTask.Result;
-                var hasViewTeamMembersPermission = viewTeamTask.Result;
-
-                List<Employee> employees = new List<Employee>();
-
-                // Step 4: Query based on permission
-                if (hasViewAllEmployeesPermission && !projectId.HasValue)
-                {
-                    // OrganizationId needs to be retrieved from loggedInEmployee or context based on your app's structure
-                    var employeeQuery = _context.Employees
-                        .AsNoTracking() // Optimize EF query for read-only operation[web:1][web:13][web:18]
-                        .Include(e => e.JobRole)
-                        .Where(e => e.OrganizationId == organizationId);
-
-                    employeeQuery = showInactive
-                        ? employeeQuery.Where(e => !e.IsActive)
-                        : employeeQuery.Where(e => e.IsActive);
-
-                    employees = await employeeQuery.ToListAsync();
-                    _logger.LogInfo("Employee list fetched with full access. Count: {Count}", employees.Count);
-                }
-                else if (hasViewTeamMembersPermission && !showInactive && !projectId.HasValue)
-                {
-                    // Only active team members with limited permission
-                    var projectIds = await _projectServices.GetMyProjectIdsAsync(tenantId, loggedInEmployee);
-
-                    employees = await _context.ProjectAllocations
-                        .AsNoTracking()
-                        .Include(pa => pa.Employee)
-                            .ThenInclude(e => e!.JobRole)
-                        .Where(pa =>
-                            projectIds.Contains(pa.ProjectId)
-                            && pa.IsActive
-                            && pa.Employee != null
-                            && pa.Employee.IsActive
-                            && pa.TenantId == tenantId)
-                        .Select(pa => pa.Employee!)
                         .Distinct()
                         .ToListAsync();
 
-                    _logger.LogInfo("Employee list fetched with limited access (active only). Count: {Count}", employees.Count);
+                result = employees.Select(e => e.ToEmployeeVMFromEmployee()).ToList();
-                }
-
-                // If a specific projectId is provided, override employee fetching to ensure strict project context
-                if (projectId.HasValue)
-                {
-                    employees = await _context.ProjectAllocations
-                        .AsNoTracking()
-                        .Include(pa => pa.Employee)
-                            .ThenInclude(e => e!.JobRole)
-                        .Where(pa =>
-                            pa.ProjectId == projectId
-                            && pa.IsActive
-                            && pa.Employee != null
-                            && pa.Employee.IsActive
-                            && pa.TenantId == tenantId)
-                        .Select(pa => pa.Employee!)
-                        .Distinct()
-                        .ToListAsync();
-
-                    _logger.LogInfo("Employee list fetched for specific project. ProjectId: {ProjectId}. Count: {Count}",
-                        projectId, employees.Count);
-                }
-
-                // Step 5: Map to view model
-                result = employees.Select(e => _mapper.Map<EmployeeVM>(e)).Distinct().ToList();
-
-                _logger.LogInfo("Employees successfully fetched. EmployeeId: {EmployeeId} for ProjectId: {ProjectId}. Final Count: {Count}",
-                    loggedInEmployee.Id, projectId ?? Guid.Empty, result.Count);
 
+                _logger.LogInfo("Employee list fetched using limited access (active only).");
+            }
+            else
+            {
+                _logger.LogWarning("Access denied for EmployeeId: {EmployeeId} - insufficient permissions.", loggedInEmployee.Id);
                 return Ok(ApiResponse<object>.SuccessResponse(result, "Filter applied.", 200));
             }
-            catch (Exception ex)
+
-            {
+            // Step 5: Log and return results
-                // Step 6: Error logging and response[web:6]
+            _logger.LogInfo("Employees fetched successfully by EmployeeId: {EmployeeId} for ProjectId: {ProjectId}. Count: {Count}",
-                _logger.LogError(ex, "Exception occurred while getting the list of employees");
+                loggedInEmployee.Id, projectid ?? Guid.Empty, result.Count);
-                return StatusCode(500, ApiResponse<object>.ErrorResponse("Internal server error. Please try again later.", null, 500));
+
-            }
+            return Ok(ApiResponse<object>.SuccessResponse(result, "Filter applied.", 200));
         }
 
-
         [HttpGet("basic")]
-        public async Task<IActionResult> GetEmployeesByProjectBasic(Guid? projectId, [FromQuery] string? searchString, [FromQuery] bool allEmployee)
+        public async Task<IActionResult> GetEmployeesByProjectBasic(Guid? projectId, [FromQuery] string? searchString)
         {
             var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-            var employeeQuery = _context.Employees.Where(e => e.IsActive);
+            var employeeQuery = _context.Employees.Where(e => e.TenantId == tenantId);
             if (projectId != null && projectId != Guid.Empty)
             {
                 var hasProjectPermission = await _permission.HasProjectPermission(loggedInEmployee, projectId.Value);
@@ -344,26 +199,13 @@ namespace MarcoBMS.Services.Controllers
                 var employeeIds = await _context.ProjectAllocations.Where(pa => pa.ProjectId == projectId && pa.IsActive && pa.TenantId == tenantId).Select(p => p.EmployeeId).ToListAsync();
                 employeeQuery = employeeQuery.Where(e => employeeIds.Contains(e.Id));
             }
-            else
-            {
-                employeeQuery = employeeQuery.Where(e => e.OrganizationId == organizationId);
-            }
             if (!string.IsNullOrWhiteSpace(searchString))
             {
                 var searchStringLower = searchString.ToLower();
                 employeeQuery = employeeQuery.Where(e => (e.FirstName + " " + e.LastName).ToLower().Contains(searchStringLower));
             }
 
-            var query = employeeQuery.OrderBy(e => e.FirstName);
+            var response = await employeeQuery.Take(10).Select(e => _mapper.Map<BasicEmployeeVM>(e)).ToListAsync();
-
-            if (!allEmployee)
-            {
-                query = (IOrderedQueryable<Employee>)query.Take(10);
-            }
-
-            var response = await query
-                .Select(e => _mapper.Map<BasicEmployeeVM>(e))
-                .ToListAsync();
             return Ok(ApiResponse<object>.SuccessResponse(response, $"{response.Count} records of employees fetched successfully", 200));
         }
 
@@ -473,7 +315,7 @@ namespace MarcoBMS.Services.Controllers
         }
 
 
-        [HttpPost("old/manage")]
+        [HttpPost("manage")]
         public async Task<IActionResult> CreateUser([FromBody] CreateUserDto model)
         {
             Guid tenantId = _userHelper.GetTenantId();
@@ -606,203 +448,6 @@ namespace MarcoBMS.Services.Controllers
             return Ok(ApiResponse<object>.SuccessResponse("Success.", responsemessage, 200));
         }
 
-        [HttpPost("manage")]
-        public async Task<IActionResult> CreateEmployeeAsync([FromBody] CreateUserDto model)
-        {
-            // Correlation and context capture for logs
-            var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-
-            {
-                if (model == null)
-                {
-                    _logger.LogWarning("Model is null in CreateEmployeeAsync");
-                    return BadRequest(ApiResponse<object>.ErrorResponse("Invalid payload", "Request body is required", 400));
-                }
-
-                // Basic validation
-                if (model.HasApplicationAccess && string.IsNullOrWhiteSpace(model.Email))
-                {
-                    _logger.LogWarning("Application access requested but email is missing");
-                    return BadRequest(ApiResponse<object>.ErrorResponse("Invalid email", "Application users must have a valid email", 400));
-                }
-
-                await using var transaction = await _context.Database.BeginTransactionAsync();
-                try
-                {
-                    // Load existing employee if updating, constrained by organization scope
-                    Employee? existingEmployee = null;
-                    if (model.Id.HasValue && model.Id.Value != Guid.Empty)
-                    {
-                        existingEmployee = await _context.Employees
-                            .FirstOrDefaultAsync(e => e.Id == model.Id);
-                        if (existingEmployee == null)
-                        {
-                            _logger.LogInfo("Employee not found for update. Id={EmployeeId}", model.Id);
-                            return NotFound(ApiResponse<object>.ErrorResponse("Employee not found", "Employee not found in database", 404));
-                        }
-                    }
-
-                    // Identity user creation path (only if needed)
-                    ApplicationUser? identityUserToCreate = null;
-                    ApplicationUser? createdIdentityUser = null;
-
-                    if (model.HasApplicationAccess)
-                    {
-                        // Only attempt identity resolution/creation if email supplied and either:
-                        // - Creating new employee, or
-                        // - Updating but existing employee does not have ApplicationUserId
-                        var needsIdentity = string.IsNullOrWhiteSpace(existingEmployee?.ApplicationUserId);
-                        if (needsIdentity && !string.IsNullOrWhiteSpace(model.Email))
-                        {
-                            var existingUser = await _userManager.FindByEmailAsync(model.Email);
-                            if (existingUser == null)
-                            {
-                                // Seat check only when provisioning a new identity user
-                                var isSeatsAvailable = await _generalHelper.CheckSeatsRemainingAsync(tenantId);
-                                if (!isSeatsAvailable)
-                                {
-                                    _logger.LogWarning("Maximum users reached for Tenant {TenantId}. Cannot create identity user for {Email}", tenantId, model.Email);
-                                    return BadRequest(ApiResponse<object>.ErrorResponse(
-                                        "Maximum number of users reached. Cannot add new user",
-                                        "Maximum number of users reached. Cannot add new user", 400));
-                                }
-
-                                identityUserToCreate = new ApplicationUser
-                                {
-                                    UserName = model.Email,
-                                    Email = model.Email,
-                                    EmailConfirmed = true
-                                };
-                            }
-                            else
-                            {
-                                // If identity exists, re-use it; do not re-create
-                                createdIdentityUser = existingUser;
-                            }
-                        }
-                    }
-
-                    // For create path: enforce uniqueness of employee email if applicable to business rules
-                    // Consider adding a unique filtered index: (OrganizationId, Email) WHERE Email IS NOT NULL
-                    if (!model.Id.HasValue || model.Id == Guid.Empty)
-                    {
-                        if (!string.IsNullOrWhiteSpace(model.Email))
-                        {
-                            var emailExists = await _context.Employees
-                                .AnyAsync(e => e.Email == model.Email);
-                            if (emailExists)
-                            {
-                                _logger.LogInfo("Employee email already exists. Email={Email}", model.Email);
-                                return StatusCode(403, ApiResponse<object>.ErrorResponse(
-                                    "Employee with email already exists",
-                                    "Employee with this email already exists", 403));
-                            }
-                        }
-                    }
-
-                    // Create identity user if needed
-                    if (identityUserToCreate != null && !string.IsNullOrWhiteSpace(identityUserToCreate.Email))
-                    {
-                        var createResult = await _userManager.CreateAsync(identityUserToCreate, "User@123");
-                        if (!createResult.Succeeded)
-                        {
-                            _logger.LogWarning("Failed to create identity user for {Email}. Errors={Errors}",
-                                identityUserToCreate.Email,
-                                string.Join(", ", createResult.Errors.Select(e => $"{e.Code}:{e.Description}")));
-                            return BadRequest(ApiResponse<object>.ErrorResponse("Failed to create user", createResult.Errors, 400));
-                        }
-
-                        createdIdentityUser = identityUserToCreate;
-                        _logger.LogInfo("Identity user created. IdentityUserId={UserId}, Email={Email}",
-                            createdIdentityUser.Id, createdIdentityUser.Email);
-                    }
-
-
-                    Guid employeeId;
-                    EmployeeVM employeeVM;
-                    string responseMessage;
-
-                    if (existingEmployee != null)
-                    {
-                        // Update flow
-                        _mapper.Map(model, existingEmployee);
-
-                        if (createdIdentityUser != null && !string.IsNullOrWhiteSpace(createdIdentityUser.Email))
-                        {
-                            existingEmployee.ApplicationUserId = createdIdentityUser.Id;
-                            await SendResetIfApplicableAsync(createdIdentityUser, existingEmployee.FirstName ?? "User");
-                        }
-
-                        await _context.SaveChangesAsync();
-
-                        employeeId = existingEmployee.Id;
-                        employeeVM = _mapper.Map<EmployeeVM>(existingEmployee);
-                        responseMessage = "Employee Updated Successfully";
-
-                        _logger.LogInfo("Employee updated. EmployeeId={EmployeeId}, Org={OrgId}", employeeId, existingEmployee.OrganizationId);
-                    }
-                    else
-                    {
-                        // Create flow
-                        var newEmployee = _mapper.Map<Employee>(model);
-                        newEmployee.IsSystem = false;
-                        newEmployee.IsActive = true;
-                        newEmployee.IsPrimary = false;
-
-                        if (createdIdentityUser != null && !string.IsNullOrWhiteSpace(createdIdentityUser.Email))
-                        {
-                            newEmployee.ApplicationUserId = createdIdentityUser.Id;
-                            await SendResetIfApplicableAsync(createdIdentityUser, newEmployee.FirstName ?? "User");
-                        }
-
-                        await _context.Employees.AddAsync(newEmployee);
-                        await _context.SaveChangesAsync();
-
-                        employeeId = newEmployee.Id;
-                        employeeVM = _mapper.Map<EmployeeVM>(newEmployee);
-                        responseMessage = "Employee Created Successfully";
-
-                        _logger.LogInfo("Employee created. EmployeeId={EmployeeId}, Org={OrgId}", employeeId, newEmployee.OrganizationId);
-                    }
-
-                    await transaction.CommitAsync();
-
-                    // SignalR notification
-                    var notification = new
-                    {
-                        LoggedInUserId = loggedInEmployee.Id,
-                        Keyword = "Employee",
-                        EmployeeId = employeeId
-                    };
-
-                    // Consider broadcasting to tenant/organization group instead of Clients.All to avoid cross-tenant leaks:
-                    // await _signalR.Clients.Group($"org:{model.OrganizationId}").SendAsync("NotificationEventHandler", notification);
-                    await _signalR.Clients.All.SendAsync("NotificationEventHandler", notification);
-
-                    _logger.LogInfo("Notification broadcasted for EmployeeId={EmployeeId}", employeeId);
-
-                    return Ok(ApiResponse<object>.SuccessResponse(employeeVM, responseMessage, 200));
-                }
-                catch (DbUpdateException dbEx)
-                {
-                    await transaction.RollbackAsync();
-                    _logger.LogError(dbEx, "Database exception occurred while managing employee");
-                    return StatusCode(500, ApiResponse<object>.ErrorResponse(
-                        "Internal exception occurred",
-                        "Internal database exception has occurred", 500));
-                }
-                catch (Exception ex)
-                {
-                    await transaction.RollbackAsync();
-                    _logger.LogError(ex, "Unhandled exception occurred while managing employee");
-                    return StatusCode(500, ApiResponse<object>.ErrorResponse(
-                        "Internal exception occurred",
-                        "Internal exception has occurred", 500));
-                }
-            }
-        }
-
-
         [HttpPost("manage-mobile")]
         public async Task<IActionResult> CreateUserMoblie([FromBody] MobileUserManageDto model)
         {
@@ -882,205 +527,10 @@ namespace MarcoBMS.Services.Controllers
             }
         }
 
-        [HttpPost("app/manage")]
-        public async Task<IActionResult> CreateUserMobileAsync([FromBody] MobileUserManageDto model)
-        {
-            var loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-            if (tenantId == Guid.Empty)
-            {
-                _logger.LogWarning("Tenant resolution failed in CreateUserMobile"); // structured warning
-                return StatusCode(403, ApiResponse<object>.ErrorResponse("Unauthorized tenant context", "Unauthorized", 403));
-            }
-
-            if (model is null)
-            {
-                _logger.LogWarning("Null payload in CreateUserMobile for Tenant {TenantId}", tenantId); // validation log
-                return BadRequest(ApiResponse<object>.ErrorResponse("Invalid data", "Invalid data", 400));
-            }
-
-            if (string.IsNullOrWhiteSpace(model.FirstName) || string.IsNullOrWhiteSpace(model.PhoneNumber))
-            {
-                _logger.LogWarning("Missing required fields FirstName/Phone for Tenant {TenantId}", tenantId); // validation log
-                return BadRequest(ApiResponse<object>.ErrorResponse("First name and phone number are required.", "Required fields missing", 400));
-            }
-
-            // Strict Base64 parse
-            byte[]? imageBytes = null;
-            if (!string.IsNullOrWhiteSpace(model.ProfileImage))
-            {
-                try
-                {
-                    imageBytes = Convert.FromBase64String(model.ProfileImage);
-                }
-                catch (FormatException ex)
-                {
-                    _logger.LogError(ex, "Invalid base64 image in CreateUserMobile for Tenant {TenantId}", tenantId); // input issue
-                    return BadRequest(ApiResponse<object>.ErrorResponse("Invalid image format.", "Invalid image", 400));
-                }
-            }
-            if (model.Id == null || model.Id == Guid.Empty)
-            {
-                var emailExists = await _context.Employees
-                .AnyAsync(e => e.Email == model.Email && e.OrganizationId == model.OrganizationId);
-
-                if (emailExists && !string.IsNullOrWhiteSpace(model.Email))
-                {
-                    _logger.LogInfo("Employee email already exists in org. Email={Email}, Org={OrgId}", model.Email, model.OrganizationId);
-                    return StatusCode(409, ApiResponse<object>.ErrorResponse("Employee with email already exists", "Employee with this email already exists", 409));
-                }
-
-                // Create path: map only allowed fields
-                var employee = new Employee
-                {
-                    Id = Guid.NewGuid(),
-                    TenantId = tenantId,
-                    FirstName = model.FirstName.Trim(),
-                    LastName = model.LastName?.Trim(),
-                    Email = model.Email,
-                    Gender = model.Gender,
-                    PhoneNumber = model.PhoneNumber,
-                    JoiningDate = model.JoiningDate,
-                    JobRoleId = model.JobRoleId,
-                    Photo = imageBytes,
-                    OrganizationId = model.OrganizationId,
-                    HasApplicationAccess = model.HasApplicationAccess,
-                };
-
-                if (!string.IsNullOrWhiteSpace(model.Email) && model.HasApplicationAccess)
-                {
-                    var existingUser = await _userManager.FindByEmailAsync(model.Email);
-                    if (existingUser == null)
-                    {
-                        existingUser = new ApplicationUser
-                        {
-                            UserName = model.Email,
-                            Email = model.Email,
-                            EmailConfirmed = true
-                        };
-                        var createResult = await _userManager.CreateAsync(existingUser, "User@123");
-                        if (!createResult.Succeeded)
-                        {
-                            _logger.LogWarning("Failed to create identity user for {Email}. Errors={Errors}",
-                                existingUser.Email,
-                                string.Join(", ", createResult.Errors.Select(e => $"{e.Code}:{e.Description}")));
-                            return BadRequest(ApiResponse<object>.ErrorResponse("Failed to create user", createResult.Errors, 400));
-                        }
-
-                        await SendResetIfApplicableAsync(existingUser, employee.FirstName ?? "User");
-                        employee.ApplicationUserId = existingUser.Id;
-                    }
-                }
-
-                _context.Employees.Add(employee);
-                await _context.SaveChangesAsync();
-
-                var employeeVM = _mapper.Map<EmployeeVM>(employee);
-
-                var notification = new
-                {
-                    LoggedInUserId = loggedInEmployee?.Id,
-                    Keyword = "Employee",
-                    EmployeeId = employee.Id
-                };
-
-                // Consider broadcasting to tenant/organization group instead of Clients.All to avoid cross-tenant leaks:
-                // await _signalR.Clients.Group($"org:{model.OrganizationId}").SendAsync("NotificationEventHandler", notification);
-                await _signalR.Clients.All.SendAsync("NotificationEventHandler", notification);
-
-                _logger.LogInfo("Employee {EmployeeId} created in Tenant {TenantId}", employee.Id, tenantId); // success
-                return Ok(ApiResponse<object>.SuccessResponse(employeeVM, "Employee created successfully", 200));
-            }
-            else
-            {
-                // Update path: fetch scoped to tenant
-                var employeeId = model.Id.Value;
-                var existingEmployee = await _context.Employees
-                    .FirstOrDefaultAsync(e => e.Id == employeeId); // tenant-safe lookup
-
-                if (existingEmployee is null)
-                {
-                    _logger.LogWarning("Update attempted for missing Employee {EmployeeId} in Tenant {TenantId}", employeeId, tenantId); // not found
-                    return NotFound(ApiResponse<object>.ErrorResponse("Employee not found", "Not found", 404));
-                }
-
-                // Update allowed fields only
-                existingEmployee.FirstName = model.FirstName.Trim();
-                existingEmployee.LastName = model.LastName?.Trim();
-                existingEmployee.Gender = model.Gender;
-                existingEmployee.PhoneNumber = model.PhoneNumber;
-                existingEmployee.JoiningDate = model.JoiningDate;
-                existingEmployee.JobRoleId = model.JobRoleId;
-                existingEmployee.OrganizationId = model.OrganizationId;
-                existingEmployee.HasApplicationAccess = model.HasApplicationAccess;
-
-                if (string.IsNullOrWhiteSpace(existingEmployee.Email) && !string.IsNullOrWhiteSpace(model.Email))
-                {
-                    var emailExists = await _context.Employees
-                        .AnyAsync(e => e.Email == model.Email);
-
-                    if (emailExists)
-                    {
-                        _logger.LogInfo("Employee email already exists in org. Email={Email}, Org={OrgId}", model.Email, model.OrganizationId);
-                        return StatusCode(409, ApiResponse<object>.ErrorResponse("Employee with email already exists", "Employee with this email already exists", 409));
-                    }
-                    existingEmployee.Email = model.Email;
-                }
-
-                if (model.HasApplicationAccess && !string.IsNullOrWhiteSpace(model.Email) && string.IsNullOrWhiteSpace(existingEmployee.ApplicationUserId))
-                {
-                    var existingUser = await _userManager.FindByEmailAsync(model.Email);
-                    if (existingUser == null)
-                    {
-                        existingUser = new ApplicationUser
-                        {
-                            UserName = model.Email,
-                            Email = model.Email,
-                            EmailConfirmed = true
-                        };
-                        var createResult = await _userManager.CreateAsync(existingUser, "User@123");
-                        if (!createResult.Succeeded)
-                        {
-                            _logger.LogWarning("Failed to create identity user for {Email}. Errors={Errors}",
-                                existingUser.Email,
-                                string.Join(", ", createResult.Errors.Select(e => $"{e.Code}:{e.Description}")));
-                            return BadRequest(ApiResponse<object>.ErrorResponse("Failed to create user", createResult.Errors, 400));
-                        }
-
-                        await SendResetIfApplicableAsync(existingUser, existingEmployee.FirstName ?? "User");
-                        existingEmployee.ApplicationUserId = existingUser.Id;
-                    }
-                }
-
-                if (imageBytes != null)
-                {
-                    existingEmployee.Photo = imageBytes;
-                }
-
-                await _context.SaveChangesAsync();
-
-                var employeeVM = _mapper.Map<EmployeeVM>(existingEmployee);
-
-                var notification = new
-                {
-                    LoggedInUserId = loggedInEmployee?.Id,
-                    Keyword = "Employee",
-                    EmployeeId = employeeId
-                };
-
-                // Consider broadcasting to tenant/organization group instead of Clients.All to avoid cross-tenant leaks:
-                // await _signalR.Clients.Group($"org:{model.OrganizationId}").SendAsync("NotificationEventHandler", notification);
-                await _signalR.Clients.All.SendAsync("NotificationEventHandler", notification);
-
-                _logger.LogInfo("Employee {EmployeeId} updated in Tenant {TenantId}", existingEmployee.Id, tenantId); // success
-                return Ok(ApiResponse<object>.SuccessResponse(employeeVM, "Employee updated successfully", 200));
-            }
-        }
-
-
         [HttpDelete("{id}")]
         public async Task<IActionResult> SuspendEmployee(Guid id, [FromQuery] bool active = false)
         {
-            using var scope = _serviceScopeFactory.CreateScope();
+            using var scope = _serviceScope.CreateScope();
 
             Guid tenantId = _userHelper.GetTenantId();
             var LoggedEmployee = await _userHelper.GetCurrentEmployeeAsync();
@@ -1111,7 +561,7 @@ namespace MarcoBMS.Services.Controllers
                     }
                 }
             }
-            var attendance = await _context.Attendes.Where(a => a.EmployeeId == employee.Id && (a.OutTime == null || a.Activity == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE)).ToListAsync();
+            var attendance = await _context.Attendes.Where(a => a.EmployeeID == employee.Id && (a.OutTime == null || a.Activity == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE)).ToListAsync();
             if (attendance.Count != 0)
             {
                 _logger.LogWarning("Employee with ID {EmployeeId} have any pending check-out or regularization requests", employee.Id);
@@ -1253,14 +703,5 @@ namespace MarcoBMS.Services.Controllers
             return info;
 
         }
-
-        // Prepare reset link sender helper
-        private async Task SendResetIfApplicableAsync(ApplicationUser u, string firstName)
-        {
-            var token = await _userManager.GeneratePasswordResetTokenAsync(u);
-            var resetLink = $"{_configuration["AppSettings:WebFrontendUrl"]}/reset-password?token={WebUtility.UrlEncode(token)}";
-            await _emailSender.SendResetPasswordEmailOnRegister(u.Email ?? "", firstName, resetLink);
-            _logger.LogInfo("Reset password email queued. Email={Email}", u.Email ?? "");
-        }
     }
 }

Marco.Pms.Services/Controllers/ImageController.cs

@@ -13,7 +13,6 @@ using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.CodeAnalysis;
 using Microsoft.EntityFrameworkCore;
-using Serilog;
 using System.Text.Json;
 
 namespace Marco.Pms.Services.Controllers
@@ -23,32 +22,25 @@ namespace Marco.Pms.Services.Controllers
     [Authorize]
     public class ImageController : ControllerBase
     {
-        private readonly IDbContextFactory<ApplicationDbContext> _dbContextFactory;
         private readonly ApplicationDbContext _context;
         private readonly S3UploadService _s3Service;
         private readonly UserHelper _userHelper;
         private readonly ILoggingService _logger;
         private readonly PermissionServices _permission;
         private readonly Guid tenantId;
-        public ImageController(IDbContextFactory<ApplicationDbContext> dbContextFactory,
+        public ImageController(ApplicationDbContext context, S3UploadService s3Service, UserHelper userHelper, ILoggingService logger, PermissionServices permission)
-            ApplicationDbContext context,
-            S3UploadService s3Service,
-            UserHelper userHelper,
-            ILoggingService logger,
-            PermissionServices permission)
         {
-            _dbContextFactory = dbContextFactory ?? throw new ArgumentNullException(nameof(dbContextFactory));
+            _context = context;
-            _context = context ?? throw new ArgumentNullException(nameof(context));
+            _s3Service = s3Service;
-            _s3Service = s3Service ?? throw new ArgumentNullException(nameof(s3Service));
+            _userHelper = userHelper;
-            _userHelper = userHelper ?? throw new ArgumentNullException(nameof(userHelper));
+            _logger = logger;
-            _logger = logger ?? throw new ArgumentNullException(nameof(logger));
-            _permission = permission ?? throw new ArgumentNullException(nameof(permission));
             tenantId = userHelper.GetTenantId();
+            _permission = permission;
         }
 
         [HttpGet("images/{projectId}")]
 
-        public async Task<IActionResult> GetImageList(Guid projectId, [FromQuery] string? filter, [FromQuery] int pageNumber = 1, [FromQuery] int pageSize = 10)
+        public async Task<IActionResult> GetImageList(Guid projectId, [FromQuery] string? filter, [FromQuery] int? pageNumber = 1, [FromQuery] int? pageSize = 10)
         {
             _logger.LogInfo("[GetImageList] Called by Employee for ProjectId: {ProjectId}", projectId);
 
@@ -87,28 +79,6 @@ namespace Marco.Pms.Services.Controllers
                 }
             }
 
-            var taskQuery = _context.TaskAllocations
-                .Include(t => t.Employee)
-                .Include(t => t.ReportedBy)
-                .Include(t => t.ApprovedBy)
-                .Include(t => t.WorkStatus)
-                .Include(t => t.WorkItem)
-                    .ThenInclude(wi => wi!.ActivityMaster)
-                        .ThenInclude(a => a!.ActivityGroup)
-                            .ThenInclude(ag => ag!.Service)
-                .Include(t => t.WorkItem)
-                    .ThenInclude(wi => wi!.WorkArea)
-                        .ThenInclude(wa => wa!.Floor)
-                            .ThenInclude(f => f!.Building)
-                .Include(t => t.WorkItem)
-                    .ThenInclude(wi => wi!.WorkCategoryMaster)
-                .Where(t => t.WorkItem != null &&
-                            t.WorkItem.WorkArea != null &&
-                            t.WorkItem.WorkArea.Floor != null &&
-                            t.WorkItem.WorkArea.Floor.Building != null &&
-                            t.WorkItem.WorkArea.Floor.Building.ProjectId == projectId &&
-                            t.TenantId == tenantId);
-
             // Step 4: Extract filter values
             var buildingIds = imageFilter?.BuildingIds;
             var floorIds = imageFilter?.FloorIds;
@@ -118,60 +88,71 @@ namespace Marco.Pms.Services.Controllers
             var startDate = imageFilter?.StartDate;
             var endDate = imageFilter?.EndDate;
             var uploadedByIds = imageFilter?.UploadedByIds;
-            var serviceIds = imageFilter?.ServiceIds;
 
-            if (buildingIds?.Any() ?? false)
+            // Step 5: Fetch building > floor > area > work item hierarchy
+            List<Building>? buildings = null;
+            List<Floor>? floors = null;
+            List<WorkArea>? workAreas = null;
+
+            if (buildingIds != null && buildingIds.Count > 0)
             {
 
-                taskQuery = taskQuery
+                buildings = await _context.Buildings
-                .Where(t => t.WorkItem != null &&
+                .Where(b => b.ProjectId == projectId && buildingIds.Contains(b.Id))
-                            t.WorkItem.WorkArea != null &&
+                .ToListAsync();
-                            t.WorkItem.WorkArea.Floor != null &&
-                            buildingIds.Contains(t.WorkItem.WorkArea.Floor.BuildingId));
             }
-
+            else
-            if (floorIds?.Any() ?? false)
             {
-                taskQuery = taskQuery
+                buildings = await _context.Buildings
-                .Where(t => t.WorkItem != null &&
+                .Where(b => b.ProjectId == projectId)
-                            t.WorkItem.WorkArea != null &&
+                .ToListAsync();
-                            floorIds.Contains(t.WorkItem.WorkArea.FloorId));
+
+                buildingIds = buildings.Select(b => b.Id).ToList();
             }
 
-            if (workAreaIds?.Any() ?? false)
+            if (floorIds != null && floorIds.Count > 0)
             {
-                taskQuery = taskQuery
+                floors = await _context.Floor
-                .Where(t => t.WorkItem != null &&
+                .Where(f => buildingIds.Contains(f.BuildingId) && floorIds.Contains(f.Id))
-                            workAreaIds.Contains(t.WorkItem.WorkAreaId));
+                .ToListAsync();
             }
-
+            else
-
-            if (activityIds?.Any() == true)
             {
-                taskQuery = taskQuery
+                floors = await _context.Floor
-                    .Where(t => t.WorkItem != null &&
+                .Where(f => buildingIds.Contains(f.BuildingId))
-                            activityIds.Contains(t.WorkItem.ActivityId));
+                .ToListAsync();
+
+                floorIds = floors.Select(f => f.Id).ToList();
             }
+            if (workAreaIds != null && workAreaIds.Count > 0)
+            {
+                workAreas = await _context.WorkAreas
+                .Where(wa => floorIds.Contains(wa.FloorId) && workAreaIds.Contains(wa.Id))
+                .ToListAsync();
+            }
+            else
+            {
+                workAreas = await _context.WorkAreas
+                .Where(wa => floorIds.Contains(wa.FloorId))
+                .ToListAsync();
+
+                workAreaIds = workAreas.Select(wa => wa.Id).ToList();
+            }
+
+            var workItemsQuery = _context.WorkItems.Include(w => w.ActivityMaster).Include(w => w.WorkCategoryMaster)
+                .Where(wi => workAreaIds.Contains(wi.WorkAreaId));
+            if (activityIds?.Any() == true) workItemsQuery = workItemsQuery.Where(wi => activityIds.Contains(wi.ActivityId));
 
             if (workCategoryIds?.Any() == true)
             {
-                taskQuery = taskQuery
+                workItemsQuery = workItemsQuery.Where(wi => wi.WorkCategoryMaster != null && workCategoryIds.Contains(wi.WorkCategoryMaster.Id));
-                    .Where(t => t.WorkItem != null &&
-                            t.WorkItem.WorkCategoryId.HasValue &&
-                            workCategoryIds.Contains(t.WorkItem.WorkCategoryId.Value));
-            }
-
-            if (serviceIds?.Any() ?? false)
-            {
-                taskQuery = taskQuery.Where(t => t.WorkItem != null &&
-                    t.WorkItem.ActivityMaster != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup != null &&
-                    serviceIds.Contains(t.WorkItem.ActivityMaster.ActivityGroup.ServiceId));
             }
+            var workItems = await workItemsQuery.ToListAsync();
+            var workItemIds = workItems.Select(wi => wi.Id).ToList();
 
             // Step 6: Fetch task allocations and comments
-            var tasks = await taskQuery.ToListAsync();
+            var tasks = await _context.TaskAllocations.Include(t => t.ReportedBy)
-
+                .Where(t => workItemIds.Contains(t.WorkItemId)).ToListAsync();
             var taskIds = tasks.Select(t => t.Id).ToList();
 
             var comments = await _context.TaskComments.Include(c => c.Employee)
@@ -191,21 +172,21 @@ namespace Marco.Pms.Services.Controllers
             {
                 docQuery = docQuery.Where(d => d.UploadedAt.Date >= startDate.Value.Date && d.UploadedAt.Date <= endDate.Value.Date);
             }
-
+            if (pageNumber != null && pageSize != null)
-            int totalRecords = await docQuery.GroupBy(d => d.BatchId).CountAsync();
-            int totalPages = (int)Math.Ceiling((double)totalRecords / pageSize);
-
-            documents = await docQuery
-            .GroupBy(d => d.BatchId)
-            .OrderByDescending(g => g.Max(d => d.UploadedAt))
-            .Skip((pageNumber - 1) * pageSize)
-            .Take(pageSize)
-            .Select(g => new DocumentBatchDto
             {
-                BatchId = g.Key,
+                documents = await docQuery
-                Documents = g.ToList()
+                .GroupBy(d => d.BatchId)
-            })
+                .OrderByDescending(g => g.Max(d => d.UploadedAt))
-            .ToListAsync();
+                .Skip((pageNumber.Value - 1) * pageSize.Value)
+                .Take(pageSize.Value)
+                .Select(g => new DocumentBatchDto
+                {
+                    BatchId = g.Key,
+                    Documents = g.ToList()
+                })
+                .ToListAsync();
+                Console.Write("Pagenation Success");
+            }
 
 
             // Step 8: Build response
@@ -228,10 +209,10 @@ namespace Marco.Pms.Services.Controllers
                     comment = comments.OrderBy(c => c.CommentDate).FirstOrDefault(c => c.TaskAllocationId == task.Id);
                 }
 
-                var workItem = task!.WorkItem;
+                var workItem = workItems.FirstOrDefault(w => w.Id == task?.WorkItemId);
-                var workArea = task!.WorkItem!.WorkArea;
+                var workArea = workAreas.FirstOrDefault(wa => wa.Id == workItem?.WorkAreaId);
-                var floor = task!.WorkItem!.WorkArea!.Floor;
+                var floor = floors.FirstOrDefault(f => f.Id == workArea?.FloorId);
-                var building = task!.WorkItem!.WorkArea!.Floor!.Building;
+                var building = buildings.FirstOrDefault(b => b.Id == floor?.BuildingId);
 
                 return new
                 {
@@ -268,17 +249,8 @@ namespace Marco.Pms.Services.Controllers
                 documentVM = documentVM.Where(d => d.Documents != null && d.Documents.Any(x => uploadedByIds.Contains(x.UploadedBy?.Id ?? Guid.Empty))).ToList();
             }
 
-            var VM = new
-            {
-                TotalCount = totalRecords,
-                TotalPages = totalPages,
-                CurrentPage = pageNumber,
-                PageSize = pageSize,
-                Data = documentVM
-            };
-
             _logger.LogInfo("[GetImageList] Fetched {Count} documents for ProjectId: {ProjectId}", documentVM.Count, projectId);
-            return Ok(ApiResponse<object>.SuccessResponse(VM, $"{documentVM.Count} image records fetched successfully", 200));
+            return Ok(ApiResponse<object>.SuccessResponse(documentVM, $"{documentVM.Count} image records fetched successfully", 200));
         }
 
         [HttpGet("batch/{batchId}")]
@@ -390,150 +362,6 @@ namespace Marco.Pms.Services.Controllers
             return Ok(ApiResponse<object>.SuccessResponse(response, "Images for provided batchId fetched successfully", 200));
         }
 
-        [HttpGet("filter/{projectId}")]
-        public async Task<IActionResult> GetFilterObjectAsync(Guid projectId, CancellationToken ct)
-        {
-            _logger.LogInfo("GetFilterObject started for ProjectId {ProjectId}", projectId); // start log [memory:1]
-
-            // Validate input early
-            if (projectId == Guid.Empty)
-            {
-                Log.Warning("GetFilterObject received empty ProjectId"); // input validation [memory:1]
-                return BadRequest(ApiResponse<object>.ErrorResponse("Invalid project id", 400));
-            }
-
-            // Load only what is needed for attachments; project scoping will be enforced downstream
-            // NOTE: Select only the columns required to reduce payload
-            var taskAttachments = await _context.TaskAttachments
-                .AsNoTracking()
-                .Select(ta => new { ta.ReferenceId, ta.DocumentId })
-                .ToListAsync(ct); // I/O bound work [memory:10]
-
-            if (taskAttachments.Count == 0)
-            {
-                Log.Information("No task attachments found for ProjectId {ProjectId}", projectId); // early exit [memory:1]
-                var emptyResponse = new
-                {
-                    Buildings = Array.Empty<object>(),
-                    Floors = Array.Empty<object>(),
-                    WorkAreas = Array.Empty<object>(),
-                    WorkCategories = Array.Empty<object>(),
-                    Activities = Array.Empty<object>(),
-                    UploadedBys = Array.Empty<object>(),
-                    Services = Array.Empty<object>()
-                };
-                return Ok(ApiResponse<object>.SuccessResponse(emptyResponse, "No data found", 200));
-            }
-
-            // HashSets for O(1) membership tests and to dedupe upfront
-            var referenceIds = new HashSet<Guid>(taskAttachments.Select(x => x.ReferenceId));
-            var documentIds = new HashSet<Guid>(taskAttachments.Select(x => x.DocumentId));
-
-            _logger.LogDebug("Collected {ReferenceCount} referenceIds and {DocumentCount} documentIds", referenceIds.Count, documentIds.Count); // metrics [memory:1]
-
-            // Load comments for the references under this tenant
-            var comments = await _context.TaskComments
-                .AsNoTracking()
-                .Where(tc => referenceIds.Contains(tc.Id) && tc.TenantId == tenantId)
-                .Select(tc => new { tc.Id, tc.TaskAllocationId })
-                .ToListAsync(ct); // filtered selection [memory:10]
-
-            var taskIds = new HashSet<Guid>(comments.Select(c => c.TaskAllocationId));
-            _logger.LogDebug("Resolved {CommentCount} comments mapping to {TaskIdCount} taskIds", comments.Count, taskIds.Count); // observation [memory:1]
-
-            // IMPORTANT: Correct project filter should be == not !=
-            // Include graph tailored to fields needed in final projection (avoid over-inclusion)
-            // Avoid Task.Run for async EF (it doesn’t add value and can harm thread pool)
-            var tasks = await _context.TaskAllocations
-                .AsNoTracking()
-                .Where(t =>
-                    t.WorkItem != null &&
-                    t.WorkItem.WorkArea != null &&
-                    t.WorkItem.WorkArea.Floor != null &&
-                    t.WorkItem.WorkArea.Floor.Building != null &&
-                    t.WorkItem.WorkArea.Floor.Building.ProjectId == projectId && // fixed filter
-                    (taskIds.Contains(t.Id) || referenceIds.Contains(t.Id)))
-                .Include(t => t.WorkItem!)
-                    .ThenInclude(wi => wi.ActivityMaster!)
-                        .ThenInclude(a => a.ActivityGroup!)
-                            .ThenInclude(ag => ag.Service)
-                .Include(t => t.WorkItem!)
-                    .ThenInclude(wi => wi.WorkArea!)
-                        .ThenInclude(wa => wa.Floor!)
-                            .ThenInclude(f => f.Building)
-                .Include(t => t.WorkItem!)
-                    .ThenInclude(wi => wi.WorkCategoryMaster)
-                // Only select fields used later to reduce tracked object size in memory
-                .Select(t => new
-                {
-                    TaskId = t.Id,
-                    Building = new { Id = t.WorkItem!.WorkArea!.Floor!.Building!.Id, Name = t.WorkItem.WorkArea.Floor.Building.Name },
-                    Floor = new { Id = t.WorkItem.WorkArea.Floor.Id, Name = t.WorkItem.WorkArea.Floor.FloorName },
-                    WorkArea = new { Id = t.WorkItem.WorkArea.Id, Name = t.WorkItem.WorkArea.AreaName },
-                    Activity = t.WorkItem.ActivityMaster == null ? null : new { Id = t.WorkItem.ActivityMaster.Id, Name = t.WorkItem.ActivityMaster.ActivityName },
-                    WorkCategory = t.WorkItem.WorkCategoryMaster == null ? null : new { Id = t.WorkItem.WorkCategoryMaster.Id, Name = t.WorkItem.WorkCategoryMaster.Name },
-                    Service = t.WorkItem.ActivityMaster!.ActivityGroup!.Service == null ? null : new { Id = t.WorkItem.ActivityMaster.ActivityGroup.Service.Id, Name = t.WorkItem.ActivityMaster.ActivityGroup.Service.Name }
-                })
-                .ToListAsync(ct); // optimized projection [memory:10]
-
-            _logger.LogDebug("Fetched {TaskCount} tasks after filtering and projection", tasks.Count); // metrics [memory:1]
-
-            // Documents query in parallel with tasks is okay; here we’ve already awaited tasks, but both can run together if needed.
-            // Only fetch uploader fields needed
-            var documents = await _context.Documents
-                .AsNoTracking()
-                .Where(d => documentIds.Contains(d.Id))
-                .Select(d => new
-                {
-                    d.Id,
-                    UploadedBy = d.UploadedBy == null ? null : new
-                    {
-                        d.UploadedBy.Id,
-                        d.UploadedBy.FirstName,
-                        d.UploadedBy.LastName
-                    }
-                })
-                .ToListAsync(ct); // minimal shape [memory:10]
-
-            _logger.LogDebug("Fetched {DocumentCount} documents for UploadedBy resolution", documents.Count); // metrics [memory:1]
-
-            // Distinct projections via HashSet to avoid custom equality or anonymous Distinct pitfalls
-            static List<T> DistinctBy<T, TKey>(IEnumerable<T> source, Func<T, TKey> keySelector)
-                => source.GroupBy(keySelector).Select(g => g.First()).ToList();
-
-            var buildings = DistinctBy(tasks.Select(t => t.Building), b => b.Id);
-            var floors = DistinctBy(tasks.Select(t => t.Floor), f => f.Id);
-            var workAreas = DistinctBy(tasks.Select(t => t.WorkArea), wa => wa.Id);
-            var activities = DistinctBy(tasks.Where(t => t.Activity != null).Select(t => t.Activity!), a => a.Id);
-            var workCategories = DistinctBy(tasks.Where(t => t.WorkCategory != null).Select(t => t.WorkCategory!), wc => wc.Id);
-            var services = DistinctBy(tasks.Where(t => t.Service != null).Select(t => t.Service!), s => s.Id);
-
-            var uploadedBys = DistinctBy(
-                documents.Where(d => d.UploadedBy != null)
-                         .Select(d => new
-                         {
-                             Id = d.UploadedBy!.Id,
-                             Name = string.Join(' ', new[] { d.UploadedBy!.FirstName, d.UploadedBy!.LastName }.Where(x => !string.IsNullOrWhiteSpace(x)))
-                         }),
-                u => u.Id);
-
-            var response = new
-            {
-                Buildings = buildings,
-                Floors = floors,
-                WorkAreas = workAreas,
-                WorkCategories = workCategories,
-                Activities = activities,
-                UploadedBys = uploadedBys,
-                Services = services
-            };
-
-            _logger.LogInfo("GetFilterObject succeeded for ProjectId {ProjectId}. Buildings={Buildings}, Floors={Floors}, WorkAreas={WorkAreas}, Activities={Activities}, WorkCategories={WorkCategories}, Services={Services}, UploadedBys={UploadedBys}",
-                projectId, buildings.Count, floors.Count, workAreas.Count, activities.Count, workCategories.Count, services.Count, uploadedBys.Count); // success log [memory:1]
-
-            return Ok(ApiResponse<object>.SuccessResponse(response, "Filter object for image gallery fetched successfully", 200));
-        }
-
         [HttpGet("{documentId}")]
         public async Task<IActionResult> GetImage(Guid documentId)
         {

Marco.Pms.Services/Controllers/ProjectController.cs

@@ -319,20 +319,12 @@ namespace MarcoBMS.Services.Controllers
             return StatusCode(response.StatusCode, response);
         }
 
-        [HttpGet("get/task/team/{projectId}")]
-        public async Task<IActionResult> GetProjectTeamByServiceAndOrganization(Guid projectId, [FromQuery] Guid? serviceId, [FromQuery] Guid? organizationId)
-        {
-            Employee loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-            var response = await _projectServices.GetProjectTeamByServiceAndOrganizationAsync(projectId, serviceId, organizationId, tenantId, loggedInEmployee);
-            return StatusCode(response.StatusCode, response);
-        }
-
         #endregion
 
         #region =================================================================== Project InfraStructure Get APIs ===================================================================
 
         [HttpGet("infra-details/{projectId}")]
-        public async Task<IActionResult> GetInfraDetails(Guid projectId, [FromQuery] Guid? serviceId)
+        public async Task<IActionResult> GetInfraDetails(Guid projectId)
         {
             // --- Step 1: Input Validation ---
             if (!ModelState.IsValid)
@@ -344,7 +336,7 @@ namespace MarcoBMS.Services.Controllers
 
             // --- Step 2: Prepare data without I/O ---
             Employee loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-            var response = await _projectServices.GetInfraDetailsAsync(projectId, serviceId, tenantId, loggedInEmployee);
+            var response = await _projectServices.GetInfraDetailsAsync(projectId, tenantId, loggedInEmployee);
             return StatusCode(response.StatusCode, response);
 
         }
@@ -365,7 +357,6 @@ namespace MarcoBMS.Services.Controllers
             var response = await _projectServices.GetWorkItemsAsync(workAreaId, serviceId, tenantId, loggedInEmployee);
             return StatusCode(response.StatusCode, response);
         }
-
         [HttpGet("tasks-employee/{employeeId}")]
         public async Task<IActionResult> GetTasksByEmployee(Guid employeeId, [FromQuery] DateTime? fromDate, DateTime? toDate)
         {
@@ -519,7 +510,6 @@ namespace MarcoBMS.Services.Controllers
 
             return StatusCode(response.StatusCode, response);
         }
-
         [HttpPost("assign/service")]
         public async Task<IActionResult> AssignServiceToProject([FromBody] AssignServiceDto model)
         {
@@ -560,13 +550,6 @@ namespace MarcoBMS.Services.Controllers
             var response = await _projectServices.GetAssignedOrganizationsToProjectAsync(projectId, tenantId, loggedInEmployee);
             return StatusCode(response.StatusCode, response);
         }
-        [HttpGet("get/assigned/organization/dropdown/{projectId}")]
-        public async Task<IActionResult> GetAssignedOrganizationsToProjectForDropdownAsync(Guid projectId)
-        {
-            Employee loggedInEmployee = await _userHelper.GetCurrentEmployeeAsync();
-            var response = await _projectServices.GetAssignedOrganizationsToProjectForDropdownAsync(projectId, tenantId, loggedInEmployee);
-            return StatusCode(response.StatusCode, response);
-        }
 
         #endregion
     }

Marco.Pms.Services/Controllers/ReportController.cs

@@ -1,5 +1,4 @@
 using Marco.Pms.DataAccess.Data;
-using Marco.Pms.Model.Dtos.Attendance;
 using Marco.Pms.Model.Dtos.Mail;
 using Marco.Pms.Model.Mail;
 using Marco.Pms.Model.MongoDBModels.Utility;
@@ -11,6 +10,7 @@ using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.CodeAnalysis;
 using Microsoft.EntityFrameworkCore;
+using MongoDB.Driver;
 using System.Data;
 using System.Globalization;
 using System.Net.Mail;
@@ -22,25 +22,27 @@ namespace Marco.Pms.Services.Controllers
     [Authorize]
     public class ReportController : ControllerBase
     {
-        private readonly IDbContextFactory<ApplicationDbContext> _dbContextFactory;
-        private readonly IServiceScopeFactory _serviceScopeFactory;
         private readonly ApplicationDbContext _context;
+        private readonly IEmailSender _emailSender;
         private readonly ILoggingService _logger;
         private readonly UserHelper _userHelper;
+        private readonly IWebHostEnvironment _env;
+        private readonly ReportHelper _reportHelper;
         private readonly IConfiguration _configuration;
-        public ReportController(IDbContextFactory<ApplicationDbContext> dbContextFactory,
+        private readonly CacheUpdateHelper _cache;
-            ApplicationDbContext context,
+        private readonly IServiceScopeFactory _serviceScopeFactory;
-            ILoggingService logger,
+        public ReportController(ApplicationDbContext context, IEmailSender emailSender, ILoggingService logger, UserHelper userHelper,
-            UserHelper userHelper,
+            IWebHostEnvironment env, ReportHelper reportHelper, IConfiguration configuration, CacheUpdateHelper cache, IServiceScopeFactory serviceScopeFactory)
-            IConfiguration configuration,
-            IServiceScopeFactory serviceScopeFactory)
         {
-            _dbContextFactory = dbContextFactory ?? throw new ArgumentNullException(nameof(dbContextFactory));
+            _context = context;
-            _serviceScopeFactory = serviceScopeFactory ?? throw new ArgumentNullException(nameof(serviceScopeFactory));
+            _emailSender = emailSender;
-            _context = context ?? throw new ArgumentNullException(nameof(context));
+            _logger = logger;
-            _logger = logger ?? throw new ArgumentNullException(nameof(logger));
+            _userHelper = userHelper;
-            _userHelper = userHelper ?? throw new ArgumentNullException(nameof(userHelper));
+            _env = env;
-            _configuration = configuration ?? throw new ArgumentNullException(nameof(configuration));
+            _reportHelper = reportHelper;
+            _configuration = configuration;
+            _cache = cache;
+            _serviceScopeFactory = serviceScopeFactory;
         }
 
         /// <summary>
@@ -433,12 +435,10 @@ namespace Marco.Pms.Services.Controllers
                 200));
         }
 
+
         [HttpGet("report-mail")]
         public async Task<IActionResult> GetProjectStatisticsFromCache()
         {
-            using var scope = _serviceScopeFactory.CreateScope();
-            var _cache = scope.ServiceProvider.GetRequiredService<CacheUpdateHelper>();
-
             var mailList = await _cache.GetProjectReportMail(false);
             if (mailList == null)
             {
@@ -447,92 +447,5 @@ namespace Marco.Pms.Services.Controllers
 
             return Ok(ApiResponse<object>.SuccessResponse(mailList, "Fetched list of mail body successfully", 200));
         }
-
-        [HttpGet("report-attendance")]
-        public async Task<IActionResult> GetAttendanceReportAsync([FromQuery] bool isCurrentMonth = false)
-        {
-            Guid tenantId = _userHelper.GetTenantId();
-            using var scope = _serviceScopeFactory.CreateScope();
-
-            DateTime today = DateTime.Today;
-            DateTime firstDayOfMonth = new DateTime(today.Year, today.Month, 1);
-            DateTime firstDayOfNextMonth = firstDayOfMonth.AddMonths(1);
-
-            if (!isCurrentMonth)
-            {
-                firstDayOfNextMonth = firstDayOfMonth;
-                firstDayOfMonth = firstDayOfMonth.AddMonths(-1);
-            }
-
-            // Generate list of all dates in the month
-            var allDates = Enumerable.Range(0, (firstDayOfNextMonth - firstDayOfMonth).Days)
-                                     .Select(offset => firstDayOfMonth.AddDays(offset))
-                                     .ToList();
-
-            var attendancesTask = Task.Run(async () =>
-            {
-                await using var context = await _dbContextFactory.CreateDbContextAsync();
-                return await context.Attendes
-                    .Where(a => a.AttendanceDate >= firstDayOfMonth && a.AttendanceDate < firstDayOfNextMonth && a.Employee != null && a.TenantId == tenantId)
-                    .GroupBy(a => a.ProjectID)
-                    .ToListAsync();
-            });
-
-            var projectAllocationTask = Task.Run(async () =>
-            {
-                await using var context = await _dbContextFactory.CreateDbContextAsync();
-                return await context.ProjectAllocations
-                    .Include(pa => pa.Employee)
-                    .Where(pa => pa.TenantId == tenantId && pa.IsActive)
-                    .ToListAsync();
-            });
-
-            await Task.WhenAll(attendancesTask, projectAllocationTask);
-
-            var attendances = attendancesTask.Result;
-            var projectAllocations = projectAllocationTask.Result;
-
-            var result = attendances.Select(g =>
-            {
-                var projectAllocation = projectAllocations.Where(pa => pa.ProjectId == g.Key && pa.Employee != null).ToList();
-                var projectAttendance = projectAllocation.Select(pa =>
-                {
-                    var attendances = g.Where(a => a.EmployeeId == pa.EmployeeId).ToList();
-                    var attendanceDate = attendances.Select(a => a.AttendanceDate.Date).ToList();
-                    return new
-                    {
-                        FirstName = pa.Employee!.FirstName,
-                        LastName = pa.Employee.LastName,
-                        Attendances = allDates.Select(d =>
-                        {
-                            var attendance = attendances.FirstOrDefault(a => a.AttendanceDate.Date == d);
-                            return new
-                            {
-                                AttendanceDate = d,
-                                CheckIn = attendance?.InTime,
-                                CheckOut = attendance?.OutTime,
-                                Activity = attendance?.Activity,
-                                IsApproved = attendance?.ApprovedById.HasValue,
-                            };
-                        }).ToList(),
-                        CheckInCheckOutDone = attendances.Where(a => a.InTime.HasValue && a.OutTime.HasValue && a.Activity == ATTENDANCE_MARK_TYPE.REGULARIZE).Count(),
-                        CheckInDone = attendances.Where(a => a.InTime.HasValue).Count(),
-                        CheckOutPending = attendances.Where(a => a.InTime.HasValue && !a.OutTime.HasValue).Count(),
-                        RejectedRegularize = attendances.Where(a => a.Activity == ATTENDANCE_MARK_TYPE.REGULARIZE_REJECT).Count(),
-                        AbsentAttendance = allDates.Where(d => !attendanceDate.Contains(d) && d.DayOfWeek != DayOfWeek.Sunday).Count()
-                    };
-                }).OrderBy(ar => ar.FirstName).ThenBy(ar => ar.LastName).ToList();
-
-                return new
-                {
-                    ProjectName = _context.Projects.Where(p => p.Id == g.Key && p.TenantId == tenantId).Select(p => p.Name).FirstOrDefault(),
-                    ProjectAttendance = projectAttendance
-                };
-            }).ToList();
-
-            var response = result.OrderBy(r => r.ProjectName).ToList();
-
-            return Ok(ApiResponse<object>.SuccessResponse(response, "Attendance Report fetched successfully", 200));
-        }
     }
 }

Marco.Pms.Services/Controllers/TaskController.cs

@@ -430,14 +430,37 @@ namespace MarcoBMS.Services.Controllers
         }
 
         [HttpGet("list")]
-        public async Task<IActionResult> GetTasksList([FromQuery] Guid projectId, [FromQuery] string? filter, [FromQuery] Guid? serviceId,
+        public async Task<IActionResult> GetTasksList([FromQuery] Guid projectId, [FromQuery] string? filter, [FromQuery] int pageNumber = 1, [FromQuery] int pageSize = 20,
-            [FromQuery] int pageNumber = 1, [FromQuery] int pageSize = 20)
+            [FromQuery] string? dateFrom = null, [FromQuery] string? dateTo = null)
         {
-            _logger.LogInfo("GetTasksList called for projectId: {ProjectId}", projectId);
+            _logger.LogInfo("GetTasksList called for projectId: {ProjectId}, dateFrom: {DateFrom}, dateTo: {DateTo}", projectId, dateFrom ?? "", dateTo ?? "");
 
             Guid tenantId = GetTenantId();
+            DateTime fromDate = new DateTime();
+            DateTime toDate = new DateTime();
 
-            // 1. Get task allocations in the specified date range
+            // 1. Parse and validate dateFrom
+            if (dateFrom != null && !DateTime.TryParse(dateFrom, out fromDate))
+            {
+                _logger.LogWarning("Invalid starting date provided: {DateFrom}", dateFrom);
+                return BadRequest(ApiResponse<object>.ErrorResponse("Invalid starting date.", "Invalid starting date.", 400));
+            }
+
+            // 2. Parse and validate dateTo
+            if (dateTo != null && !DateTime.TryParse(dateTo, out toDate))
+            {
+                _logger.LogWarning("Invalid ending date provided: {DateTo}", dateTo);
+                return BadRequest(ApiResponse<object>.ErrorResponse("Invalid ending date.", "Invalid ending date.", 400));
+            }
+
+            // 3. Set default date range if not provided
+            fromDate = dateFrom == null ? DateTime.UtcNow.Date : fromDate;
+            toDate = dateTo == null ? fromDate.AddDays(1) : toDate;
+
+
+            _logger.LogInfo("Fetching task allocations between {FromDate} and {ToDate}", fromDate, toDate);
+
+            // 4. Get task allocations in the specified date range
             var taskAllocationQuery = _context.TaskAllocations
                 .Include(t => t.Employee)
                 .Include(t => t.ReportedBy)
@@ -457,7 +480,9 @@ namespace MarcoBMS.Services.Controllers
                             t.WorkItem.WorkArea != null &&
                             t.WorkItem.WorkArea.Floor != null &&
                             t.WorkItem.WorkArea.Floor.Building != null &&
-                            t.WorkItem.WorkArea.Floor.Building.ProjectId == projectId &&
+                            t.WorkItem.WorkArea.Floor.Building.ProjectId != projectId &&
+                            t.AssignmentDate.Date >= fromDate.Date &&
+                            t.AssignmentDate.Date <= toDate.Date &&
                             t.TenantId == tenantId);
 
             var taskFilter = TryDeserializeFilter(filter);
@@ -485,23 +510,10 @@ namespace MarcoBMS.Services.Controllers
                 if (taskFilter.ServiceIds?.Any() ?? false)
                 {
                     taskAllocationQuery = taskAllocationQuery.Where(t => t.WorkItem != null &&
-                    t.WorkItem.ActivityMaster != null &&
+                        t.WorkItem.ActivityMaster != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup != null &&
+                        t.WorkItem.ActivityMaster.ActivityGroup != null &&
-                    taskFilter.ServiceIds.Contains(t.WorkItem.ActivityMaster.ActivityGroup.ServiceId));
+                        taskFilter.ServiceIds.Contains(t.WorkItem.ActivityMaster.ActivityGroup.ServiceId));
                 }
-                if (taskFilter.dateFrom.HasValue && taskFilter.dateTo.HasValue)
-                {
-                    taskAllocationQuery = taskAllocationQuery.Where(t => t.AssignmentDate.Date >= taskFilter.dateFrom.Value.Date &&
-                            t.AssignmentDate.Date <= taskFilter.dateTo.Value.Date);
-                }
-            }
-
-            if (serviceId.HasValue)
-            {
-                taskAllocationQuery = taskAllocationQuery.Where(t => t.WorkItem != null &&
-                    t.WorkItem.ActivityMaster != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup.ServiceId == serviceId);
             }
 
 
@@ -516,14 +528,14 @@ namespace MarcoBMS.Services.Controllers
 
             var taskIds = taskAllocations.Select(t => t.Id).ToList();
 
-            // 2. Load team members
+            // 5. Load team members
             _logger.LogInfo("Loading task members and related employee data.");
             var teamMembers = await _context.TaskMembers
                 .Include(t => t.Employee)
                 .Where(t => taskIds.Contains(t.TaskAllocationId))
                 .ToListAsync();
 
-            // 3. Load task comments
+            // 6. Load task comments
             _logger.LogInfo("Fetching comments and attachments.");
             var allComments = await _context.TaskComments
                 .Include(c => c.Employee)
@@ -531,14 +543,14 @@ namespace MarcoBMS.Services.Controllers
                 .ToListAsync();
             var commentIds = allComments.Select(c => c.Id).ToList();
 
-            // 4. Load all attachments (task and comment)
+            // 7. Load all attachments (task and comment)
             var attachments = await _context.TaskAttachments
                 .Where(t => taskIds.Contains(t.ReferenceId) || commentIds.Contains(t.ReferenceId))
                 .ToListAsync();
 
             var documentIds = attachments.Select(t => t.DocumentId).ToList();
 
-            // 5. Load actual documents from attachment references
+            // 8. Load actual documents from attachment references
             var documents = await _context.Documents
                 .Where(d => documentIds.Contains(d.Id))
                 .ToListAsync();
@@ -752,97 +764,6 @@ namespace MarcoBMS.Services.Controllers
             return Ok(ApiResponse<object>.SuccessResponse(taskVM, "Success", 200));
         }
 
-        [HttpGet("filter/{projectId}")]
-        public async Task<IActionResult> GetTaskFilterObject(Guid projectId)
-        {
-            // Get the current tenant from claims/context
-            Guid tenantId = GetTenantId();
-
-            // Log API invocation with the project and tenant for traceability
-            _logger.LogInfo("Fetching filter objects for ProjectId={ProjectId}, TenantId={TenantId}", projectId, tenantId);
-
-            try
-            {
-                // AsNoTracking for improved performance—no intention to update these records
-                // Only fetch & project properties actually required (DTO projection)
-                var tasks = await _context.TaskAllocations
-                .Include(t => t.WorkItem)
-                    .ThenInclude(wi => wi!.WorkArea)
-                        .ThenInclude(wa => wa!.Floor)
-                            .ThenInclude(f => f!.Building)
-                .Include(t => t.WorkItem)
-                    .ThenInclude(wi => wi!.ActivityMaster)
-                        .ThenInclude(a => a!.ActivityGroup)
-                            .ThenInclude(ag => ag!.Service)
-                .Where(t => t.WorkItem != null &&
-                    t.WorkItem.WorkArea != null &&
-                    t.WorkItem.WorkArea.Floor != null &&
-                    t.WorkItem.WorkArea.Floor.Building != null &&
-                    t.WorkItem.WorkArea.Floor.Building.ProjectId == projectId &&
-                    t.TenantId == tenantId).ToListAsync();
-
-                // Distinct by Id (since projection doesn't guarantee uniqueness across different allocations)
-                var buildings = tasks.Where(t => t.WorkItem != null && t.WorkItem.WorkArea != null && t.WorkItem.WorkArea.Floor != null && t.WorkItem.WorkArea.Floor.Building != null)
-                .Select(t => t.WorkItem!.WorkArea!.Floor!.Building!)
-                .Select(b => new
-                {
-                    Id = b.Id,
-                    Name = b.Name
-                }).Distinct().ToList();
-
-                var floors = tasks.Where(t => t.WorkItem != null && t.WorkItem.WorkArea != null && t.WorkItem.WorkArea.Floor != null)
-                    .Select(t => t.WorkItem!.WorkArea!.Floor!)
-                    .Select(f => new
-                    {
-                        Id = f.Id,
-                        Name = f.FloorName,
-                        BuildingId = f.BuildingId
-                    }).Distinct().ToList();
-
-                var activities = tasks.Where(t => t.WorkItem != null &&
-                    t.WorkItem.ActivityMaster != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup.Service != null)
-                        .Select(t => t.WorkItem!.ActivityMaster!)
-                    .Select(a => new
-                    {
-                        Id = a.Id,
-                        Name = a.ActivityName
-                    }).Distinct().ToList();
-
-                var services = tasks.Where(t => t.WorkItem != null &&
-                    t.WorkItem.ActivityMaster != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup != null &&
-                    t.WorkItem.ActivityMaster.ActivityGroup.Service != null)
-                    .Select(t => t.WorkItem!.ActivityMaster!.ActivityGroup!.Service!)
-                    .Select(s => new
-                    {
-                        Id = s.Id,
-                        Name = s.Name
-                    }).Distinct().ToList();
-
-                var response = new
-                {
-                    Buildings = buildings,
-                    Floors = floors,
-                    Activities = activities,
-                    Services = services
-                };
-
-                _logger.LogInfo("Successfully fetched filter objects for ProjectId={ProjectId}, TenantId={TenantId}", projectId, tenantId);
-
-                // Use DTO in API response for clarity and easier frontend usage
-                return Ok(ApiResponse<object>.SuccessResponse(response, "Filter object for task fetched successfully", 200));
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "Failed to fetch filter objects for ProjectId={ProjectId}, TenantId={TenantId}", projectId, tenantId);
-                // Return a standard error result
-                return StatusCode(500, ApiResponse<object>.ErrorResponse("Failed to fetch filter object.", 500));
-            }
-        }
-
-
         /// <summary>
         /// Approves a reported task after validation, updates status, and stores attachments/comments.
         /// </summary>

Marco.Pms.Services/Controllers/TenantController.cs

@@ -6,7 +6,6 @@ using Marco.Pms.Model.Employees;
 using Marco.Pms.Model.Entitlements;
 using Marco.Pms.Model.Filters;
 using Marco.Pms.Model.MongoDBModels.Utility;
-using Marco.Pms.Model.OrganizationModel;
 using Marco.Pms.Model.Projects;
 using Marco.Pms.Model.Roles;
 using Marco.Pms.Model.TenantModels;
@@ -474,27 +473,6 @@ namespace Marco.Pms.Services.Controllers
             await using var transaction = await _context.Database.BeginTransactionAsync();
             try
             {
-                // Get last SPRID and increment for new organization
-                var lastOrganization = await _context.Organizations.OrderByDescending(sp => sp.SPRID).FirstOrDefaultAsync();
-                double lastSPRID = lastOrganization?.SPRID ?? 5400;
-
-                // Map DTO to entity and set defaults
-                Organization organization = new Organization
-                {
-                    Name = model.OrganizationName,
-                    Email = model.Email,
-                    ContactPerson = $"{model.FirstName} {model.LastName}",
-                    Address = model.BillingAddress,
-                    ContactNumber = model.ContactNumber,
-                    SPRID = lastSPRID + 1,
-                    logoImage = model.logoImage,
-                    CreatedAt = DateTime.UtcNow,
-                    CreatedById = loggedInEmployee.Id,
-                    IsActive = true
-                };
-
-                _context.Organizations.Add(organization);
-
                 // Create the primary Tenant entity
 
                 var tenant = _mapper.Map<Tenant>(model);
@@ -502,7 +480,6 @@ namespace Marco.Pms.Services.Controllers
                 tenant.TenantStatusId = tenantActiveStatus;
                 tenant.CreatedById = loggedInEmployee.Id;
                 tenant.IsSuperTenant = false;
-                tenant.OrganizationId = organization.Id;
 
                 _context.Tenants.Add(tenant);
 
@@ -549,11 +526,7 @@ namespace Marco.Pms.Services.Controllers
                     ApplicationUserId = applicationUser.Id,
                     JobRole = adminJobRole, // Link to the newly created role
                     CurrentAddress = model.BillingAddress,
-                    IsActive = true,
+                    TenantId = tenant.Id
-                    IsSystem = false,
-                    IsPrimary = true,
-                    OrganizationId = organization.Id,
-                    HasApplicationAccess = true
                 };
                 _context.Employees.Add(employeeUser);
 
@@ -586,21 +559,6 @@ namespace Marco.Pms.Services.Controllers
                     {
                         ApplicationRoleId = applicationRole.Id,
                         FeaturePermissionId = PermissionsMaster.ViewMasters
-                    },
-                    new RolePermissionMappings
-                    {
-                        ApplicationRoleId = applicationRole.Id,
-                        FeaturePermissionId = PermissionsMaster.ViewOrganization
-                    },
-                    new RolePermissionMappings
-                    {
-                        ApplicationRoleId = applicationRole.Id,
-                        FeaturePermissionId = PermissionsMaster.AddOrganization
-                    },
-                    new RolePermissionMappings
-                    {
-                        ApplicationRoleId = applicationRole.Id,
-                        FeaturePermissionId = PermissionsMaster.EditOrganization
                     }
                 };
                 _context.RolePermissionMappings.AddRange(rolePermissionMappigs);
@@ -621,8 +579,6 @@ namespace Marco.Pms.Services.Controllers
                     ProjectAddress = model.BillingAddress,
                     StartDate = model.OnBoardingDate,
                     EndDate = DateTime.MaxValue,
-                    PromoterId = organization.Id,
-                    PMCId = organization.Id,
                     ContactPerson = tenant.ContactName,
                     TenantId = tenant.Id
                 };
@@ -639,18 +595,6 @@ namespace Marco.Pms.Services.Controllers
                 };
                 _context.ProjectAllocations.Add(projectAllocation);
 
-                // Map organization services
-                if (model.ServiceIds?.Any() ?? false)
-                {
-                    var serviceOrgMappings = model.ServiceIds.Select(s => new OrgServiceMapping
-                    {
-                        ServiceId = s,
-                        OrganizationId = organization.Id
-                    }).ToList();
-
-                    _context.OrgServiceMappings.AddRange(serviceOrgMappings);
-                }
-
                 // All entities are now added to the context. Save them all in a single database operation.
                 await _context.SaveChangesAsync();
 
@@ -1013,6 +957,10 @@ namespace Marco.Pms.Services.Controllers
 
             try
             {
+                _ = Task.Run(async () =>
+                {
+                    await ClearPermissionForTenant();
+                });
                 var features = await _featureDetailsHelper.GetFeatureDetails(subscriptionPlan.FeaturesId);
                 if (features == null)
                 {
@@ -1065,7 +1013,7 @@ namespace Marco.Pms.Services.Controllers
                 // Get root employee and role for this tenant
                 var rootEmployee = await _context.Employees
                     .Include(e => e.ApplicationUser)
-                    .FirstOrDefaultAsync(e => e.ApplicationUser != null && (e.ApplicationUser.IsRootUser ?? false) && e.OrganizationId == tenant.OrganizationId);
+                    .FirstOrDefaultAsync(e => e.ApplicationUser != null && (e.ApplicationUser.IsRootUser ?? false) && e.TenantId == model.TenantId);
 
                 if (rootEmployee == null)
                 {
@@ -1123,9 +1071,6 @@ namespace Marco.Pms.Services.Controllers
                     _logger.LogInfo("Removed {Count} role permission mappings for role {RoleId}", deleteMappings.Count, roleId);
                 }
 
-                var _cache = scope.ServiceProvider.GetRequiredService<CacheUpdateHelper>();
-                await _cache.ClearAllEmployeesFromCacheByTenantId(tenant.Id);
-
                 var _masteData = scope.ServiceProvider.GetRequiredService<MasterDataService>();
 
                 if (features.Modules?.ProjectManagement?.Enabled ?? false)
@@ -1324,6 +1269,10 @@ namespace Marco.Pms.Services.Controllers
                 _logger.LogInfo("Subscription plan changed: Tenant={TenantId}, NewPlan={PlanId}",
                     model.TenantId, model.PlanId);
 
+                _ = Task.Run(async () =>
+                {
+                    await ClearPermissionForTenant();
+                });
 
                 // 8. Update tenant permissions based on subscription features.
                 var features = await _featureDetailsHelper.GetFeatureDetails(subscriptionPlan.FeaturesId);
@@ -1358,7 +1307,7 @@ namespace Marco.Pms.Services.Controllers
                 // 8c. Find root employee & role for this tenant.
                 var rootEmployee = await context.Employees
                     .Include(e => e.ApplicationUser)
-                    .FirstOrDefaultAsync(e => e.ApplicationUser != null && (e.ApplicationUser.IsRootUser ?? false) && e.OrganizationId == tenant.OrganizationId);
+                    .FirstOrDefaultAsync(e => e.ApplicationUser != null && (e.ApplicationUser.IsRootUser ?? false) && e.TenantId == model.TenantId);
 
                 if (rootEmployee == null)
                 {
@@ -1369,8 +1318,7 @@ namespace Marco.Pms.Services.Controllers
 
                 var rootRoleId = await context.EmployeeRoleMappings
                     .AsNoTracking()
-                    .Include(er => er.Role)
+                    .Where(er => er.EmployeeId == rootEmployee.Id && er.TenantId == model.TenantId)
-                    .Where(er => er.EmployeeId == rootEmployee.Id && er.TenantId == model.TenantId && er.Role != null && er.Role.Role == "Super User")
                     .Select(er => er.RoleId)
                     .FirstOrDefaultAsync();
 
@@ -1435,9 +1383,6 @@ namespace Marco.Pms.Services.Controllers
                     _logger.LogInfo("Permissions revoked: {Count} for Role={RoleId}", mappingsToRemove.Count, rootRoleId);
                 }
 
-                var _cache = scope.ServiceProvider.GetRequiredService<CacheUpdateHelper>();
-                await _cache.ClearAllEmployeesFromCacheByTenantId(tenant.Id);
-
                 var _masteData = scope.ServiceProvider.GetRequiredService<MasterDataService>();
 
                 if (features.Modules?.ProjectManagement?.Enabled ?? false)
@@ -1822,6 +1767,19 @@ namespace Marco.Pms.Services.Controllers
             return ApiResponse<SubscriptionPlanVM>.SuccessResponse(VM, "Success", 200);
         }
 
+        private async Task ClearPermissionForTenant()
+        {
+            await using var _context = await _dbContextFactory.CreateDbContextAsync();
+            using var scope = _serviceScopeFactory.CreateScope();
+
+            var _cache = scope.ServiceProvider.GetRequiredService<CacheUpdateHelper>();
+            var _cacheLogger = scope.ServiceProvider.GetRequiredService<ILoggingService>();
+
+            var employeeIds = await _context.Employees.Where(e => e.TenantId == tenantId).Select(e => e.Id).ToListAsync();
+            await _cache.ClearAllEmployeesFromCacheByEmployeeIds(employeeIds, tenantId);
+            _cacheLogger.LogInfo("{EmployeeCount} number of employee deleted", employeeIds.Count);
+        }
+
         #endregion
     }
 }

Marco.Pms.Services/Helpers/CacheUpdateHelper.cs

@@ -10,7 +10,6 @@ using Marco.Pms.Model.MongoDBModels.Expenses;
 using Marco.Pms.Model.MongoDBModels.Masters;
 using Marco.Pms.Model.MongoDBModels.Project;
 using Marco.Pms.Model.MongoDBModels.Utility;
-using Marco.Pms.Model.OrganizationModel;
 using Marco.Pms.Model.Projects;
 using Marco.Pms.Model.Utilities;
 using MarcoBMS.Services.Service;
@@ -70,45 +69,6 @@ namespace Marco.Pms.Services.Helpers
                     .Where(s => s.Id == project.ProjectStatusId)
                     .Select(s => new { s.Id, s.Status }) // Projection
                     .FirstOrDefaultAsync();
-
-            });
-
-            var promotorTask = Task.Run(async () =>
-            {
-                using var context = _dbContextFactory.CreateDbContext();
-                return await context.Organizations
-                    .AsNoTracking()
-                    .Where(o => o.Id == project.PromoterId)
-                    .Select(o => new OrganizationMongoDB
-                    {
-                        Id = o.Id.ToString(),
-                        Name = o.Name,
-                        ContactPerson = o.ContactPerson,
-                        Email = o.Email,
-                        Address = o.Address,
-                        ContactNumber = o.ContactNumber,
-                        SPRID = o.SPRID
-                    }) // Projection
-                    .FirstOrDefaultAsync();
-            });
-
-            var pmcTask = Task.Run(async () =>
-            {
-                using var context = _dbContextFactory.CreateDbContext();
-                return await context.Organizations
-                    .AsNoTracking()
-                    .Where(o => o.Id == project.PMCId)
-                    .Select(o => new OrganizationMongoDB
-                    {
-                        Id = o.Id.ToString(),
-                        Name = o.Name,
-                        ContactPerson = o.ContactPerson,
-                        Email = o.Email,
-                        Address = o.Address,
-                        ContactNumber = o.ContactNumber,
-                        SPRID = o.SPRID
-                    }) // Projection
-                    .FirstOrDefaultAsync();
             });
 
             var teamSizeTask = Task.Run(async () =>
@@ -160,15 +120,12 @@ namespace Marco.Pms.Services.Helpers
             });
 
             // Wait for all parallel database operations to complete.
-            await Task.WhenAll(statusTask, teamSizeTask, infrastructureTask, promotorTask, pmcTask);
+            await Task.WhenAll(statusTask, teamSizeTask, infrastructureTask);
 
             // Get the results from the completed tasks.
-
+            var status = await statusTask;
-            var status = statusTask.Result;
+            var teamSize = await teamSizeTask;
-            var teamSize = teamSizeTask.Result;
+            var (allBuildings, allFloors, allWorkAreas, workSummariesByWorkAreaId) = await infrastructureTask;
-            var (allBuildings, allFloors, allWorkAreas, workSummariesByWorkAreaId) = infrastructureTask.Result;
-            var promotor = promotorTask.Result;
-            var pmc = pmcTask.Result;
 
             // --- Step 2: Process the fetched data and build the MongoDB model ---
 
@@ -259,8 +216,6 @@ namespace Marco.Pms.Services.Helpers
             projectDetails.Buildings = buildingMongoList;
             projectDetails.PlannedWork = totalPlannedWork;
             projectDetails.CompletedWork = totalCompletedWork;
-            projectDetails.Promoter = promotor;
-            projectDetails.PMC = pmc;
 
             try
             {
@@ -279,8 +234,6 @@ namespace Marco.Pms.Services.Helpers
                 return; // Nothing to do
             }
             var projectStatusIds = projects.Select(p => p.ProjectStatusId).Distinct().ToList();
-            var promotorIds = projects.Select(p => p.PromoterId).Distinct().ToList();
-            var pmcsIds = projects.Select(p => p.PMCId).Distinct().ToList();
 
             // --- Step 1: Fetch all required data in maximum parallel ---
             // Each task uses its own DbContext and selects only the required columns (projection).
@@ -295,25 +248,6 @@ namespace Marco.Pms.Services.Helpers
                     .ToDictionaryAsync(s => s.Id);
             });
 
-            var organizationTask = Task.Run(async () =>
-            {
-                using var context = _dbContextFactory.CreateDbContext();
-                return await context.Organizations
-                    .AsNoTracking()
-                    .Where(o => promotorIds.Contains(o.Id) || pmcsIds.Contains(o.Id))
-                    .Select(o => new OrganizationMongoDB
-                    {
-                        Id = o.Id.ToString(),
-                        Name = o.Name,
-                        ContactPerson = o.ContactPerson,
-                        Email = o.Email,
-                        Address = o.Address,
-                        ContactNumber = o.ContactNumber,
-                        SPRID = o.SPRID
-                    }) // Projection
-                    .ToListAsync();
-            });
-
             var teamSizeTask = Task.Run(async () =>
             {
                 using var context = _dbContextFactory.CreateDbContext();
@@ -388,22 +322,20 @@ namespace Marco.Pms.Services.Helpers
             });
 
             // Await the remaining parallel tasks.
-            await Task.WhenAll(statusTask, teamSizeTask, workAreasTask, workSummaryTask, organizationTask);
+            await Task.WhenAll(statusTask, teamSizeTask, workAreasTask, workSummaryTask);
 
             // --- Step 2: Process the fetched data and build the MongoDB models ---
 
-            var allStatuses = statusTask.Result;
+            var allStatuses = await statusTask;
-            var teamSizesByProjectId = teamSizeTask.Result;
+            var teamSizesByProjectId = await teamSizeTask;
-            var allWorkAreas = workAreasTask.Result;
+            var allWorkAreas = await workAreasTask;
-            var workSummariesByWorkAreaId = workSummaryTask.Result;
+            var workSummariesByWorkAreaId = await workSummaryTask;
-            var organizations = organizationTask.Result;
 
             // Create fast in-memory lookups for hierarchical data
             var buildingsByProjectId = allBuildings.ToLookup(b => b.ProjectId);
             var floorsByBuildingId = allFloors.ToLookup(f => f.BuildingId);
             var workAreasByFloorId = allWorkAreas.ToLookup(wa => wa.FloorId);
 
-
             var projectDetailsList = new List<ProjectMongoDB>(projects.Count);
             foreach (var project in projects)
             {
@@ -495,8 +427,6 @@ namespace Marco.Pms.Services.Helpers
                 projectDetails.Buildings = buildingMongoList;
                 projectDetails.PlannedWork = totalPlannedWork;
                 projectDetails.CompletedWork = totalCompletedWork;
-                projectDetails.Promoter = organizations.FirstOrDefault(o => o.Id == project.PromoterId.ToString());
-                projectDetails.PMC = organizations.FirstOrDefault(o => o.Id == project.PMCId.ToString());
 
                 projectDetailsList.Add(projectDetails);
             }
@@ -513,32 +443,11 @@ namespace Marco.Pms.Services.Helpers
         }
         public async Task<bool> UpdateProjectDetailsOnly(Project project)
         {
-            var projectStatusTask = Task.Run(async () =>
+            StatusMaster projectStatus = await _context.StatusMasters
-            {
-                await using var context = await _dbContextFactory.CreateDbContextAsync();
-                return await context.StatusMasters
                 .FirstOrDefaultAsync(s => s.Id == project.ProjectStatusId) ?? new StatusMaster();
-            });
-            var promotorTask = Task.Run(async () =>
-            {
-                await using var context = await _dbContextFactory.CreateDbContextAsync();
-                return await context.Organizations.FirstOrDefaultAsync(o => o.Id == project.PromoterId) ?? new Organization();
-            });
-            var pmcTask = Task.Run(async () =>
-            {
-                await using var context = await _dbContextFactory.CreateDbContextAsync();
-                return await context.Organizations.FirstOrDefaultAsync(o => o.Id == project.PMCId) ?? new Organization();
-            });
-
-            await Task.WhenAll(projectStatusTask, promotorTask, pmcTask);
-
-            var projectStatus = projectStatusTask.Result;
-            var promotor = promotorTask.Result;
-            var pmc = pmcTask.Result;
-
             try
             {
-                bool response = await _projectCache.UpdateProjectDetailsOnlyToCache(project, projectStatus, promotor, pmc);
+                bool response = await _projectCache.UpdateProjectDetailsOnlyToCache(project, projectStatus);
                 return response;
             }
             catch (Exception ex)
@@ -956,28 +865,6 @@ namespace Marco.Pms.Services.Helpers
                 _logger.LogError(ex, "Error occured while deleting all employees from Cache");
             }
         }
-        public async Task ClearAllEmployeesFromCacheByOnlyEmployeeId(Guid employeeId)
-        {
-            try
-            {
-                var response = await _employeeCache.ClearAllEmployeesFromCacheByOnlyEmployeeId(employeeId);
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "Error occured while deleting all employees from Cache");
-            }
-        }
-        public async Task ClearAllEmployeesFromCacheByTenantId(Guid tenantId)
-        {
-            try
-            {
-                var response = await _employeeCache.ClearAllEmployeesFromCacheByTenantId(tenantId);
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "Error occured while deleting all employees from Cache");
-            }
-        }
         public async Task ClearAllEmployees()
         {
             try

Marco.Pms.Services/Helpers/EmployeeHelper.cs

@@ -1,5 +1,4 @@
 
-using AutoMapper;
 using Marco.Pms.DataAccess.Data;
 using Marco.Pms.Model.Employees;
 using Marco.Pms.Model.Mapper;
@@ -14,12 +13,10 @@ namespace MarcoBMS.Services.Helpers
     {
         private readonly ApplicationDbContext _context;
         private readonly ILoggingService _logger;
-        private readonly IMapper _mapper;
+        public EmployeeHelper(ApplicationDbContext context, ILoggingService logger)
-        public EmployeeHelper(ApplicationDbContext context, ILoggingService logger, IMapper mapper)
         {
             _context = context;
             _logger = logger;
-            _mapper = mapper;
         }
         public async Task<Employee> GetEmployeeByID(Guid EmployeeID)
         {
@@ -75,36 +72,38 @@ namespace MarcoBMS.Services.Helpers
             }
         }
 
-        public async Task<List<EmployeeVM>> GetEmployeeByProjectId(Guid organizationId, Guid tenantId, Guid? projectId, bool ShowInActive)
+        public async Task<List<EmployeeVM>> GetEmployeeByProjectId(Guid tenantId, Guid? projectId, bool ShowInActive)
         {
             try
             {
-                List<Employee> employees = new List<Employee>();
+                List<EmployeeVM> result = new List<EmployeeVM>();
                 if (projectId.HasValue)
                 {
-                    employees = await _context.ProjectAllocations
+                    var employeeIds = await _context.ProjectAllocations
-                        .Include(pa => pa.Employee)
+                    .Where(pa => projectId == pa.ProjectId && pa.IsActive && pa.TenantId == tenantId)
-                            .ThenInclude(e => e!.JobRole)
+                    .Select(pa => pa.EmployeeId)
-                        .Where(pa => projectId == pa.ProjectId && pa.IsActive && pa.TenantId == tenantId && pa.Employee != null && pa.Employee.IsActive)
+                    .Distinct()
-                        .Select(pa => pa.Employee!)
+                    .ToListAsync();
+
+                    var employees = await _context.Employees
+                        .Include(fp => fp.JobRole)
+                        .Where(e => employeeIds.Contains(e.Id) && e.JobRole != null && e.IsActive && e.TenantId == tenantId)
                         .Distinct()
                         .ToListAsync();
+                    result = employees.Select(e => e.ToEmployeeVMFromEmployee()).ToList();
+
                 }
                 else if (ShowInActive)
                 {
-                    employees = await _context.Employees
+                    result = await _context.Employees.Where(c => c.TenantId == tenantId && c.IsActive == false).Include(fp => fp.JobRole)
-                        .Include(fp => fp.JobRole)
+                            .Select(c => c.ToEmployeeVMFromEmployee()).ToListAsync();
-                        .Where(c => c.OrganizationId == organizationId && c.IsActive == false)
-                        .ToListAsync();
                 }
                 else
                 {
-                    employees = await _context.Employees
+                    result = await _context.Employees.Where(c => c.TenantId == tenantId && c.IsActive == true).Include(fp => fp.JobRole)
-                        .Include(fp => fp.JobRole)
+                            .Select(c => c.ToEmployeeVMFromEmployee()).ToListAsync();
-                        .Where(c => c.OrganizationId == organizationId && c.IsActive == true)
-                        .ToListAsync();
                 }
-                var result = employees.Select(e => _mapper.Map<EmployeeVM>(e)).Distinct().ToList();
+
                 return result;
             }
             catch (Exception ex)

Marco.Pms.Services/Helpers/ReportHelper.cs

@@ -71,11 +71,11 @@ namespace Marco.Pms.Services.Helpers
                     .Where(a => a.ProjectID == projectId && a.InTime != null && a.InTime.Value.Date == reportDate)
                     .ToListAsync();
 
-                var checkedInEmployeeIds = attendances.Select(a => a.EmployeeId).Distinct().ToHashSet();
+                var checkedInEmployeeIds = attendances.Select(a => a.EmployeeID).Distinct().ToHashSet();
-                var checkoutPendingIds = attendances.Where(a => a.OutTime == null).Select(a => a.EmployeeId).Distinct().ToHashSet();
+                var checkoutPendingIds = attendances.Where(a => a.OutTime == null).Select(a => a.EmployeeID).Distinct().ToHashSet();
                 var regularizationIds = attendances
                     .Where(a => a.Activity == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE)
-                    .Select(a => a.EmployeeId).Distinct().ToHashSet();
+                    .Select(a => a.EmployeeID).Distinct().ToHashSet();
 
                 // Preload buildings, floors, areas
                 List<BuildingMongoDBVM>? buildings = null;
@@ -190,7 +190,6 @@ namespace Marco.Pms.Services.Helpers
                 double totalCompletedWork = workItems.Sum(w => w.CompletedWork);
 
                 var todayAssignedTasks = tasks.Where(t => t.AssignmentDate.Date == reportDate).ToList();
-                var todaysCompletedTasks = tasks.Where(t => t.AssignmentDate.Date == reportDate && t.ReportedById != null).ToList();
                 var reportPending = tasks.Where(t => t.ReportedDate == null).ToList();
 
                 double totalPlannedTask = todayAssignedTasks.Sum(t => t.PlannedTask);
@@ -247,7 +246,7 @@ namespace Marco.Pms.Services.Helpers
                 // Attendance details
                 var performedAttendance = attendances.Select(att =>
                 {
-                    var alloc = projectAllocations.FirstOrDefault(p => p.EmployeeId == att.EmployeeId);
+                    var alloc = projectAllocations.FirstOrDefault(p => p.EmployeeId == att.EmployeeID);
                     var role = jobRoles.FirstOrDefault(r => r.Id == alloc?.JobRoleId);
                     string name = $"{alloc?.Employee?.FirstName ?? ""} {alloc?.Employee?.LastName ?? ""}";
 
@@ -264,17 +263,14 @@ namespace Marco.Pms.Services.Helpers
                 // Fill report
                 statisticReport.TodaysAttendances = checkedInEmployeeIds.Count;
                 statisticReport.TotalEmployees = assignedEmployeeIds.Count;
-                statisticReport.AttendancePercentage = assignedEmployeeIds.Count > 0 ? (checkedInEmployeeIds.Count / assignedEmployeeIds.Count) * 100 : 0;
                 statisticReport.RegularizationPending = regularizationIds.Count;
                 statisticReport.CheckoutPending = checkoutPendingIds.Count;
                 statisticReport.TotalPlannedWork = totalPlannedWork;
                 statisticReport.TotalCompletedWork = totalCompletedWork;
-                statisticReport.CompletionStatus = totalPlannedWork > 0 ? (totalCompletedWork / totalPlannedWork) * 100 : 0;
                 statisticReport.TotalPlannedTask = totalPlannedTask;
                 statisticReport.TotalCompletedTask = totalCompletedTask;
-                statisticReport.AttendancePercentage = totalCompletedTask > 0 ? (totalCompletedTask / totalPlannedTask) * 100 : 0;
+                statisticReport.CompletionStatus = totalPlannedWork > 0 ? totalCompletedWork / totalPlannedWork : 0;
                 statisticReport.TodaysAssignTasks = todayAssignedTasks.Count;
-                statisticReport.TodaysCompletedTasks = todaysCompletedTasks.Count;
                 statisticReport.ReportPending = reportPending.Count;
                 statisticReport.TeamOnSite = teamOnSite;
                 statisticReport.PerformedTasks = performedTasks;
@@ -330,11 +326,11 @@ namespace Marco.Pms.Services.Helpers
                     .Where(a => a.ProjectID == projectId && a.InTime != null && a.InTime.Value.Date == reportDate)
                     .ToListAsync();
 
-                var checkedInEmployeeIds = attendances.Select(a => a.EmployeeId).Distinct().ToHashSet();
+                var checkedInEmployeeIds = attendances.Select(a => a.EmployeeID).Distinct().ToHashSet();
-                var checkoutPendingIds = attendances.Where(a => a.OutTime == null).Select(a => a.EmployeeId).Distinct().ToHashSet();
+                var checkoutPendingIds = attendances.Where(a => a.OutTime == null).Select(a => a.EmployeeID).Distinct().ToHashSet();
                 var regularizationIds = attendances
                     .Where(a => a.Activity == ATTENDANCE_MARK_TYPE.REQUEST_REGULARIZE)
-                    .Select(a => a.EmployeeId).Distinct().ToHashSet();
+                    .Select(a => a.EmployeeID).Distinct().ToHashSet();
 
                 // Preload buildings, floors, areas
                 List<BuildingMongoDBVM>? buildings = null;
@@ -505,7 +501,7 @@ namespace Marco.Pms.Services.Helpers
                 // Attendance details
                 var performedAttendance = attendances.Select(att =>
                 {
-                    var alloc = projectAllocations.FirstOrDefault(p => p.EmployeeId == att.EmployeeId);
+                    var alloc = projectAllocations.FirstOrDefault(p => p.EmployeeId == att.EmployeeID);
                     var role = jobRoles.FirstOrDefault(r => r.Id == alloc?.JobRoleId);
                     string name = $"{alloc?.Employee?.FirstName ?? ""} {alloc?.Employee?.LastName ?? ""}";
 

Marco.Pms.Services/Helpers/RolesHelper.cs

@@ -37,8 +37,7 @@ namespace MarcoBMS.Services.Helpers
                 // --- Step 1: Define the subquery using the main thread's context ---
                 // This is safe because the query is not executed yet.
                 var employeeRoleIdsQuery = _context.EmployeeRoleMappings
-                    .AsNoTracking()
+                    .Where(erm => erm.EmployeeId == EmployeeId && erm.IsEnabled)
-                    .Where(erm => erm.EmployeeId == EmployeeId && erm.IsEnabled && erm.TenantId == tenantId)
                     .Select(erm => erm.RoleId);
 
                 // --- Step 2: Asynchronously update the cache using the DbContextFactory ---
@@ -51,8 +50,7 @@ namespace MarcoBMS.Services.Helpers
 
                         // Now, re-create and execute the query using this new, isolated context.
                         var roleIds = await contextForCache.EmployeeRoleMappings
-                            .AsNoTracking()
+                            .Where(erm => erm.EmployeeId == EmployeeId && erm.IsEnabled)
-                            .Where(erm => erm.EmployeeId == EmployeeId && erm.IsEnabled && erm.TenantId == tenantId)
                             .Select(erm => erm.RoleId)
                             .ToListAsync();
 
@@ -75,12 +73,9 @@ namespace MarcoBMS.Services.Helpers
                 var roleIds = await employeeRoleIdsQuery.ToListAsync();
 
                 var permissionIds = await _context.RolePermissionMappings
-                    .AsNoTracking()
                     .Where(rp => roleIds.Contains(rp.ApplicationRoleId)).Select(rp => rp.FeaturePermissionId).ToListAsync();
 
-                var permissions = await _context.FeaturePermissions
+                var permissions = await _context.FeaturePermissions.Include(f => f.Feature)
-                    .AsNoTracking()
-                    .Include(f => f.Feature)
                     .Where(fp => permissionIds.Contains(fp.Id))
                     .Distinct()
                     .ToListAsync();

Marco.Pms.Services/Helpers/UserHelper.cs

@@ -26,17 +26,12 @@ namespace MarcoBMS.Services.Helpers
             var tenant = _httpContextAccessor.HttpContext?.User.FindFirst("TenantId")?.Value;
             return (tenant != null ? Guid.Parse(tenant) : Guid.Empty);
         }
-        public Guid GetCurrentOrganizationId()
-        {
-            var tenant = _httpContextAccessor.HttpContext?.User.FindFirst("OrganizationId")?.Value;
-            return (tenant != null ? Guid.Parse(tenant) : Guid.Empty);
-        }
         public async Task<Tenant?> GetCurrentTenant()
         {
             var tenantId = _httpContextAccessor.HttpContext?.User.FindFirst("TenantId")?.Value;
             if (tenantId != null)
             {
-                return await _context.Tenants.AsNoTracking().FirstOrDefaultAsync(t => t.Id == Guid.Parse(tenantId));
+                return await _context.Tenants.FirstOrDefaultAsync(t => t.Id == Guid.Parse(tenantId));
             }
             return null;
         }
@@ -54,7 +49,7 @@ namespace MarcoBMS.Services.Helpers
         {
             var user = await GetCurrentUserAsync();
             if (user == null) return new Employee { };
-            var Employee = await _context.Employees.AsNoTracking().Include(a => a.ApplicationUser).Include(e => e.JobRole).FirstOrDefaultAsync(e => e.ApplicationUserId == user.Id && e.IsActive);
+            var Employee = await _context.Employees.Include(e => e.JobRole).FirstOrDefaultAsync(e => e.ApplicationUserId == user.Id && e.IsActive);
             return Employee ?? new Employee { };
         }
 

Marco.Pms.Services/Hub/MarcoHub.cs

@@ -1,8 +1,10 @@
 using MarcoBMS.Services.Service;
+using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.SignalR;
 
 namespace Marco.Pms.Services.Hubs
 {
+    [Authorize]
     public class MarcoHub : Hub
     {
         private readonly ILoggingService _logger;

Marco.Pms.Services/MappingProfiles/MappingProfile.cs

@@ -6,7 +6,6 @@ using Marco.Pms.Model.Dtos.Activities;
 using Marco.Pms.Model.Dtos.AppMenu;
 using Marco.Pms.Model.Dtos.Directory;
 using Marco.Pms.Model.Dtos.DocumentManager;
-using Marco.Pms.Model.Dtos.Employees;
 using Marco.Pms.Model.Dtos.Expenses;
 using Marco.Pms.Model.Dtos.Master;
 using Marco.Pms.Model.Dtos.Organization;
@@ -47,13 +46,6 @@ namespace Marco.Pms.Services.MappingProfiles
             CreateMap<CreateOrganizationDto, Organization>();
             CreateMap<Organization, OrganizationVM>();
             CreateMap<Organization, BasicOrganizationVm>();
-            CreateMap<Organization, OrganizationDetailsVM>();
-            CreateMap<OrganizationMongoDB, BasicOrganizationVm>()
-                .ForMember(
-               dest => dest.Id,
-               // Explicitly and safely convert string Id to Guid Id
-               opt => opt.MapFrom(src => new Guid(src.Id))
-           );
 
             #endregion
 
@@ -128,7 +120,6 @@ namespace Marco.Pms.Services.MappingProfiles
             CreateMap<Project, ProjectListVM>();
             CreateMap<Project, ProjectDto>();
             CreateMap<ProjectMongoDB, ProjectListVM>();
-            CreateMap<ProjectServiceMapping, ProjectServiceMappingVM>();
             CreateMap<ProjectMongoDB, ProjectVM>()
            .ForMember(
                dest => dest.Id,
@@ -185,13 +176,7 @@ namespace Marco.Pms.Services.MappingProfiles
 
             #region ======================================================= Employee =======================================================
 
-            CreateMap<Employee, EmployeeVM>()
+            CreateMap<Employee, EmployeeVM>();
-                .ForMember(
-                dest => dest.JobRole,
-                opt => opt.MapFrom(src => src.JobRole != null ? src.JobRole.Name : ""));
-            CreateMap<CreateUserDto, Employee>();
-            CreateMap<MobileUserManageDto, Employee>();
-
             CreateMap<Employee, BasicEmployeeVM>()
                 .ForMember(
                 dest => dest.JobRoleName,
@@ -379,27 +364,6 @@ namespace Marco.Pms.Services.MappingProfiles
 
             #endregion
 
-            #region ======================================================= Contact Category Master =======================================================
-            CreateMap<CreateContactCategoryDto, ContactCategoryMaster>();
-            CreateMap<UpdateContactCategoryDto, ContactCategoryMaster>();
-            CreateMap<ContactCategoryMaster, ContactCategoryVM>();
-            #endregion
-            #region ======================================================= Contact Tag Master =======================================================
-            CreateMap<CreateContactTagDto, ContactTagMaster>();
-            CreateMap<UpdateContactTagDto, ContactTagMaster>();
-            CreateMap<ContactTagMaster, ContactTagVM>();
-            #endregion
-            #region ======================================================= Expenses Status Master =======================================================
-            #endregion
-            #region ======================================================= Expenses Status Master =======================================================
-            #endregion
-            #region ======================================================= Expenses Status Master =======================================================
-            #endregion
-            #region ======================================================= Expenses Status Master =======================================================
-            #endregion
-            #region ======================================================= Expenses Status Master =======================================================
-            #endregion
-
             #endregion
 
             #region ======================================================= Document =======================================================

Marco.Pms.Services/Service/DirectoryService.cs

@@ -956,14 +956,10 @@ namespace Marco.Pms.Services.Service
             try
             {
                 var contact = _mapper.Map<Contact>(createContact);
-                if (string.IsNullOrWhiteSpace(createContact.Description))
+                if (string.IsNullOrWhiteSpace(createContact.Name))
                 {
                     contact.Description = string.Empty;
                 }
-                if (string.IsNullOrWhiteSpace(createContact.Designation))
-                {
-                    contact.Designation = string.Empty;
-                }
                 contact.CreatedAt = DateTime.UtcNow;
                 contact.CreatedById = loggedInEmployeeId;
                 contact.TenantId = tenantId;
@@ -2035,11 +2031,9 @@ namespace Marco.Pms.Services.Service
 
             try
             {
-                var (hasAdminPermission, hasManagerPermission, hasUserPermission) = await CheckPermissionsAsync(loggedInEmployee.Id);
+                var bucketIds = await _context.ContactBucketMappings.Where(cb => cb.ContactId == noteDto.ContactId).Select(cb => cb.BucketId).ToListAsync();
-
+                var hasContactAccess = await _context.EmployeeBucketMappings.AnyAsync(eb => bucketIds.Contains(eb.BucketId) && eb.EmployeeId == loggedInEmployee.Id);
-                var bucketIds = await _context.ContactBucketMappings.AsNoTracking().Where(cb => cb.ContactId == noteDto.ContactId).Select(cb => cb.BucketId).ToListAsync();
+                if (!hasContactAccess)
-                var hasContactAccess = await _context.EmployeeBucketMappings.AsNoTracking().AnyAsync(eb => bucketIds.Contains(eb.BucketId) && eb.EmployeeId == loggedInEmployee.Id);
-                if (!hasAdminPermission && !hasContactAccess)
                 {
                     _logger.LogWarning("Employee {EmployeeId} does not have permission to delete contact {ContactId}",
                          loggedInEmployee.Id, noteDto.ContactId);
@@ -2271,11 +2265,9 @@ namespace Marco.Pms.Services.Service
                 return ApiResponse<object>.ErrorResponse("Note not found", "Note not found", 404);
             }
 
-            var (hasAdminPermission, hasManagerPermission, hasUserPermission) = await CheckPermissionsAsync(loggedInEmployee.Id);
+            var bucketIds = await _context.ContactBucketMappings.Where(cb => cb.ContactId == note.ContactId).Select(cb => cb.BucketId).ToListAsync();
-
+            var hasContactAccess = await _context.EmployeeBucketMappings.AnyAsync(eb => bucketIds.Contains(eb.BucketId) && eb.EmployeeId == loggedInEmployee.Id);
-            var bucketIds = await _context.ContactBucketMappings.AsNoTracking().Where(cb => cb.ContactId == note.ContactId).Select(cb => cb.BucketId).ToListAsync();
+            if (hasContactAccess)
-            var hasContactAccess = await _context.EmployeeBucketMappings.AsNoTracking().AnyAsync(eb => bucketIds.Contains(eb.BucketId) && eb.EmployeeId == loggedInEmployee.Id);
-            if (!hasAdminPermission && !hasContactAccess)
             {
                 _logger.LogWarning("Employee {EmployeeId} does not have permission to delete contact {ContactId}",
                      loggedInEmployee.Id, note.ContactId);
@@ -2619,7 +2611,6 @@ namespace Marco.Pms.Services.Service
                 var bucketTask = bucketContext.Buckets
                     .Include(b => b.CreatedBy)
                         .ThenInclude(e => e!.JobRole)
-                    .AsNoTracking()
                     .FirstOrDefaultAsync(b => b.Id == bucketDto.Id && b.TenantId == tenantId);
 
                 // Await all parallel tasks

Marco.Pms.Services/Service/ExpensesService.cs

@@ -23,7 +23,6 @@ using MarcoBMS.Services.Service;
 using Microsoft.CodeAnalysis;
 using Microsoft.EntityFrameworkCore;
 using System.Text.Json;
-using System.Text.RegularExpressions;
 using Document = Marco.Pms.Model.DocumentManager.Document;
 
 namespace Marco.Pms.Services.Service
@@ -120,12 +119,8 @@ namespace Marco.Pms.Services.Service
                 // 2. --- Deserialize Filter and Apply ---
                 ExpensesFilter? expenseFilter = TryDeserializeFilter(filter);
 
-                //var (totalPages, totalCount, cacheList) = await _cache.GetExpenseListAsync(tenantId, loggedInEmployeeId, hasViewAllPermissionTask.Result, hasViewSelfPermissionTask.Result,
+                var (totalPages, totalCount, cacheList) = await _cache.GetExpenseListAsync(tenantId, loggedInEmployeeId, hasViewAllPermissionTask.Result, hasViewSelfPermissionTask.Result,
-                //    pageNumber, pageSize, expenseFilter, searchString);
+                    pageNumber, pageSize, expenseFilter, searchString);
-
-                List<ExpenseDetailsMongoDB>? cacheList = null;
-                var totalPages = 0;
-                var totalCount = 0;
 
                 // 3. --- Build Base Query and Apply Permissions ---
                 // Start with a base IQueryable. Filters will be chained onto this.
@@ -273,8 +268,7 @@ namespace Marco.Pms.Services.Service
         {
             try
             {
-                //ExpenseDetailsMongoDB? expenseDetails = await _cache.GetExpenseDetailsById(id, tenantId);
+                var expenseDetails = await _cache.GetExpenseDetailsById(id, tenantId);
-                ExpenseDetailsMongoDB? expenseDetails = null;
                 if (expenseDetails == null)
                 {
                     var expense = await _context.Expenses.AsNoTracking().FirstOrDefaultAsync(e => e.Id == id && e.TenantId == tenantId);
@@ -319,13 +313,6 @@ namespace Marco.Pms.Services.Service
                     {
                         status.PermissionIds = permissionStatusMappings.Where(ps => ps.StatusId == status.Id).Select(ps => ps.PermissionIds).FirstOrDefault();
                     }
-                    int index = vm.NextStatus.FindIndex(ns => ns.DisplayName == "Reject");
-                    if (index > -1)
-                    {
-                        var item = vm.NextStatus[index];
-                        vm.NextStatus.RemoveAt(index);
-                        vm.NextStatus.Insert(0, item);
-                    }
                 }
                 vm.ExpensesReimburse = _mapper.Map<ExpensesReimburseVM>(expensesReimburse);
 
@@ -495,15 +482,6 @@ namespace Marco.Pms.Services.Service
                     await using var dbContext = await _dbContextFactory.CreateDbContextAsync();
                     return await dbContext.PaymentModeMatser.AsNoTracking().FirstOrDefaultAsync(pm => pm.Id == dto.PaymentModeId);
                 });
-                var expenseUIdTask = Task.Run(async () =>
-                {
-                    await using var dbContext = await _dbContextFactory.CreateDbContextAsync();
-                    var result = await dbContext.Expenses
-                    .Where(e => !string.IsNullOrWhiteSpace(e.ExpenseUId)).ToListAsync();
-                    return result
-                    .Select(e => ExtractNumber(e.ExpenseUId))
-                    .OrderByDescending(id => id).FirstOrDefault();
-                });
                 var statusMappingTask = Task.Run(async () =>
                 {
                     await using var dbContext = await _dbContextFactory.CreateDbContextAsync();
@@ -523,7 +501,10 @@ namespace Marco.Pms.Services.Service
 
 
                 // Await all prerequisite checks at once.
-                await Task.WhenAll(hasUploadPermissionTask, hasProjectPermissionTask, projectTask, expenseTypeTask, paymentModeTask, statusMappingTask, paidByTask, expenseUIdTask);
+                await Task.WhenAll(
+                    hasUploadPermissionTask, hasProjectPermissionTask,
+                    projectTask, expenseTypeTask, paymentModeTask, statusMappingTask, paidByTask
+                );
 
                 // 2. Aggregate and Check Results
                 if (!await hasUploadPermissionTask || !await hasProjectPermissionTask)
@@ -533,12 +514,11 @@ namespace Marco.Pms.Services.Service
                 }
 
                 var validationErrors = new List<string>();
-                var project = projectTask.Result;
+                var project = await projectTask;
-                var expenseType = expenseTypeTask.Result;
+                var expenseType = await expenseTypeTask;
-                var paymentMode = paymentModeTask.Result;
+                var paymentMode = await paymentModeTask;
-                var statusMapping = statusMappingTask.Result;
+                var statusMapping = await statusMappingTask;
-                var paidBy = paidByTask.Result;
+                var paidBy = await paidByTask;
-                var lastExpenseUId = expenseUIdTask.Result;
 
                 if (project == null) validationErrors.Add("Project not found.");
                 if (paidBy == null) validationErrors.Add("Paid by employee not found");
@@ -554,10 +534,9 @@ namespace Marco.Pms.Services.Service
                     _logger.LogWarning("Expense creation failed due to validation errors: {ValidationErrors}", errorMessage);
                     return ApiResponse<object>.ErrorResponse("Invalid input data.", errorMessage, 400);
                 }
-                var currentexpenseUId = (lastExpenseUId + 1).ToString("D5");
+
                 // 3. Entity Creation
                 var expense = _mapper.Map<Expenses>(dto);
-                expense.ExpenseUId = $"EX-{currentexpenseUId}";
                 expense.CreatedById = loggedInEmployee.Id;
                 expense.CreatedAt = DateTime.UtcNow;
                 expense.TenantId = tenantId;
@@ -1100,13 +1079,6 @@ namespace Marco.Pms.Services.Service
         #endregion
 
         #region =================================================================== Helper Functions ===================================================================
-
-        private int ExtractNumber(string id)
-        {
-            // Extract trailing number; handles EX_0001, EX-0001, EX0001
-            var m = Regex.Match(id ?? string.Empty, @"(\d+)$");
-            return m.Success ? int.Parse(m.Value) : int.MinValue; // put invalid IDs at the bottom
-        }
         private static object ExceptionMapper(Exception ex)
         {
             return new

Marco.Pms.Services/Service/PermissionServices.cs

@@ -79,7 +79,6 @@ namespace Marco.Pms.Services.Service
             {
                 // Fetch permissions explicitly assigned to this employee in the project.
                 var projectLevelPermissionIds = await _context.ProjectLevelPermissionMappings
-                    .AsNoTracking()
                     .Where(pl => pl.ProjectId == projectId.Value && pl.EmployeeId == employeeId)
                     .Select(pl => pl.PermissionId)
                     .ToListAsync();
@@ -98,7 +97,6 @@ namespace Marco.Pms.Services.Service
 
                     // Get all feature permissions under those modules where the user didn't have explicit project-level grants.
                     var allOverriddenPermissions = await _context.FeaturePermissions
-                        .AsNoTracking()
                         .Where(fp => projectLevelModuleIds.Contains(fp.FeatureId) &&
                                      !projectLevelPermissionIds.Contains(fp.Id))
                         .Select(fp => fp.Id)
@@ -140,12 +138,12 @@ namespace Marco.Pms.Services.Service
                 var hasPermission = await HasPermission(PermissionsMaster.ManageProject, employeeId);
                 if (hasPermission)
                 {
-                    var projects = await _context.Projects.AsNoTracking().Where(c => c.TenantId == LoggedInEmployee.TenantId).ToListAsync();
+                    var projects = await _context.Projects.Where(c => c.TenantId == LoggedInEmployee.TenantId).ToListAsync();
                     projectIds = projects.Select(p => p.Id).ToList();
                 }
                 else
                 {
-                    var allocation = await _context.ProjectAllocations.AsNoTracking().Where(c => c.EmployeeId == employeeId && c.IsActive).ToListAsync();
+                    var allocation = await _context.ProjectAllocations.Where(c => c.EmployeeId == employeeId && c.IsActive).ToListAsync();
                     if (!allocation.Any())
                     {
                         return false;

Marco.Pms.Services/Service/ServiceInterfaces/IProjectServices.cs

@@ -23,9 +23,8 @@ namespace Marco.Pms.Services.Service.ServiceInterfaces
         Task<ApiResponse<object>> GetProjectsByEmployeeAsync(Guid employeeId, Guid tenantId, Employee loggedInEmployee);
         Task<ApiResponse<List<ProjectAllocationVM>>> AssigneProjectsToEmployeeAsync(List<ProjectsAllocationDto> projectAllocationDtos, Guid employeeId, Guid tenantId, Employee loggedInEmployee);
         Task<ApiResponse<object>> GetProjectByEmployeeBasicAsync(Guid employeeId, Guid tenantId, Employee loggedInEmployee);
-        Task<ApiResponse<object>> GetProjectTeamByServiceAndOrganizationAsync(Guid projectId, Guid? serviceId, Guid? organizationId, Guid tenantId, Employee loggedInEmployee);
 
-        Task<ApiResponse<object>> GetInfraDetailsAsync(Guid projectId, Guid? serviceId, Guid tenantId, Employee loggedInEmployee);
+        Task<ApiResponse<object>> GetInfraDetailsAsync(Guid projectId, Guid tenantId, Employee loggedInEmployee);
         Task<ApiResponse<object>> GetWorkItemsAsync(Guid workAreaId, Guid? serviceId, Guid tenantId, Employee loggedInEmployee);
         Task<ApiResponse<object>> GetTasksByEmployeeAsync(Guid employeeId, DateTime fromDate, DateTime toDate, Guid tenantId, Employee loggedInEmployee);
         Task<ServiceResponse> ManageProjectInfraAsync(List<InfraDto> infraDtos, Guid tenantId, Employee loggedInEmployee);
@@ -49,7 +48,6 @@ namespace Marco.Pms.Services.Service.ServiceInterfaces
         Task<ApiResponse<object>> DeassignServiceToProjectAsync(DeassignServiceDto model, Guid tenantId, Employee loggedInEmployee);
 
         Task<ApiResponse<object>> GetAssignedOrganizationsToProjectAsync(Guid projectId, Guid tenantId, Employee loggedInEmployee);
-        Task<ApiResponse<object>> GetAssignedOrganizationsToProjectForDropdownAsync(Guid projectId, Guid tenantId, Employee loggedInEmployee);
 
     }
 }