From 273a45ff1f5a7b19bebd34296ba3d0a15c5b3ff1 Mon Sep 17 00:00:00 2001 From: PesBandi <127593627+PesBandi@users.noreply.github.com> Date: Fri, 21 Feb 2025 09:26:52 +0100 Subject: [PATCH] [PTRun][Calc]Add list separator handling for different cultures (#36735) --- .../NumberTranslatorTests.cs | 1 + .../NumberTranslator.cs | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator.UnitTest/NumberTranslatorTests.cs b/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator.UnitTest/NumberTranslatorTests.cs index 4a597cf917..c3f5de7f02 100644 --- a/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator.UnitTest/NumberTranslatorTests.cs +++ b/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator.UnitTest/NumberTranslatorTests.cs @@ -74,6 +74,7 @@ namespace Microsoft.PowerToys.Run.Plugin.Calculator.UnitTests [DataRow("5,2+6", "5.2+6")] [DataRow("round(2,5)", "round(2.5)")] [DataRow("3,3333", "3.3333")] + [DataRow("max(2;3)", "max(2,3)")] public void Translate_NoErrors_WhenCalled(string input, string expectedResult) { // Arrange diff --git a/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator/NumberTranslator.cs b/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator/NumberTranslator.cs index 5de28ba775..946198e122 100644 --- a/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator/NumberTranslator.cs +++ b/src/modules/launcher/Plugins/Microsoft.PowerToys.Run.Plugin.Calculator/NumberTranslator.cs @@ -123,7 +123,7 @@ namespace Microsoft.PowerToys.Run.Plugin.Calculator outputBuilder.Append( decimal.TryParse(token, NumberStyles.Number, cultureFrom, out number) ? (new string('0', leadingZeroCount) + number.ToString(cultureTo)) - : token); + : token.Replace(cultureFrom.TextInfo.ListSeparator, cultureTo.TextInfo.ListSeparator)); } }