fix bug.

hargata · Nov 12, 2024 · e59e33b · e59e33b
1 parent a671dc1
commit e59e33b
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/Controllers/Vehicle/ReportController.cs b/Controllers/Vehicle/ReportController.cs
@@ -107,15 +107,15 @@ public IActionResult GetReportPartialView(int vehicleId)
             }).ToList();
             if (invertedFuelMileageUnit)
             {
-                foreach(CostForVehicleByMonth monthMileage in monthlyMileageData)
+                foreach (CostForVehicleByMonth monthMileage in monthlyMileageData)
                 {
                     if (monthMileage.Cost != default)
                     {
                         monthMileage.Cost = 100 / monthMileage.Cost;
                     }
                 }
             }
-            var mpgViewModel = new MPGForVehicleByMonth { 
+            var mpgViewModel = new MPGForVehicleByMonth {
                 CostData = monthlyMileageData,
                 Unit = invertedFuelMileageUnit ? preferredFuelMileageUnit : fuelEconomyMileageUnit,
                 SortedCostData = (userConfig.UseMPG || invertedFuelMileageUnit) ? monthlyMileageData.OrderByDescending(x => x.Cost).ToList() : monthlyMileageData.OrderBy(x => x.Cost).ToList()
@@ -322,7 +322,16 @@ public IActionResult GetVehicleAttachments(int vehicleId, List<ImportMode> expor
         }
         public IActionResult GetReportParameters()
         {
-            var viewModel = new ReportParameter();
+            var viewModel = new ReportParameter() { 
+                VisibleColumns = new List<string> {
+                    nameof(GenericReportModel.DataType),
+                    nameof(GenericReportModel.Date),
+                    nameof(GenericReportModel.Odometer),
+                    nameof(GenericReportModel.Description),
+                    nameof(GenericReportModel.Cost),
+                    nameof(GenericReportModel.Notes) 
+                }
+            };
             //get all extra fields from service records, repairs, upgrades, and tax records.
             var recordTypes = new List<int>() { 0, 1, 3, 4 };
             var extraFields = new List<string>();

diff --git a/Models/Report/ReportParameter.cs b/Models/Report/ReportParameter.cs
@@ -2,13 +2,7 @@
 {
     public class ReportParameter
     {
-       public List<string> VisibleColumns { get; set; } = new List<string>() { 
-           nameof(GenericReportModel.DataType), 
-           nameof(GenericReportModel.Date), 
-           nameof(GenericReportModel.Odometer), 
-           nameof(GenericReportModel.Description), 
-           nameof(GenericReportModel.Cost),
-           nameof(GenericReportModel.Notes) };
+       public List<string> VisibleColumns { get; set; } = new List<string>();
        public List<string> ExtraFields { get; set; } = new List<string>();
     }
 }