Rollback on migration Adding_ChangeLog and fixed. ChangeLogItem description and ChangeLog response mapping fixed. New check workitem change to don't save duplicate.

Author: DiegoGalante

AzureDevopsTracker/Data/Context/AzureDevopsTrackerContext.cs

@@ -23,6 +23,8 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
                 property.SetColumnType("varchar(200)");
 
             modelBuilder.HasDefaultSchema(DataBaseConfig.SchemaName);
+
+            modelBuilder.ApplyConfigurationsFromAssembly(typeof(AzureDevopsTrackerContext).Assembly);
         }
 
         protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)

AzureDevopsTracker/Data/Mapping/ChangeLogItemMapping.cs

@@ -0,0 +1,15 @@
+using AzureDevopsTracker.Entities;
+using Microsoft.EntityFrameworkCore;
+using Microsoft.EntityFrameworkCore.Metadata.Builders;
+
+namespace AzureDevopsTracker.Data.Mapping
+{
+    public class ChangeLogItemMapping : IEntityTypeConfiguration<ChangeLogItem>
+    {
+        public void Configure(EntityTypeBuilder<ChangeLogItem> builder)
+        {
+            builder.Property(d => d.Description)
+                   .HasColumnType("varchar(max)");
+        }
+    }
+}

AzureDevopsTracker/Data/Mapping/ChangeLogMapping.cs

@@ -0,0 +1,15 @@
+using AzureDevopsTracker.Entities;
+using Microsoft.EntityFrameworkCore;
+using Microsoft.EntityFrameworkCore.Metadata.Builders;
+
+namespace AzureDevopsTracker.Data.Mapping
+{
+    public class ChangeLogMapping : IEntityTypeConfiguration<ChangeLog>
+    {
+        public void Configure(EntityTypeBuilder<ChangeLog> builder)
+        {
+            builder.Property(d => d.Response)
+                   .HasColumnType("varchar(max)");
+        }
+    }
+}

AzureDevopsTracker/Data/WorkItemRepository.cs

@@ -17,6 +17,7 @@ public async Task<WorkItem> GetByWorkItemId(string workItemId)
             return await DbSet
                           .Include(x => x.WorkItemsChanges)
                           .Include(x => x.TimeByStates)
+                          .Include(x => x.ChangeLogItem)
                           .FirstOrDefaultAsync(x => x.Id == workItemId);
         }
 
@@ -25,6 +26,7 @@ public async Task<IEnumerable<WorkItem>> ListByWorkItemId(IEnumerable<string> wo
             return await DbSet
                           .Include(x => x.WorkItemsChanges)
                           .Include(x => x.TimeByStates)
+                          .Include(x => x.ChangeLogItem)
                           .Where(x => workItemsId.Contains(x.Id))
                           .ToListAsync();
         }
@@ -34,6 +36,7 @@ public async Task<IEnumerable<WorkItem>> ListByIterationPath(string iterationPat
             return await DbSet
                           .Include(x => x.WorkItemsChanges)
                           .Include(x => x.TimeByStates)
+                          .Include(x => x.ChangeLogItem)
                           .Where(x => x.IterationPath == iterationPath)
                           .ToListAsync();
         }

AzureDevopsTracker/Entities/ChangeLogItem.cs

@@ -15,16 +15,23 @@ public class ChangeLogItem : Entity
         public ChangeLog ChangeLog { get; private set; }
         private ChangeLogItem() { }
 
-        public ChangeLogItem(string workItemId, string title, string description, string workitemType)
+        public ChangeLogItem(string workItemId, string title, string description, string workItemType)
         {
             WorkItemId = workItemId;
             Title = title;
             Description = description;
-            WorkItemType = workitemType;
+            WorkItemType = workItemType;
 
             Validate();
         }
 
+        public void Update(string title, string workItemType, string description)
+        {
+            Title = title;
+            WorkItemType = workItemType;
+            Description = description;
+        }
+
         public void Release(string changeLogId)
         {
             ChangeLogId = changeLogId;

AzureDevopsTracker/Entities/WorkItem.cs

@@ -22,7 +22,6 @@ public class WorkItem : Entity
         public string Activity { get; private set; }
         public string Lancado { get; private set; }
 
-        public string ChangeLogItemId { get; private set; }
         public ChangeLogItem ChangeLogItem { get; private set; }
 
         private readonly List<WorkItemChange> _workItemsChanges;
@@ -111,10 +110,9 @@ public void ClearTimesByState()
 
         public void VinculateChangeLogItem(ChangeLogItem changeLogItem)
         {
-            if (ChangeLogItem == null)
+            if (changeLogItem == null)
                 throw new Exception("ChangeLogItem is null");
 
-            ChangeLogItemId = changeLogItem.Id;
             ChangeLogItem = changeLogItem;
         }
 

…11206211800_Adding_ChangeLog.Designer.cs …11213213716_Adding_ChangeLog.Designer.csAzureDevopsTracker/Migrations/20211206211800_Adding_ChangeLog.Designer.cs renamed to AzureDevopsTracker/Migrations/20211213213716_Adding_ChangeLog.Designer.cs AzureDevopsTracker/Migrations/20211206211800_Adding_ChangeLog.Designer.cs renamed to AzureDevopsTracker/Migrations/20211213213716_Adding_ChangeLog.Designer.cs

@@ -8,28 +8,14 @@ public partial class Adding_ChangeLog : Migration
     {
         protected override void Up(MigrationBuilder migrationBuilder)
         {
-            migrationBuilder.AddColumn<string>(
-                name: "ChangeLogItemId",
-                schema: DataBaseConfig.SchemaName,
-                table: "WorkItems",
-                type: "varchar(200)",
-                nullable: true);
-
-            migrationBuilder.AddColumn<string>(
-                name: "ChangeLogItemId1",
-                schema: DataBaseConfig.SchemaName,
-                table: "WorkItems",
-                type: "varchar(200)",
-                nullable: true);
-
             migrationBuilder.CreateTable(
                 name: "ChangeLogs",
                 schema: DataBaseConfig.SchemaName,
                 columns: table => new
                 {
                     Id = table.Column<string>(type: "varchar(200)", nullable: false),
                     CreatedAt = table.Column<DateTime>(nullable: false),
-                    Response = table.Column<string>(type: "varchar(200)", nullable: true),
+                    Response = table.Column<string>(type: "varchar(max)", nullable: true),
                     Revision = table.Column<int>(nullable: false)
                 },
                 constraints: table =>
@@ -46,7 +32,7 @@ protected override void Up(MigrationBuilder migrationBuilder)
                     CreatedAt = table.Column<DateTime>(nullable: false),
                     WorkItemId = table.Column<string>(type: "varchar(200)", nullable: true),
                     Title = table.Column<string>(type: "varchar(200)", nullable: true),
-                    Description = table.Column<string>(type: "varchar(200)", nullable: true),
+                    Description = table.Column<string>(type: "varchar(max)", nullable: true),
                     WorkItemType = table.Column<string>(type: "varchar(200)", nullable: true),
                     ChangeLogId = table.Column<string>(type: "varchar(200)", nullable: true)
                 },
@@ -60,60 +46,39 @@ protected override void Up(MigrationBuilder migrationBuilder)
                         principalTable: "ChangeLogs",
                         principalColumn: "Id",
                         onDelete: ReferentialAction.Restrict);
+                    table.ForeignKey(
+                        name: "FK_ChangeLogItems_WorkItems_WorkItemId",
+                        column: x => x.WorkItemId,
+                        principalSchema: DataBaseConfig.SchemaName,
+                        principalTable: "WorkItems",
+                        principalColumn: "Id",
+                        onDelete: ReferentialAction.Restrict);
                 });
 
-            migrationBuilder.CreateIndex(
-                name: "IX_WorkItems_ChangeLogItemId1",
-                schema: DataBaseConfig.SchemaName,
-                table: "WorkItems",
-                column: "ChangeLogItemId1");
-
             migrationBuilder.CreateIndex(
                 name: "IX_ChangeLogItems_ChangeLogId",
                 schema: DataBaseConfig.SchemaName,
                 table: "ChangeLogItems",
                 column: "ChangeLogId");
 
-            migrationBuilder.AddForeignKey(
-                name: "FK_WorkItems_ChangeLogItems_ChangeLogItemId1",
+            migrationBuilder.CreateIndex(
+                name: "IX_ChangeLogItems_WorkItemId",
                 schema: DataBaseConfig.SchemaName,
-                table: "WorkItems",
-                column: "ChangeLogItemId1",
-                principalSchema: DataBaseConfig.SchemaName,
-                principalTable: "ChangeLogItems",
-                principalColumn: "Id",
-                onDelete: ReferentialAction.Restrict);
+                table: "ChangeLogItems",
+                column: "WorkItemId",
+                unique: true,
+                filter: "[WorkItemId] IS NOT NULL");
         }
 
         protected override void Down(MigrationBuilder migrationBuilder)
         {
-            migrationBuilder.DropForeignKey(
-                name: "FK_WorkItems_ChangeLogItems_ChangeLogItemId1",
-                schema: DataBaseConfig.SchemaName,
-                table: "WorkItems");
-
             migrationBuilder.DropTable(
                 name: "ChangeLogItems",
                 schema: DataBaseConfig.SchemaName);
 
             migrationBuilder.DropTable(
                 name: "ChangeLogs",
                 schema: DataBaseConfig.SchemaName);
-
-            migrationBuilder.DropIndex(
-                name: "IX_WorkItems_ChangeLogItemId1",
-                schema: DataBaseConfig.SchemaName,
-                table: "WorkItems");
-
-            migrationBuilder.DropColumn(
-                name: "ChangeLogItemId",
-                schema: DataBaseConfig.SchemaName,
-                table: "WorkItems");
-
-            migrationBuilder.DropColumn(
-                name: "ChangeLogItemId1",
-                schema: DataBaseConfig.SchemaName,
-                table: "WorkItems");
         }
     }
 }

…tions/20211206211800_Adding_ChangeLog.cs …tions/20211213213716_Adding_ChangeLog.csAzureDevopsTracker/Migrations/20211206211800_Adding_ChangeLog.cs renamed to AzureDevopsTracker/Migrations/20211213213716_Adding_ChangeLog.cs AzureDevopsTracker/Migrations/20211206211800_Adding_ChangeLog.cs renamed to AzureDevopsTracker/Migrations/20211213213716_Adding_ChangeLog.cs

@@ -30,7 +30,7 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .HasColumnType("datetime2");
 
                     b.Property<string>("Response")
-                        .HasColumnType("varchar(200)");
+                        .HasColumnType("varchar(max)");
 
                     b.Property<int>("Revision")
                         .HasColumnType("int");
@@ -52,7 +52,7 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .HasColumnType("datetime2");
 
                     b.Property<string>("Description")
-                        .HasColumnType("varchar(200)");
+                        .HasColumnType("varchar(max)");
 
                     b.Property<string>("Title")
                         .HasColumnType("varchar(200)");
@@ -67,6 +67,10 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 
                     b.HasIndex("ChangeLogId");
 
+                    b.HasIndex("WorkItemId")
+                        .IsUnique()
+                        .HasFilter("[WorkItemId] IS NOT NULL");
+
                     b.ToTable("ChangeLogItems");
                 });
 
@@ -111,12 +115,6 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                     b.Property<string>("AssignedTo")
                         .HasColumnType("varchar(200)");
 
-                    b.Property<string>("ChangeLogItemId")
-                        .HasColumnType("varchar(200)");
-
-                    b.Property<string>("ChangeLogItemId1")
-                        .HasColumnType("varchar(200)");
-
                     b.Property<DateTime>("CreatedAt")
                         .HasColumnType("datetime2");
 
@@ -155,8 +153,6 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 
                     b.HasKey("Id");
 
-                    b.HasIndex("ChangeLogItemId1");
-
                     b.ToTable("WorkItems");
                 });
 
@@ -204,6 +200,10 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                     b.HasOne("AzureDevopsTracker.Entities.ChangeLog", "ChangeLog")
                         .WithMany("ChangeLogItems")
                         .HasForeignKey("ChangeLogId");
+
+                    b.HasOne("AzureDevopsTracker.Entities.WorkItem", null)
+                        .WithOne("ChangeLogItem")
+                        .HasForeignKey("AzureDevopsTracker.Entities.ChangeLogItem", "WorkItemId");
                 });
 
             modelBuilder.Entity("AzureDevopsTracker.Entities.TimeByState", b =>
@@ -213,13 +213,6 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .HasForeignKey("WorkItemId");
                 });
 
-            modelBuilder.Entity("AzureDevopsTracker.Entities.WorkItem", b =>
-                {
-                    b.HasOne("AzureDevopsTracker.Entities.ChangeLogItem", "ChangeLogItem")
-                        .WithMany()
-                        .HasForeignKey("ChangeLogItemId1");
-                });
-
             modelBuilder.Entity("AzureDevopsTracker.Entities.WorkItemChange", b =>
                 {
                     b.HasOne("AzureDevopsTracker.Entities.WorkItem", "WorkItem")