improvement in permission handling

Author: Abizer-R

quickedit/src/main/java/com/abizer_r/quickedit/ui/mainScreen/MainScreen.kt

@@ -59,7 +59,7 @@ fun MainScreen(
                 isGranted = perms[permission] == true
             )
         }
-        if (perms.values.all { true }) {
+        if (perms.values.all { granted -> granted }) {
             viewModel.permissionsGranted.value = true
         }
     }

quickedit/src/main/java/com/abizer_r/quickedit/utils/FileUtils.kt

@@ -9,6 +9,7 @@ import android.os.Environment
 import android.provider.MediaStore
 import androidx.annotation.RequiresApi
 import androidx.core.content.FileProvider
+import com.abizer_r.quickedit.R
 import java.io.File
 import java.io.FileInputStream
 
@@ -65,7 +66,8 @@ object FileUtils {
             copyFileToUri(context.contentResolver, file, uri)
             onSuccess()
         } else {
-            onFailure(null)
+            onFailure(context.getString(R.string.fail_to_insert_uri_in_content_resolver))
+            return
         }
     }
 
@@ -89,6 +91,7 @@ object FileUtils {
             copyFile(file, externalStorageDir)
         } catch (e: Exception) {
             onFailure(e.message)
+            return
         }
 
         // Make the file visible in the gallery
@@ -103,7 +106,8 @@ object FileUtils {
             copyFileToUri(context.contentResolver, file, uri)
             onSuccess()
         } else {
-            onFailure(null)
+            onFailure(context.getString(R.string.fail_to_insert_uri_in_content_resolver))
+            return
         }
     }
 

quickedit/src/main/res/values/strings.xml

@@ -34,4 +34,5 @@
     <string name="following_permissions_are_required">Following permissions are required to perform this action:\n</string>
     <string name="perm_item_storage">- Storage\n</string>
     <string name="app_not_found">App not found</string>
+    <string name="fail_to_insert_uri_in_content_resolver">Couldn\'t insert uri in content resolver</string>
 </resources>