@capgo/capacitor-updater 5.7.4 → 5.7.9

package/android/src/main/java/ee/forgr/capacitor_updater/Callback.java

@@ -0,0 +1,7 @@
+package ee.forgr.capacitor_updater;
+
+import com.getcapacitor.JSObject;
+
+public interface Callback {
+  void callback(JSObject jsoObject);
+}

package/android/src/main/java/ee/forgr/capacitor_updater/CapacitorUpdater.java

@@ -23,7 +23,6 @@ import com.android.volley.DefaultRetryPolicy;
 import com.android.volley.NetworkResponse;
 import com.android.volley.Request;
 import com.android.volley.RequestQueue;
-import com.android.volley.Response;
 import com.android.volley.VolleyError;
 import com.android.volley.toolbox.HttpHeaderParser;
 import com.android.volley.toolbox.JsonObjectRequest;
@@ -48,6 +47,7 @@ import java.util.ArrayList;
 import java.util.Date;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Objects;
 import java.util.zip.CRC32;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipInputStream;
@@ -55,10 +55,6 @@ import javax.crypto.SecretKey;
 import org.json.JSONException;
 import org.json.JSONObject;
 
-interface Callback {
-  void callback(JSObject jsoObject);
-}
-
 public class CapacitorUpdater {
 
   private static final String AB =
@@ -94,16 +90,13 @@ public class CapacitorUpdater {
   public String deviceID = "";
   public int timeout = 20000;
 
-  private final FilenameFilter filter = new FilenameFilter() {
-    @Override
-    public boolean accept(final File f, final String name) {
-      // ignore directories generated by mac os x
-      return (
-        !name.startsWith("__MACOSX") &&
-        !name.startsWith(".") &&
-        !name.startsWith(".DS_Store")
-      );
-    }
+  private final FilenameFilter filter = (f, name) -> {
+    // ignore directories generated by mac os x
+    return (
+      !name.startsWith("__MACOSX") &&
+      !name.startsWith(".") &&
+      !name.startsWith(".DS_Store")
+    );
   };
 
   private boolean isProd() {
@@ -142,23 +135,15 @@ public class CapacitorUpdater {
     return (percent * (max - min)) / 100 + min;
   }
 
-  void notifyDownload(final String id, final int percent) {
-    return;
-  }
+  void notifyDownload(final String id, final int percent) {}
 
-  void directUpdateFinish(final BundleInfo latest) {
-    return;
-  }
+  void directUpdateFinish(final BundleInfo latest) {}
 
-  void notifyListeners(final String id, final JSObject res) {
-    return;
-  }
+  void notifyListeners(final String id, final JSObject res) {}
 
-  private String randomString(final int len) {
-    final StringBuilder sb = new StringBuilder(len);
-    for (int i = 0; i < len; i++) sb.append(
-      AB.charAt(rnd.nextInt(AB.length()))
-    );
+  private String randomString() {
+    final StringBuilder sb = new StringBuilder(10);
+    for (int i = 0; i < 10; i++) sb.append(AB.charAt(rnd.nextInt(AB.length())));
     return sb.toString();
   }
 
@@ -204,8 +189,8 @@ public class CapacitorUpdater {
           );
         }
 
+        assert dir != null;
         if (!dir.isDirectory() && !dir.mkdirs()) {
-          final JSObject ret = new JSObject();
           this.sendStats("directory_path_fail");
           throw new FileNotFoundException(
             "Failed to ensure directory: " + dir.getAbsolutePath()
@@ -247,7 +232,7 @@ public class CapacitorUpdater {
       );
     }
     final File destinationFile = new File(this.documentsDir, dest);
-    destinationFile.getParentFile().mkdirs();
+    Objects.requireNonNull(destinationFile.getParentFile()).mkdirs();
     final String[] entries = sourceFile.list(this.filter);
     if (entries == null || entries.length == 0) {
       throw new IOException(
@@ -282,17 +267,17 @@ public class CapacitorUpdater {
     this.activity.unregisterReceiver(receiver);
   }
 
-  private BroadcastReceiver receiver = new BroadcastReceiver() {
+  private final BroadcastReceiver receiver = new BroadcastReceiver() {
     @Override
     public void onReceive(Context context, Intent intent) {
       String action = intent.getAction();
       Bundle bundle = intent.getExtras();
       if (bundle != null) {
-        if (action == DownloadService.PERCENTDOWNLOAD) {
+        if (Objects.equals(action, DownloadService.PERCENTDOWNLOAD)) {
           String id = bundle.getString(DownloadService.ID);
           int percent = bundle.getInt(DownloadService.PERCENT);
           CapacitorUpdater.this.notifyDownload(id, percent);
-        } else if (action == DownloadService.NOTIFICATION) {
+        } else if (Objects.equals(action, DownloadService.NOTIFICATION)) {
           String id = bundle.getString(DownloadService.ID);
           String dest = bundle.getString(DownloadService.FILEDEST);
           String version = bundle.getString(DownloadService.VERSION);
@@ -357,7 +342,7 @@ public class CapacitorUpdater {
       final String checksum;
       checksum = this.getChecksum(downloaded);
       this.notifyDownload(id, 71);
-      final File unzipped = this.unzip(id, downloaded, this.randomString(10));
+      final File unzipped = this.unzip(id, downloaded, this.randomString());
       downloaded.delete();
       this.notifyDownload(id, 91);
       final String idName = bundleDirectory + "/" + id;
@@ -438,7 +423,7 @@ public class CapacitorUpdater {
     this.activity.startService(intent);
   }
 
-  private File downloadFile(
+  private void downloadFile(
     final String id,
     final String url,
     final String dest
@@ -447,7 +432,7 @@ public class CapacitorUpdater {
     final URLConnection connection = u.openConnection();
 
     final File target = new File(this.documentsDir, dest);
-    target.getParentFile().mkdirs();
+    Objects.requireNonNull(target.getParentFile()).mkdirs();
     target.createNewFile();
 
     final long totalLength = connection.getContentLength();
@@ -477,7 +462,6 @@ public class CapacitorUpdater {
       this.sendStats("low_mem_fail");
       throw new IOException("OutOfMemoryError while downloading file");
     }
-    return target;
   }
 
   private void deleteDirectory(final File file) throws IOException {
@@ -574,7 +558,7 @@ public class CapacitorUpdater {
     final String sessionKey,
     final String checksum
   ) {
-    final String id = this.randomString(10);
+    final String id = this.randomString();
     this.saveBundleInfo(
         id,
         new BundleInfo(
@@ -593,7 +577,7 @@ public class CapacitorUpdater {
         version,
         sessionKey,
         checksum,
-        this.randomString(10)
+        this.randomString()
       );
   }
 
@@ -603,7 +587,7 @@ public class CapacitorUpdater {
     final String sessionKey,
     final String checksum
   ) throws IOException {
-    final String id = this.randomString(10);
+    final String id = this.randomString();
     this.saveBundleInfo(
         id,
         new BundleInfo(
@@ -615,10 +599,9 @@ public class CapacitorUpdater {
         )
       );
     this.notifyDownload(id, 0);
-    final String idName = bundleDirectory + "/" + id;
     this.notifyDownload(id, 5);
-    final String dest = this.randomString(10);
-    final File downloaded = this.downloadFile(id, url, dest);
+    final String dest = this.randomString();
+    this.downloadFile(id, url, dest);
     final Boolean finished =
       this.finishDownload(id, dest, version, sessionKey, checksum, false);
     final BundleStatus status = finished
@@ -640,7 +623,7 @@ public class CapacitorUpdater {
     final File destHot = new File(this.documentsDir, bundleDirectory);
     Log.d(TAG, "list File : " + destHot.getPath());
     if (destHot.exists()) {
-      for (final File i : destHot.listFiles()) {
+      for (final File i : Objects.requireNonNull(destHot.listFiles())) {
         final String id = i.getName();
         res.add(this.getBundleInfo(id));
       }
@@ -683,15 +666,12 @@ public class CapacitorUpdater {
   private boolean bundleExists(final String id) {
     final File bundle = this.getBundleDirectory(id);
     final BundleInfo bundleInfo = this.getBundleInfo(id);
-    if (
+    return (
       bundle.isDirectory() &&
       bundle.exists() &&
       new File(bundle.getPath(), "/index.html").exists() &&
       !bundleInfo.isDeleted()
-    ) {
-      return true;
-    }
-    return false;
+    );
   }
 
   public Boolean set(final BundleInfo bundle) {
@@ -803,10 +783,10 @@ public class CapacitorUpdater {
         );
         retError.put("message", message + ": " + json);
       } catch (UnsupportedEncodingException e) {
-        retError.put("message", message + ": " + e.toString());
+        retError.put("message", message + ": " + e);
       }
     } else {
-      retError.put("message", message + ": " + error.toString());
+      retError.put("message", message + ": " + error);
     }
     Log.e(TAG, message + ": " + retError);
     return retError;
@@ -820,7 +800,7 @@ public class CapacitorUpdater {
       Log.e(TAG, "Error getLatest JSONException", e);
       e.printStackTrace();
       final JSObject retError = new JSObject();
-      retError.put("message", "Cannot get info: " + e.toString());
+      retError.put("message", "Cannot get info: " + e);
       retError.put("error", "json_error");
       callback.callback(retError);
       return;
@@ -832,49 +812,40 @@ public class CapacitorUpdater {
       Request.Method.POST,
       updateUrl,
       json,
-      new Response.Listener<JSONObject>() {
-        @Override
-        public void onResponse(JSONObject res) {
-          final JSObject ret = new JSObject();
-          Iterator<String> keys = res.keys();
-          while (keys.hasNext()) {
-            String key = keys.next();
-            if (res.has(key)) {
-              try {
-                if ("session_key".equals(key)) {
-                  ret.put("sessionKey", res.get(key));
-                } else {
-                  ret.put(key, res.get(key));
-                }
-              } catch (JSONException e) {
-                e.printStackTrace();
-                final JSObject retError = new JSObject();
-                retError.put("message", "Cannot set info: " + e.toString());
-                retError.put("error", "response_error");
-                callback.callback(retError);
+      res -> {
+        final JSObject ret = new JSObject();
+        Iterator<String> keys = res.keys();
+        while (keys.hasNext()) {
+          String key = keys.next();
+          if (res.has(key)) {
+            try {
+              if ("session_key".equals(key)) {
+                ret.put("sessionKey", res.get(key));
+              } else {
+                ret.put(key, res.get(key));
               }
+            } catch (JSONException e) {
+              e.printStackTrace();
+              final JSObject retError = new JSObject();
+              retError.put("message", "Cannot set info: " + e);
+              retError.put("error", "response_error");
+              callback.callback(retError);
             }
           }
-          callback.callback(ret);
         }
+        callback.callback(ret);
       },
-      new Response.ErrorListener() {
-        @Override
-        public void onErrorResponse(VolleyError error) {
-          callback.callback(
-            CapacitorUpdater.this.createError("Error get latest", error)
-          );
-        }
-      }
+      error ->
+        callback.callback(
+          CapacitorUpdater.this.createError("Error get latest", error)
+        )
     );
     this.requestQueue.add(setRetryPolicy(request));
   }
 
   public void unsetChannel(final Callback callback) {
     String channelUrl = this.channelUrl;
-    if (
-      channelUrl == null || "".equals(channelUrl) || channelUrl.length() == 0
-    ) {
+    if (channelUrl == null || channelUrl.isEmpty()) {
       Log.e(TAG, "Channel URL is not set");
       final JSObject retError = new JSObject();
       retError.put("message", "channelUrl missing");
@@ -882,14 +853,14 @@ public class CapacitorUpdater {
       callback.callback(retError);
       return;
     }
-    JSONObject json = null;
+    JSONObject json;
     try {
       json = this.createInfoObject();
     } catch (JSONException e) {
       Log.e(TAG, "Error unsetChannel JSONException", e);
       e.printStackTrace();
       final JSObject retError = new JSObject();
-      retError.put("message", "Cannot get info: " + e.toString());
+      retError.put("message", "Cannot get info: " + e);
       retError.put("error", "json_error");
       callback.callback(retError);
       return;
@@ -899,49 +870,37 @@ public class CapacitorUpdater {
       Request.Method.DELETE,
       channelUrl,
       json,
-      new Response.Listener<JSONObject>() {
-        @Override
-        public void onResponse(JSONObject res) {
-          final JSObject ret = new JSObject();
-          Iterator<String> keys = res.keys();
-          while (keys.hasNext()) {
-            String key = keys.next();
-            if (res.has(key)) {
-              try {
-                ret.put(key, res.get(key));
-              } catch (JSONException e) {
-                e.printStackTrace();
-                final JSObject retError = new JSObject();
-                retError.put(
-                  "message",
-                  "Cannot unset channel: " + e.toString()
-                );
-                retError.put("error", "response_error");
-                callback.callback(ret);
-              }
+      res -> {
+        final JSObject ret = new JSObject();
+        Iterator<String> keys = res.keys();
+        while (keys.hasNext()) {
+          String key = keys.next();
+          if (res.has(key)) {
+            try {
+              ret.put(key, res.get(key));
+            } catch (JSONException e) {
+              e.printStackTrace();
+              final JSObject retError = new JSObject();
+              retError.put("message", "Cannot unset channel: " + e);
+              retError.put("error", "response_error");
+              callback.callback(ret);
             }
           }
-          Log.i(TAG, "Channel unset");
-          callback.callback(ret);
         }
+        Log.i(TAG, "Channel unset");
+        callback.callback(ret);
       },
-      new Response.ErrorListener() {
-        @Override
-        public void onErrorResponse(VolleyError error) {
-          callback.callback(
-            CapacitorUpdater.this.createError("Error unset channel", error)
-          );
-        }
-      }
+      error ->
+        callback.callback(
+          CapacitorUpdater.this.createError("Error unset channel", error)
+        )
     );
     this.requestQueue.add(setRetryPolicy(request));
   }
 
   public void setChannel(final String channel, final Callback callback) {
     String channelUrl = this.channelUrl;
-    if (
-      channelUrl == null || "".equals(channelUrl) || channelUrl.length() == 0
-    ) {
+    if (channelUrl == null || channelUrl.isEmpty()) {
       Log.e(TAG, "Channel URL is not set");
       final JSObject retError = new JSObject();
       retError.put("message", "channelUrl missing");
@@ -949,7 +908,7 @@ public class CapacitorUpdater {
       callback.callback(retError);
       return;
     }
-    JSONObject json = null;
+    JSONObject json;
     try {
       json = this.createInfoObject();
       json.put("channel", channel);
@@ -957,7 +916,7 @@ public class CapacitorUpdater {
       Log.e(TAG, "Error setChannel JSONException", e);
       e.printStackTrace();
       final JSObject retError = new JSObject();
-      retError.put("message", "Cannot get info: " + e.toString());
+      retError.put("message", "Cannot get info: " + e);
       retError.put("error", "json_error");
       callback.callback(retError);
       return;
@@ -967,46 +926,37 @@ public class CapacitorUpdater {
       Request.Method.POST,
       channelUrl,
       json,
-      new Response.Listener<JSONObject>() {
-        @Override
-        public void onResponse(JSONObject res) {
-          final JSObject ret = new JSObject();
-          Iterator<String> keys = res.keys();
-          while (keys.hasNext()) {
-            String key = keys.next();
-            if (res.has(key)) {
-              try {
-                ret.put(key, res.get(key));
-              } catch (JSONException e) {
-                e.printStackTrace();
-                final JSObject retError = new JSObject();
-                retError.put("message", "Cannot set channel: " + e.toString());
-                retError.put("error", "response_error");
-                callback.callback(ret);
-              }
+      res -> {
+        final JSObject ret = new JSObject();
+        Iterator<String> keys = res.keys();
+        while (keys.hasNext()) {
+          String key = keys.next();
+          if (res.has(key)) {
+            try {
+              ret.put(key, res.get(key));
+            } catch (JSONException e) {
+              e.printStackTrace();
+              final JSObject retError = new JSObject();
+              retError.put("message", "Cannot set channel: " + e);
+              retError.put("error", "response_error");
+              callback.callback(ret);
             }
           }
-          Log.i(TAG, "Channel set to \"" + channel);
-          callback.callback(ret);
         }
+        Log.i(TAG, "Channel set to \"" + channel);
+        callback.callback(ret);
       },
-      new Response.ErrorListener() {
-        @Override
-        public void onErrorResponse(VolleyError error) {
-          callback.callback(
-            CapacitorUpdater.this.createError("Error set channel", error)
-          );
-        }
-      }
+      error ->
+        callback.callback(
+          CapacitorUpdater.this.createError("Error set channel", error)
+        )
     );
     this.requestQueue.add(setRetryPolicy(request));
   }
 
   public void getChannel(final Callback callback) {
     String channelUrl = this.channelUrl;
-    if (
-      channelUrl == null || "".equals(channelUrl) || channelUrl.length() == 0
-    ) {
+    if (channelUrl == null || channelUrl.isEmpty()) {
       Log.e(TAG, "Channel URL is not set");
       final JSObject retError = new JSObject();
       retError.put("message", "Channel URL is not set");
@@ -1014,14 +964,14 @@ public class CapacitorUpdater {
       callback.callback(retError);
       return;
     }
-    JSONObject json = null;
+    JSONObject json;
     try {
       json = this.createInfoObject();
     } catch (JSONException e) {
       Log.e(TAG, "Error getChannel JSONException", e);
       e.printStackTrace();
       final JSObject retError = new JSObject();
-      retError.put("message", "Cannot get info: " + e.toString());
+      retError.put("message", "Cannot get info: " + e);
       retError.put("error", "json_error");
       callback.callback(retError);
       return;
@@ -1031,33 +981,26 @@ public class CapacitorUpdater {
       Request.Method.PUT,
       channelUrl,
       json,
-      new Response.Listener<JSONObject>() {
-        @Override
-        public void onResponse(JSONObject res) {
-          final JSObject ret = new JSObject();
-          Iterator<String> keys = res.keys();
-          while (keys.hasNext()) {
-            String key = keys.next();
-            if (res.has(key)) {
-              try {
-                ret.put(key, res.get(key));
-              } catch (JSONException e) {
-                e.printStackTrace();
-              }
+      res -> {
+        final JSObject ret = new JSObject();
+        Iterator<String> keys = res.keys();
+        while (keys.hasNext()) {
+          String key = keys.next();
+          if (res.has(key)) {
+            try {
+              ret.put(key, res.get(key));
+            } catch (JSONException e) {
+              e.printStackTrace();
             }
           }
-          Log.i(TAG, "Channel get to \"" + ret);
-          callback.callback(ret);
         }
+        Log.i(TAG, "Channel get to \"" + ret);
+        callback.callback(ret);
       },
-      new Response.ErrorListener() {
-        @Override
-        public void onErrorResponse(VolleyError error) {
-          callback.callback(
-            CapacitorUpdater.this.createError("Error get channel", error)
-          );
-        }
-      }
+      error ->
+        callback.callback(
+          CapacitorUpdater.this.createError("Error get channel", error)
+        )
     );
     this.requestQueue.add(setRetryPolicy(request));
   }
@@ -1068,10 +1011,10 @@ public class CapacitorUpdater {
 
   public void sendStats(final String action, final String versionName) {
     String statsUrl = this.statsUrl;
-    if (statsUrl == null || "".equals(statsUrl) || statsUrl.length() == 0) {
+    if (statsUrl == null || statsUrl.isEmpty()) {
       return;
     }
-    JSONObject json = null;
+    JSONObject json;
     try {
       json = this.createInfoObject();
       json.put("action", action);
@@ -1085,21 +1028,9 @@ public class CapacitorUpdater {
       Request.Method.POST,
       statsUrl,
       json,
-      new Response.Listener<JSONObject>() {
-        @Override
-        public void onResponse(JSONObject response) {
-          Log.i(
-            TAG,
-            "Stats send for \"" + action + "\", version " + versionName
-          );
-        }
-      },
-      new Response.ErrorListener() {
-        @Override
-        public void onErrorResponse(VolleyError error) {
-          CapacitorUpdater.this.createError("Error send stats", error);
-        }
-      }
+      response ->
+        Log.i(TAG, "Stats send for \"" + action + "\", version " + versionName),
+      error -> CapacitorUpdater.this.createError("Error send stats", error)
     );
     this.requestQueue.add(setRetryPolicy(request));
   }
@@ -1161,14 +1092,6 @@ public class CapacitorUpdater {
     this.editor.commit();
   }
 
-  public void setVersionName(final String id, final String name) {
-    if (id != null) {
-      Log.d(TAG, "Setting name for bundle [" + id + "] to " + name);
-      BundleInfo info = this.getBundleInfo(id);
-      this.saveBundleInfo(id, info.setVersionName(name));
-    }
-  }
-
   private void setBundleStatus(final String id, final BundleStatus status) {
     if (id != null && status != null) {
       BundleInfo info = this.getBundleInfo(id);
@@ -1192,7 +1115,7 @@ public class CapacitorUpdater {
 
   public String getCurrentBundlePath() {
     String path = this.prefs.getString(WebView.CAP_SERVER_PATH, "public");
-    if ("".equals(path.trim())) {
+    if (path.trim().isEmpty()) {
       return "public";
     }
     return path;

package/android/src/main/java/ee/forgr/capacitor_updater/CapacitorUpdaterPlugin.java

@@ -8,7 +8,6 @@ package ee.forgr.capacitor_updater;
 
 import android.app.Activity;
 import android.app.ActivityManager;
-import android.app.Application;
 import android.content.Context;
 import android.content.SharedPreferences;
 import android.content.pm.PackageInfo;
@@ -36,6 +35,7 @@ import java.util.ArrayList;
 import java.util.Date;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Objects;
 import java.util.Timer;
 import java.util.TimerTask;
 import java.util.UUID;
@@ -55,7 +55,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
   private static final String channelUrlDefault =
     "https://api.capgo.app/channel_self";
 
-  private final String PLUGIN_VERSION = "5.7.4";
+  private final String PLUGIN_VERSION = "5.7.9";
   private static final String DELAY_CONDITION_PREFERENCES = "";
 
   private SharedPreferences.Editor editor;
@@ -70,7 +70,6 @@ public class CapacitorUpdaterPlugin extends Plugin {
   private Boolean autoUpdate = false;
   private String updateUrl = "";
   private Version currentVersionNative;
-  private Boolean resetWhenUpdate = true;
   private Thread backgroundTask;
   private Boolean taskRunning = false;
 
@@ -167,7 +166,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
     this.implementation.appId =
       this.getConfig().getString("appId", this.implementation.appId);
     if (
-      this.implementation.appId == null || "".equals(this.implementation.appId)
+      this.implementation.appId == null || this.implementation.appId.isEmpty()
     ) {
       // crash the app
       throw new RuntimeException(
@@ -215,32 +214,29 @@ public class CapacitorUpdaterPlugin extends Plugin {
     this.appReadyTimeout = this.getConfig().getInt("appReadyTimeout", 10000);
     this.implementation.timeout =
       this.getConfig().getInt("responseTimeout", 20) * 1000;
-    this.resetWhenUpdate = this.getConfig().getBoolean("resetWhenUpdate", true);
+    boolean resetWhenUpdate =
+      this.getConfig().getBoolean("resetWhenUpdate", true);
 
-    if (this.resetWhenUpdate) {
+    if (resetWhenUpdate) {
       this.cleanupObsoleteVersions();
     }
-    final Application application = (Application) this.getContext()
-      .getApplicationContext();
 
     this.checkForUpdateAfterDelay();
-    //    application.registerActivityLifecycleCallbacks(this);
   }
 
   private void semaphoreWait(Number waitTime) {
     Log.i(CapacitorUpdater.TAG, "semaphoreWait " + waitTime);
     try {
       //        Log.i(CapacitorUpdater.TAG, "semaphoreReady count " + CapacitorUpdaterPlugin.this.semaphoreReady.getCount());
-      CapacitorUpdaterPlugin.this.semaphoreReady.awaitAdvanceInterruptibly(
-          CapacitorUpdaterPlugin.this.semaphoreReady.getPhase(),
-          waitTime.longValue(),
-          TimeUnit.SECONDS
-        );
+      semaphoreReady.awaitAdvanceInterruptibly(
+        semaphoreReady.getPhase(),
+        waitTime.longValue(),
+        TimeUnit.SECONDS
+      );
       //        Log.i(CapacitorUpdater.TAG, "semaphoreReady await " + res);
       Log.i(
         CapacitorUpdater.TAG,
-        "semaphoreReady count " +
-        CapacitorUpdaterPlugin.this.semaphoreReady.getPhase()
+        "semaphoreReady count " + semaphoreReady.getPhase()
       );
     } catch (InterruptedException e) {
       Log.i(CapacitorUpdater.TAG, "semaphoreWait InterruptedException");
@@ -252,17 +248,16 @@ public class CapacitorUpdaterPlugin extends Plugin {
 
   private void semaphoreUp() {
     Log.i(CapacitorUpdater.TAG, "semaphoreUp");
-    CapacitorUpdaterPlugin.this.semaphoreReady.register();
+    semaphoreReady.register();
   }
 
   private void semaphoreDown() {
     Log.i(CapacitorUpdater.TAG, "semaphoreDown");
     Log.i(
       CapacitorUpdater.TAG,
-      "semaphoreDown count " +
-      CapacitorUpdaterPlugin.this.semaphoreReady.getPhase()
+      "semaphoreDown count " + semaphoreReady.getPhase()
     );
-    CapacitorUpdaterPlugin.this.semaphoreReady.arriveAndDeregister();
+    semaphoreReady.arriveAndDeregister();
   }
 
   private void sendReadyToJs(final BundleInfo current, final String msg) {
@@ -292,8 +287,10 @@ public class CapacitorUpdaterPlugin extends Plugin {
       try {
         if (
           !"".equals(previous.getOriginalString()) &&
-          !this.currentVersionNative.getOriginalString()
-            .equals(previous.getOriginalString())
+          !Objects.equals(
+            this.currentVersionNative.getOriginalString(),
+            previous.getOriginalString()
+          )
         ) {
           Log.i(
             CapacitorUpdater.TAG,
@@ -483,7 +480,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
             } else {
               if (
                 CapacitorUpdaterPlugin.this._isAutoUpdateEnabled() &&
-                triggerAutoUpdate
+                Boolean.TRUE.equals(triggerAutoUpdate)
               ) {
                 Log.i(
                   CapacitorUpdater.TAG,
@@ -528,7 +525,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
               } else {
                 if (
                   CapacitorUpdaterPlugin.this._isAutoUpdateEnabled() &&
-                  triggerAutoUpdate
+                  Boolean.TRUE.equals(triggerAutoUpdate)
                 ) {
                   Log.i(
                     CapacitorUpdater.TAG,
@@ -606,7 +603,12 @@ public class CapacitorUpdaterPlugin extends Plugin {
           final JSObject ret = new JSObject();
           ret.put("version", version);
           CapacitorUpdaterPlugin.this.notifyListeners("downloadFailed", ret);
-          CapacitorUpdaterPlugin.this.implementation.sendStats("download_fail");
+          final BundleInfo current =
+            CapacitorUpdaterPlugin.this.implementation.getCurrentBundle();
+          CapacitorUpdaterPlugin.this.implementation.sendStats(
+              "download_fail",
+              current.getVersionName()
+            );
         }
       });
     } catch (final Exception e) {
@@ -615,7 +617,12 @@ public class CapacitorUpdaterPlugin extends Plugin {
       final JSObject ret = new JSObject();
       ret.put("version", version);
       CapacitorUpdaterPlugin.this.notifyListeners("downloadFailed", ret);
-      CapacitorUpdaterPlugin.this.implementation.sendStats("download_fail");
+      final BundleInfo current =
+        CapacitorUpdaterPlugin.this.implementation.getCurrentBundle();
+      CapacitorUpdaterPlugin.this.implementation.sendStats(
+          "download_fail",
+          current.getVersionName()
+        );
     }
   }
 
@@ -822,9 +829,6 @@ public class CapacitorUpdaterPlugin extends Plugin {
     if (this.periodCheckDelay == 0 || !this._isAutoUpdateEnabled()) {
       return;
     }
-    // The delay should be equal to this.periodCheckDelay.
-    // In appMovedToForeground we schedule a backgroundDownload and it happens BEFORE this scheduleAtFixedRate
-    // As such we get a race condition
     final Timer timer = new Timer();
     timer.scheduleAtFixedRate(
       new TimerTask() {
@@ -835,14 +839,17 @@ public class CapacitorUpdaterPlugin extends Plugin {
                 CapacitorUpdaterPlugin.this.updateUrl,
                 res -> {
                   if (res.has("error")) {
-                    Log.e(CapacitorUpdater.TAG, res.getString("error"));
+                    Log.e(
+                      CapacitorUpdater.TAG,
+                      Objects.requireNonNull(res.getString("error"))
+                    );
                     return;
                   } else if (res.has("version")) {
                     String newVersion = res.getString("version");
                     String currentVersion = String.valueOf(
                       CapacitorUpdaterPlugin.this.implementation.getCurrentBundle()
                     );
-                    if (!newVersion.equals(currentVersion)) {
+                    if (!Objects.equals(newVersion, currentVersion)) {
                       Log.i(
                         CapacitorUpdater.TAG,
                         "New version found: " + newVersion
@@ -857,7 +864,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
           }
         }
       },
-      this.periodCheckDelay,
+      0,
       this.periodCheckDelay
     );
   }
@@ -967,7 +974,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
     for (DelayCondition condition : delayConditionList) {
       String kind = condition.getKind().toString();
       String value = condition.getValue();
-      if (!"".equals(kind)) {
+      if (!kind.isEmpty()) {
         switch (kind) {
           case "background":
             if (!killed) {
@@ -1020,7 +1027,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
   private Boolean _isAutoUpdateEnabled() {
     final CapConfig config = CapConfig.loadDefault(this.getActivity());
     String serverUrl = config.getServerUrl();
-    if (serverUrl != null && !"".equals(serverUrl)) {
+    if (serverUrl != null && !serverUrl.isEmpty()) {
       // log warning autoupdate disabled when serverUrl is set
       Log.w(
         CapacitorUpdater.TAG,
@@ -1030,7 +1037,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
     return (
       CapacitorUpdaterPlugin.this.autoUpdate &&
       !"".equals(CapacitorUpdaterPlugin.this.updateUrl) &&
-      (serverUrl == null || "".equals(serverUrl))
+      (serverUrl == null || serverUrl.isEmpty())
     );
   }
 
@@ -1077,7 +1084,10 @@ public class CapacitorUpdaterPlugin extends Plugin {
     Boolean error
   ) {
     if (error) {
-      Log.i(CapacitorUpdater.TAG, "endBackGroundTaskWithNotif error" + error);
+      Log.i(
+        CapacitorUpdater.TAG,
+        "endBackGroundTaskWithNotif error" + error.toString()
+      );
       this.implementation.sendStats("download_fail", current.getVersionName());
       final JSObject ret = new JSObject();
       ret.put("version", latestVersionName);
@@ -1149,7 +1159,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
 
               if (
                 latestVersionName != null &&
-                !"".equals(latestVersionName) &&
+                !latestVersionName.isEmpty() &&
                 !current.getVersionName().equals(latestVersionName)
               ) {
                 final BundleInfo latest =
@@ -1308,7 +1318,7 @@ public class CapacitorUpdaterPlugin extends Plugin {
         delayUpdatePreferences,
         type
       );
-      if (delayConditionList != null && delayConditionList.size() != 0) {
+      if (delayConditionList != null && !delayConditionList.isEmpty()) {
         Log.i(CapacitorUpdater.TAG, "Update delayed to next backgrounding");
         return;
       }
@@ -1442,8 +1452,11 @@ public class CapacitorUpdaterPlugin extends Plugin {
   }
 
   public void appMovedToBackground() {
+    final BundleInfo current =
+      CapacitorUpdaterPlugin.this.implementation.getCurrentBundle();
     CapacitorUpdaterPlugin.this.implementation.sendStats(
-        "app_moved_to_background"
+        "app_moved_to_background",
+        current.getVersionName()
       );
     Log.i(CapacitorUpdater.TAG, "Checking for pending update");
     try {
@@ -1504,10 +1517,12 @@ public class CapacitorUpdaterPlugin extends Plugin {
       ActivityManager.RecentTaskInfo runningTask = runningTasks
         .get(0)
         .getTaskInfo();
-      String className = runningTask.baseIntent.getComponent().getClassName();
+      String className = Objects
+        .requireNonNull(runningTask.baseIntent.getComponent())
+        .getClassName();
+      assert runningTask.topActivity != null;
       String runningActivity = runningTask.topActivity.getClassName();
-      boolean isThisAppActivity = className.equals(runningActivity);
-      return isThisAppActivity;
+      return className.equals(runningActivity);
     } catch (NullPointerException e) {
       return false;
     }

package/ios/Plugin/CapacitorUpdater.swift

@@ -950,12 +950,6 @@ extension CustomError: LocalizedError {
         UserDefaults.standard.synchronize()
     }
 
-    public func setVersionName(id: String, version: String) {
-        print("\(self.TAG) Setting version for folder [\(id)] to \(version)")
-        let info = self.getBundleInfo(id: id)
-        self.saveBundleInfo(id: id, bundle: info.setVersionName(version: version))
-    }
-
     private func setBundleStatus(id: String, status: BundleStatus) {
         print("\(self.TAG) Setting status for bundle [\(id)] to \(status)")
         let info = self.getBundleInfo(id: id)

package/ios/Plugin/CapacitorUpdaterPlugin.swift

@@ -15,7 +15,7 @@ import Version
 @objc(CapacitorUpdaterPlugin)
 public class CapacitorUpdaterPlugin: CAPPlugin {
     public var implementation = CapacitorUpdater()
-    private let PLUGIN_VERSION: String = "5.7.4"
+    private let PLUGIN_VERSION: String = "5.7.9"
     static let updateUrlDefault = "https://api.capgo.app/updates"
     static let statsUrlDefault = "https://api.capgo.app/stats"
     static let channelUrlDefault = "https://api.capgo.app/channel_self"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@capgo/capacitor-updater",
-  "version": "5.7.4",
+  "version": "5.7.9",
   "packageManager": "pnpm@8.15.4",
   "license": "MPL-2.0",
   "description": "Live update for capacitor apps",