From 13f92d49ccf7e2772d3bdd569ccc8a39c6ecbe00 Mon Sep 17 00:00:00 2001 From: quentin Date: Mon, 15 Jul 2024 17:51:03 -0500 Subject: [PATCH] Auditing is now the responsibility of the database. Removed foregin key from audit tables to parent --- API/Services/ServiceBase.cs | 12 +---------- DAL/Contexts/SASGContext.cs | 42 ------------------------------------- 2 files changed, 1 insertion(+), 53 deletions(-) diff --git a/API/Services/ServiceBase.cs b/API/Services/ServiceBase.cs index 4cc277d..7e1c8fc 100644 --- a/API/Services/ServiceBase.cs +++ b/API/Services/ServiceBase.cs @@ -76,8 +76,6 @@ namespace API.Services if (origModel == null) return null; - copyToAudit(origModel); - model.updateModel(ref origModel); origModel.updated = DateTime.Now; @@ -97,22 +95,14 @@ namespace API.Services if (origModel == null) return null; - copyToAudit(origModel); - origModel.updated = DateTime.Now; origModel.updater = user.id; - - copyToAudit(origModel); + Context.SaveChanges(); Context.Remove(origModel); Context.SaveChanges(); return origModel.adaptToAudit(); } - - private void copyToAudit(TModel model) - { - Context.Set().Add(model.adaptToAudit()); - } } } diff --git a/DAL/Contexts/SASGContext.cs b/DAL/Contexts/SASGContext.cs index f21a7f1..442ccab 100644 --- a/DAL/Contexts/SASGContext.cs +++ b/DAL/Contexts/SASGContext.cs @@ -98,48 +98,6 @@ namespace DAL.Contexts .HasForeignKey(e => e.updater).HasConstraintName("events_users_id_fk"); entity.Property(e => e.hashingType).HasConversion(); }); - - builder.Entity(entity => - { - entity.HasOne().WithMany(e => e.audits) - .HasForeignKey(e => e.originalId).IsRequired(); - }); - - builder.Entity(entity => - { - entity.HasOne().WithMany(e => e.audits) - .HasForeignKey(e => e.originalId).IsRequired(); - }); - - builder.Entity(entity => - { - entity.HasOne().WithMany(e => e.audits) - .HasForeignKey(e => e.originalId).IsRequired(); - }); - - builder.Entity(entity => - { - entity.HasOne().WithMany(e => e.audits) - .HasForeignKey(e => e.originalId).IsRequired(); - }); - - builder.Entity(entity => - { - entity.HasOne().WithMany(e => e.audits) - .HasForeignKey(e => e.originalId).IsRequired(); - }); - - builder.Entity(entity => - { - entity.HasOne().WithMany(e => e.audits) - .HasForeignKey(e => e.originalId).IsRequired(); - }); - - builder.Entity(entity => - { - entity.HasOne().WithMany(e => e.audits) - .HasForeignKey(e => e.originalId).IsRequired(); - }); } } }