Merge pull request #96 from OP-Engineering/osp/fixes-for-moveAssetsDa…

…tabase Minor fixes for moveAssetsDatabase
OP-Engineering · May 13, 2024 · ca4cb31 · ca4cb31
2 parents c6aa5f0 + b07c01d
commit ca4cb31
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/android/src/main/java/com/op/sqlite/OPSQLiteModule.kt b/android/src/main/java/com/op/sqlite/OPSQLiteModule.kt
@@ -59,10 +59,7 @@ internal class OPSQLiteModule(context: ReactApplicationContext?) : ReactContextB
         val assetsManager = context.assets
 
         try {
-            // Open the input stream for the asset file
-            val inputStream: InputStream = assetsManager.open("$path/$filename")
 
-            // Create the output file in the documents directory
             val databasesFolder =
                     context.getDatabasePath("defaultDatabase")
                             .absolutePath
@@ -79,6 +76,8 @@ internal class OPSQLiteModule(context: ReactApplicationContext?) : ReactContextB
                 }
             }
 
+            val inputStream: InputStream = assetsManager.open("$path/$filename")
+
             // Open the output stream for the output file
             val outputStream: OutputStream = FileOutputStream(outputFile)
 

diff --git a/src/index.ts b/src/index.ts
@@ -489,11 +489,11 @@ export const open = (options: {
   };
 };
 
-export const moveAssetsDatabase = (args: {
+export const moveAssetsDatabase = async (args: {
   filename: string;
   path?: string;
   overwrite?: boolean;
-}): boolean => {
+}): Promise<boolean> => {
   return NativeModules.OPSQLite.moveAssetsDatabase(args);
 };