feat(InAppUpdateManager): Add kdoc for the inAppUpdate manager

Infomaniak · Nov 15, 2024 · 89e83f0 · 89e83f0
1 parent c7e849e
commit 89e83f0
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/Stores/src/main/java/com/infomaniak/lib/stores/updaterequired/UpdateRequiredActivity.kt b/Stores/src/main/java/com/infomaniak/lib/stores/updaterequired/UpdateRequiredActivity.kt
@@ -33,7 +33,6 @@ import com.infomaniak.lib.core.utils.goToPlayStore
 import com.infomaniak.lib.core.utils.showToast
 import com.infomaniak.lib.stores.databinding.ActivityUpdateRequiredBinding
 import com.infomaniak.lib.stores.updatemanagers.InAppUpdateManager
-import io.sentry.Sentry
 import kotlin.system.exitProcess
 import com.infomaniak.lib.core.R as RCore
 

diff --git a/Stores/src/standard/java/com/infomaniak/lib/stores/updatemanagers/InAppUpdateManager.kt b/Stores/src/standard/java/com/infomaniak/lib/stores/updatemanagers/InAppUpdateManager.kt
@@ -38,6 +38,14 @@ import com.infomaniak.lib.stores.StoreUtils
 import com.infomaniak.lib.stores.StoresSettingsRepository
 import io.sentry.Sentry
 
+/**
+ * Manager encapsulating all the needed logic for Google Play's in-app update api
+ * Implements [BaseInAppUpdateManager] to add compatibility with fDroid
+ *
+ * @param activity: Activity on which the lifecycleObserver will be bound
+ * @param appId: Parameter needed for the fDroid manager, won't compile without it
+ * @param versionCode: Parameter needed for the fDroid manager, won't compile without it
+ */
 class InAppUpdateManager(
     private val activity: FragmentActivity,
     appId: String,
@@ -149,7 +157,7 @@ class InAppUpdateManager(
             .addOnSuccessListener { onInstallSuccess?.invoke() }
             .addOnFailureListener {
                 viewModel.resetUpdateSettings()
-                onInstallFailure?.invoke(it)
+                onInstallFailure?.invoke(AppUpdateException(it.message))
             }
     }
 
@@ -196,4 +204,6 @@ class InAppUpdateManager(
             AppUpdateOptions.newBuilder(updateType).build(),
         )
     }
+
+    private class AppUpdateException(override val message: String?) : Exception()
 }