Updates from review

Gordon360/Controllers/LostAndFoundController.cs

@@ -60,10 +60,9 @@ public async Task<ActionResult> UpdateReportStatus(int id, string status)
         }
 
         /// <summary>
-        /// Get all missing items, only for data entry level users
+        /// Get all missing item reports, only for data entry level users
         /// </summary>
-        /// <param name="id">The id</param>
-        /// <returns></returns>
+        /// <returns>ObjectResult - an http status code, with an array of MissingItem objects in the body </returns>
         [HttpGet]
         [Route("missingitemsall")]
         [StateYourBusiness(operation = Static.Names.Operation.READ_ALL, resource = Resource.LOST_AND_FOUND_MISSING_REPORT)]
@@ -80,6 +79,10 @@ public ActionResult<IEnumerable<MissingItemReportViewModel>> GetAllMissingItems(
             }
         }
 
+        /// <summary>
+        /// Get the list of missing item reports for the currently authenticated user.
+        /// </summary>
+        /// <returns>ObjectResult - an http status code, with an array of MissingItem objects in the body </returns>
         [HttpGet]
         [Route("missingitems")]
         public ActionResult<IEnumerable<MissingItemReportViewModel>> GetMissingItems()
@@ -96,13 +99,18 @@ public ActionResult<IEnumerable<MissingItemReportViewModel>> GetMissingItems()
                 return NotFound();
             }
         }
-
-        /// <param name="id">The id</param>
+        /// <summary>
+        /// Get a missing item report with given ID.
+        /// </summary>
+        /// <param name="id">The id of the report to get</param>
+        /// <returns>ObjectResult - an http status code, with a MissingItem object in the body </returns>
         [HttpGet]
         [Route("missingitemsbyid/{id}")]
         public ActionResult<MissingItemReportViewModel> GetMissingItem(int id)
         {
-            MissingItemReportViewModel? result = lostAndFoundService.GetMissingItem(id);
+            var authenticatedUserUsername = AuthUtils.GetUsername(User);
+
+            MissingItemReportViewModel? result = lostAndFoundService.GetMissingItem(id, authenticatedUserUsername);
             if (result != null)
             {
                 return Ok(result);

Gordon360/Models/CCT/Context/CCTContext.cs

@@ -38,8 +38,6 @@ public CCTContext(DbContextOptions<CCTContext> options)
 
     public virtual DbSet<CUSTOM_PROFILE> CUSTOM_PROFILE { get; set; }
 
-    public virtual DbSet<Categories> Categories { get; set; }
-
     public virtual DbSet<ChapelEvent> ChapelEvent { get; set; }
 
     public virtual DbSet<Clifton_Strengths> Clifton_Strengths { get; set; }
@@ -58,8 +56,6 @@ public CCTContext(DbContextOptions<CCTContext> options)
 
     public virtual DbSet<FacStaff> FacStaff { get; set; }
 
-    public virtual DbSet<FoundItems> FoundItems { get; set; }
-
     public virtual DbSet<GuestUsers> GuestUsers { get; set; }
 
     public virtual DbSet<Housing_Applicants> Housing_Applicants { get; set; }
@@ -96,8 +92,6 @@ public CCTContext(DbContextOptions<CCTContext> options)
 
     public virtual DbSet<Minors> Minors { get; set; }
 
-    public virtual DbSet<Missing> Missing { get; set; }
-
     public virtual DbSet<MissingItemData> MissingItemData { get; set; }
 
     public virtual DbSet<MissingReports> MissingReports { get; set; }
@@ -148,8 +142,6 @@ public CCTContext(DbContextOptions<CCTContext> options)
 
     public virtual DbSet<Statistic> Statistic { get; set; }
 
-    public virtual DbSet<Statuses> Statuses { get; set; }
-
     public virtual DbSet<Student> Student { get; set; }
 
     public virtual DbSet<StudentNewsExpiration> StudentNewsExpiration { get; set; }
@@ -246,11 +238,6 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
             entity.Property(e => e.SESS_CDE).IsFixedLength();
         });
 
-        modelBuilder.Entity<Categories>(entity =>
-        {
-            entity.HasKey(e => e.CategoryName).HasName("PK__Categori__8517B2E195D5FEE5");
-        });
-
         modelBuilder.Entity<ChapelEvent>(entity =>
         {
             entity.ToView("ChapelEvent", "dbo");
@@ -316,13 +303,6 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
             entity.Property(e => e.BuildingDescription).IsFixedLength();
         });
 
-        modelBuilder.Entity<FoundItems>(entity =>
-        {
-            entity.HasKey(e => e.ID).HasName("PK__FoundIte__3214EC277B2634DE");
-
-            entity.Property(e => e.ID).ValueGeneratedNever();
-        });
-
         modelBuilder.Entity<GuestUsers>(entity =>
         {
             entity.HasKey(e => e.ID).HasName("PK__GuestUse__3214EC2774F2F95F");
@@ -444,11 +424,6 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
             entity.ToView("Minors", "dbo");
         });
 
-        modelBuilder.Entity<Missing>(entity =>
-        {
-            entity.HasKey(e => e.recordID).HasName("PK__Missing__D825197E645524A6");
-        });
-
         modelBuilder.Entity<MissingItemData>(entity =>
         {
             entity.ToView("MissingItemData", "LostAndFound");
@@ -457,14 +432,6 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
         modelBuilder.Entity<MissingReports>(entity =>
         {
             entity.HasKey(e => e.ID).HasName("PK__MissingR__3214EC27985CEC3D");
-
-            entity.HasOne(d => d.categoryNavigation).WithMany(p => p.MissingReports)
-                .OnDelete(DeleteBehavior.ClientSetNull)
-                .HasConstraintName("FK__MissingRe__categ__0AB36FCB");
-
-            entity.HasOne(d => d.statusNavigation).WithMany(p => p.MissingReports)
-                .OnDelete(DeleteBehavior.ClientSetNull)
-                .HasConstraintName("FK__MissingRe__statu__0BA79404");
         });
 
         modelBuilder.Entity<PART_DEF>(entity =>
@@ -627,11 +594,6 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
                 .HasConstraintName("FK_Statistic_ParticipantTeam");
         });
 
-        modelBuilder.Entity<Statuses>(entity =>
-        {
-            entity.HasKey(e => e.StatusName).HasName("PK__Statuses__05E7698B0A94483B");
-        });
-
         modelBuilder.Entity<Student>(entity =>
         {
             entity.ToView("Student", "dbo");

Gordon360/Models/CCT/Context/efpt.CCT.config.json

@@ -74,30 +74,14 @@
          "Name": "[LostAndFound].[ActionsTaken]",
          "ObjectType": 0
       },
-      {
-         "Name": "[LostAndFound].[Categories]",
-         "ObjectType": 0
-      },
-      {
-         "Name": "[LostAndFound].[FoundItems]",
-         "ObjectType": 0
-      },
       {
          "Name": "[LostAndFound].[GuestUsers]",
          "ObjectType": 0
       },
-      {
-         "Name": "[LostAndFound].[Missing]",
-         "ObjectType": 0
-      },
       {
          "Name": "[LostAndFound].[MissingReports]",
          "ObjectType": 0
       },
-      {
-         "Name": "[LostAndFound].[Statuses]",
-         "ObjectType": 0
-      },
       {
          "Name": "[RecIM].[Activity]",
          "ObjectType": 0

Gordon360/Models/CCT/LostAndFound/MissingReports.cs

@@ -67,12 +67,4 @@ public partial class MissingReports
 
     [InverseProperty("missing")]
     public virtual ICollection<GuestUsers> GuestUsers { get; set; } = new List<GuestUsers>();
-
-    [ForeignKey("category")]
-    [InverseProperty("MissingReports")]
-    public virtual Categories categoryNavigation { get; set; }
-
-    [ForeignKey("status")]
-    [InverseProperty("MissingReports")]
-    public virtual Statuses statusNavigation { get; set; }
 }