Feature/permission fixes (#39)

* - Provided "android.permission.MANAGE_EXTERNAL_STORAGE" permission access to the user and removed from the library

* Update README.md

Co-authored-by: AKASH PATEL <akash.patel@mindinventory.com>

Author: chiragmi

README.md

@@ -51,7 +51,7 @@ Lassi is simplest way to pick media (either image, video, audio or doc)
     ```groovy
         dependencies {
             ...
-            implementation 'com.github.Mindinventory:Lassi:0.2.1'
+            implementation 'com.github.Mindinventory:Lassi:0.2.2'
         }
     ``` 
 
@@ -108,6 +108,8 @@ Lassi is simplest way to pick media (either image, video, audio or doc)
                 }
     ```
 
+### Document access permission note
+If Android device SDK is >= 30 and wants to access document (only for choose the non media file) then add ```android.permission.MANAGE_EXTERNAL_STORAGE``` permission in your app otherwise library won't allow to access documents. Kindly check sample app for more detail.
 
 ### Guideline for contributors
 Contribution towards our repository is always welcome, we request contributors to create a pull request to the **develop** branch only.  

app/src/main/AndroidManifest.xml

@@ -2,14 +2,17 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="com.lassi.app">
 
+    <!--        This permission require only for document picker feature and if Android device SDK is >= 30-->
+    <uses-permission android:name="android.permission.MANAGE_EXTERNAL_STORAGE" />
+
     <application
         android:allowBackup="true"
         android:icon="@mipmap/ic_launcher"
         android:label="@string/app_name"
+        android:preserveLegacyExternalStorage="true"
         android:requestLegacyExternalStorage="true"
         android:roundIcon="@mipmap/ic_launcher_round"
         android:supportsRtl="true"
-        android:preserveLegacyExternalStorage="true"
         android:theme="@style/AppTheme">
 
         <activity

app/src/main/java/com/lassi/app/MainActivity.kt

@@ -4,8 +4,10 @@ import android.app.Activity
 import android.content.ContentResolver
 import android.content.Intent
 import android.net.Uri
+import android.os.Build
 import android.os.Bundle
-import android.util.Log
+import android.os.Environment
+import android.provider.Settings
 import android.view.View
 import android.webkit.MimeTypeMap
 import androidx.activity.result.contract.ActivityResultContracts
@@ -104,36 +106,40 @@ class MainActivity : AppCompatActivity(), View.OnClickListener {
                 receiveData.launch(intent)
             }
             R.id.btnDocPicker -> {
-                val intent = Lassi(this)
-                    .setMediaType(MediaType.DOC)
-                    .setMaxCount(4)
-                    .setGridSize(2)
-                    .setPlaceHolder(R.drawable.ic_document_placeholder)
-                    .setErrorDrawable(R.drawable.ic_document_placeholder)
-                    .setSelectionDrawable(R.drawable.ic_checked_media)
-                    .setStatusBarColor(R.color.colorPrimaryDark)
-                    .setToolbarColor(R.color.colorPrimary)
-                    .setToolbarResourceColor(android.R.color.white)
-                    .setSupportedFileTypes(
-                        "pdf",
-                        "odt",
-                        "doc",
-                        "docs",
-                        "docx",
-                        "txt",
-                        "ppt",
-                        "pptx",
-                        "rtf",
-                        "xlsx",
-                        "xls"
-                    )
-                    .setProgressBarColor(R.color.colorAccent)
-                    .build()
-                receiveData.launch(intent)
+                requestPermissionForDocument()
             }
         }
     }
 
+    private fun launchDocPicker() {
+        val intent = Lassi(this)
+            .setMediaType(MediaType.DOC)
+            .setMaxCount(4)
+            .setGridSize(2)
+            .setPlaceHolder(R.drawable.ic_document_placeholder)
+            .setErrorDrawable(R.drawable.ic_document_placeholder)
+            .setSelectionDrawable(R.drawable.ic_checked_media)
+            .setStatusBarColor(R.color.colorPrimaryDark)
+            .setToolbarColor(R.color.colorPrimary)
+            .setToolbarResourceColor(android.R.color.white)
+            .setSupportedFileTypes(
+                "pdf",
+                "odt",
+                "doc",
+                "docs",
+                "docx",
+                "txt",
+                "ppt",
+                "pptx",
+                "rtf",
+                "xlsx",
+                "xls"
+            )
+            .setProgressBarColor(R.color.colorAccent)
+            .build()
+        receiveData.launch(intent)
+    }
+
     private val receiveData =
         registerForActivityResult(ActivityResultContracts.StartActivityForResult()) {
             if (it.resultCode == Activity.RESULT_OK) {
@@ -175,4 +181,40 @@ class MainActivity : AppCompatActivity(), View.OnClickListener {
             startActivity(Intent.createChooser(intent, "Open file"))
         }
     }
+
+    /**
+     *   If Android device SDK is >= 30 and wants to access document (only for choose the non media file)
+     *   then ask for "android.permission.MANAGE_EXTERNAL_STORAGE" permission
+     */
+    private fun requestPermissionForDocument() {
+        when {
+            Build.VERSION.SDK_INT >= Build.VERSION_CODES.R -> {
+                if (Environment.isExternalStorageManager()) {
+                    launchDocPicker()
+                } else {
+                    try {
+                        val intent = Intent(Settings.ACTION_MANAGE_APP_ALL_FILES_ACCESS_PERMISSION)
+                        intent.addCategory("android.intent.category.DEFAULT")
+                        intent.data = Uri.parse(
+                            String.format("package:%s", applicationContext?.packageName)
+                        )
+                        mPermissionSettingResult.launch(intent)
+                    } catch (e: Exception) {
+                        val intent = Intent()
+                        intent.action = Settings.ACTION_MANAGE_ALL_FILES_ACCESS_PERMISSION
+                        mPermissionSettingResult.launch(intent)
+                    }
+                }
+            }
+
+            else -> {
+                launchDocPicker()
+            }
+        }
+    }
+
+    private val mPermissionSettingResult =
+        registerForActivityResult(ActivityResultContracts.StartActivityForResult()) {
+            requestPermissionForDocument()
+        }
 }

lassi/build.gradle

@@ -12,8 +12,8 @@ android {
     defaultConfig {
         minSdkVersion 17
         targetSdkVersion 30
-        versionCode 19
-        versionName "0.2.1"
+        versionCode 20
+        versionName "0.2.2"
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         vectorDrawables.useSupportLibrary = true
         multiDexEnabled true

lassi/src/main/AndroidManifest.xml

@@ -4,7 +4,6 @@
 
     <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
     <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
-    <uses-permission android:name="android.permission.MANAGE_EXTERNAL_STORAGE" />
     <uses-permission android:name="android.permission.CAMERA" />
     <uses-permission android:name="android.permission.RECORD_AUDIO" />
 

lassi/src/main/java/com/lassi/presentation/docs/DocsFragment.kt

@@ -88,19 +88,6 @@ class DocsFragment : LassiBaseViewModelFragment<DocsViewModel>() {
             Build.VERSION.SDK_INT >= Build.VERSION_CODES.R -> {
                 if (Environment.isExternalStorageManager()) {
                     fetchDocs()
-                } else {
-                    try {
-                        val intent = Intent(Settings.ACTION_MANAGE_APP_ALL_FILES_ACCESS_PERMISSION)
-                        intent.addCategory("android.intent.category.DEFAULT")
-                        intent.data = Uri.parse(
-                            String.format("package:%s", context?.applicationContext?.packageName)
-                        )
-                        mPermissionSettingResult.launch(intent)
-                    } catch (e: Exception) {
-                        val intent = Intent()
-                        intent.action = Settings.ACTION_MANAGE_ALL_FILES_ACCESS_PERMISSION
-                        mPermissionSettingResult.launch(intent)
-                    }
                 }
             }
             Build.VERSION.SDK_INT == Build.VERSION_CODES.Q -> {