Merge pull request #1541 from TeamAmaze/fix-1525

Fix 1525
TeamAmaze · Dec 17, 2018 · 069095e · 069095e
1 parent 6488833
commit 069095e
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/app/src/main/java/com/amaze/filemanager/utils/ServiceWatcherUtil.java b/app/src/main/java/com/amaze/filemanager/utils/ServiceWatcherUtil.java
@@ -96,7 +96,8 @@ public void run() {
                         // we passed at the beginning is never reached
                         // we try to get a less precise size and make our decision based on that
                         progressHandler.addWrittenLength(progressHandler.getTotalSize());
-                        pendingIntents.remove();
+                        if (!pendingIntents.isEmpty())
+                            pendingIntents.remove();
                         handler.removeCallbacks(this);
                         handlerThread.quit();
                         return;
@@ -127,7 +128,8 @@ public void run() {
                 if (position == progressHandler.getTotalSize() || progressHandler.getCancelled()) {
                     // process complete, free up resources
                     // we've finished the work or process cancelled
-                    pendingIntents.remove();
+                    if (!pendingIntents.isEmpty())
+                        pendingIntents.remove();
                     handler.removeCallbacks(this);
                     handlerThread.quit();
                     return;
@@ -158,15 +160,23 @@ public void stopWatch() {
      * as there are higher chances for android system to GC the thread when it is running low on memory
      */
     public static synchronized void runService(final Context context, final Intent intent) {
-        pendingIntents.add(intent);
         switch (pendingIntents.size()) {
+            case 0:
+                context.startService(intent);
+                break;
             case 1:
                 // initialize waiting handlers
+                pendingIntents.add(intent);
                 postWaiting(context);
                 break;
             case 2:
                 // to avoid notifying repeatedly
+                pendingIntents.add(intent);
                 notificationManager.notify(NotificationConstants.WAIT_ID, builder.build());
+                break;
+            default:
+                pendingIntents.add(intent);
+                break;
         }
     }