Resolve unhandled exception on iOS (#1298)

Baseflow · Mar 27, 2024 · bbcd26c · bbcd26c
1 parent 2db062f
commit bbcd26c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/permission_handler_platform_interface/CHANGELOG.md b/permission_handler_platform_interface/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 4.2.1
+
+* Resolves an error that occurred when calling the `shouldShowRequestPermissionRationale` on iOS.
+
 ## 4.2.0
 
 * Adds a new permission `Permission.backgroundRefresh` to check the background refresh permission status on iOS & macOS platforms. This is a no-op on all other platforms.

diff --git a/..._handler_platform_interface/lib/src/method_channel/method_channel_permission_handler.dart b/..._handler_platform_interface/lib/src/method_channel/method_channel_permission_handler.dart
@@ -1,5 +1,6 @@
 import 'dart:async';
 
+import 'package:flutter/foundation.dart';
 import 'package:flutter/services.dart';
 
 import '../../permission_handler_platform_interface.dart';
@@ -88,6 +89,10 @@ class MethodChannelPermissionHandler extends PermissionHandlerPlatform {
   @override
   Future<bool> shouldShowRequestPermissionRationale(
       Permission permission) async {
+    if (defaultTargetPlatform != TargetPlatform.android) {
+      return false;
+    }
+
     final shouldShowRationale = await _methodChannel.invokeMethod(
         'shouldShowRequestPermissionRationale', permission.value);
 

diff --git a/permission_handler_platform_interface/pubspec.yaml b/permission_handler_platform_interface/pubspec.yaml
@@ -3,7 +3,7 @@ description: A common platform interface for the permission_handler plugin.
 homepage: https://github.com/baseflow/flutter-permission-handler/tree/master/permission_handler_platform_interface
 # NOTE: We strongly prefer non-breaking changes, even at the expense of a
 # less-clean API. See https://flutter.dev/go/platform-interface-breaking-changes
-version: 4.2.0
+version: 4.2.1
 
 dependencies:
   flutter: