From 67022c8d51fe3d07c1f571001f30ac4ffd7e4e92 Mon Sep 17 00:00:00 2001 From: Fadi George Date: Tue, 10 Mar 2026 13:39:47 -0700 Subject: [PATCH] fix: use versionName instead of versionCode for app version reporting Aligns Android with iOS, which reports CFBundleShortVersionString (display version). Previously Android reported versionCode (integer build number), causing inconsistency for cross-platform frameworks like .NET MAUI. Falls back to versionCode if versionName is null. Made-with: Cursor --- .../main/java/com/onesignal/common/AndroidUtils.kt | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/common/AndroidUtils.kt b/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/common/AndroidUtils.kt index 1e9114adb..7750d9358 100644 --- a/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/common/AndroidUtils.kt +++ b/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/common/AndroidUtils.kt @@ -80,14 +80,12 @@ object AndroidUtils { } fun getAppVersion(context: Context): String? { - val appVersion: Int? = - try { - context.packageManager.getPackageInfo(context.packageName, 0).versionCode - } catch (e: PackageManager.NameNotFoundException) { - null - } - - return appVersion?.toString() + return try { + val packageInfo = context.packageManager.getPackageInfo(context.packageName, 0) + packageInfo.versionName ?: packageInfo.versionCode.toString() + } catch (e: PackageManager.NameNotFoundException) { + null + } } fun getManifestMeta(