Faction Id shouldn't be Required (#150)

- Faction Id should no longer be required - Fixed error message on the Background textbox on edit
Society-In-Shadow · Sep 17, 2024 · 8892ff7 · 8892ff7
1 parent 34cd275
commit 8892ff7
Show file tree

Hide file tree

Showing 17 changed files with 1,066 additions and 43 deletions.
diff --git a/api/ExpressedRealms.DB/Characters/Character.cs b/api/ExpressedRealms.DB/Characters/Character.cs
@@ -21,7 +21,7 @@ public class Character : ISoftDelete
     public byte IntelligenceId { get; set; }
     public byte WillpowerId { get; set; }
 
-    public int FactionId { get; set; }
+    public int? FactionId { get; set; }
 
     public int StatExperiencePoints { get; set; }
 
@@ -37,6 +37,6 @@ public class Character : ISoftDelete
     public virtual StatLevel StrengthStatLevel { get; set; } = null!;
     public virtual StatLevel IntelligenceStatLevel { get; set; } = null!;
     public virtual StatLevel WillpowerStatLevel { get; set; } = null!;
-    public virtual ExpressionSection FactionInfo { get; set; } = null!;
+    public virtual ExpressionSection? FactionInfo { get; set; } = null!;
     public virtual List<CharacterSkillsMapping> CharacterSkillsMappings { get; set; } = null!;
 }
diff --git a/api/ExpressedRealms.DB/Characters/CharacterConfiguration.cs b/api/ExpressedRealms.DB/Characters/CharacterConfiguration.cs
@@ -21,7 +21,7 @@ public void Configure(EntityTypeBuilder<Character> builder)
         builder.Property(x => x.IntelligenceId).IsRequired().HasDefaultValue(1);
         builder.Property(x => x.WillpowerId).IsRequired().HasDefaultValue(1);
 
-        builder.Property(x => x.FactionId).IsRequired().HasDefaultValue(1);
+        builder.Property(x => x.FactionId);
 
         builder.Property(x => x.StatExperiencePoints).IsRequired().HasDefaultValue(72);
 
@@ -87,7 +87,6 @@ public void Configure(EntityTypeBuilder<Character> builder)
             .HasOne(x => x.FactionInfo)
             .WithMany(x => x.CharactersList)
             .HasForeignKey(x => x.FactionId)
-            .OnDelete(DeleteBehavior.Restrict)
-            .IsRequired();
+            .OnDelete(DeleteBehavior.Restrict);
     }
 }