Move shared helpers into Geocoding.Extensions

Root cause: the first split kept a compatibility shim and legacy JSON method casing, which preserved old entry points but did not match the requested .Extensions namespace layout or the desired .NET-style naming for JSON helpers.

Author: niemyjski

src/Geocoding.Core/Extensions.cs

@@ -1,6 +1,6 @@
 using System.Diagnostics.CodeAnalysis;
 
-namespace Geocoding.Collections;
+namespace Geocoding.Extensions;
 
 /// <summary>
 /// Collection-related helpers.

src/Geocoding.Core/Extensions/CollectionExtensions.cs

@@ -1,4 +1,4 @@
-namespace Geocoding.Collections;
+namespace Geocoding.Extensions;
 
 /// <summary>
 /// Enumerable-related helpers.

src/Geocoding.Core/Extensions/EnumerableExtensions.cs

@@ -1,6 +1,8 @@
 using System.Text.Json;
 using System.Text.Json.Serialization;
-namespace Geocoding.Serialization;
+using Geocoding.Serialization;
+
+namespace Geocoding.Extensions;
 
 /// <summary>
 /// JSON serialization helpers and shared serializer options.
@@ -19,7 +21,7 @@ public static class JsonExtensions
     /// </summary>
     /// <param name="value">The object to serialize.</param>
     /// <returns>The JSON payload, or an empty string when the input is null.</returns>
-    public static string ToJSON(object? value)
+    public static string ToJson(object? value)
     {
         if (value is null)
             return String.Empty;
@@ -33,7 +35,7 @@ public static string ToJSON(object? value)
     /// <typeparam name="T">The destination type.</typeparam>
     /// <param name="json">The JSON payload.</param>
     /// <returns>A deserialized instance, or default value for blank input.</returns>
-    public static T? FromJSON<T>(string? json)
+    public static T? FromJson<T>(string? json)
     {
         if (String.IsNullOrWhiteSpace(json))
             return default;

src/Geocoding.Core/Extensions/JsonExtensions.cs

@@ -4,7 +4,7 @@
 using System.Text;
 using System.Text.Json;
 using System.Text.Json.Serialization;
-using Geocoding.Serialization;
+using Geocoding.Extensions;
 
 namespace Geocoding.Here;
 

src/Geocoding.Here/HereGeocoder.cs

@@ -1,6 +1,6 @@
 using System.Text;
 using System.Text.Json.Serialization;
-using Geocoding.Serialization;
+using Geocoding.Extensions;
 
 namespace Geocoding.MapQuest;
 
@@ -135,7 +135,7 @@ public virtual string RequestBody
     {
         get
         {
-            return JsonExtensions.ToJSON(this);
+            return JsonExtensions.ToJson(this);
         }
     }
 

src/Geocoding.MapQuest/BaseRequest.cs

@@ -1,5 +1,5 @@
 using System.Text.Json.Serialization;
-using Geocoding.Collections;
+using Geocoding.Extensions;
 
 namespace Geocoding.MapQuest;
 

src/Geocoding.MapQuest/BatchGeocodeRequest.cs

@@ -1,8 +1,7 @@
 using System.Net;
 using System.Net.Http;
 using System.Text;
-using Geocoding.Collections;
-using Geocoding.Serialization;
+using Geocoding.Extensions;
 
 namespace Geocoding.MapQuest;
 
@@ -227,7 +226,7 @@ private async Task<MapQuestResponse> Parse(HttpClient client, HttpRequestMessage
             if (String.IsNullOrWhiteSpace(json))
                 throw new Exception("Remote system response with blank: " + requestInfo);
 
-            MapQuestResponse? o = JsonExtensions.FromJSON<MapQuestResponse>(json);
+            MapQuestResponse? o = JsonExtensions.FromJson<MapQuestResponse>(json);
             if (o is null)
                 throw new Exception("Unable to deserialize remote response: " + requestInfo);
 

src/Geocoding.MapQuest/MapQuestGeocoder.cs

@@ -4,7 +4,7 @@
 using System.Net.Http;
 using System.Text.Json;
 using System.Text.Json.Serialization;
-using Geocoding.Serialization;
+using Geocoding.Extensions;
 
 namespace Geocoding.Microsoft;
 

src/Geocoding.Microsoft/AzureMapsGeocoder.cs

@@ -4,8 +4,7 @@
 using System.Linq;
 using System.Text;
 using System.Text.Json;
-using Geocoding.Collections;
-using Geocoding.Serialization;
+using Geocoding.Extensions;
 
 namespace Geocoding.Microsoft;