Fixes #26 by only asking user for camera permissions only

IzHoBX · Jun 28, 2020 · 40a187c · 40a187c
1 parent 8e04fcf
commit 40a187c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/app/src/main/java/com/izho/saveentry/MainActivity.kt b/app/src/main/java/com/izho/saveentry/MainActivity.kt
@@ -58,10 +58,7 @@ class MainActivity : AppCompatActivity() {
 
     override fun onResume() {
         super.onResume()
-        if (!Utils.allPermissionsGranted(this)) {
-            Utils.requestRuntimePermissions(this)
-        }
-
+        Utils.requestCameraPermissionIfNotGranted(this)
         setSelectedTab()
     }
 

diff --git a/app/src/main/java/com/izho/saveentry/utils/Utils.kt b/app/src/main/java/com/izho/saveentry/utils/Utils.kt
@@ -37,7 +37,10 @@ import android.net.Uri
 import android.os.Build
 import android.util.Log
 import androidx.core.app.ActivityCompat
+import androidx.core.app.ActivityCompat.requestPermissions
+import androidx.core.content.ContextCompat
 import androidx.exifinterface.media.ExifInterface
+import com.izho.saveentry.CAMERA_PERMISSSION_REQUEST_CODE
 import com.izho.saveentry.camera.CameraSizePair
 import java.io.IOException
 import java.io.InputStream
@@ -71,6 +74,12 @@ object Utils {
         }
     }
 
+    fun requestCameraPermissionIfNotGranted(activity: Activity) {
+        if(ContextCompat.checkSelfPermission(activity, android.Manifest.permission.CAMERA) != PackageManager.PERMISSION_GRANTED) {
+            activity.requestPermissions(Array<String>(1, {index -> android.Manifest.permission.CAMERA}), CAMERA_PERMISSSION_REQUEST_CODE)
+        }
+    }
+
     internal fun allPermissionsGranted(context: Context): Boolean = getRequiredPermissions(context)
             .all { checkSelfPermission(context, it) == PackageManager.PERMISSION_GRANTED }