From 122cd1160b0f0dbd8dd6c0797855cba1c67ea36f Mon Sep 17 00:00:00 2001
From: tombogle <tom_bogle@sil.org>
Date: Mon, 15 Dec 2025 17:53:02 -0500
Subject: [PATCH 1/2] Added DeviceUILanguage application property in support of
 "Top 5 Device Lang" metric

---
 DesktopAnalytics.sln.DotSettings  |  1 +
 src/DesktopAnalytics/Analytics.cs | 16 ++++++++++++----
 src/SampleApp/Program.cs          |  5 +++--
 3 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/DesktopAnalytics.sln.DotSettings b/DesktopAnalytics.sln.DotSettings
index 484cf45..ad4e755 100644
--- a/DesktopAnalytics.sln.DotSettings
+++ b/DesktopAnalytics.sln.DotSettings
@@ -1,6 +1,7 @@
 <wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation">
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=API/@EntryIndexedValue">API</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=IO/@EntryIndexedValue">IO</s:String>
+	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=UI/@EntryIndexedValue">UI</s:String>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Analytics/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Mixpanel/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=netapi/@EntryIndexedValue">True</s:Boolean>
diff --git a/src/DesktopAnalytics/Analytics.cs b/src/DesktopAnalytics/Analytics.cs
index adf58ad..b06a4be 100644
--- a/src/DesktopAnalytics/Analytics.cs
+++ b/src/DesktopAnalytics/Analytics.cs
@@ -220,8 +220,8 @@ public Analytics(string apiSecret, UserInfo userInfo, Dictionary<string,
 			ReportIpAddressOfThisMachineAsync(); //this will take a while and may fail, so just do it when/if we can
 			
 			var assembly = assemblyToUseForVersion ?? Assembly.GetEntryAssembly();
-			var versionNumberWithBuild = assembly.GetName().Version?.ToString() ?? "";
-			string versionNumber = versionNumberWithBuild.Split('.').Take(2).Aggregate((a, b) => a + "." + b);
+			var versionNumberWithBuild = assembly?.GetName().Version?.ToString() ?? "";
+			var versionNumber = versionNumberWithBuild.Split('.').Take(2).Aggregate((a, b) => a + "." + b);
 			SetApplicationProperty("Version", versionNumber);
 			SetApplicationProperty("FullVersion", versionNumberWithBuild);
 			SetApplicationProperty("UserName", GetUserNameForEvent());
@@ -229,8 +229,16 @@ public Analytics(string apiSecret, UserInfo userInfo, Dictionary<string,
 			SetApplicationProperty("OS Version Number", GetOperatingSystemVersionLabel());
 			SetApplicationProperty("64bit OS", Environment.Is64BitOperatingSystem.ToString());
 			SetApplicationProperty("64bit App", Environment.Is64BitProcess.ToString());
-
-
+			// This (and "64bit OS" above) really belong in Context, but segment.io doesn't seem
+			// to convey context to Mixpanel in a reliable/predictable form.
+			var ci = CultureInfo.InstalledUICulture;
+			const string invariantCulture = "iv";
+			var installedUICulture = !IsNullOrEmpty(ci.TwoLetterISOLanguageName) &&
+				ci.TwoLetterISOLanguageName != invariantCulture
+				? ci.TwoLetterISOLanguageName
+				: ci.ThreeLetterISOLanguageName;
+			SetApplicationProperty("DeviceUILanguage", installedUICulture);
+			
 			if (IsNullOrEmpty(AnalyticsSettings.Default.LastVersionLaunched))
 			{
 				//"Created" is a special property that segment.io understands and coverts to equivalents in various analytics services
diff --git a/src/SampleApp/Program.cs b/src/SampleApp/Program.cs
index 830f837..33c6e87 100644
--- a/src/SampleApp/Program.cs
+++ b/src/SampleApp/Program.cs
@@ -18,7 +18,7 @@ static int Main(string[] args)
 
 			if (!Enum.TryParse<ClientType>(args[1], true, out var clientType))
 			{
-				Console.WriteLine($"Usage: SampleApp <analyticsApiSecret> <Segment|Mixpanel|???>{Environment.NewLine}Unrecoginzed client type: {args[1]}");
+				Console.WriteLine($"Usage: SampleApp <analyticsApiSecret> <Segment|Mixpanel|???>{Environment.NewLine}Unrecognized client type: {args[1]}");
 				return 1;
 			}
 
@@ -29,7 +29,8 @@ static int Main(string[] args)
 				Email="john@example.com",
 				UILanguageCode= "fr"
 			};
-			userInfo.OtherProperties.Add("HowIUseIt","This is a really long explanation of how I use this product to see how much you would be able to extract from Mixpanel.\r\nAnd a second line of it.");
+			userInfo.OtherProperties.Add("HowIUseIt",
+				"This is a really long explanation of how I use this product to see how much you would be able to extract from Mixpanel.\r\nAnd a second line of it.");
 
 			var propsForEveryEvent = new Dictionary<string, string> {{"channel", "beta"}};
 			using (new Analytics(args[0], userInfo, propertiesThatGoWithEveryEvent: propsForEveryEvent, clientType: clientType))

From b42c993123a32076d781b2268efb8d0ee5b49ca2 Mon Sep 17 00:00:00 2001
From: tombogle <tom_bogle@sil.org>
Date: Tue, 6 Jan 2026 16:21:14 -0500
Subject: [PATCH 2/2] Decided that CurrentUICulture is probably a better fit
 for our needs than InstalledUICulture

---
 src/DesktopAnalytics/Analytics.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/DesktopAnalytics/Analytics.cs b/src/DesktopAnalytics/Analytics.cs
index b06a4be..2c5e5c4 100644
--- a/src/DesktopAnalytics/Analytics.cs
+++ b/src/DesktopAnalytics/Analytics.cs
@@ -231,7 +231,7 @@ public Analytics(string apiSecret, UserInfo userInfo, Dictionary<string,
 			SetApplicationProperty("64bit App", Environment.Is64BitProcess.ToString());
 			// This (and "64bit OS" above) really belong in Context, but segment.io doesn't seem
 			// to convey context to Mixpanel in a reliable/predictable form.
-			var ci = CultureInfo.InstalledUICulture;
+			var ci = CultureInfo.CurrentUICulture;
 			const string invariantCulture = "iv";
 			var installedUICulture = !IsNullOrEmpty(ci.TwoLetterISOLanguageName) &&
 				ci.TwoLetterISOLanguageName != invariantCulture