[share_plus] Fix types, lower Android minSDK

packages/share_plus/share_plus/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 4.0.2
+
+- Fix type mismatch on Android for some users
+- Set min Flutter to 1.20.0 for all platforms
+- Lower Android minSdkVersion to 22
+
 ## 4.0.1
 
 - Hotfix dependencies

packages/share_plus/share_plus/android/build.gradle

@@ -28,7 +28,7 @@ android {
   compileSdkVersion 31
 
   defaultConfig {
-    minSdkVersion 23
+    minSdkVersion 22
     testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
   }
   lintOptions {

packages/share_plus/share_plus/android/src/main/kotlin/dev/fluttercommunity/plus/share/MethodCallHandler.kt

@@ -5,71 +5,74 @@ import io.flutter.plugin.common.MethodChannel
 import java.io.IOException
 
 /** Handles the method calls for the plugin.  */
-internal class MethodCallHandler(private val share: Share, private val manager: ShareSuccessManager) : MethodChannel.MethodCallHandler {
+internal class MethodCallHandler(
+    private val share: Share,
+    private val manager: ShareSuccessManager
+) : MethodChannel.MethodCallHandler {
 
-  override fun onMethodCall(call: MethodCall, result: MethodChannel.Result) {
-    when (call.method) {
-      "share" -> {
-        expectMapArguments(call)
-        // Android does not support showing the share sheet at a particular point on screen.
-        share.share(
-          call.argument<Any>("text") as String,
-          call.argument<Any>("subject") as String?,
-          false,
-        )
-        result.success(null)
-      }
-      "shareFiles" -> {
-        expectMapArguments(call)
+    override fun onMethodCall(call: MethodCall, result: MethodChannel.Result) {
+        when (call.method) {
+            "share" -> {
+                expectMapArguments(call)
+                // Android does not support showing the share sheet at a particular point on screen.
+                share.share(
+                    call.argument<Any>("text") as String,
+                    call.argument<Any>("subject") as String?,
+                    false,
+                )
+                result.success(null)
+            }
+            "shareFiles" -> {
+                expectMapArguments(call)
 
-        // Android does not support showing the share sheet at a particular point on screen.
-        try {
-          share.shareFiles(
-            call.argument<List<String>>("paths")!!,
-            call.argument<List<String>?>("mimeTypes"),
-            call.argument<String?>("text"),
-            call.argument<String?>("subject"),
-            false,
-          )
-          result.success(null)
-        } catch (e: IOException) {
-          result.error(e.message, null, null)
-        }
-      }
-      "shareWithResult" -> {
-        expectMapArguments(call)
-        if(!manager.setCallback(result)) return
+                // Android does not support showing the share sheet at a particular point on screen.
+                try {
+                    share.shareFiles(
+                        call.argument<List<String>>("paths")!!,
+                        call.argument<List<String>?>("mimeTypes"),
+                        call.argument<String?>("text"),
+                        call.argument<String?>("subject"),
+                        false,
+                    )
+                    result.success(null)
+                } catch (e: IOException) {
+                    result.error("Share failed", e.message, null)
+                }
+            }
+            "shareWithResult" -> {
+                expectMapArguments(call)
+                if (!manager.setCallback(result)) return
 
-        // Android does not support showing the share sheet at a particular point on screen.
-        share.share(
-          call.argument<Any>("text") as String,
-          call.argument<Any>("subject") as String?,
-          true,
-        )
-      }
-      "shareFilesWithResult" -> {
-        expectMapArguments(call)
-        if(!manager.setCallback(result)) return
+                // Android does not support showing the share sheet at a particular point on screen.
+                share.share(
+                    call.argument<Any>("text") as String,
+                    call.argument<Any>("subject") as String?,
+                    true,
+                )
+            }
+            "shareFilesWithResult" -> {
+                expectMapArguments(call)
+                if (!manager.setCallback(result)) return
 
-        // Android does not support showing the share sheet at a particular point on screen.
-        try {
-          share.shareFiles(
-            call.argument<List<String>>("paths")!!,
-            call.argument<List<String>?>("mimeTypes"),
-            call.argument<String?>("text"),
-            call.argument<String?>("subject"),
-            true,
-          )
-        } catch (e: IOException) {
-          result.error(e.message, null, null)
+                // Android does not support showing the share sheet at a particular point on screen.
+                try {
+                    share.shareFiles(
+                        call.argument<List<String>>("paths")!!,
+                        call.argument<List<String>?>("mimeTypes"),
+                        call.argument<String?>("text"),
+                        call.argument<String?>("subject"),
+                        true,
+                    )
+                } catch (e: IOException) {
+                    result.error("Share failed", e.message, null)
+                }
+            }
+            else -> result.notImplemented()
         }
-      }
-      else -> result.notImplemented()
     }
-  }
 
-  @Throws(IllegalArgumentException::class)
-  private fun expectMapArguments(call: MethodCall) {
-    require(call.arguments is Map<*, *>) { "Map arguments expected" }
-  }
+    @Throws(IllegalArgumentException::class)
+    private fun expectMapArguments(call: MethodCall) {
+        require(call.arguments is Map<*, *>) { "Map arguments expected" }
+    }
 }