improvements

Author: Krzysztof Borowy

android/src/main/java/com/reactnativecommunity/asyncstorage/AsyncLocalStorageUtil.java

@@ -155,27 +155,29 @@ private static void deepMergeInto(JSONObject oldJSON, JSONObject newJSON)
    * a "checkpoint" and is done automatically (by default) when the WAL file reaches a threshold
    * size of 1000 pages.
    * More here: https://sqlite.org/wal.html
-   * <p>
+   *
    * This helper will force checkpoint on RKStorage, if Next storage file does not exists yet.
    */
   public static void verifyAndForceSqliteCheckpoint(Context ctx) {
+    if(Build.VERSION.SDK_INT < Build.VERSION_CODES.P) {
+        Log.i("AsyncStorage_Next", "SQLite checkpoint not required on this API version.");
+    }
+
     File nextStorageFile = ctx.getDatabasePath("AsyncStorage");
     File currentStorageFile = ctx.getDatabasePath(ReactDatabaseSupplier.DATABASE_NAME);
-    boolean isCheckpointRequired = Build.VERSION.SDK_INT >= Build.VERSION_CODES.P
-            && !nextStorageFile.exists()
-            && currentStorageFile.exists();
+    boolean isCheckpointRequired = !nextStorageFile.exists() && currentStorageFile.exists();
     if (!isCheckpointRequired) {
       Log.i("AsyncStorage_Next", "SQLite checkpoint not required.");
       return;
     }
 
     try {
       ReactDatabaseSupplier supplier = ReactDatabaseSupplier.getInstance(ctx);
-      supplier.get().execSQL("PRAGMA wal_checkpoint");
+      supplier.get().rawQuery("PRAGMA wal_checkpoint", null).close();
       supplier.closeDatabase();
       Log.i("AsyncStorage_Next", "Forcing SQLite checkpoint successful.");
     } catch (Exception e) {
-      Log.w("AsyncStorage_Next", "Could not force checkpoint on RKStorage: " + e.getMessage());
+      Log.w("AsyncStorage_Next", "Data migration to Next storage failed.\n" + "Could not force checkpoint on RKStorage: " + e.getMessage());
     }
   }
 }

android/src/main/java/com/reactnativecommunity/asyncstorage/next/StorageSupplier.kt

@@ -1,6 +1,7 @@
 package com.reactnativecommunity.asyncstorage.next
 
 import android.content.Context
+import android.util.Log
 import androidx.room.ColumnInfo
 import androidx.room.Dao
 import androidx.room.Database
@@ -100,6 +101,7 @@ private object MIGRATION_TO_NEXT : Migration(1, 2) {
             FROM $oldTableName;
         """.trimIndent()
         )
+        Log.e("AsyncStorage_Next", "Migration to Next storage completed.")
     }
 }