WarningHelper: make warning counts consistent and more configurable

Erisa · Erisa · commit 482bf03b69da · 2025-01-17T00:07:20.000Z
Fixes #250
diff --git a/Helpers/WarningHelpers.cs b/Helpers/WarningHelpers.cs
@@ -34,11 +34,12 @@ await LykosAvatarMethods.UserOrMemberAvatarURL(targetUser, Program.homeGuild, "p
                     .WithColor(color: DiscordColor.DarkGreen);
             else
             {
+                TimeSpan timeToCheck = TimeSpan.FromDays(Program.cfgjson.WarningDaysThreshold);
                 foreach (string key in keys)
                 {
                     UserWarning warning = warningsOutput[key];
                     TimeSpan span = DateTime.Now - warning.WarnTimestamp;
-                    if (span.Days < 31)
+                    if (span <= timeToCheck)
                     {
                         recentCount += 1;
                     }
@@ -77,7 +78,7 @@ await LykosAvatarMethods.UserOrMemberAvatarURL(targetUser, Program.homeGuild, "p
 
                     embed.AddField($"Last {Program.cfgjson.RecentWarningsPeriodHours} hours", hourRecentMatches.Count().ToString(), true);
 
-                    embed.AddField("Last 30 days", recentCount.ToString(), true)
+                    embed.AddField($"Last {Program.cfgjson.WarningDaysThreshold} days", recentCount.ToString(), true)
                         .AddField("Total", keys.Count().ToString(), true);
                 }
 

Original file line number	Diff line number	Diff line change
`@@ -34,11 +34,12 @@ await LykosAvatarMethods.UserOrMemberAvatarURL(targetUser, Program.homeGuild, "p`
`34`	`34`	`.WithColor(color: DiscordColor.DarkGreen);`
`35`	`35`	`else`
`36`	`36`	`{`
	`37`	`+ TimeSpan timeToCheck = TimeSpan.FromDays(Program.cfgjson.WarningDaysThreshold);`
`37`	`38`	`foreach (string key in keys)`
`38`	`39`	`{`
`39`	`40`	`UserWarning warning = warningsOutput[key];`
`40`	`41`	`TimeSpan span = DateTime.Now - warning.WarnTimestamp;`
`41`		`- if (span.Days < 31)`
	`42`	`+ if (span <= timeToCheck)`
`42`	`43`	`{`
`43`	`44`	`recentCount += 1;`
`44`	`45`	`}`
`@@ -77,7 +78,7 @@ await LykosAvatarMethods.UserOrMemberAvatarURL(targetUser, Program.homeGuild, "p`
`77`	`78`
`78`	`79`	`embed.AddField($"Last {Program.cfgjson.RecentWarningsPeriodHours} hours", hourRecentMatches.Count().ToString(), true);`
`79`	`80`
`80`		`- embed.AddField("Last 30 days", recentCount.ToString(), true)`
	`81`	`+ embed.AddField($"Last {Program.cfgjson.WarningDaysThreshold} days", recentCount.ToString(), true)`
`81`	`82`	`.AddField("Total", keys.Count().ToString(), true);`
`82`	`83`	`}`
`83`	`84`