onReceives() causes java.lang.IllegalStateException ? #154

Author: EddyVerbruggen

demo/nsconfig.json

@@ -0,0 +1,3 @@
+{
+	"useLegacyWorkflow": true
+}

native-src/android/README.md

@@ -3,5 +3,5 @@
 ## Building the framework
 - Import this folder as a gradle project in Android Studio
 - Change anything needed
-- In the Gradle build window run app > build > build
-- Grab the release .aar from native-src/android/app/build/outputs/aar: `cp native-src/android/app/build/outputs/aar/app-release.aar src/platforms/android/` 
+- In the Gradle build window run `app > build > clean` and `app > build > build`
+- Copy the release .aar: `cp native-src/android/app/build/outputs/aar/app-release.aar src/platforms/android/` 

native-src/android/app/app.iml

@@ -17,7 +17,7 @@
         <option name="ALLOW_USER_CONFIGURATION" value="false" />
         <option name="MANIFEST_FILE_RELATIVE_PATH" value="/src/main/AndroidManifest.xml" />
         <option name="RES_FOLDER_RELATIVE_PATH" value="/src/main/res" />
-        <option name="RES_FOLDERS_RELATIVE_PATH" value="file://$MODULE_DIR$/src/main/res;file://$MODULE_DIR$/build/generated/res/rs/debug;file://$MODULE_DIR$/build/generated/res/resValues/debug" />
+        <option name="RES_FOLDERS_RELATIVE_PATH" value="file://$MODULE_DIR$/src/main/res" />
         <option name="ASSETS_FOLDER_RELATIVE_PATH" value="/src/main/assets" />
         <option name="PROJECT_TYPE" value="1" />
       </configuration>
@@ -95,7 +95,6 @@
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/intermediate-jars" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/javaPrecompile" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/jniLibs" />
-      <excludeFolder url="file://$MODULE_DIR$/build/intermediates/lint" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/manifests" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/res" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/rs" />

native-src/android/app/src/main/java/com/telerik/localnotifications/NotificationClearedReceiver.java

@@ -27,11 +27,13 @@ public void onReceive(Context context, @Nullable Intent intent) {
     final int id = intent.getIntExtra(Builder.NOTIFICATION_ID, 0);
     final JSONObject opts = Store.get(context, id);
 
-    if (opts.optInt("repeatInterval", 0) == 0) {
-      // Remove the persisted notification data if it's not repeating:
-      Store.remove(context, id);
-    }
+    if (opts != null) {
+      if (opts.optInt("repeatInterval", 0) == 0) {
+        // Remove the persisted notification data if it's not repeating:
+        Store.remove(context, id);
+      }
 
-    LocalNotificationsPlugin.executeOnMessageClearedCallback(opts);
+      LocalNotificationsPlugin.executeOnMessageClearedCallback(opts);
+    }
   }
 }

native-src/android/app/src/main/java/com/telerik/localnotifications/NotificationRestoreReceiver.java

@@ -28,16 +28,16 @@ public void onReceive(Context context, Intent intent) {
       return;
     }
 
-    for (Map.Entry<String, String> entry : Store.getAll(context).entrySet()) {
-      final String notificationString = entry.getValue();
+    try {
+      for (Map.Entry<String, String> entry : Store.getAll(context).entrySet()) {
+        final String notificationString = entry.getValue();
 
-      Log.e(TAG, "Will restore previously scheduled notification: " + notificationString);
+        Log.e(TAG, "Will restore previously scheduled notification: " + notificationString);
 
-      try {
         scheduleNotification(new JSONObject(notificationString), context);
-      } catch (JSONException e) {
-        Log.e(TAG, "Notification could not be scheduled! " + e.getMessage(), e);
       }
+    } catch (IllegalStateException | JSONException e) {
+      Log.e(TAG, "Notification could not be scheduled! " + e.getMessage(), e);
     }
   }