From e6cc3c2a3c9188fe6e5c6abfa679eb48307cb43e Mon Sep 17 00:00:00 2001 From: Alex Mickelson Date: Fri, 5 Jan 2024 08:24:51 -0700 Subject: [PATCH] got assignment edit page working again --- .../AssignmentForm/AssignmentForm.razor | 10 +- .../AssignmentMarkdownEditor.razor | 2 +- .../Assignment/AssignmentListItem.razor | 11 +- Management.Web/appsettings.Development.json | 2 +- Management.Web/appsettings.json | 2 +- .../Assignment/AssignemntParseExceptions.cs | 10 ++ .../Local/Assignment/LocalAssignment.cs | 119 +---------------- .../LocalAssignmentMarkdownParser.cs | 120 ++++++++++++++++++ docker-compose.yml | 1 + 9 files changed, 148 insertions(+), 129 deletions(-) create mode 100644 Management/Models/Local/Assignment/AssignemntParseExceptions.cs create mode 100644 Management/Models/Local/Assignment/LocalAssignmentMarkdownParser.cs diff --git a/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor b/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor index 1b2d518..f39b278 100644 --- a/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor +++ b/Management.Web/Shared/Components/AssignmentForm/AssignmentForm.razor @@ -34,6 +34,9 @@ private string name { get; set; } = String.Empty; private bool addingAssignmentToCanvas = false; private bool deletingAssignmentFromCanvas = false; + private bool showHelp = false; + + private void toggleHelp() => showHelp = !showHelp; private void submitHandler() { @@ -157,9 +160,6 @@ deletingAssignmentFromCanvas = false; StateHasChanged(); } - - - private bool showHelp { get; set; } = false; }
@@ -170,7 +170,7 @@
@if (assignmentContext.Assignment != null) { - @* *@ + }
@@ -182,7 +182,7 @@
} - diff --git a/Management.Web/Shared/Components/AssignmentForm/AssignmentMarkdownEditor.razor b/Management.Web/Shared/Components/AssignmentForm/AssignmentMarkdownEditor.razor index c5775e0..0c197aa 100644 --- a/Management.Web/Shared/Components/AssignmentForm/AssignmentMarkdownEditor.razor +++ b/Management.Web/Shared/Components/AssignmentForm/AssignmentMarkdownEditor.razor @@ -64,7 +64,7 @@ private MarkupString preview { get => (MarkupString)Markdown.ToHtml(assignmentContext?.Assignment?.Description ?? ""); } private string HelpText() { - var groupNames = string.Join(", " , planner.LocalCourse?.Settings.AssignmentGroups.Select(g => g.Name)); + var groupNames = string.Join(", " , planner.LocalCourse?.Settings.AssignmentGroups.Select(g => g.Name) ?? []); return $@" SubmissionTypes: - {AssignmentSubmissionType.ONLINE_TEXT_ENTRY} diff --git a/Management.Web/Shared/Module/Assignment/AssignmentListItem.razor b/Management.Web/Shared/Module/Assignment/AssignmentListItem.razor index 4cf7b7c..e60f12c 100644 --- a/Management.Web/Shared/Module/Assignment/AssignmentListItem.razor +++ b/Management.Web/Shared/Module/Assignment/AssignmentListItem.razor @@ -98,12 +98,17 @@ - @if(existsInCanvas && NeedsToBeUpdatedInCanvas) + @if( + planner.LocalCourse != null + && existsInCanvas + && NeedsToBeUpdatedInCanvas + && assignmentInCanvas != null + ) {
@Assignment.GetUpdateReason( - (CanvasAssignment)assignmentInCanvas, - Assignment.GetCanvasAssignmentGroupId(planner.LocalCourse.Settings.AssignmentGroups)) + (CanvasAssignment)assignmentInCanvas, + Assignment.GetCanvasAssignmentGroupId(planner.LocalCourse.Settings.AssignmentGroups))
} @if(!existsInCanvas) diff --git a/Management.Web/appsettings.Development.json b/Management.Web/appsettings.Development.json index f042c67..253cd2d 100644 --- a/Management.Web/appsettings.Development.json +++ b/Management.Web/appsettings.Development.json @@ -2,7 +2,7 @@ "DetailedErrors": true, "Logging": { "LogLevel": { - "Default": "Information", + "Default": "Error", "Microsoft.AspNetCore": "Warning" } } diff --git a/Management.Web/appsettings.json b/Management.Web/appsettings.json index d9e88ac..04986a1 100644 --- a/Management.Web/appsettings.json +++ b/Management.Web/appsettings.json @@ -1,7 +1,7 @@ { "Logging": { "LogLevel": { - "Default": "Trace", + "Default": "Error", "Microsoft.AspNetCore": "Warning" } }, diff --git a/Management/Models/Local/Assignment/AssignemntParseExceptions.cs b/Management/Models/Local/Assignment/AssignemntParseExceptions.cs new file mode 100644 index 0000000..bb54163 --- /dev/null +++ b/Management/Models/Local/Assignment/AssignemntParseExceptions.cs @@ -0,0 +1,10 @@ + +namespace LocalModels; +public class RubricMarkdownParseException : Exception +{ + public RubricMarkdownParseException(string message) : base(message) { } +} +public class AssignmentMarkdownParseException : Exception +{ + public AssignmentMarkdownParseException(string message) : base(message) { } +} \ No newline at end of file diff --git a/Management/Models/Local/Assignment/LocalAssignment.cs b/Management/Models/Local/Assignment/LocalAssignment.cs index 919920e..c31d0dd 100644 --- a/Management/Models/Local/Assignment/LocalAssignment.cs +++ b/Management/Models/Local/Assignment/LocalAssignment.cs @@ -57,89 +57,6 @@ public record LocalAssignment return yaml; } - public static LocalAssignment ParseMarkdown(string input) - { - var settingsString = input.Split("---")[0]; - var (name, localAssignmentGroupName, submissionTypes, dueAt, lockAt) = parseSettings(settingsString); - - var description = input.Split("---\n")[1].Split("## Rubric")[0]; - - var rubricString = input.Split("## Rubric\n")[1]; - var rubric = ParseRubricMarkdown(rubricString); - return new LocalAssignment() - { - Name = name.Trim(), - LocalAssignmentGroupName = localAssignmentGroupName.Trim(), - SubmissionTypes = submissionTypes, - DueAt = dueAt, - LockAt = lockAt, - Rubric = rubric, - Description = description.Trim() - }; - } - - private static (string name, string assignmentGroupName, List submissionTypes, DateTime dueAt, DateTime? lockAt) parseSettings(string input) - { - var name = extractLabelValue(input, "Name"); - var rawLockAt = extractLabelValue(input, "LockAt"); - var rawDueAt = extractLabelValue(input, "DueAt"); - var localAssignmentGroupName = extractLabelValue(input, "AssignmentGroupName"); - var submissionTypes = parseSubmissionTypes(input); - - DateTime? lockAt = DateTime.TryParse(rawLockAt, out DateTime parsedLockAt) - ? parsedLockAt - : null; - var dueAt = DateTime.TryParse(rawDueAt, out DateTime parsedDueAt) - ? parsedDueAt - : throw new QuizMarkdownParseException($"Error with DueAt: {rawDueAt}"); - - return (name, localAssignmentGroupName, submissionTypes, dueAt, lockAt); - - - } - - private static List parseSubmissionTypes(string input) - { - input = input.Replace("\r\n", "\n"); - List submissionTypes = new List(); - - // Define a regular expression pattern to match the bulleted list items - string startOfTypePattern = @"- (.+)"; - Regex regex = new Regex(startOfTypePattern); - - var words = input.Split("SubmissionTypes:"); - var inputAfterSubmissionTypes = words[1]; - - string[] lines = inputAfterSubmissionTypes.Split("\n", StringSplitOptions.RemoveEmptyEntries); - - foreach (string line in lines) - { - string trimmedLine = line.Trim(); - Match match = regex.Match(trimmedLine); - - if (!match.Success) - break; - - string type = match.Groups[1].Value.Trim(); - submissionTypes.Add(type); - } - - return submissionTypes; - } - - static string extractLabelValue(string input, string label) - { - string pattern = $@"{label}: (.*?)\n"; - Match match = Regex.Match(input, pattern); - - if (match.Success) - { - return match.Groups[1].Value; - } - - return string.Empty; - } - public string ToMarkdown() { var settingsMarkdown = settingsToMarkdown(); @@ -180,39 +97,5 @@ public record LocalAssignment return builder.ToString(); } - public static IEnumerable ParseRubricMarkdown(string rawMarkdown) - { - if (rawMarkdown.Trim() == string.Empty) - return []; - var lines = rawMarkdown.Trim().Split("\n"); - var items = lines.Select(parseIndividualRubricItemMarkdown).ToArray(); - return items; - } - - private static RubricItem parseIndividualRubricItemMarkdown(string rawMarkdown) - { - var pointsPattern = @"\s*-\s*(\d+)\s*pt(s)?:"; - var match = Regex.Match(rawMarkdown, pointsPattern); - if (!match.Success) - throw new RubricMarkdownParseException($"points not found: {rawMarkdown}"); - - var points = int.Parse(match.Groups[1].Value); - - var label = string.Join(": ", rawMarkdown.Split(": ").Skip(1)); - - return new RubricItem() - { - Points = points, - Label = label - }; - } + public static LocalAssignment ParseMarkdown(string input) => LocalAssignmentMarkdownParser.ParseMarkdown(input); } - -public class RubricMarkdownParseException : Exception -{ - public RubricMarkdownParseException(string message) : base(message) { } -} -public class AssignmentMarkdownParseException : Exception -{ - public AssignmentMarkdownParseException(string message) : base(message) { } -} \ No newline at end of file diff --git a/Management/Models/Local/Assignment/LocalAssignmentMarkdownParser.cs b/Management/Models/Local/Assignment/LocalAssignmentMarkdownParser.cs new file mode 100644 index 0000000..3b0c66a --- /dev/null +++ b/Management/Models/Local/Assignment/LocalAssignmentMarkdownParser.cs @@ -0,0 +1,120 @@ + + +using System.Text.RegularExpressions; + +namespace LocalModels; +public static class LocalAssignmentMarkdownParser +{ + + public static LocalAssignment ParseMarkdown(string input) + { + var settingsString = input.Split("---")[0]; + var (name, localAssignmentGroupName, submissionTypes, dueAt, lockAt) = parseSettings(settingsString); + + var description = input.Split("---\n")[1].Split("## Rubric")[0]; + + var rubricString = input.Split("## Rubric\n")[1]; + var rubric = ParseRubricMarkdown(rubricString); + return new LocalAssignment() + { + Name = name.Trim(), + LocalAssignmentGroupName = localAssignmentGroupName.Trim(), + SubmissionTypes = submissionTypes, + DueAt = dueAt, + LockAt = lockAt, + Rubric = rubric, + Description = description.Trim() + }; + } + + private static (string name, string assignmentGroupName, List submissionTypes, DateTime dueAt, DateTime? lockAt) parseSettings(string input) + { + var name = extractLabelValue(input, "Name"); + var rawLockAt = extractLabelValue(input, "LockAt"); + var rawDueAt = extractLabelValue(input, "DueAt"); + var localAssignmentGroupName = extractLabelValue(input, "AssignmentGroupName"); + var submissionTypes = parseSubmissionTypes(input); + + DateTime? lockAt = DateTime.TryParse(rawLockAt, out DateTime parsedLockAt) + ? parsedLockAt + : null; + var dueAt = DateTime.TryParse(rawDueAt, out DateTime parsedDueAt) + ? parsedDueAt + : throw new QuizMarkdownParseException($"Error with DueAt: {rawDueAt}"); + + return (name, localAssignmentGroupName, submissionTypes, dueAt, lockAt); + + + } + + private static List parseSubmissionTypes(string input) + { + input = input.Replace("\r\n", "\n"); + List submissionTypes = new List(); + + // Define a regular expression pattern to match the bulleted list items + string startOfTypePattern = @"- (.+)"; + Regex regex = new Regex(startOfTypePattern); + + var words = input.Split("SubmissionTypes:"); + var inputAfterSubmissionTypes = words[1]; + + string[] lines = inputAfterSubmissionTypes.Split("\n", StringSplitOptions.RemoveEmptyEntries); + + foreach (string line in lines) + { + string trimmedLine = line.Trim(); + Match match = regex.Match(trimmedLine); + + if (!match.Success) + break; + + string type = match.Groups[1].Value.Trim(); + submissionTypes.Add(type); + } + + return submissionTypes; + } + + private static string extractLabelValue(string input, string label) + { + string pattern = $@"{label}: (.*?)\n"; + Match match = Regex.Match(input, pattern); + + if (match.Success) + { + return match.Groups[1].Value; + } + + return string.Empty; + } + + + public static IEnumerable ParseRubricMarkdown(string rawMarkdown) + { + if (rawMarkdown.Trim() == string.Empty) + return []; + var lines = rawMarkdown.Trim().Split("\n"); + var items = lines.Select(parseIndividualRubricItemMarkdown).ToArray(); + return items; + } + + private static RubricItem parseIndividualRubricItemMarkdown(string rawMarkdown) + { + var pointsPattern = @"\s*-\s*(\d+)\s*pt(s)?:"; + var match = Regex.Match(rawMarkdown, pointsPattern); + if (!match.Success) + throw new RubricMarkdownParseException($"points not found: {rawMarkdown}"); + + var points = int.Parse(match.Groups[1].Value); + + var label = string.Join(": ", rawMarkdown.Split(": ").Skip(1)); + + return new RubricItem() + { + Points = points, + Label = label + }; + } + +} \ No newline at end of file diff --git a/docker-compose.yml b/docker-compose.yml index 028b34d..f4a9770 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -16,6 +16,7 @@ services: - ~/projects/faculty/1400/2024_spring_alex/modules:/app/storage/1400 - ~/projects/faculty/1405/2024_spring_alex/modules:/app/storage/1405 - ~/projects/faculty/4620_Distributed/2024Spring/modules:/app/storage/distributed + - ~/projects/faculty/3840_Telemetry/2024Spring_alex/modules:/app/storage/telemetry_and_operations collector: image: otel/opentelemetry-collector-contrib