Merge pull request #98 from solrudev/develop

0.9.2
solrudev · Dec 19, 2024 · 4bca22f · 4bca22f
2 parents 0265f0f + b01b4a7
commit 4bca22f
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -32,7 +32,7 @@ Ackpine depends on Jetpack libraries, so it's necessary to declare the `google()
 
 ```kotlin
 dependencies {
-    val ackpineVersion = "0.9.1"
+    val ackpineVersion = "0.9.2"
     implementation("ru.solrudev.ackpine:ackpine-core:$ackpineVersion")
 
     // optional - Kotlin extensions and Coroutines support

diff --git a/...in/ru/solrudev/ackpine/impl/installer/activity/SessionBasedInstallConfirmationActivity.kt b/...in/ru/solrudev/ackpine/impl/installer/activity/SessionBasedInstallConfirmationActivity.kt
@@ -135,8 +135,6 @@ internal class SessionBasedInstallConfirmationActivity : InstallActivity(TAG, st
 		when {
 			// Confirmation is a preapproval on API >= 34.
 			isPreapproval -> finish()
-			// User has cancelled install permission request or hasn't granted permission.
-			!canInstallPackages -> abortSession("Install permission denied")
 			// User hasn't confirmed installation because confirmation activity didn't appear after permission request.
 			isSessionStuck && isInstallPermissionStatusChanged && wasOnTopOnStart -> launchInstallActivity()
 			// Session proceeded normally.
@@ -145,6 +143,8 @@ internal class SessionBasedInstallConfirmationActivity : InstallActivity(TAG, st
 			// gets updated almost like the installation was confirmed even though it wasn't and no result is received
 			// from PackageInstallerStatusReceiver.
 			isSessionAlive && !isSessionStuck -> finish()
+			// User has cancelled install permission request or hasn't granted permission.
+			!canInstallPackages -> abortSession("Install permission denied")
 			// User has dismissed confirmation activity.
 			isSessionAlive && isActivityCancelled -> abortSession()
 			// There was some error while installing which is not handled in PackageInstallerStatusReceiver,

diff --git a/docs/changelog.md b/docs/changelog.md
@@ -1,6 +1,13 @@
 Change Log
 ==========
 
+Version 0.9.2 (2024-12-19)
+--------------------------
+
+### Bug fixes and improvements
+
+- Fix `SESSION_BASED` installer session completing with "Install permission denied" failure when performing a self-update if install permission was not granted because it was unnecessary.
+
 Version 0.9.1 (2024-12-14)
 --------------------------
 

diff --git a/docs/index.md b/docs/index.md
@@ -27,7 +27,7 @@ Ackpine depends on Jetpack libraries, so it's necessary to declare the `google()
 
 ```kotlin
 dependencies {
-    val ackpineVersion = "0.9.1"
+    val ackpineVersion = "0.9.2"
     implementation("ru.solrudev.ackpine:ackpine-core:$ackpineVersion")
 
     // optional - Kotlin extensions and Coroutines support

diff --git a/version.properties b/version.properties
@@ -1,5 +1,5 @@
 MAJOR_VERSION=0
 MINOR_VERSION=9
-PATCH_VERSION=1
+PATCH_VERSION=2
 SUFFIX=
 SNAPSHOT=false