Config setting to default date range

ucdavis · Mar 11, 2024 · c50c922 · c50c922
1 parent 3be2cd7
commit c50c922
Show file tree

Hide file tree

Showing 6 changed files with 31 additions and 11 deletions.
diff --git a/Sloth.Web/Controllers/ReportsController.cs b/Sloth.Web/Controllers/ReportsController.cs
@@ -21,15 +21,20 @@
 using Sloth.Web.Models.TransactionViewModels;
 using Sloth.Web.Resources;
 using Sloth.Web.Helpers;
+using Sloth.Web.Models;
+using Microsoft.Extensions.Options;
 
 namespace Sloth.Web.Controllers
 {
 
     [Authorize(Policy = PolicyCodes.TeamAnyRole)]
     public class ReportsController : SuperController
     {
-        public ReportsController(ApplicationUserManager userManager, SlothDbContext dbContext) : base(userManager, dbContext)
-        { }
+        private readonly DataLimitingOptions _dataLimitingOptions;
+        public ReportsController(ApplicationUserManager userManager, SlothDbContext dbContext, IOptions<DataLimitingOptions> dataLimitingOptions) : base(userManager, dbContext)
+        {
+            _dataLimitingOptions = dataLimitingOptions.Value;
+        }
 
         public IActionResult Index()
         {
@@ -78,7 +83,7 @@ public async Task<IActionResult> DownloadableTransactions(TransactionsFilterMode
             if (filter == null)
                 filter = new TransactionsFilterModel();
 
-            FilterHelpers.SanitizeTransactionsFilter(filter);
+            FilterHelpers.SanitizeTransactionsFilter(filter, _dataLimitingOptions.DefaultDateRange);
 
             var model = new TransfersReportViewModel
             {

diff --git a/Sloth.Web/Controllers/TransactionsController.cs b/Sloth.Web/Controllers/TransactionsController.cs
@@ -21,6 +21,8 @@
 using Sloth.Web.Models.TransactionViewModels;
 using Sloth.Web.Resources;
 using Sloth.Web.Helpers;
+using Microsoft.Extensions.Options;
+using Sloth.Web.Models;
 
 namespace Sloth.Web.Controllers
 {
@@ -29,12 +31,14 @@ public class TransactionsController : SuperController
     {
         private readonly IWebHookService _webHookService;
         private readonly IAggieEnterpriseService _aggieEnterpriseService;
+        private readonly DataLimitingOptions _dataLimitingOptions;
 
         public TransactionsController(ApplicationUserManager userManager, SlothDbContext dbContext, IWebHookService webHookService,
-            IAggieEnterpriseService aggieEnterpriseService) : base(userManager, dbContext)
+            IAggieEnterpriseService aggieEnterpriseService, IOptions<DataLimitingOptions> dataLimitingOptions) : base(userManager, dbContext)
         {
             _webHookService = webHookService;
             _aggieEnterpriseService = aggieEnterpriseService;
+            _dataLimitingOptions = dataLimitingOptions.Value;
         }
 
 
@@ -45,7 +49,7 @@ public async Task<IActionResult> Index(TransactionsFilterModel filter = null)
             if (filter == null)
                 filter = new TransactionsFilterModel();
 
-            FilterHelpers.SanitizeTransactionsFilter(filter);
+            FilterHelpers.SanitizeTransactionsFilter(filter, _dataLimitingOptions.DefaultDateRange);
 
             IQueryable<Transaction> query;
 

diff --git a/Sloth.Web/Helpers/FilterHelpers.cs b/Sloth.Web/Helpers/FilterHelpers.cs
@@ -5,15 +5,15 @@ namespace Sloth.Web.Helpers
 {
     public class FilterHelpers
     {
-        public static void SanitizeTransactionsFilter(TransactionsFilterModel model)
+        public static void SanitizeTransactionsFilter(TransactionsFilterModel model, int defaultDays)
         {
-            var fromUtc = (model.From ?? DateTime.Now.AddMonths(-1)).ToUniversalTime().Date;
+            var fromUtc = (model.From ?? DateTime.Now.AddDays(defaultDays * -1)).ToUniversalTime().Date;
             var throughUtc = (model.To ?? DateTime.Now).ToUniversalTime().AddDays(1).Date;
 
             if (fromUtc > DateTime.UtcNow || fromUtc < DateTime.UtcNow.AddYears(-100))
             {
-                // invalid, so default to filtering from one month ago
-                var from = DateTime.Now.AddMonths((-1)).Date;
+                // invalid, so default to filtering from one month ago -- Use parameter defaultDays instead
+                var from = DateTime.Now.AddDays(defaultDays * -1).Date;
                 model.From = from;
                 fromUtc = from.ToUniversalTime();
             }
@@ -24,8 +24,8 @@ public static void SanitizeTransactionsFilter(TransactionsFilterModel model)
 
             if (fromUtc >= throughUtc)
             {
-                // invalid, so default to filtering through one month after fromUtc
-                throughUtc = fromUtc.AddMonths(1).AddDays(1).Date;
+                // invalid, so default to filtering through one month after fromUtc -- Use parameter defaultDays instead
+                throughUtc = fromUtc.AddDays(defaultDays * -1).Date;
                 model.To = throughUtc.AddDays(-1).ToLocalTime();
             }
             else

diff --git a/Sloth.Web/Models/DataLimitingOptions.cs b/Sloth.Web/Models/DataLimitingOptions.cs
@@ -0,0 +1,7 @@
+namespace Sloth.Web.Models
+{
+    public class DataLimitingOptions
+    {
+        public int DefaultDateRange { get; set; }
+    }
+}
diff --git a/Sloth.Web/Startup.cs b/Sloth.Web/Startup.cs
@@ -60,6 +60,7 @@ public void ConfigureServices(IServiceCollection services)
             services.Configure<SparkpostOptions>(Configuration.GetSection("SparkPost"));
             services.Configure<NotificationOptions>(Configuration.GetSection("Notifications"));
             services.Configure<KfsOptions>(Configuration.GetSection("Kfs"));
+            services.Configure<DataLimitingOptions>(Configuration.GetSection("DataLimiting"));
 
 
             // add infrastructure services

diff --git a/Sloth.Web/appsettings.json b/Sloth.Web/appsettings.json
@@ -58,6 +58,9 @@
     "MaxRetryDelaySeconds": 32,
     "MinFetchAgeMinutes": 10
   },
+  "DataLimiting": {
+    "DefaultDateRange": 30,
+  },
   "RebuildDb": false,
   "AutoMigrateDb": false,
   "CasBaseUrl": "https://ssodev.ucdavis.edu/cas"