refactor: Simplify date parsing logic in AgentTelemetryService and improve condition checks in MapRotationCfgParser

Author: frasermolyneux

src/XtremeIdiots.Portal.Web/Services/AgentTelemetryService.cs

@@ -190,7 +190,7 @@ private string GetAppInsightsResourceId()
         var value = row[index];
         return value is DateTimeOffset dto
             ? dto.UtcDateTime
-            : value is DateTime dt ? dt : (DateTime?)(DateTime.TryParse(value?.ToString(), out var parsed) ? parsed : null);
+            : value is DateTime dt ? dt : DateTime.TryParse(value?.ToString(), out var parsed) ? parsed : null;
     }
 
     private static int GetIntValue(LogsTableRow row, IReadOnlyList<LogsTableColumn> columns, string columnName)

src/XtremeIdiots.Portal.Web/Services/MapRotationCfgParser.cs

@@ -112,9 +112,11 @@ private static bool IsContinuationVariable(string baseVar, int suffixIndex, stri
     {
         // sv_maprotation_1, sv_maprotation_2 are continuations of sv_maprotation
         // But scr_aacp_maps_1 is NOT a continuation — it's the base variable
-        return baseVar.Equals("sv_maprotation", StringComparison.OrdinalIgnoreCase) && suffixIndex > 0
-            ? true
-            : baseVar.Equals("scr_small_rotation", StringComparison.OrdinalIgnoreCase) && suffixIndex > 0 || (baseVar.Equals("scr_med_rotation", StringComparison.OrdinalIgnoreCase) && suffixIndex > 0) || (baseVar.Equals("scr_large_rotation", StringComparison.OrdinalIgnoreCase) && suffixIndex > 0);
+        return suffixIndex > 0
+            && (baseVar.Equals("sv_maprotation", StringComparison.OrdinalIgnoreCase)
+                || baseVar.Equals("scr_small_rotation", StringComparison.OrdinalIgnoreCase)
+                || baseVar.Equals("scr_med_rotation", StringComparison.OrdinalIgnoreCase)
+                || baseVar.Equals("scr_large_rotation", StringComparison.OrdinalIgnoreCase));
     }
 
     private static (string BaseVar, int SuffixIndex) ParseVariableSuffix(string varName)