fix: update IAP analytics keys for price

- Added lms_usd_price, localized_price, localized_currency_code in IAP analytics. fix: LEARNER-10094
edx · Jul 22, 2024 · 24086ac · 24086ac
1 parent b206ef2
commit 24086ac
Show file tree

Hide file tree

Showing 7 changed files with 34 additions and 6 deletions.
diff --git a/core/src/main/java/org/openedx/core/data/model/CourseStructureModel.kt b/core/src/main/java/org/openedx/core/data/model/CourseStructureModel.kt
@@ -77,7 +77,11 @@ data class CourseStructureModel(
             }?.takeIf {
                 it.androidSku.isNullOrEmpty().not() && it.storeSku.isNullOrEmpty().not()
             }?.run {
-                ProductInfo(courseSku = androidSku!!, storeSku = storeSku!!)
+                ProductInfo(
+                    courseSku = androidSku!!,
+                    storeSku = storeSku!!,
+                    lmsUSDPrice = price ?: 0.0
+                )
             }
         )
     }

diff --git a/core/src/main/java/org/openedx/core/data/model/EnrolledCourse.kt b/core/src/main/java/org/openedx/core/data/model/EnrolledCourse.kt
@@ -47,7 +47,11 @@ data class EnrolledCourse(
             }?.takeIf {
                 it.androidSku.isNullOrEmpty().not() && it.storeSku.isNullOrEmpty().not()
             }?.run {
-                ProductInfo(courseSku = androidSku!!, storeSku = storeSku!!)
+                ProductInfo(
+                    courseSku = androidSku!!,
+                    storeSku = storeSku!!,
+                    lmsUSDPrice = price ?: 0.0
+                )
             }
         )
     }

diff --git a/core/src/main/java/org/openedx/core/domain/model/iap/ProductInfo.kt b/core/src/main/java/org/openedx/core/domain/model/iap/ProductInfo.kt
@@ -7,4 +7,5 @@ import kotlinx.parcelize.Parcelize
 data class ProductInfo(
     val courseSku: String,
     val storeSku: String,
+    val lmsUSDPrice: Double,
 ) : Parcelable
diff --git a/core/src/main/java/org/openedx/core/extension/DoubleExt.kt b/core/src/main/java/org/openedx/core/extension/DoubleExt.kt
@@ -0,0 +1,5 @@
+package org.openedx.core.extension
+
+fun Double.nonZero(): Double? {
+    return if (this != 0.0) this else null
+}
diff --git a/core/src/main/java/org/openedx/core/presentation/IAPAnalytics.kt b/core/src/main/java/org/openedx/core/presentation/IAPAnalytics.kt
@@ -53,7 +53,9 @@ enum class IAPAnalyticsKeys(val key: String) {
     SELF("self"),
     INSTRUCTOR("instructor"),
     IAP_FLOW_TYPE("flow_type"),
-    PRICE("price"),
+    LMS_USD_PRICE("lms_usd_price"),
+    LOCALIZED_PRICE("localized_price"),
+    CURRENCY_CODE("localized_currency_code"),
     COMPONENT_ID("component_id"),
     ELAPSED_TIME("elapsed_time"),
     ERROR("error"),

diff --git a/core/src/main/java/org/openedx/core/presentation/iap/IAPViewModel.kt b/core/src/main/java/org/openedx/core/presentation/iap/IAPViewModel.kt
@@ -24,6 +24,8 @@ import org.openedx.core.data.storage.CorePreferences
 import org.openedx.core.domain.interactor.IAPInteractor
 import org.openedx.core.domain.model.iap.PurchaseFlowData
 import org.openedx.core.exception.iap.IAPException
+import org.openedx.core.extension.nonZero
+import org.openedx.core.extension.takeIfNotEmpty
 import org.openedx.core.module.billing.BillingProcessor
 import org.openedx.core.module.billing.getCourseSku
 import org.openedx.core.module.billing.getPriceAmount
@@ -348,8 +350,14 @@ class IAPViewModel(
                     if (purchaseFlowData.isSelfPaced == true) IAPAnalyticsKeys.SELF.key else IAPAnalyticsKeys.INSTRUCTOR.key
                 )
             }
-            purchaseFlowData.formattedPrice?.takeIf { it.isNotBlank() }?.let { formattedPrice ->
-                put(IAPAnalyticsKeys.PRICE.key, formattedPrice)
+            purchaseFlowData.productInfo?.lmsUSDPrice?.nonZero()?.let { lmsUSDPrice ->
+                put(IAPAnalyticsKeys.LMS_USD_PRICE.key, lmsUSDPrice)
+            }
+            purchaseFlowData.price.nonZero()?.let { localizedPrice ->
+                put(IAPAnalyticsKeys.LOCALIZED_PRICE.key, localizedPrice)
+            }
+            purchaseFlowData.currencyCode.takeIfNotEmpty()?.let { currencyCode ->
+                put(IAPAnalyticsKeys.CURRENCY_CODE.key, currencyCode)
             }
             purchaseFlowData.componentId?.takeIf { it.isNotBlank() }?.let { componentId ->
                 put(IAPAnalyticsKeys.COMPONENT_ID.key, componentId)

diff --git a/dashboard/src/main/java/org/openedx/dashboard/presentation/DashboardListFragment.kt b/dashboard/src/main/java/org/openedx/dashboard/presentation/DashboardListFragment.kt
@@ -736,5 +736,9 @@ private val mockCourseEnrolled = EnrolledCourse(
         videoOutline = "",
         isSelfPaced = false
     ),
-    productInfo = ProductInfo(courseSku = "example_sku", storeSku = "mobile.android.example_100")
+    productInfo = ProductInfo(
+        courseSku = "example_sku",
+        storeSku = "mobile.android.example_100",
+        lmsUSDPrice = 99.9
+    )
 )