Marco.Pms.DataAccess/Data/ApplicationDbContext.cs

@@ -44,7 +44,6 @@ namespace Marco.Pms.DataAccess.Data
         public DbSet<TaskAllocation> TaskAllocations { get; set; }
         public DbSet<TaskComment> TaskComments { get; set; }
         public DbSet<TaskMembers> TaskMembers { get; set; }
-        public DbSet<TaskAttachment> TaskAttachments { get; set; }
         public DbSet<Attendance> Attendes { get; set; }
         public DbSet<AttendanceLog> AttendanceLogs { get; set; }
         public DbSet<Employee> Employees { get; set; }

Marco.Pms.DataAccess/Migrations/20250612094243_Added_TaskAttachments_Table.cs

@@ -1,36 +0,0 @@
-using System;
-using Microsoft.EntityFrameworkCore.Migrations;
-
-#nullable disable
-
-namespace Marco.Pms.DataAccess.Migrations
-{
-    /// <inheritdoc />
-    public partial class Added_TaskAttachments_Table : Migration
-    {
-        /// <inheritdoc />
-        protected override void Up(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.CreateTable(
-                name: "TaskAttachments",
-                columns: table => new
-                {
-                    Id = table.Column<Guid>(type: "char(36)", nullable: false, collation: "ascii_general_ci"),
-                    ReferenceId = table.Column<Guid>(type: "char(36)", nullable: false, collation: "ascii_general_ci"),
-                    DocumentId = table.Column<Guid>(type: "char(36)", nullable: false, collation: "ascii_general_ci")
-                },
-                constraints: table =>
-                {
-                    table.PrimaryKey("PK_TaskAttachments", x => x.Id);
-                })
-                .Annotation("MySql:CharSet", "utf8mb4");
-        }
-
-        /// <inheritdoc />
-        protected override void Down(MigrationBuilder migrationBuilder)
-        {
-            migrationBuilder.DropTable(
-                name: "TaskAttachments");
-        }
-    }
-}

Marco.Pms.DataAccess/Migrations/ApplicationDbContextModelSnapshot.cs

@@ -63,23 +63,6 @@ namespace Marco.Pms.DataAccess.Migrations
                     b.ToTable("TaskAllocations");
                 });
 
-            modelBuilder.Entity("Marco.Pms.Model.Activities.TaskAttachment", b =>
-                {
-                    b.Property<Guid>("Id")
-                        .ValueGeneratedOnAdd()
-                        .HasColumnType("char(36)");
-
-                    b.Property<Guid>("DocumentId")
-                        .HasColumnType("char(36)");
-
-                    b.Property<Guid>("ReferenceId")
-                        .HasColumnType("char(36)");
-
-                    b.HasKey("Id");
-
-                    b.ToTable("TaskAttachments");
-                });
-
             modelBuilder.Entity("Marco.Pms.Model.Activities.TaskComment", b =>
                 {
                     b.Property<Guid>("Id")

Marco.Pms.Model/Activities/TaskAttachment.cs

@@ -1,9 +0,0 @@
-namespace Marco.Pms.Model.Activities
-{
-    public class TaskAttachment
-    {
-        public Guid Id { get; set; }
-        public Guid ReferenceId { get; set; }
-        public Guid DocumentId { get; set; }
-    }
-}

Marco.Pms.Model/Activities/TaskImages.cs

@@ -0,0 +1,17 @@
+using System.ComponentModel.DataAnnotations.Schema;
+using Microsoft.AspNetCore.Mvc.ModelBinding.Validation;
+
+namespace Marco.Pms.Model.Activities
+{
+    public class TaskImages
+    {
+        public Guid Id { get; set; }
+
+        public Guid TaskAllocationId { get; set; }
+        [ValidateNever]
+        [ForeignKey(nameof(TaskAllocationId))]
+        public TaskAllocation? TaskAllocation { get; set; }
+
+        public string? ImagePath { get; set; }
+    }
+}

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

@@ -1,12 +1,11 @@
-using Marco.Pms.Model.Utilities;
+namespace Marco.Pms.Model.Dtos.Activities
-
-namespace Marco.Pms.Model.Dtos.Activities
 {
     public class CreateCommentDto
     {
         public Guid TaskAllocationId { get; set; }
+
         public DateTime CommentDate { get; set; }
         public string? Comment { get; set; }
-        public List<FileUploadModel>? Images { get; set; }
+
     }
 }

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

@@ -1,6 +1,4 @@
-using Marco.Pms.Model.Utilities;
+namespace Marco.Pms.Model.Dtos.Activities
-
-namespace Marco.Pms.Model.Dtos.Activities
 {
     public class ReportTaskDto
     {
@@ -9,6 +7,5 @@ namespace Marco.Pms.Model.Dtos.Activities
         public DateTime ReportedDate { get; set; }
         public string? Comment { get; set; }
         public List<ReportCheckListDto>? CheckList { get; set; }
-        public List<FileUploadModel>? Images { get; set; }
     }
 }

Marco.Pms.Model/Dtos/Directory/UpdateContactDto.cs

@@ -8,7 +8,7 @@
         public List<UpdateContactPhoneDto>? ContactPhones { get; set; }
         public List<UpdateContactEmailDto>? ContactEmails { get; set; }
         public List<Guid>? BucketIds { get; set; }
-        public Guid? ContactCategoryId { get; set; }
+        public Guid ContactCategoryId { get; set; }
         public string? Description { get; set; }
         public string? Organization { get; set; }
         public string? Address { get; set; }

Marco.Pms.Model/Mapper/ActivitiesMapper.cs

@@ -101,7 +101,6 @@ namespace Marco.Pms.Model.Mapper
             {
                 Id = taskAllocation.Id,
                 AssignmentDate = taskAllocation.AssignmentDate,
-                Description = taskAllocation.Description,
                 PlannedTask = taskAllocation.PlannedTask,
                 ReportedDate = taskAllocation.ReportedDate,
                 CompletedTask = taskAllocation.CompletedTask,

Marco.Pms.Model/ViewModels/Activities/CommentVM.cs

@@ -8,6 +8,5 @@
         public string? Comment { get; set; }
         public Guid CommentedBy { get; set; }
         public BasicEmployeeVM? Employee { get; set; }
-        public List<string>? PreSignedUrls { get; set; }
     }
 }

Marco.Pms.Model/ViewModels/Activities/ListTaskVM.cs

@@ -10,9 +10,7 @@ namespace Marco.Pms.Model.ViewModels.Activities
         public double PlannedTask { get; set; }
         public double CompletedTask { get; set; }
         public BasicEmployeeVM? AssignedBy { get; set; }
-        public string? Description { get; set; }
         public Guid WorkItemId { get; set; }
-        public List<string>? ReportedPreSignedUrls { get; set; }
         public WorkItem? WorkItem { get; set; }
         public List<BasicEmployeeVM>? teamMembers { get; set; }
         public List<CommentVM>? comments { get; set; }

Marco.Pms.Model/ViewModels/Activities/ReportTaskVM.cs

@@ -10,6 +10,7 @@
         public string? Description { get; set; }
         public Guid AssignedBy { get; set; }
         public Guid WorkItemId { get; set; }
+
         public List<CommentVM>? Comments { get; set; }
         public List<CheckListVM>? checkList { get; set; }
     }

Marco.Pms.Model/ViewModels/Activities/TaskVM.cs

@@ -13,7 +13,6 @@ namespace Marco.Pms.Model.ViewModels.Activities
         public string? Description { get; set; }
         public string? AssignBy { get; set; }
         public WorkItem? WorkItem { get; set; }
-        public List<string>? PreSignedUrls { get; set; }
         public List<CommentVM>? Comments { get; set; }
         public List<EmployeeVM>? TeamMembers { get; set; }
     }

Marco.Pms.Services/Controllers/AttendanceController.cs

@@ -699,6 +699,7 @@ namespace MarcoBMS.Services.Controllers
                 }
                 Document? document = null;
                 var Image = recordAttendanceDot.Image;
+                var objectKey = string.Empty;
                 var preSignedUrl = string.Empty;
 
                 if (Image != null && Image.ContentType != null)
@@ -707,16 +708,7 @@ namespace MarcoBMS.Services.Controllers
                     if (string.IsNullOrEmpty(Image.Base64Data))
                         return BadRequest(ApiResponse<object>.ErrorResponse("Base64 data is missing", "Base64 data is missing", 400));
 
-                    //If base64 has a data URI prefix, strip it
+                    objectKey = await _s3Service.UploadFileAsync(Image.Base64Data, TenantId, "attendance");
-                    var base64 = Image.Base64Data.Contains(",")
-                        ? Image.Base64Data.Substring(Image.Base64Data.IndexOf(",") + 1)
-                        : Image.Base64Data;
-
-                    string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                    string fileName = _s3Service.GenerateFileName(fileType, TenantId, "attendance");
-
-                    string objectKey = $"tenant-{TenantId}/Employee/{recordAttendanceDot.EmployeeID}/Attendance/{fileName}";
-                    await _s3Service.UploadFileAsync(base64, fileType, objectKey);
                     preSignedUrl = _s3Service.GeneratePreSignedUrlAsync(objectKey);
 
                     document = new Document
@@ -731,7 +723,6 @@ namespace MarcoBMS.Services.Controllers
                     };
                     _context.Documents.Add(document);
 
-
                     await _context.SaveChangesAsync();
                 }
 

Marco.Pms.Services/Controllers/ForumController.cs

@@ -68,16 +68,7 @@ namespace Marco.Pms.Services.Controllers
                         return BadRequest(ApiResponse<object>.ErrorResponse("Base64 data is missing", "Base64 data is missing", 400));
                     }
 
-                    //If base64 has a data URI prefix, strip it
+                    var objectKey = await _s3Service.UploadFileAsync(Image.Base64Data, tenantId, "forum");
-                    var base64 = Image.Base64Data.Contains(",")
-                        ? Image.Base64Data.Substring(Image.Base64Data.IndexOf(",") + 1)
-                        : Image.Base64Data;
-
-                    string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                    string fileName = _s3Service.GenerateFileName(fileType, tenantId, "forum");
-
-                    string objectKey = $"tenant-{tenantId}/project-{createTicketDto.LinkedProjectId}/froum/{fileName}";
-                    await _s3Service.UploadFileAsync(base64, fileType, objectKey);
 
                     Document document = attachmentDto.ToDocumentFromForumAttachmentDto(objectKey, objectKey, createTicketDto.CreatedAt, tenantId);
                     _context.Documents.Add(document);
@@ -191,16 +182,7 @@ namespace Marco.Pms.Services.Controllers
                                 return BadRequest(ApiResponse<object>.ErrorResponse("Base64 data is missing", "Base64 data is missing", 400));
                             }
 
-                            //If base64 has a data URI prefix, strip it
+                            var objectKey = await _s3Service.UploadFileAsync(Image.Base64Data, tenantId, "forum");
-                            var base64 = Image.Base64Data.Contains(",")
-                                ? Image.Base64Data.Substring(Image.Base64Data.IndexOf(",") + 1)
-                                : Image.Base64Data;
-
-                            string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                            string fileName = _s3Service.GenerateFileName(fileType, tenantId, "forum");
-
-                            string objectKey = $"tenant-{tenantId}/project-{updateTicketDto.LinkedProjectId}/froum/{fileName}";
-                            await _s3Service.UploadFileAsync(base64, fileType, objectKey);
 
                             Document document = attachmentDto.ToDocumentFromUpdateAttachmentDto(objectKey, objectKey, updateTicketDto.CreatedAt, tenantId);
                             _context.Documents.Add(document);
@@ -347,14 +329,6 @@ namespace Marco.Pms.Services.Controllers
             List<TicketAttachment> attachments = new List<TicketAttachment>();
             List<Document> documents = new List<Document>();
 
-            TicketForum? ticket = await _context.Tickets.FirstOrDefaultAsync(t => t.Id == addCommentDto.TicketId);
-
-            if (ticket == null)
-            {
-                _logger.LogError("Ticket {TicketId} not Found in database", addCommentDto.TicketId);
-                return NotFound(ApiResponse<object>.ErrorResponse("Ticket not Found", "Ticket not Found", 404));
-            }
-
             TicketComment comment = addCommentDto.ToTicketCommentFromAddCommentDto(tenantId);
             _context.TicketComments.Add(comment);
             await _context.SaveChangesAsync();
@@ -370,16 +344,7 @@ namespace Marco.Pms.Services.Controllers
                         return BadRequest(ApiResponse<object>.ErrorResponse("Base64 data is missing", "Base64 data is missing", 400));
                     }
 
-                    //If base64 has a data URI prefix, strip it
+                    var objectKey = await _s3Service.UploadFileAsync(Image.Base64Data, tenantId, "forum");
-                    var base64 = Image.Base64Data.Contains(",")
-                        ? Image.Base64Data.Substring(Image.Base64Data.IndexOf(",") + 1)
-                        : Image.Base64Data;
-
-                    string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                    string fileName = _s3Service.GenerateFileName(fileType, tenantId, "forum");
-
-                    string objectKey = $"tenant-{tenantId}/project-{ticket.LinkedProjectId}/froum/{fileName}";
-                    await _s3Service.UploadFileAsync(base64, fileType, objectKey);
 
                     Document document = attachmentDto.ToDocumentFromForumAttachmentDto(objectKey, objectKey, addCommentDto.SentAt, tenantId);
                     _context.Documents.Add(document);
@@ -431,14 +396,6 @@ namespace Marco.Pms.Services.Controllers
             Guid tenantId = _userHelper.GetTenantId();
             List<TicketAttachment> attachments = new List<TicketAttachment>();
 
-            TicketForum? ticket = await _context.Tickets.FirstOrDefaultAsync(t => t.Id == updateCommentDto.TicketId);
-
-            if (ticket == null)
-            {
-                _logger.LogError("Ticket {TicketId} not Found in database", updateCommentDto.TicketId);
-                return NotFound(ApiResponse<object>.ErrorResponse("Ticket not Found", "Ticket not Found", 404));
-            }
-
             TicketComment existingComment = await _context.TicketComments.AsNoTracking().FirstOrDefaultAsync(c => c.Id == updateCommentDto.Id) ?? new TicketComment();
             TicketComment updateComment = updateCommentDto.ToTicketCommentFromUpdateCommentDto(tenantId, existingComment);
             _context.TicketComments.Update(updateComment);
@@ -462,16 +419,7 @@ namespace Marco.Pms.Services.Controllers
                             return BadRequest(ApiResponse<object>.ErrorResponse("Base64 data is missing", "Base64 data is missing", 400));
                         }
 
-                        //If base64 has a data URI prefix, strip it
+                        var objectKey = await _s3Service.UploadFileAsync(Image.Base64Data, tenantId, "forum");
-                        var base64 = Image.Base64Data.Contains(",")
-                            ? Image.Base64Data.Substring(Image.Base64Data.IndexOf(",") + 1)
-                            : Image.Base64Data;
-
-                        string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                        string fileName = _s3Service.GenerateFileName(fileType, tenantId, "forum");
-
-                        string objectKey = $"tenant-{tenantId}/project-{ticket.LinkedProjectId}/froum/{fileName}";
-                        await _s3Service.UploadFileAsync(base64, fileType, objectKey);
 
                         Document document = attachmentDto.ToDocumentFromUpdateAttachmentDto(objectKey, objectKey, existingComment.SentAt, tenantId);
                         _context.Documents.Add(document);
@@ -543,16 +491,6 @@ namespace Marco.Pms.Services.Controllers
             Guid tenantId = _userHelper.GetTenantId();
             List<TicketAttachmentVM> ticketAttachmentVMs = new List<TicketAttachmentVM>();
 
-            List<Guid> ticketIds = forumAttachmentDtos.Select(f => f.TicketId.HasValue ? f.TicketId.Value : Guid.Empty).ToList();
-
-            List<TicketForum> tickets = await _context.Tickets.Where(t => ticketIds.Contains(t.Id)).ToListAsync();
-
-            if (tickets == null || tickets.Count > 0)
-            {
-                _logger.LogError("Tickets not Found in database");
-                return NotFound(ApiResponse<object>.ErrorResponse("Ticket not Found", "Ticket not Found", 404));
-            }
-
             TicketAttachment attachment = new TicketAttachment();
 
             foreach (var forumAttachmentDto in forumAttachmentDtos)
@@ -567,17 +505,7 @@ namespace Marco.Pms.Services.Controllers
                     _logger.LogError("ticket ID is missing");
                     return BadRequest(ApiResponse<object>.ErrorResponse("ticket ID is missing", "ticket ID is missing", 400));
                 }
-                var ticket = tickets.FirstOrDefault(t => t.Id == forumAttachmentDto.TicketId);
+                var objectKey = await _s3Service.UploadFileAsync(forumAttachmentDto.Base64Data, tenantId, "forum");
-                //If base64 has a data URI prefix, strip it
-                var base64 = forumAttachmentDto.Base64Data.Contains(",")
-                    ? forumAttachmentDto.Base64Data.Substring(forumAttachmentDto.Base64Data.IndexOf(",") + 1)
-                    : forumAttachmentDto.Base64Data;
-
-                string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                string fileName = _s3Service.GenerateFileName(fileType, tenantId, "forum");
-
-                string objectKey = $"tenant-{tenantId}/project-{ticket?.LinkedProjectId}/froum/{fileName}";
-                await _s3Service.UploadFileAsync(base64, fileType, objectKey);
 
                 Document document = forumAttachmentDto.ToDocumentFromForumAttachmentDto(objectKey, objectKey, forumAttachmentDto.SentAt, tenantId);
                 _context.Documents.Add(document);

Marco.Pms.Services/Controllers/TaskController.cs

@@ -8,13 +8,10 @@ using Marco.Pms.Model.Projects;
 using Marco.Pms.Model.Utilities;
 using Marco.Pms.Model.ViewModels.Activities;
 using Marco.Pms.Model.ViewModels.Employee;
-using Marco.Pms.Services.Service;
 using MarcoBMS.Services.Helpers;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
-using Microsoft.CodeAnalysis;
 using Microsoft.EntityFrameworkCore;
-using Document = Marco.Pms.Model.DocumentManager.Document;
 
 namespace MarcoBMS.Services.Controllers
 {
@@ -26,14 +23,12 @@ namespace MarcoBMS.Services.Controllers
     {
         private readonly ApplicationDbContext _context;
         private readonly UserHelper _userHelper;
-        private readonly S3UploadService _s3Service;
 
 
-        public TaskController(ApplicationDbContext context, UserHelper userHelper, S3UploadService s3Service)
+        public TaskController(ApplicationDbContext context, UserHelper userHelper)
         {
             _context = context;
             _userHelper = userHelper;
-            _s3Service = s3Service;
         }
 
         private Guid GetTenantId()
@@ -89,6 +84,7 @@ namespace MarcoBMS.Services.Controllers
             return Ok(ApiResponse<object>.SuccessResponse(response, "Task assignned successfully", 200));
         }
 
+
         [HttpPost("report")]
         public async Task<IActionResult> ReportTaskProgress([FromBody] ReportTaskDto reportTask)
         {
@@ -108,13 +104,10 @@ namespace MarcoBMS.Services.Controllers
 
             var checkListIds = reportTask.CheckList != null ? reportTask.CheckList.Select(c => c.Id).ToList() : new List<Guid>();
             var checkList = await _context.ActivityCheckLists.Where(c => checkListIds.Contains(c.Id)).ToListAsync();
-            if (taskAllocation == null || taskAllocation.WorkItem == null)
+            if (taskAllocation == null)
             {
                 return BadRequest(ApiResponse<object>.ErrorResponse("No such task has been allocated.", "No such task has been allocated.", 400));
             }
-            WorkArea workArea = await _context.WorkAreas.Include(a => a.Floor).FirstOrDefaultAsync(a => a.Id == taskAllocation.WorkItem.WorkAreaId) ?? new WorkArea();
-
-            var bulding = await _context.Buildings.FirstOrDefaultAsync(b => b.Id == (workArea.Floor != null ? workArea.Floor.BuildingId : Guid.Empty));
             if (taskAllocation.WorkItem != null)
             {
                 if (taskAllocation.CompletedTask != 0)
@@ -150,49 +143,6 @@ namespace MarcoBMS.Services.Controllers
             _context.CheckListMappings.AddRange(checkListMappings);
             var comment = reportTask.ToCommentFromReportTaskDto(tenantId, Employee.Id);
 
-            var Images = reportTask.Images;
-
-            if (Images != null && Images.Count > 0)
-            {
-
-                foreach (var Image in Images)
-                {
-
-                    if (string.IsNullOrEmpty(Image.Base64Data))
-                        return BadRequest(ApiResponse<object>.ErrorResponse("Base64 data is missing", "Base64 data is missing", 400));
-
-                    //If base64 has a data URI prefix, strip it
-                    var base64 = Image.Base64Data.Contains(",")
-                        ? Image.Base64Data.Substring(Image.Base64Data.IndexOf(",") + 1)
-                        : Image.Base64Data;
-
-                    string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                    string fileName = _s3Service.GenerateFileName(fileType, tenantId, "task_report");
-
-                    string objectKey = $"tenant-{tenantId}/project-{bulding?.ProjectId}/Actitvity/{fileName}";
-                    await _s3Service.UploadFileAsync(base64, fileType, objectKey);
-
-                    Document document = new Document
-                    {
-                        FileName = Image.FileName ?? "",
-                        ContentType = Image.ContentType ?? "",
-                        S3Key = objectKey,
-                        Base64Data = Image.Base64Data,
-                        FileSize = Image.FileSize,
-                        UploadedAt = DateTime.UtcNow,
-                        TenantId = tenantId
-                    };
-                    _context.Documents.Add(document);
-                    TaskAttachment attachment = new TaskAttachment
-                    {
-                        DocumentId = document.Id,
-                        ReferenceId = reportTask.Id
-                    };
-                    _context.TaskAttachments.Add(attachment);
-                }
-                await _context.SaveChangesAsync();
-            }
-
             _context.TaskComments.Add(comment);
             await _context.SaveChangesAsync();
 
@@ -214,62 +164,10 @@ namespace MarcoBMS.Services.Controllers
             var tenantId = GetTenantId();
             var Employee = await _userHelper.GetCurrentEmployeeAsync();
 
-            var taskAllocation = await _context.TaskAllocations.Include(t => t.WorkItem).FirstOrDefaultAsync(t => t.Id == createComment.TaskAllocationId);
-            if (taskAllocation == null || taskAllocation.WorkItem == null)
-            {
-                return BadRequest(ApiResponse<object>.ErrorResponse("No such task has been allocated.", "No such task has been allocated.", 400));
-            }
-            WorkArea workArea = await _context.WorkAreas.Include(a => a.Floor).FirstOrDefaultAsync(a => a.Id == taskAllocation.WorkItem.WorkAreaId) ?? new WorkArea();
-
-            var bulding = await _context.Buildings.FirstOrDefaultAsync(b => b.Id == (workArea.Floor != null ? workArea.Floor.BuildingId : Guid.Empty));
-
             var comment = createComment.ToCommentFromCommentDto(tenantId, Employee.Id);
             _context.TaskComments.Add(comment);
             await _context.SaveChangesAsync();
 
-            var Images = createComment.Images;
-
-            if (Images != null && Images.Count > 0)
-            {
-
-                foreach (var Image in Images)
-                {
-
-                    if (string.IsNullOrEmpty(Image.Base64Data))
-                        return BadRequest(ApiResponse<object>.ErrorResponse("Base64 data is missing", "Base64 data is missing", 400));
-
-                    //If base64 has a data URI prefix, strip it
-                    var base64 = Image.Base64Data.Contains(",")
-                        ? Image.Base64Data.Substring(Image.Base64Data.IndexOf(",") + 1)
-                        : Image.Base64Data;
-
-                    string fileType = _s3Service.GetContentTypeFromBase64(base64);
-                    string fileName = _s3Service.GenerateFileName(fileType, tenantId, "task_report");
-
-                    string objectKey = $"tenant-{tenantId}/project-{bulding?.ProjectId}/Actitvity/{fileName}";
-                    await _s3Service.UploadFileAsync(base64, fileType, objectKey);
-
-                    Document document = new Document
-                    {
-                        FileName = Image.FileName ?? "",
-                        ContentType = Image.ContentType ?? "",
-                        S3Key = objectKey,
-                        Base64Data = Image.Base64Data,
-                        FileSize = Image.FileSize,
-                        UploadedAt = DateTime.UtcNow,
-                        TenantId = tenantId
-                    };
-                    _context.Documents.Add(document);
-                    TaskAttachment attachment = new TaskAttachment
-                    {
-                        DocumentId = document.Id,
-                        ReferenceId = comment.Id
-                    };
-                    _context.TaskAttachments.Add(attachment);
-                }
-                await _context.SaveChangesAsync();
-            }
-
             CommentVM response = comment.ToCommentVMFromTaskComment();
             return Ok(ApiResponse<object>.SuccessResponse(response, "Comment saved successfully", 200));
         }
@@ -317,13 +215,6 @@ namespace MarcoBMS.Services.Controllers
 
             List<Employee> employees = await _context.Employees.Where(e => employeeIdList.Contains(e.Id)).Include(e => e.JobRole).ToListAsync();
 
-            List<TaskComment> allComments = await _context.TaskComments.Include(c => c.Employee).Where(c => taskIdList.Contains(c.TaskAllocationId)).ToListAsync();
-            var allCommentIds = allComments.Select(c => c.Id).ToList();
-
-            var taskAttachments = await _context.TaskAttachments.Where(t => taskIdList.Contains(t.ReferenceId) || allCommentIds.Contains(t.ReferenceId)).ToListAsync();
-            var documentIds = taskAttachments.Select(t => t.DocumentId).ToList();
-            var documents = await _context.Documents.Where(d => documentIds.Contains(d.Id)).ToListAsync();
-
             List<ListTaskVM> tasks = new List<ListTaskVM>();
             //foreach (var workItem in workItems)
             //{
@@ -332,21 +223,10 @@ namespace MarcoBMS.Services.Controllers
 
                 var response = taskAllocation.ToListTaskVMFromTaskAllocation();
 
-                List<TaskComment> comments = allComments.Where(c => c.TaskAllocationId == taskAllocation.Id).ToList();
+                List<TaskComment> comments = await _context.TaskComments.Where(c => c.TaskAllocationId == taskAllocation.Id).ToListAsync();
                 List<BasicEmployeeVM> team = new List<BasicEmployeeVM>();
                 List<TaskMembers> taskMembers = teamMembers.Where(m => m.TaskAllocationId == taskAllocation.Id).ToList();
 
-                var taskDocumentIds = taskAttachments.Where(t => t.ReferenceId == taskAllocation.Id).Select(t => t.DocumentId).ToList();
-                var taskDocuments = documents.Where(d => taskDocumentIds.Contains(d.Id)).ToList();
-
-                List<string> taskPreSignedUrls = new List<string>();
-                foreach (var document in taskDocuments)
-                {
-                    string preSignedUrl = _s3Service.GeneratePreSignedUrlAsync(document.S3Key);
-                    taskPreSignedUrls.Add(preSignedUrl);
-                }
-                response.ReportedPreSignedUrls = taskPreSignedUrls;
-
                 foreach (var taskMember in taskMembers)
                 {
                     var teamMember = employees.Find(e => e.Id == taskMember.EmployeeId);
@@ -358,18 +238,7 @@ namespace MarcoBMS.Services.Controllers
                 List<CommentVM> commentVM = new List<CommentVM> { };
                 foreach (var comment in comments)
                 {
-                    var commentDocumentIds = taskAttachments.Where(t => t.ReferenceId == comment.Id).Select(t => t.DocumentId).ToList();
+                    commentVM.Add(comment.ToCommentVMFromTaskComment());
-                    var commentDocuments = documents.Where(d => commentDocumentIds.Contains(d.Id)).ToList();
-                    List<string> commentPreSignedUrls = new List<string>();
-                    foreach (var document in commentDocuments)
-                    {
-                        string preSignedUrl = _s3Service.GeneratePreSignedUrlAsync(document.S3Key);
-                        commentPreSignedUrls.Add(preSignedUrl);
-                    }
-                    CommentVM commentVm = comment.ToCommentVMFromTaskComment();
-                    commentVm.PreSignedUrls = commentPreSignedUrls;
-
-                    commentVM.Add(commentVm);
                 }
                 List<ActivityCheckList> checkLists = await _context.ActivityCheckLists.Where(x => x.ActivityId == (taskAllocation.WorkItem != null ? taskAllocation.WorkItem.ActivityId : Guid.Empty)).ToListAsync();
                 List<CheckListMappings> checkListMappings = await _context.CheckListMappings.Where(c => c.TaskAllocationId == taskAllocation.Id).ToListAsync();
@@ -411,23 +280,8 @@ namespace MarcoBMS.Services.Controllers
                 if (taskAllocation == null) return NotFound(ApiResponse<object>.ErrorResponse("Task Not Found", "Task not found", 404));
                 var taskVM = taskAllocation.TaskAllocationToTaskVM(employeeName);
                 var comments = await _context.TaskComments.Where(c => c.TaskAllocationId == taskAllocation.Id).ToListAsync();
-                var commentIds = comments.Select(c => c.Id).ToList();
-                var taskAttachments = await _context.TaskAttachments.Where(t => t.ReferenceId == taskAllocation.Id || commentIds.Contains(t.ReferenceId)).ToListAsync();
-                var documentIds = taskAttachments.Select(t => t.DocumentId).ToList();
-                var documents = await _context.Documents.Where(d => documentIds.Contains(d.Id)).ToListAsync();
                 var team = await _context.TaskMembers.Where(m => m.TaskAllocationId == taskAllocation.Id).Include(m => m.Employee).ToListAsync();
                 var teamMembers = new List<EmployeeVM> { };
-
-                var taskDocumentIds = taskAttachments.Where(t => t.ReferenceId == taskAllocation.Id).Select(t => t.DocumentId).ToList();
-                var taskDocuments = documents.Where(d => taskDocumentIds.Contains(d.Id)).ToList();
-
-                List<string> taskPreSignedUrls = new List<string>();
-                foreach (var document in taskDocuments)
-                {
-                    string preSignedUrl = _s3Service.GeneratePreSignedUrlAsync(document.S3Key);
-                    taskPreSignedUrls.Add(preSignedUrl);
-                }
-                taskVM.PreSignedUrls = taskPreSignedUrls;
                 foreach (var member in team)
                 {
                     var result = member.Employee != null ? member.Employee.ToEmployeeVMFromEmployee() : new EmployeeVM();
@@ -436,17 +290,7 @@ namespace MarcoBMS.Services.Controllers
                 List<CommentVM> Comments = new List<CommentVM> { };
                 foreach (var comment in comments)
                 {
-                    var commentDocumentIds = taskAttachments.Where(t => t.ReferenceId == comment.Id).Select(t => t.DocumentId).ToList();
+                    Comments.Add(comment.ToCommentVMFromTaskComment());
-                    var commentDocuments = documents.Where(d => commentDocumentIds.Contains(d.Id)).ToList();
-                    List<string> commentPreSignedUrls = new List<string>();
-                    foreach (var document in commentDocuments)
-                    {
-                        string preSignedUrl = _s3Service.GeneratePreSignedUrlAsync(document.S3Key);
-                        commentPreSignedUrls.Add(preSignedUrl);
-                    }
-                    CommentVM commentVM = comment.ToCommentVMFromTaskComment();
-                    commentVM.PreSignedUrls = commentPreSignedUrls;
-                    Comments.Add(commentVM);
                 }
                 taskVM.Comments = Comments;
                 taskVM.TeamMembers = teamMembers;

Marco.Pms.Services/Service/S3UploadService.cs

@@ -28,46 +28,53 @@ namespace Marco.Pms.Services.Service
             _s3Client = new AmazonS3Client(settings.AccessKey, settings.SecretKey, region);
         }
         //public async Task<string> UploadFileAsync(string fileName, string contentType)
-        public async Task UploadFileAsync(string base64, string fileType, string objectKey)
+        public async Task<string> UploadFileAsync(string base64Data, Guid tenantId, string tag)
         {
             byte[] fileBytes;
 
-
+            //If base64 has a data URI prefix, strip it
+            var base64 = base64Data.Contains(",")
+                ? base64Data.Substring(base64Data.IndexOf(",") + 1)
+                : base64Data;
             var allowedFilesType = _configuration.GetSection("WhiteList:ContentType")
                                                 .GetChildren()
                                                 .Select(x => x.Value)
                                                 .ToList();
-
+            string fileType = GetContentTypeFromBase64(base64);
-            if (allowedFilesType == null || !allowedFilesType.Contains(fileType))
+            if (allowedFilesType != null && allowedFilesType.Contains(fileType))
             {
-                throw new InvalidOperationException("Unsupported file type.");
+                string fileName = GenerateFileName(fileType, tenantId, tag);
+
+                fileBytes = Convert.FromBase64String(base64);
+
+
+                using var fileStream = new MemoryStream(fileBytes);
+
+                // Generate a unique object key (you can customize this)
+                var objectKey = $"{fileName}";
+
+                var uploadRequest = new TransferUtilityUploadRequest
+                {
+                    InputStream = fileStream,
+                    Key = objectKey,
+                    BucketName = _bucketName,
+                    ContentType = fileType,
+                    AutoCloseStream = true
+                };
+                try
+                {
+                    var transferUtility = new TransferUtility(_s3Client);
+                    await transferUtility.UploadAsync(uploadRequest);
+                    _logger.LogInfo("File uploaded to Amazon S3");
+                    return objectKey;
+                }
+                catch (Exception ex)
+                {
+                    _logger.LogError("{error} while uploading file to S3", ex.Message);
+                    return string.Empty;
+                }
             }
-
+            throw new InvalidOperationException("Unsupported file type.");
-            fileBytes = Convert.FromBase64String(base64);
-
-
-            using var fileStream = new MemoryStream(fileBytes);
-
-            var uploadRequest = new TransferUtilityUploadRequest
-            {
-                InputStream = fileStream,
-                Key = objectKey,
-                BucketName = _bucketName,
-                ContentType = fileType,
-                AutoCloseStream = true
-            };
-            try
-            {
-                var transferUtility = new TransferUtility(_s3Client);
-                await transferUtility.UploadAsync(uploadRequest);
-                _logger.LogInfo("File uploaded to Amazon S3");
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError("{error} while uploading file to S3", ex.Message);
-            }
-
-
         }
         public string GeneratePreSignedUrlAsync(string objectKey)
         {