status-im
/
react-native
mirror of https://github.com/status-im/react-native.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Make ReactDatabaseSupplier use application context 

Summary: public
We pass in a `ReactContext` but we really only need a context. Make sure we're using the application one.

Reviewed By: astreet

Differential Revision: D2690692

fb-gh-sync-id: 857d6571c9c01d35e12f09be4c8733cca007306f
This commit is contained in:
Alexander Blom 2015-11-24 09:46:58 -08:00 committed by facebook-github-bot-7
parent 3a00545bc7
commit 5387df8dc5
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
ReactAndroid/src/main/java/com/facebook/react/modules/storage/ReactDatabaseSupplier.java
View File

@ -42,9 +42,10 @@ public class ReactDatabaseSupplier extends SQLiteOpenHelper {
VALUE_COLUMN + " TEXT NOT NULL" +
")";
private static @Nullable ReactDatabaseSupplier sReactDatabaseSupplierInstance;
private Context mContext;
private @Nullable SQLiteDatabase mDb;
private static @Nullable ReactDatabaseSupplier mReactDatabaseSupplierInstance;
private ReactDatabaseSupplier(Context context) {
super(context, DATABASE_NAME, null, DATABASE_VERSION);
@ -52,10 +53,10 @@ public class ReactDatabaseSupplier extends SQLiteOpenHelper {
}
public static ReactDatabaseSupplier getInstance(Context context) {
if (mReactDatabaseSupplierInstance == null) {
mReactDatabaseSupplierInstance = new ReactDatabaseSupplier(context);
if (sReactDatabaseSupplierInstance == null) {
sReactDatabaseSupplierInstance = new ReactDatabaseSupplier(context.getApplicationContext());
}
return mReactDatabaseSupplierInstance;
return sReactDatabaseSupplierInstance;
}
@Override
@ -150,6 +151,6 @@ public class ReactDatabaseSupplier extends SQLiteOpenHelper {
// For testing purposes only!
public static void deleteInstance() {
mReactDatabaseSupplierInstance = null;
sReactDatabaseSupplierInstance = null;
}
}