From 6a76172231a9853b8dcdc49dce1eeb9ca3d8cf41 Mon Sep 17 00:00:00 2001
From: Alex Mickelson <alexmickelson96@gmail.com>
Date: Fri, 18 Aug 2023 14:32:08 -0600
Subject: [PATCH] need to fix quiz assignment bug

---
 .../Shared/Components/Quiz/QuizSettings.razor |  2 +-
 .../CoursePlannerValidationExtensions.cs      | 55 +++++++++----------
 .../AssignmentSyncronizationExtensions.cs     | 55 ++++++++++++++++---
 .../ModuleSyncronizationExtensions.cs         | 20 +++++++
 .../Canvas/CanvasAssignmentService.cs         |  4 +-
 .../Services/Canvas/CanvasQuizService.cs      |  5 +-
 requests/assignment.http                      |  6 +-
 requests/semester.http                        |  8 ++-
 8 files changed, 109 insertions(+), 46 deletions(-)

diff --git a/Management.Web/Shared/Components/Quiz/QuizSettings.razor b/Management.Web/Shared/Components/Quiz/QuizSettings.razor
index c800868..d7836c2 100644
--- a/Management.Web/Shared/Components/Quiz/QuizSettings.razor
+++ b/Management.Web/Shared/Components/Quiz/QuizSettings.razor
@@ -64,7 +64,7 @@
       type="checkbox"
       role="switch" 
       id="lockAtDueDate"
-      checked="lockAtDueDate"
+      checked="@lockAtDueDate"
       @onchange="handleLockAtDueDateChange">
     <label 
       class="form-check-label" for="lockAtDueDate">
diff --git a/Management/Features/Configuration/CoursePlannerValidationExtensions.cs b/Management/Features/Configuration/CoursePlannerValidationExtensions.cs
index 1c6f95f..96b5323 100644
--- a/Management/Features/Configuration/CoursePlannerValidationExtensions.cs
+++ b/Management/Features/Configuration/CoursePlannerValidationExtensions.cs
@@ -9,18 +9,20 @@ public static class CoursePlannerExtensions
 {
   public static LocalCourse GeneralCourseCleanup(this LocalCourse incomingCourse)
   {
-    var cleanModules = incomingCourse.Modules.Select(
-      module =>
-        module with
-        {
-          Assignments = module.Assignments
-            .OrderBy(a => a.DueAt)
-            .DistinctBy(a => a.Id)
-            .Select(a => a.validateSubmissionTypes())
-            .Select(a => a.validateDates())
-            .ToArray()
-        }
-    ).ToArray();
+    var cleanModules = incomingCourse.Modules
+      .Select(
+        module =>
+          module with
+          {
+            Assignments = module.Assignments
+              .OrderBy(a => a.DueAt)
+              .DistinctBy(a => a.Id)
+              .Select(a => a.validateSubmissionTypes())
+              .Select(a => a.validateDates())
+              .ToArray()
+          }
+      )
+      .ToArray();
 
     var cleanStartDay = new DateTime(
       incomingCourse.StartDate.Year,
@@ -102,18 +104,16 @@ public static class CoursePlannerExtensions
     }
     return assignment;
   }
+
   private static LocalQuiz validateQuizForCanvasId(
     this LocalQuiz quiz,
     IEnumerable<CanvasQuiz> canvasQuizzes
   )
   {
-    var assignmentIdInCanvas =
-      canvasQuizzes.FirstOrDefault(cq => cq.Id == quiz.CanvasId) != null;
+    var assignmentIdInCanvas = canvasQuizzes.FirstOrDefault(cq => cq.Id == quiz.CanvasId) != null;
     if (!assignmentIdInCanvas)
     {
-      Console.WriteLine(
-        $"no id in canvas for quiz, removing old canvas id: {quiz.Name}"
-      );
+      Console.WriteLine($"no id in canvas for quiz, removing old canvas id: {quiz.Name}");
       return quiz with { CanvasId = null };
     }
     return quiz;
@@ -125,19 +125,18 @@ public static class CoursePlannerExtensions
       assignment.SubmissionTypes.FirstOrDefault(t => t == SubmissionType.DISCUSSION_TOPIC) != null;
 
     if (containsDiscussion)
-      return assignment with
-      {
-        SubmissionTypes = new string[] { SubmissionType.DISCUSSION_TOPIC }
-      };
+      return assignment with { SubmissionTypes = new string[] { SubmissionType.DISCUSSION_TOPIC } };
     return assignment;
   }
 
-    public static LocalAssignment validateDates(this LocalAssignment assignment)
+  public static LocalAssignment validateDates(this LocalAssignment assignment)
+  {
+    var dueAt = assignment.DueAt.AddMilliseconds(0).AddMilliseconds(0);
+    var lockAt = assignment.LockAt?.AddMilliseconds(0).AddMilliseconds(0);
+    return assignment with
     {
-      return assignment with 
-      {
-        DueAt=assignment.DueAt.AddMilliseconds(0).AddMilliseconds(0),
-        LockAt=assignment.LockAt?.AddMilliseconds(0).AddMilliseconds(0)
-      };
-    }
+      DueAt = dueAt,
+      LockAt = assignment.LockAtDueDate ? dueAt : lockAt
+    };
+  }
 }
diff --git a/Management/Features/Configuration/Synchronization/AssignmentSyncronizationExtensions.cs b/Management/Features/Configuration/Synchronization/AssignmentSyncronizationExtensions.cs
index ea7eb0e..a360127 100644
--- a/Management/Features/Configuration/Synchronization/AssignmentSyncronizationExtensions.cs
+++ b/Management/Features/Configuration/Synchronization/AssignmentSyncronizationExtensions.cs
@@ -78,7 +78,9 @@ public static partial class AssignmentSyncronizationExtensions
       .Replace(">", "")
       .Replace("<", "")
       .Replace("&quot;", "")
-      .Replace("\"", "");
+      .Replace("\"", "")
+      .Replace("&amp;", "")
+      .Replace("&", "");
 
     var canvasHtmlDescription = canvasAssignment.Description;
     canvasHtmlDescription = CanvasScriptTagRegex().Replace(canvasHtmlDescription, "");
@@ -90,7 +92,9 @@ public static partial class AssignmentSyncronizationExtensions
       .Replace(">", "")
       .Replace("<", "")
       .Replace("&quot;", "")
-      .Replace("\"", "");
+      .Replace("\"", "")
+      .Replace("&amp;", "")
+      .Replace("&", "");
 
     var canvasComparisonDueDate =
       canvasAssignment.DueAt != null
@@ -115,12 +119,37 @@ public static partial class AssignmentSyncronizationExtensions
         )
         : new DateTime();
 
+    var canvasComparisonLockDate =
+      canvasAssignment.LockAt != null
+        ? new DateTime(
+          year: canvasAssignment.LockAt.Value.Year,
+          month: canvasAssignment.LockAt.Value.Month,
+          day: canvasAssignment.LockAt.Value.Day,
+          hour: canvasAssignment.LockAt.Value.Hour,
+          minute: canvasAssignment.LockAt.Value.Minute,
+          second: canvasAssignment.LockAt.Value.Second
+        )
+        : new DateTime();
+    var localComparisonLockDate = localAssignment.LockAtDueDate
+      ? localComparisonDueDate
+      : canvasAssignment.LockAt != null
+        ? new DateTime(
+          year: localAssignment.LockAt?.Year ?? 0,
+          month: localAssignment.LockAt?.Month ?? 0,
+          day: localAssignment.LockAt?.Day ?? 0,
+          hour: localAssignment.LockAt?.Hour ?? 0,
+          minute: localAssignment.LockAt?.Minute ?? 0,
+          second: localAssignment.LockAt?.Second ?? 0
+        )
+        : new DateTime();
+
     var dueDatesSame =
       canvasAssignment.DueAt != null && canvasComparisonDueDate == localComparisonDueDate;
+    var lockDatesSame = canvasAssignment.LockAt != null && canvasComparisonLockDate == localComparisonLockDate;
+
 
     var descriptionSame = canvasHtmlDescription == localHtmlDescription;
     var nameSame = canvasAssignment.Name == localAssignment.Name;
-    var lockDateSame = canvasAssignment.LockAt == localAssignment.LockAt;
     var pointsSame = canvasAssignment.PointsPossible == localAssignment.PointsPossible;
     var submissionTypesSame = canvasAssignment.SubmissionTypes.SequenceEqual(
       localAssignment.SubmissionTypes.Select(t => t.ToString())
@@ -134,12 +163,24 @@ public static partial class AssignmentSyncronizationExtensions
         Console.WriteLine(canvasComparisonDueDate);
         Console.WriteLine(localComparisonDueDate);
         Console.WriteLine(
-          $"Due dates different for {localAssignment.Name}, local: {localAssignment.DueAt}, in canvas {canvasAssignment.DueAt}"
+          $"Due dates different for assignment {localAssignment.Name}, local: {localAssignment.DueAt}, in canvas {canvasAssignment.DueAt}"
         );
         Console.WriteLine(JsonSerializer.Serialize(localAssignment.DueAt));
         Console.WriteLine(JsonSerializer.Serialize(canvasAssignment.DueAt));
       }
 
+      if (!lockDatesSame)
+      {
+        Console.WriteLine(JsonSerializer.Serialize(canvasAssignment));
+        Console.WriteLine(canvasComparisonLockDate);
+        Console.WriteLine(localComparisonLockDate);
+        Console.WriteLine(
+          $"Lock dates different for assignment {localAssignment.Name}, local: {localAssignment.LockAt}, in canvas {canvasAssignment.LockAt}"
+        );
+        Console.WriteLine(JsonSerializer.Serialize(localAssignment.LockAt));
+        Console.WriteLine(JsonSerializer.Serialize(canvasAssignment.LockAt));
+      }
+
       if (!descriptionSame)
       {
         Console.WriteLine();
@@ -161,10 +202,6 @@ public static partial class AssignmentSyncronizationExtensions
         Console.WriteLine(
           $"names different for {localAssignment.Name}, local: {localAssignment.Name}, in canvas {canvasAssignment.Name}"
         );
-      if (!lockDateSame)
-        Console.WriteLine(
-          $"Lock Dates different for {localAssignment.Name}, local: {localAssignment.LockAt}, in canvas {canvasAssignment.LockAt}"
-        );
       if (!pointsSame)
         Console.WriteLine(
           $"Points different for {localAssignment.Name}, local: {localAssignment.PointsPossible}, in canvas {canvasAssignment.PointsPossible}"
@@ -177,7 +214,7 @@ public static partial class AssignmentSyncronizationExtensions
 
     return !nameSame
       || !dueDatesSame
-      || !lockDateSame
+      || !lockDatesSame
       || !descriptionSame
       || !pointsSame
       || !submissionTypesSame;
diff --git a/Management/Features/Configuration/Synchronization/ModuleSyncronizationExtensions.cs b/Management/Features/Configuration/Synchronization/ModuleSyncronizationExtensions.cs
index 1a550c7..8306b4d 100644
--- a/Management/Features/Configuration/Synchronization/ModuleSyncronizationExtensions.cs
+++ b/Management/Features/Configuration/Synchronization/ModuleSyncronizationExtensions.cs
@@ -134,6 +134,26 @@ public static partial class ModuleSyncronizationExtensions
       }
     }
 
+    foreach(var localQuiz in localModule.Quizzes)
+    {
+      
+      var canvasModuleItemContentIds = canvasModulesItems[moduleCanvasId].Select(i => i.ContentId);
+      if (!canvasModuleItemContentIds.Contains(localQuiz.CanvasId))
+      {
+        var canvasAssignmentId =
+          localQuiz.CanvasId
+          ?? throw new Exception("cannot create module item if assignment does not have canvas id");
+        await canvas.CreateModuleItem(
+          canvasId,
+          moduleCanvasId,
+          localQuiz.Name,
+          "Quiz",
+          canvasAssignmentId
+        );
+        anyUpdated = true;
+      }
+    }
+
     return anyUpdated;
   }
 
diff --git a/Management/Services/Canvas/CanvasAssignmentService.cs b/Management/Services/Canvas/CanvasAssignmentService.cs
index d7ff29a..970a4c1 100644
--- a/Management/Services/Canvas/CanvasAssignmentService.cs
+++ b/Management/Services/Canvas/CanvasAssignmentService.cs
@@ -44,7 +44,7 @@ public class CanvasAssignmentService
       submission_types = localAssignment.SubmissionTypes.Select(t => t.ToString()),
       description = htmlDescription,
       due_at = localAssignment.DueAt,
-      lock_at = localAssignment.LockAt,
+      lock_at = localAssignment.LockAtDueDate ? localAssignment.DueAt : localAssignment.LockAt,
       points_possible = localAssignment.PointsPossible
     };
     var bodyObj = new { assignment = body };
@@ -71,7 +71,7 @@ public class CanvasAssignmentService
       submission_types = localAssignment.SubmissionTypes.Select(t => t.ToString()),
       description = htmlDescription,
       due_at = localAssignment.DueAt,
-      lock_at = localAssignment.LockAt,
+      lock_at = localAssignment.LockAtDueDate ? localAssignment.DueAt : localAssignment.LockAt,
       points_possible = localAssignment.PointsPossible
     };
     var bodyObj = new { assignment = body };
diff --git a/Management/Services/Canvas/CanvasQuizService.cs b/Management/Services/Canvas/CanvasQuizService.cs
index 7afd5f9..2f0fff8 100644
--- a/Management/Services/Canvas/CanvasQuizService.cs
+++ b/Management/Services/Canvas/CanvasQuizService.cs
@@ -39,7 +39,7 @@ public class CanvasQuizService
       {
         title = localQuiz.Name,
         description = localQuiz.Description,
-        // assignment_group_id = "quiz", TODO: support specific assignment groups
+        // assignment_group_id = "quiz", // TODO: support specific assignment groups
         // time_limit = localQuiz.TimeLimit,
         shuffle_answers = localQuiz.ShuffleAnswers,
         // hide_results = localQuiz.HideResults,
@@ -47,7 +47,7 @@ public class CanvasQuizService
         one_question_at_a_time = true,
         cant_go_back = false,
         due_at = localQuiz.DueAt,
-        lock_at = localQuiz.LockAt,
+        lock_at = localQuiz.LockAtDueDate ? localQuiz.DueAt : localQuiz.LockAt,
       }
     };
     var request = new RestRequest(url);
@@ -60,7 +60,6 @@ public class CanvasQuizService
     var updatedQuiz = localQuiz with { CanvasId = canvasQuiz.Id };
     var quizWithQuestions = await CreateQuizQuestions(canvasCourseId, updatedQuiz);
 
-
     return quizWithQuestions;
   }
 
diff --git a/requests/assignment.http b/requests/assignment.http
index 71d6b17..8ba6c7d 100644
--- a/requests/assignment.http
+++ b/requests/assignment.http
@@ -1,9 +1,13 @@
 
 ###
-get https://snow.instructure.com/api/v1/courses
+GET https://snow.instructure.com/api/v1/courses
 Authorization: Bearer {{$dotenv CANVAS_TOKEN}}
 
 
+###
+GET https://snow.instructure.com/api/v1/courses/871954/assignments
+Authorization: Bearer {{$dotenv CANVAS_TOKEN}}
+
 ###
 
 GET https://snow.instructure.com/api/v1/courses/705168
diff --git a/requests/semester.http b/requests/semester.http
index b6aaf71..ad60117 100644
--- a/requests/semester.http
+++ b/requests/semester.http
@@ -23,5 +23,9 @@ GET https://snow.instructure.com/api/v1/courses/855351/modules
 Authorization: Bearer {{$dotenv CANVAS_TOKEN}}
 ###
 
-GET https://snow.instructure.com/api/v1/courses/872095/modules 
-Authorization: Bearer {{$dotenv CANVAS_TOKEN}}
\ No newline at end of file
+GET https://snow.instructure.com/api/v1/courses/871954/modules 
+Authorization: Bearer {{$dotenv CANVAS_TOKEN}}
+
+###
+GET https://snow.instructure.com/api/v1/courses/871954/modules/2972989/items
+Authorization: Bearer {{$dotenv CANVAS_TOKEN}}