From b7e65e39147c7c075c70f2fd93a8b8f4a6bc661d Mon Sep 17 00:00:00 2001
From: Alex Mickelson <alexmickelson96@gmail.com>
Date: Tue, 8 Aug 2023 16:40:16 -0600
Subject: [PATCH] updated how assignment form persists data

---
 .../AssignmentDescriptionEditor.razor         |   9 +-
 .../AssignmentForm/AssignmentForm.razor       | 157 ++++++++----------
 Management.Web/Shared/Components/Modal.razor  |  38 +++--
 3 files changed, 95 insertions(+), 109 deletions(-)

diff --git a/Management.Web/Shared/Components/AssignmentForm/AssignmentDescriptionEditor.razor b/Management.Web/Shared/Components/AssignmentForm/AssignmentDescriptionEditor.razor
index 74948f6..bb56db9 100644
--- a/Management.Web/Shared/Components/AssignmentForm/AssignmentDescriptionEditor.razor
+++ b/Management.Web/Shared/Components/AssignmentForm/AssignmentDescriptionEditor.razor
@@ -31,7 +31,10 @@
 
   protected override void OnParametersSet()
   {
-    Description = Assignment.Description;
+    if (Description == "")
+    {
+      Description = Assignment.Description;
+    }
     TemplateId = Assignment.TemplateId;
     UseTemplate = Assignment.TemplateId != null && Assignment.TemplateId != "";
   }
@@ -87,6 +90,7 @@
   role="switch" 
   id="useTemplateForDescription"
   @bind="UseTemplate"
+  />
   <label class="form-check-label" for="useTemplateForDescription">
     use template for description
   </label>
@@ -172,7 +176,8 @@ else
       <textarea 
         id="description" 
         class="form-control" 
-        rows=12 
+        rows=12
+        value="@Description"
         @oninput="@((e) => 
           {
             var newDescription = e.Value?.ToString();
diff --git a/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor b/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor
index 61ef800..62c3c9a 100644
--- a/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor
+++ b/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor
@@ -29,8 +29,8 @@
   [EditorRequired]
   public bool Show { get; set; }
 
-  [Parameter] 
-  public Action OnHide { get; set; } = () => { }; 
+  [Parameter]
+  public Action OnHide { get; set; } = () => { };
   public Modal? AssignmentModal { get; set; } = null;
   private string name { get; set; } = String.Empty;
   private bool lockAtDueDate { get; set; }
@@ -39,7 +39,7 @@
 
   protected override void OnParametersSet()
   {
-    if(Show)
+    if (Show)
     {
       AssignmentModal?.Show();
     }
@@ -52,45 +52,45 @@
   private void submitHandler()
   {
     var totalRubricPoints = rubric
-      .Where(r => !r.Label.Contains(RubricItem.extraCredit))
-      .Select(s => s.Points)
-      .Sum();
+    .Where(r => !r.Label.Contains(RubricItem.extraCredit))
+    .Select(s => s.Points)
+    .Sum();
 
-    var newAssignment = Assignment with 
-    {
-      Name=name,
-      LockAtDueDate=lockAtDueDate,
-      Rubric=rubric,
-      PointsPossible=totalRubricPoints,
-      SubmissionTypes=submissionTypes,
-    };
-
-    if(planner.LocalCourse != null)
-    {
-       var currentModule = planner
-          .LocalCourse
-          .Modules
-          .First(m => 
-            m.Assignments
-              .Select(a => a.Id)
-              .Contains(Assignment.Id)
-          ) ?? throw new Exception("could not find current module in assignment form");
-      var updatedModules = planner.LocalCourse.Modules.Select(m => 
-        m.Name == currentModule.Name 
-          ? currentModule with 
-          {
-            Assignments=currentModule.Assignments.Select(a => 
-              a.Id == newAssignment.Id
-                ? newAssignment
-                : a
-            ).ToArray()
-          }
-          : m
-        ).ToArray();
-      planner.LocalCourse = planner.LocalCourse with 
+    var newAssignment = Assignment with
       {
-        Modules=updatedModules
+        Name = name,
+        LockAtDueDate = lockAtDueDate,
+        Rubric = rubric,
+        PointsPossible = totalRubricPoints,
+        SubmissionTypes = submissionTypes,
       };
+
+    if (planner.LocalCourse != null)
+    {
+      var currentModule = planner
+      .LocalCourse
+      .Modules
+      .First(m =>
+      m.Assignments
+      .Select(a => a.Id)
+      .Contains(Assignment.Id)
+      ) ?? throw new Exception("could not find current module in assignment form");
+      var updatedModules = planner.LocalCourse.Modules.Select(m =>
+      m.Name == currentModule.Name
+      ? currentModule with
+        {
+          Assignments = currentModule.Assignments.Select(a =>
+  a.Id == newAssignment.Id
+  ? newAssignment
+  : a
+        ).ToArray()
+        }
+      : m
+      ).ToArray();
+      planner.LocalCourse = planner.LocalCourse with
+        {
+          Modules = updatedModules
+        };
     }
     AssignmentModal?.Hide();
   }
@@ -101,38 +101,38 @@
     StateHasChanged();
   }
   private void SetTypes(IEnumerable<string> newTypes)
-  { 
+  {
     submissionTypes = newTypes;
     StateHasChanged();
   }
 
   private async Task HandleDelete()
   {
-    if(planner.LocalCourse != null)
+    if (planner.LocalCourse != null)
     {
       var assignment = Assignment;
       var currentModule = planner
-        .LocalCourse
-        .Modules
-        .First(m => 
-          m.Assignments
-            .Select(a => a.Id)
-            .Contains(Assignment.Id)
-        ) ?? throw new Exception("handling assignment delete, could not find module");
-      var newModules = planner.LocalCourse.Modules.Select(m => 
-        m.Name == currentModule.Name
-          ? m with 
-          {
-            Assignments = m.Assignments.Where(a => a.Id != assignment.Id).ToArray()
-          }
-          : m
+      .LocalCourse
+      .Modules
+      .First(m =>
+      m.Assignments
+      .Select(a => a.Id)
+      .Contains(Assignment.Id)
+      ) ?? throw new Exception("handling assignment delete, could not find module");
+      var newModules = planner.LocalCourse.Modules.Select(m =>
+      m.Name == currentModule.Name
+      ? m with
+        {
+          Assignments = m.Assignments.Where(a => a.Id != assignment.Id).ToArray()
+        }
+      : m
       ).ToArray();
 
-      planner.LocalCourse = planner.LocalCourse with 
-      {
-        Modules = newModules
-      };
-      if(assignment.CanvasId != null && planner.LocalCourse.CanvasId != null)
+      planner.LocalCourse = planner.LocalCourse with
+        {
+          Modules = newModules
+        };
+      if (assignment.CanvasId != null && planner.LocalCourse.CanvasId != null)
       {
         ulong courseId = planner.LocalCourse.CanvasId ?? throw new Exception("cannot delete if no course id");
         await canvas.Assignments.Delete(courseId, assignment);
@@ -145,52 +145,31 @@
   <Title>
     @Assignment.Name
   </Title>
+
   <Body>
     <form @onsubmit:preventDefault="true" @onsubmit="submitHandler">
       <div class="m-1">
-        <label 
-          class="form-label"
-        >
+        <label class="form-label">
           Name
         </label>
-        <input 
-          class="form-control"
-          @bind="name"
-        />
+        <input class="form-control" @bind="name" />
       </div>
       <div class="m-1">
-        <AssignmentDescriptionEditor
-          Assignment="Assignment"
-        />
+        <AssignmentDescriptionEditor Assignment="Assignment" />
       </div>
 
       <div class="form-check m-1">
-        <input 
-          class="form-check-input"
-          id="lockAtDueDate"
-          type="checkbox"
-        />
-        <label 
-          class="form-check-label"
-          for="lockAtDueDate"
-          @bind="lockAtDueDate"
-        >
+        <input class="form-check-input" id="lockAtDueDate" type="checkbox" />
+        <label class="form-check-label" for="lockAtDueDate" @bind="lockAtDueDate">
           Lock At Due Date
         </label>
       </div>
       <RubricEditor Rubric="rubric" SetRubric="updateRubric" />
-      <SubmissionTypeSelector
-        Types="submissionTypes"
-        SetTypes="SetTypes"
-      />
+      <SubmissionTypeSelector Types="submissionTypes" SetTypes="SetTypes" />
     </form>
   </Body>
   <Footer>
-    <ConfirmationModal 
-      Label="Delete" 
-      Class="btn btn-danger"
-      OnConfirmAsync="HandleDelete"
-    />
+    <ConfirmationModal Label="Delete" Class="btn btn-danger" OnConfirmAsync="HandleDelete" />
     <button class="btn btn-primary" @onclick="@(() => AssignmentModal?.Hide())">
       Save
     </button>
diff --git a/Management.Web/Shared/Components/Modal.razor b/Management.Web/Shared/Components/Modal.razor
index a531a6d..abcbd38 100644
--- a/Management.Web/Shared/Components/Modal.razor
+++ b/Management.Web/Shared/Components/Modal.razor
@@ -1,38 +1,38 @@
 @code {
-  [Parameter, EditorRequired] 
+  [Parameter, EditorRequired]
   public RenderFragment? Title { get; set; }
 
-  [Parameter, EditorRequired] 
+  [Parameter, EditorRequired]
   public RenderFragment? Body { get; set; }
 
-  [Parameter, EditorRequired] 
+  [Parameter, EditorRequired]
   public RenderFragment? Footer { get; set; }
 
-  [Parameter] 
-  public Action OnShow { get; set; } = () => { }; 
+  [Parameter]
+  public Action OnShow { get; set; } = () => { };
 
-  [Parameter] 
-  public Action OnHide { get; set; } = () => { }; 
-  
-  private string modalClass = "hide-modal"; 
-  private bool showBackdrop = false; 
+  [Parameter]
+  public Action OnHide { get; set; } = () => { };
+
+  private string modalClass = "hide-modal";
+  private bool showBackdrop = false;
   public void Show()
-  { 
-    modalClass = "show-modal"; 
-    showBackdrop = true; 
+  {
+    modalClass = "show-modal";
+    showBackdrop = true;
     OnShow();
   }
 
   public void Hide()
   {
-    modalClass = "hide-modal"; 
+    modalClass = "hide-modal";
     showBackdrop = false;
     OnHide();
   }
 }
 
-<div class="modal @modalClass">
-  <div class="modal-dialog modal-xl" role="document">
+<div class="modal @modalClass" @onclick="Hide">
+  <div class="modal-dialog modal-xl" role="document" @onclick:stopPropagation="true">
     <div class="modal-content">
       <div class="modal-header">
         <h4 class="modal-title text-center w-100">@Title</h4>
@@ -46,5 +46,7 @@
 
 @if (showBackdrop)
 {
-  <div class="modal-backdrop fade show"></div>
-} 
+  <div 
+    class="modal-backdrop fade show"
+  ></div>
+}