Skip to content

Commit

Permalink
Refactor serializer options retrieval method name.
Browse files Browse the repository at this point in the history
Renamed GetPayloadSerializerOptions to GetOptions for simplicity and updated method accessibility. This change ensures that the method name is more intuitive and aligns with common naming conventions. Additionally, it introduces a new public GetOptions method in the IPayloadSerializer interface.
  • Loading branch information
sfmskywalker committed Sep 27, 2024
1 parent 2e1ed99 commit 7d45db3
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -48,4 +48,9 @@ public interface IPayloadSerializer
/// <param name="serializedData">The serialized state.</param>
/// <returns>The deserialized state.</returns>
T Deserialize<T>(JsonElement serializedData);

/// <summary>
/// Gets the JSON serializer options.
/// </summary>
JsonSerializerOptions GetOptions();
}
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,14 @@ public JsonPayloadSerializer(IServiceProvider serviceProvider)
/// <inheritdoc />
public string Serialize(object payload)
{
var options = GetPayloadSerializerOptions();
var options = GetOptions();
return JsonSerializer.Serialize(payload, options);
}

/// <inheritdoc />
public JsonElement SerializeToElement(object payload)
{
var options = GetPayloadSerializerOptions();
var options = GetOptions();
return JsonSerializer.SerializeToElement(payload, options);
}

Expand All @@ -51,18 +51,19 @@ public object Deserialize(JsonElement payload)
/// <inheritdoc />
public T Deserialize<T>(string payload)
{
var options = GetPayloadSerializerOptions();
var options = GetOptions();
return JsonSerializer.Deserialize<T>(payload, options)!;
}

/// <inheritdoc />
public T Deserialize<T>(JsonElement payload)
{
var options = GetPayloadSerializerOptions();
var options = GetOptions();
return payload.Deserialize<T>(options)!;
}

private JsonSerializerOptions GetPayloadSerializerOptions()
/// <inheritdoc />
public JsonSerializerOptions GetOptions()
{
var options = new JsonSerializerOptions
{
Expand Down

0 comments on commit 7d45db3

Please sign in to comment.