fix the culture info issue in LocationMetadata (Azure#49560)

Author: ArcturusZhang

sdk/resourcemanager/Azure.ResourceManager/CHANGELOG.md

@@ -1,14 +1,10 @@
 # Release History
 
-## 1.14.0-beta.1 (Unreleased)
-
-### Features Added
-
-### Breaking Changes
+## 1.13.1 (2025-04-24)
 
 ### Bugs Fixed
 
-### Other Changes
+- Fixed the `Longitude` and `Latitude` in `LocationMetadata` class to use invariant culture during serialization and deserialization. (https://github.com/Azure/azure-sdk-for-net/issues/43116)
 
 ## 1.13.0 (2024-09-03)
 

sdk/resourcemanager/Azure.ResourceManager/docs/MigrationGuide.md

@@ -68,7 +68,7 @@ As you can see, now authentication is handled by Azure.Identity, and just a sing
 
 **Old**
 ```C#
-ServiceClientCredentials credentials = getMyCredentials(); 
+ServiceClientCredentials credentials = getMyCredentials();
 ResourceManagementClient resourcesClient = new ResourceManagementClient(credentials);
 
 string rgName = "QuickStartRG";
@@ -112,7 +112,7 @@ AvailabilitySet inputAvailabilitySet = new AvailabilitySet
     PlatformUpdateDomainCount = 1,
     Sku = new CM.Sku
     {
-        Name = AvailabilitySetSkuTypes.Aligned 
+        Name = AvailabilitySetSkuTypes.Aligned
     }
 };
 string aSetName = "quickstartvm_aSet";
@@ -130,7 +130,7 @@ ArmOperation<AvailabilitySetResource> availabilitySetOperation = await availabil
 AvailabilitySetResource availabilitySet = availabilitySetOperation.Value;
 ```
 
-Parameters can be specified via the `AvailabilitySetData` object, in here, the basic default only requires the location. The availability set is created using  the AvailabilitySetsCollection returned from the `GetAvailabilitySets()` extension method instead of using another client. 
+Parameters can be specified via the `AvailabilitySetData` object, in here, the basic default only requires the location. The availability set is created using  the AvailabilitySetsCollection returned from the `GetAvailabilitySets()` extension method instead of using another client.
 
 ### Create a Security Group
 
@@ -256,7 +256,7 @@ ArmOperation<NetworkInterfaceResource> networkInterfaceOperation = await network
 NetworkInterfaceResource networkInterface = networkInterfaceOperation.Value;
 ```
 
-This step is similar to the old SDK, however, notice that the `CreateOrUpdateAsync()` method returns the network interface that has been created. 
+This step is similar to the old SDK, however, notice that the `CreateOrUpdateAsync()` method returns the network interface that has been created.
 
 ### Create a Virtual Machine
 
@@ -510,7 +510,7 @@ In both libraries, subnets are defined inside virtual networks, however, with th
 
 **Old**
 ```C#
-var virutalMachine = azure.VirtualMachines.Define(virtualMachineName)
+var virtualMachine = azure.VirtualMachines.Define(virtualMachineName)
         .WithRegion(Region.USEast)
         .WithExistingResourceGroup(ResourceGroupName)
         .WithExistingPrimaryNetwork(virtualNetwork)

sdk/resourcemanager/Azure.ResourceManager/src/Azure.ResourceManager.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <Version>1.14.0-beta.1</Version>
+    <Version>1.13.1</Version>
     <!--The ApiCompatVersion is managed automatically and should not generally be modified manually.-->
     <ApiCompatVersion>1.13.0</ApiCompatVersion>
     <PackageId>Azure.ResourceManager</PackageId>

sdk/resourcemanager/Azure.ResourceManager/src/Resources/Custom/Models/LocationMetadata.cs

@@ -4,6 +4,7 @@
 #nullable disable
 
 using System.ClientModel.Primitives;
+using System.Globalization;
 using System.Runtime.CompilerServices;
 using System.Text.Json;
 using Azure.Core;
@@ -22,8 +23,14 @@ public partial class LocationMetadata
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         internal void WriteLongitude(Utf8JsonWriter writer, ModelReaderWriterOptions options)
         {
-            if (Longitude != null)
-                writer.WriteStringValue(Longitude.ToString());
+            if (Longitude.HasValue)
+            {
+                writer.WriteStringValue(Longitude.Value.ToString(CultureInfo.InvariantCulture));
+            }
+            else
+            {
+                writer.WriteNullValue();
+            }
         }
 
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -32,14 +39,20 @@ internal static void ReadLongitude(JsonProperty property, ref double? longitude)
             if (property.Value.ValueKind == JsonValueKind.Null)
                 return;
 
-            longitude = double.Parse(property.Value.GetString());
+            longitude = double.Parse(property.Value.GetString(), CultureInfo.InvariantCulture);
         }
 
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         internal void WriteLatitude(Utf8JsonWriter writer, ModelReaderWriterOptions options)
         {
-            if (Latitude != null)
-                writer.WriteStringValue(Latitude.ToString());
+            if (Latitude.HasValue)
+            {
+                writer.WriteStringValue(Latitude.Value.ToString(CultureInfo.InvariantCulture));
+            }
+            else
+            {
+                writer.WriteNullValue();
+            }
         }
 
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -48,7 +61,7 @@ internal static void ReadLatitude(JsonProperty property, ref double? latitude)
             if (property.Value.ValueKind == JsonValueKind.Null)
                 return;
 
-            latitude = double.Parse(property.Value.GetString());
+            latitude = double.Parse(property.Value.GetString(), CultureInfo.InvariantCulture);
         }
     }
 }

sdk/resourcemanager/Azure.ResourceManager/tests/Unit/LocationMetadataTests.cs

@@ -0,0 +1,40 @@
+using System;
+using System.ClientModel.Primitives;
+using System.Globalization;
+using System.Threading;
+using Azure.ResourceManager.Models;
+using Azure.ResourceManager.Resources.Models;
+using NUnit.Framework;
+
+namespace Azure.ResourceManager.Tests
+{
+    public class LocationMetadataTests
+    {
+        private const double _longitude = -122.1215;
+        private const double _latitude = 47.6740;
+        private static readonly string _payload = $@"{{""longitude"":""{_longitude}"",""latitude"":""{_latitude}""}}";
+
+        [TestCase("en-US")] // English (United States) - uses a dot as the decimal separator
+        [TestCase("fr-FR")] // French (France) - uses a comma as the decimal separator
+        [TestCase("de-DE")] // German (Germany) - uses a comma as the decimal separator
+        [TestCase("es-ES")] // Spanish (Spain) - uses a comma as the decimal separator
+        [TestCase("it-IT")] // Italian (Italy) - uses a comma as the decimal separator
+        [TestCase("ja-JP")] // Japanese (Japan) - uses a dot as the decimal separator
+        [TestCase("zh-CN")] // Chinese (China) - uses a dot as the decimal separator
+        [TestCase("ru-RU")] // Russian (Russia) - uses a comma as the decimal separator
+        [TestCase("pt-BR")] // Portuguese (Brazil) - uses a comma as the decimal separator
+        [TestCase("ar-SA")] // Arabic (Saudi Arabia) - uses a dot as the decimal separator
+        public void ValidateLocationMetadataInCulture(string culture)
+        {
+            Thread.CurrentThread.CurrentCulture = new CultureInfo(culture);
+
+            var location = ModelReaderWriter.Read<LocationMetadata>(new BinaryData(_payload));
+
+            Assert.AreEqual(_longitude, location.Longitude);
+            Assert.AreEqual(_latitude, location.Latitude);
+
+            var json = ModelReaderWriter.Write(location);
+            Assert.AreEqual(_payload, json.ToString());
+        }
+    }
+}