Merge pull request #31 from IowaComputerGurus/feature/chart-by-action

Fixes #25 by validating 0 length files
IowaComputerGurus · Jan 10, 2024 · 136b58f · 136b58f
2 parents 0d43ca5 + 2f96015
commit 136b58f
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/GitHubCostVisualizer.Web/Controllers/HomeController.cs b/src/GitHubCostVisualizer.Web/Controllers/HomeController.cs
@@ -43,6 +43,11 @@ public IActionResult Report(HomeViewModel data)
                 using (var csvData = new CsvReader(reader, CultureInfo.InvariantCulture))
                 {
                     var rawReport = csvData.GetRecords<GithubUsageEntry>().ToList();
+                    if (rawReport.Count == 0)
+                    {
+                        ModelState.AddModelError(nameof(data.UploadFile), "The uploaded file did not contain any records, please try with a different file.");
+                        return View(nameof(Index), data);
+                    }
                     var model = _processor.ProcessUsageReport(rawReport);
                     return View(model);
                 }