Changeset 20df1a3 for iknow-api/Data/AppDbContext.cs

Timestamp:

09/16/26 14:48:55 (7 days ago)

Author:

Stefan-Saveski <stefansaveski19@…>

Branches:

master

Children:

f7c8acf

Parents:

1b20b22

Message:

Refactor payment model to remove redundant user_id, addressing BCNF violation in Phase 5 normalization

File:

• iknow-api/Data/AppDbContext.cs (modified) (1 diff)

iknow-api/Data/AppDbContext.cs

@@ -290 +290 @@
                 e.HasKey(p => p.Id);
                 e.Property(p => p.Id).HasColumnName("id");
-                e.Property(p => p.UserId).HasColumnName("user_id");
                 e.Property(p => p.EnrollmentInfoId).HasColumnName("enrollment_id");
                 e.Property(p => p.Amount).HasColumnName("amount");
 
-                e.HasOne(p => p.User)
-                 .WithMany(u => u.Payments)
-                 .HasForeignKey(p => p.UserId)
-                 .OnDelete(DeleteBehavior.Cascade);
-
+                // No user_id: the student comes from the enrolment. Removing it
+                // was the BCNF fix from Phase 5 (enrolled_id -> user_id).
                 e.HasOne(p => p.EnrolledSemesters)
                  .WithMany(es => es.Users)