Permission and error handling

library/attachment-viewer/src/main/java/im/vector/lib/attachmentviewer/AttachmentViewerActivity.kt

@@ -45,6 +45,8 @@ import kotlin.math.abs
 
 abstract class AttachmentViewerActivity : AppCompatActivity(), AttachmentEventListener {
 
+    protected val rootView: View
+        get() = views.rootContainer
     protected val pager2: ViewPager2
         get() = views.attachmentPager
     protected val imageTransitionView: ImageView
@@ -298,10 +300,11 @@ abstract class AttachmentViewerActivity : AppCompatActivity(), AttachmentEventLi
 
     private fun createSwipeToDismissHandler(): SwipeToDismissHandler =
             SwipeToDismissHandler(
-            swipeView = views.dismissContainer,
-            shouldAnimateDismiss = { shouldAnimateDismiss() },
-            onDismiss = { animateClose() },
-            onSwipeViewMove = ::handleSwipeViewMove)
+                    swipeView = views.dismissContainer,
+                    shouldAnimateDismiss = { shouldAnimateDismiss() },
+                    onDismiss = { animateClose() },
+                    onSwipeViewMove = ::handleSwipeViewMove
+            )
 
     private fun createSwipeDirectionDetector() =
             SwipeDirectionDetector(this) { swipeDirection = it }

vector/src/main/java/im/vector/app/features/home/room/detail/TimelineFragment.kt

@@ -2111,7 +2111,6 @@ class TimelineFragment @Inject constructor(
         }
     }
 
-    // TODO mutualize permission checking creating activity extension or delegation interface?
     private fun onSaveActionClicked(action: EventSharedAction.Save) {
         if (Build.VERSION.SDK_INT < Build.VERSION_CODES.Q &&
                 !checkPermissions(PERMISSIONS_FOR_WRITING_FILES, requireActivity(), saveActionActivityResultLauncher)) {

vector/src/main/java/im/vector/app/features/media/VectorAttachmentViewerActivity.kt

@@ -17,6 +17,7 @@ package im.vector.app.features.media
 
 import android.content.Context
 import android.content.Intent
+import android.os.Build
 import android.os.Bundle
 import android.os.Parcelable
 import android.view.View
@@ -34,7 +35,13 @@ import com.airbnb.mvrx.viewModel
 import dagger.hilt.android.AndroidEntryPoint
 import im.vector.app.R
 import im.vector.app.core.di.ActiveSessionHolder
+import im.vector.app.core.extensions.singletonEntryPoint
 import im.vector.app.core.intent.getMimeTypeFromUri
+import im.vector.app.core.platform.showOptimizedSnackbar
+import im.vector.app.core.utils.PERMISSIONS_FOR_WRITING_FILES
+import im.vector.app.core.utils.checkPermissions
+import im.vector.app.core.utils.onPermissionDeniedDialog
+import im.vector.app.core.utils.registerForPermissionsResult
 import im.vector.app.core.utils.shareMedia
 import im.vector.app.features.themes.ActivityOtherThemes
 import im.vector.app.features.themes.ThemeUtils
@@ -81,9 +88,20 @@ class VectorAttachmentViewerActivity : AttachmentViewerActivity(), AttachmentInt
      * ========================================================================================== */
 
     private val viewModel: VectorAttachmentViewerViewModel by viewModel()
+    private val errorFormatter by lazy(LazyThreadSafetyMode.NONE) { singletonEntryPoint().errorFormatter() }
     private var initialIndex = 0
     private var isAnimatingOut = false
     private var currentSourceProvider: BaseAttachmentProvider<*>? = null
+    private val downloadActionResultLauncher = registerForPermissionsResult { allGranted, deniedPermanently ->
+        if (allGranted) {
+            viewModel.pendingAction?.let {
+                viewModel.handle(it)
+            }
+        } else if (deniedPermanently) {
+            onPermissionDeniedDialog(R.string.denied_permission_generic)
+        }
+        viewModel.pendingAction = null
+    }
 
     /* ==========================================================================================
      * Lifecycle
@@ -254,14 +272,18 @@ class VectorAttachmentViewerActivity : AttachmentViewerActivity(), AttachmentInt
 
     private fun handleViewEvents(event: VectorAttachmentViewerViewEvents) {
         when (event) {
-            is VectorAttachmentViewerViewEvents.DownloadingMedia      -> Unit // TODO show loader?
-            is VectorAttachmentViewerViewEvents.ErrorDownloadingMedia -> {
-                // TODO show snackbar
-                Timber.e("failure saving file: ${event.error}")
-            }
+            is VectorAttachmentViewerViewEvents.ErrorDownloadingMedia -> showSnackBarError(event.error)
         }
     }
 
+    private fun showSnackBarError(error: Throwable) {
+        rootView.showOptimizedSnackbar(errorFormatter.toHumanReadable(error))
+    }
+
+    private fun hasWritePermission() =
+            Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q ||
+                    checkPermissions(PERMISSIONS_FOR_WRITING_FILES, this, downloadActionResultLauncher)
+
     /* ==========================================================================================
      * Specialization AttachmentInteractionListener
      * ========================================================================================== */
@@ -294,15 +316,23 @@ class VectorAttachmentViewerActivity : AttachmentViewerActivity(), AttachmentInt
 
     override fun onDownload() {
         // TODO
-        //  show message on error event: see TimelineFragment
-        //  check write file permissions: see TimelineFragment
-        //  should we check if media is saveable?
+        //  test snackbar error in OnCreate()
+        //  test write permission checking with Android 9
         //  check if it is already possible to save from menu with long press on video
         //  check if it works for video or other media type as well
+        //  reorder action for a message according to issue requirements
         //  add unit tests for usecase? what is the used mock library?
         lifecycleScope.launch(Dispatchers.IO) {
+            val hasWritePermission = withContext(Dispatchers.Main) {
+                hasWritePermission()
+            }
+
             val file = currentSourceProvider?.getFileForSharing(currentPosition) ?: return@launch
-            viewModel.handle(VectorAttachmentViewerAction.DownloadMedia(file))
+            if (hasWritePermission) {
+                viewModel.handle(VectorAttachmentViewerAction.DownloadMedia(file))
+            } else {
+                viewModel.pendingAction = VectorAttachmentViewerAction.DownloadMedia(file)
+            }
         }
     }
 

vector/src/main/java/im/vector/app/features/media/VectorAttachmentViewerViewEvents.kt

@@ -19,6 +19,5 @@ package im.vector.app.features.media
 import im.vector.app.core.platform.VectorViewEvents
 
 sealed class VectorAttachmentViewerViewEvents : VectorViewEvents {
-    object DownloadingMedia : VectorAttachmentViewerViewEvents()
     data class ErrorDownloadingMedia(val error: Throwable) : VectorAttachmentViewerViewEvents()
 }

vector/src/main/java/im/vector/app/features/media/VectorAttachmentViewerViewModel.kt

@@ -44,6 +44,12 @@ class VectorAttachmentViewerViewModel @AssistedInject constructor(
 
     companion object : MavericksViewModelFactory<VectorAttachmentViewerViewModel, VectorDummyViewState> by hiltMavericksViewModelFactory()
 
+    /* ==========================================================================================
+     * Public Api
+     * ========================================================================================== */
+
+    var pendingAction: VectorAttachmentViewerAction? = null
+
     /* ==========================================================================================
      * Specialization
      * ========================================================================================== */
@@ -60,9 +66,7 @@ class VectorAttachmentViewerViewModel @AssistedInject constructor(
 
     private fun handleDownloadAction(action: VectorAttachmentViewerAction.DownloadMedia) {
         viewModelScope.launch {
-            _viewEvents.post(VectorAttachmentViewerViewEvents.DownloadingMedia)
-
-            // Success event is handled via a notification inside use case
+            // Success event is handled via a notification inside the use case
             downloadMediaUseCase.execute(action.file)
                     .onFailure { _viewEvents.post(VectorAttachmentViewerViewEvents.ErrorDownloadingMedia(it)) }
         }