Should fix error retrieving submissions for too many users

UvA-FNWI · Nov 24, 2024 · 0211ba4 · 0211ba4
1 parent f38082f
commit 0211ba4
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/IguideME.Web/Services/LMSHandlers/CanvasHandler.cs b/IguideME.Web/Services/LMSHandlers/CanvasHandler.cs
@@ -185,7 +185,17 @@ private AppGradingType mapGradingType(GradingType type)
         /// <inheritdoc />
         public IEnumerable<AssignmentSubmission> GetSubmissions(int courseID, List<User> users)
         {
-            _logger.LogInformation("Getting submissions for users:\n{}", users.Select(user => user.UserID));
+            List<AssignmentSubmission> submissions = new();
+            for (int i = 0; i < users.Count; i += 100)
+            {
+                submissions.Concat(GetSubmissionsBatch(courseID, users.GetRange(i, Math.Min(100, users.Count - i))));
+            };
+            return submissions;
+
+        }
+
+        public IEnumerable<AssignmentSubmission> GetSubmissionsBatch(int courseID, List<User> users)
+        {
             return Connector
                 .FindCourseById(courseID)
                 .GetSubmissions(users.Select(user => user.UserID).ToArray(), false)

diff --git a/charts/iguideme/Chart.yaml b/charts/iguideme/Chart.yaml
@@ -2,5 +2,5 @@ apiVersion: v2
 name: iguideme
 description: IguideME
 type: application
-version: 0.3.239
-appVersion: "0.3.239"
+version: 0.3.240
+appVersion: "0.3.240"