From 7d45db355d88c24c8ab2170ff753df323a6fa77c Mon Sep 17 00:00:00 2001
From: Sipke Schoorstra <sipkeschoorstra@outlook.com>
Date: Fri, 27 Sep 2024 13:07:08 +0200
Subject: [PATCH] Refactor serializer options retrieval method name.

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.
---
 .../Contracts/IPayloadSerializer.cs                   |  5 +++++
 .../Serializers/JsonPayloadSerializer.cs              | 11 ++++++-----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/modules/Elsa.Workflows.Core/Contracts/IPayloadSerializer.cs b/src/modules/Elsa.Workflows.Core/Contracts/IPayloadSerializer.cs
index 5768049ea..762765bf4 100644
--- a/src/modules/Elsa.Workflows.Core/Contracts/IPayloadSerializer.cs
+++ b/src/modules/Elsa.Workflows.Core/Contracts/IPayloadSerializer.cs
@@ -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();
 }
\ No newline at end of file
diff --git a/src/modules/Elsa.Workflows.Core/Serialization/Serializers/JsonPayloadSerializer.cs b/src/modules/Elsa.Workflows.Core/Serialization/Serializers/JsonPayloadSerializer.cs
index cba5c7c01..820893889 100644
--- a/src/modules/Elsa.Workflows.Core/Serialization/Serializers/JsonPayloadSerializer.cs
+++ b/src/modules/Elsa.Workflows.Core/Serialization/Serializers/JsonPayloadSerializer.cs
@@ -25,14 +25,14 @@ public class JsonPayloadSerializer : IPayloadSerializer
     /// <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);
     }
 
@@ -51,18 +51,19 @@ public class JsonPayloadSerializer : IPayloadSerializer
     /// <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
         {