aboutsummaryrefslogtreecommitdiff
path: root/core/src
diff options
context:
space:
mode:
authorAllan Wang <me@allanwang.ca>2019-05-22 18:02:32 -0700
committerAllan Wang <me@allanwang.ca>2019-05-22 18:02:32 -0700
commit8591040162b6147344aa82a2e4a308319e62c2d4 (patch)
tree94e5bd7a31e4615b724195d1c81b29a6407e0f83 /core/src
parent20516059b0bdd98e88fade05cc7d1aadf3bee340 (diff)
downloadkau-8591040162b6147344aa82a2e4a308319e62c2d4.tar.gz
kau-8591040162b6147344aa82a2e4a308319e62c2d4.tar.bz2
kau-8591040162b6147344aa82a2e4a308319e62c2d4.zip
Update permission manager docs
Diffstat (limited to 'core/src')
-rw-r--r--core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionManager.kt22
-rw-r--r--core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionResult.kt4
2 files changed, 19 insertions, 7 deletions
diff --git a/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionManager.kt b/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionManager.kt
index 4d97070..96adfa3 100644
--- a/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionManager.kt
+++ b/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionManager.kt
@@ -31,8 +31,8 @@ import java.lang.ref.WeakReference
/**
* Created by Allan Wang on 2017-07-03.
*
- * Permission manager that is decoupled from activities
- * Keeps track of pending requests, and warns about invalid requests
+ * Permission manager that is decoupled from activities.
+ * Keeps track of pending requests, and warns about invalid requests.
*/
internal object PermissionManager {
@@ -43,13 +43,20 @@ internal object PermissionManager {
*/
private val manifestPermission = lazyContext<Set<String>> {
try {
- it.packageManager.getPackageInfo(it.packageName, PackageManager.GET_PERMISSIONS)?.requestedPermissions?.toSet()
+ it.packageManager.getPackageInfo(
+ it.packageName,
+ PackageManager.GET_PERMISSIONS
+ )?.requestedPermissions?.toSet()
?: emptySet()
} catch (e: Exception) {
emptySet()
}
}
+ /**
+ * Registers a new permission request.
+ * It is expected that the callback will be called eventually, unless the parent activity is destroyed.
+ */
operator fun invoke(
context: Context,
permissions: Array<out String>,
@@ -67,6 +74,9 @@ internal object PermissionManager {
}
}
+ /**
+ * Checks that the listed permissions can be requested, and submits the request to the provided activity
+ */
private fun requestPermissions(context: Context, permissions: Array<out String>) {
permissions.forEach {
if (!manifestPermission(context).contains(it)) {
@@ -82,8 +92,8 @@ internal object PermissionManager {
}
/**
- * Handles permission result by allowing accepted permissions for all pending requests
- * Also cleans up destroyed or completed pending requests
+ * Handles permission result by allowing accepted permissions for all pending requests.
+ * Also cleans up destroyed or completed pending requests.
*/
fun onRequestPermissionsResult(context: Context, permissions: Array<out String>, grantResults: IntArray) {
KL.i { "On permission result: pending ${pendingResults.size}" }
@@ -93,7 +103,7 @@ internal object PermissionManager {
action == null || (0 until count).any { i -> action.onResult(permissions[i], grantResults[i]) }
}
val action = pendingResults.asSequence().map { it.get() }.firstOrNull { it != null }
- if (action == null) { //actions have been unlinked from their weak references
+ if (action == null) { // actions have been unlinked from their weak references
pendingResults.clear()
return
}
diff --git a/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionResult.kt b/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionResult.kt
index 8888c91..599f6e8 100644
--- a/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionResult.kt
+++ b/core/src/main/kotlin/ca/allanwang/kau/permissions/PermissionResult.kt
@@ -36,7 +36,9 @@ class PermissionResult(permissions: Array<out String>, val callback: (granted: B
return true
}
permissions.remove(permission)
- if (permissions.isNotEmpty()) return false
+ if (permissions.isNotEmpty()) {
+ return false
+ }
callback(true, null)
return true
}