Fixed crash: No Activity found to handle Intent { act=android.intent.…

…action.CREATE_DOCUMENT
spacecowboy · Sep 23, 2023 · 724d0b6 · 724d0b6
1 parent 477e3b3
commit 724d0b6
Showing 1 changed file with 32 additions and 13 deletions.
diff --git a/app/src/main/java/com/nononsenseapps/feeder/ui/compose/feed/FeedScreen.kt b/app/src/main/java/com/nononsenseapps/feeder/ui/compose/feed/FeedScreen.kt
@@ -137,6 +137,7 @@ import com.nononsenseapps.feeder.ui.compose.utils.isCompactDevice
 import com.nononsenseapps.feeder.ui.compose.utils.onKeyEventLikeEscape
 import com.nononsenseapps.feeder.ui.compose.utils.rememberIsItemMostlyVisible
 import com.nononsenseapps.feeder.ui.compose.utils.rememberIsItemVisible
+import com.nononsenseapps.feeder.util.ToastMaker
 import com.nononsenseapps.feeder.util.emailBugReportIntent
 import com.nononsenseapps.feeder.util.logDebug
 import com.nononsenseapps.feeder.util.openLinkInBrowser
@@ -148,6 +149,7 @@ import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.delay
 import kotlinx.coroutines.launch
 import org.kodein.di.compose.LocalDI
+import org.kodein.di.compose.instance
 import org.kodein.di.instance
 
 private const val LOG_TAG = "FEEDER_FEEDSCREEN"
@@ -162,6 +164,7 @@ fun FeedScreen(
     viewModel: FeedArticleViewModel,
     navDrawerListState: LazyListState,
 ) {
+    val toastMaker: ToastMaker by instance()
     val viewState: FeedArticleScreenViewState by viewModel.viewState.collectAsStateWithLifecycle()
     val pagedFeedItems = viewModel.currentFeedListItems.collectAsLazyPagingItems()
 
@@ -293,20 +296,36 @@ fun FeedScreen(
                 context.startActivity(emailBugReportIntent())
             },
             onImport = {
-                opmlImporter.launch(
-                    arrayOf(
-                        // All valid for OPML files
-                        "text/xml",
-                        "text/x-opml",
-                        "application/xml",
-                        // This is the mimetype the file actually gets when exported
-                        "application/octet-stream",
-                        // But just in case a file isn't named right etc, accept all
-                        "*/*",
-                    ),
-                )
+                try {
+                    opmlImporter.launch(
+                        arrayOf(
+                            // All valid for OPML files
+                            "text/xml",
+                            "text/x-opml",
+                            "application/xml",
+                            // This is the mimetype the file actually gets when exported
+                            "application/octet-stream",
+                            // But just in case a file isn't named right etc, accept all
+                            "*/*",
+                        ),
+                    )
+                } catch (e: Exception) {
+                    // ActivityNotFoundException in particular
+                    coroutineScope.launch {
+                        toastMaker.makeToast(R.string.failed_to_import_OPML)
+                    }
+                }
+            },
+            onExport = {
+                try {
+                    opmlExporter.launch("feeder-export-${LocalDateTime.now()}.opml")
+                } catch (e: Exception) {
+                    // ActivityNotFoundException in particular
+                    coroutineScope.launch {
+                        toastMaker.makeToast(R.string.failed_to_export_OPML)
+                    }
+                }
             },
-            onExport = { opmlExporter.launch("feeder-export-${LocalDateTime.now()}.opml") },
             drawerState = drawerState,
             markAsUnread = { itemId, unread, feedOrTag ->
                 if (unread) {