Handle nullable links in share handler

msasikanth · Apr 16, 2024 · f8ef24d · f8ef24d
1 parent cc22f34
commit f8ef24d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/shared/src/androidMain/kotlin/dev/sasikanth/rss/reader/share/AndroidShareHandler.kt b/shared/src/androidMain/kotlin/dev/sasikanth/rss/reader/share/AndroidShareHandler.kt
@@ -25,7 +25,9 @@ import me.tatarka.inject.annotations.Inject
 @ActivityScope
 class AndroidShareHandler(private val activity: ComponentActivity) : ShareHandler {
 
-  override fun share(content: String) {
+  override fun share(content: String?) {
+    if (content.isNullOrBlank()) return
+
     val sendIntent =
       Intent().apply {
         action = Intent.ACTION_SEND

diff --git a/shared/src/commonMain/kotlin/dev/sasikanth/rss/reader/share/ShareHandler.kt b/shared/src/commonMain/kotlin/dev/sasikanth/rss/reader/share/ShareHandler.kt
@@ -19,7 +19,7 @@ package dev.sasikanth.rss.reader.share
 import androidx.compose.runtime.staticCompositionLocalOf
 
 interface ShareHandler {
-  fun share(content: String)
+  fun share(content: String?)
 }
 
 val LocalShareHandler =

diff --git a/shared/src/iosMain/kotlin/dev/sasikanth/rss/reader/share/IOSShareHandler.kt b/shared/src/iosMain/kotlin/dev/sasikanth/rss/reader/share/IOSShareHandler.kt
@@ -31,7 +31,9 @@ import platform.UIKit.popoverPresentationController
 class IOSShareHandler(private val viewControllerProvider: () -> UIViewController) : ShareHandler {
 
   @OptIn(ExperimentalForeignApi::class)
-  override fun share(content: String) {
+  override fun share(content: String?) {
+    if (content.isNullOrBlank()) return
+
     val viewController = viewControllerProvider()
     val items = listOf(content)
     val activityController = UIActivityViewController(items, null)