fix requestNotificationPermissions api (flutter#761)

* fix requestNotificationPermissions api
imenuo · Oct 18, 2018 · 50a283b · 50a283b
1 parent b235438
commit 50a283b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/packages/firebase_messaging/CHANGELOG.md b/packages/firebase_messaging/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 2.0.2
+
+* Use boolean values when checking for notification types on iOS.
+
 ## 2.0.1
 
 * Bump Android dependencies to latest.

diff --git a/packages/firebase_messaging/ios/Classes/FirebaseMessagingPlugin.m b/packages/firebase_messaging/ios/Classes/FirebaseMessagingPlugin.m
@@ -46,13 +46,13 @@ - (void)handleMethodCall:(FlutterMethodCall *)call result:(FlutterResult)result
   if ([@"requestNotificationPermissions" isEqualToString:method]) {
     UIUserNotificationType notificationTypes = 0;
     NSDictionary *arguments = call.arguments;
-    if (arguments[@"sound"]) {
+    if ([arguments[@"sound"] boolValue]) {
       notificationTypes |= UIUserNotificationTypeSound;
     }
-    if (arguments[@"alert"]) {
+    if ([arguments[@"alert"] boolValue]) {
       notificationTypes |= UIUserNotificationTypeAlert;
     }
-    if (arguments[@"badge"]) {
+    if ([arguments[@"badge"] boolValue]) {
       notificationTypes |= UIUserNotificationTypeBadge;
     }
     UIUserNotificationSettings *settings =

diff --git a/packages/firebase_messaging/pubspec.yaml b/packages/firebase_messaging/pubspec.yaml
@@ -3,7 +3,7 @@ description: Flutter plugin for Firebase Cloud Messaging, a cross-platform
   messaging solution that lets you reliably deliver messages on Android and iOS.
 author: Flutter Team <[email protected]>
 homepage: https://github.com/flutter/plugins/tree/master/packages/firebase_messaging
-version: 2.0.1
+version: 2.0.2
 
 flutter:
   plugin: