fix: 500 error if paginated_stats are empty

openedx · Nov 21, 2023 · f94ffaa · f94ffaa
1 parent f408aa2
commit f94ffaa
Showing 1 changed file with 13 additions and 6 deletions.
diff --git a/api/users.rb b/api/users.rb
@@ -71,12 +71,19 @@
                                          ]
                                        }}
                                      ]).to_a[0]
-    total_count = paginated_stats["pagination"][0]["total_count"]
-    num_pages = [1, (total_count / per_page.to_f).ceil].max
-    data = paginated_stats["data"].map do |user_stats|
-      {
-        :username => user_stats["username"]
-      }.merge(user_stats["course_stats"].except(*exclude_from_stats))
+    if paginated_stats["pagination"].empty?
+      data = []
+      num_pages = 0
+      page = 0
+      total_count = 0
+    else
+      total_count = paginated_stats["pagination"][0]["total_count"]
+      num_pages = [1, (total_count / per_page.to_f).ceil].max
+      data = paginated_stats["data"].map do |user_stats|
+        {
+          :username => user_stats["username"]
+        }.merge(user_stats["course_stats"].except(*exclude_from_stats))
+      end
     end
   else
     # If a list of usernames is provided, then sort by the order in which those names appear