lgladsden/jellyfin-server
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix JsonConverter recursion

Browse Source
This commit is contained in:
crobibero 2020-12-08 07:18:25 -07:00
parent 80ff564143
commit 26e5aacb90
3 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Emby.Server.Implementations/LiveTv/TunerHosts/HdHomerun/HdHomerunHost.cs
View File

@ -342,10 +342,8 @@ namespace Emby.Server.Implementations.LiveTv.TunerHosts.HdHomerun
public string URL { get; set; } public string URL { get; set; }
[JsonConverter(typeof(JsonBoolNumberConverter))]
public bool Favorite { get; set; } public bool Favorite { get; set; }
[JsonConverter(typeof(JsonBoolNumberConverter))]
public bool DRM { get; set; } public bool DRM { get; set; }
public int HD { get; set; } public int HD { get; set; }

2
MediaBrowser.Common/Json/Converters/JsonBoolNumberConverter.cs
View File

@ -24,7 +24,7 @@ namespace MediaBrowser.Common.Json.Converters
/// <inheritdoc /> /// <inheritdoc />
public override void Write(Utf8JsonWriter writer, bool value, JsonSerializerOptions options) public override void Write(Utf8JsonWriter writer, bool value, JsonSerializerOptions options)
{ {
JsonSerializer.Serialize(writer, value, options); writer.WriteBooleanValue(value);
} }
} }
} }

11
tests/Jellyfin.Common.Tests/Json/JsonBoolNumberTests.cs
View File

@ -19,5 +19,16 @@ namespace Jellyfin.Common.Tests.Json
var value = JsonSerializer.Deserialize<bool>(input, options); var value = JsonSerializer.Deserialize<bool>(input, options);
Assert.Equal(value, output); Assert.Equal(value, output);
} }
[Theory]
[InlineData(true, "true")]
[InlineData(false, "false")]
public static void Serialize_Bool_Success(bool input, string output)
{
var options = new JsonSerializerOptions();
options.Converters.Add(new JsonBoolNumberConverter());
var value = JsonSerializer.Serialize(input, options);
Assert.Equal(value, output);
}
} }
} }