Return empty string instead of null to fix read error in KWGT

sboulema · Nov 4, 2024 · 311c084 · 311c084
1 parent c4fc174
commit 311c084
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/Converters/DateTimeJsonConverter.cs b/src/Converters/DateTimeJsonConverter.cs
@@ -4,12 +4,12 @@
 
 namespace F1Widget.Converters;
 
-public class DateTimeJsonConverter : JsonConverter<DateTime>
+public class DateTimeJsonConverter : JsonConverter<DateTime?>
 {
     private static readonly CultureInfo Culture = CultureInfo.InvariantCulture;
     private const string Format = "dd MMM HH:mm";
 
-    public override DateTime Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+    public override DateTime? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
     {
         if (!DateTime.TryParseExact(reader.GetString(), Format, Culture, DateTimeStyles.None, out var dateTime))
         {
@@ -19,8 +19,10 @@ public override DateTime Read(ref Utf8JsonReader reader, Type typeToConvert, Jso
         return dateTime;
     }
 
-    public override void Write(Utf8JsonWriter writer, DateTime value, JsonSerializerOptions options)
+    public override void Write(Utf8JsonWriter writer, DateTime? value, JsonSerializerOptions options)
     {
-        writer.WriteStringValue(value.ToString("dd MMM HH:mm"));
+        writer.WriteStringValue(value == null ? string.Empty : value.Value.ToString(Format));
     }
+
+    public override bool HandleNull => true;
 }
diff --git a/widgets/F1Widget_Mclaren_2024_Chrome.kwgt b/widgets/F1Widget_Mclaren_2024_Chrome.kwgt