Merge pull request #633 from akshetpandey/fixNotificationCancelAllCrash

[android][messaging] Handle SecuritException in cancelAllLocalNotific…
This commit is contained in:
Michael Diarmid 2017-11-21 07:39:12 +00:00 committed by GitHub
commit a95c91e453
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 1 deletions

View File

@ -104,7 +104,13 @@ public class RNFirebaseMessaging extends ReactContextBaseJavaModule implements L
@ReactMethod
public void cancelAllLocalNotifications() {
try {
mRNFirebaseLocalMessagingHelper.cancelAllLocalNotifications();
} catch (SecurityException e) {
// In some devices/situations cancelAllLocalNotifications will throw a SecurityException
// We can safely ignore this error for now as the UX impact of this not working is minor.
Log.w(TAG, e.getMessage());
}
}
@ReactMethod