status-im
/
react-native-firebase
mirror of
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge pull request #1073 from dluksza/notification-schedule-classcastexception 

Fix ClassCastException while parsing fireDate
This commit is contained in:
Chris Bianca 2018-05-11 14:26:46 +01:00 committed by GitHub
parent 13dd662903 dbc5a2d54e
commit 6848044f13
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
android/src/main/java/io/invertase/firebase/notifications/RNFirebaseNotificationManager.java
View File

@ -309,17 +309,19 @@ public class RNFirebaseNotificationManager {
// fireDate is stored in the Bundle as Long after notifications are rescheduled. // fireDate is stored in the Bundle as Long after notifications are rescheduled.
// This would lead to a fireDate of 0.0 when trying to extract a Double from the bundle. // This would lead to a fireDate of 0.0 when trying to extract a Double from the bundle.
// Instead always try extract a Long // Instead always try extract a Long
Long fireDate = schedule.getLong("fireDate", -1); Long fireDate = -1L;
if (fireDate == -1) { try {
fireDate = (long) schedule.getDouble("fireDate", -1); fireDate = (long) schedule.getDouble("fireDate", -1);
if (fireDate == -1) { } catch (ClassCastException e) {
if (promise == null) { fireDate = schedule.getLong("fireDate", -1);
Log.e(TAG, "Missing schedule information"); }
} else { if (fireDate == -1) {
promise.reject("notification/schedule_notification_error", "Missing fireDate information"); if (promise == null) {
} Log.e(TAG, "Missing schedule information");
return; } else {
promise.reject("notification/schedule_notification_error", "Missing fireDate information");
} }
return;
} }
// Scheduled alarms are cleared on restart // Scheduled alarms are cleared on restart