Merge pull request #89 from mikelambert/master

Stop passing getApplicationContext() in to the constructor

Author: auser

README.md

@@ -149,7 +149,7 @@ public class MainApplication extends Application implements ReactApplication {
     protected List<ReactPackage> getPackages() {
       return Arrays.<ReactPackage>asList(
           new MainReactPackage(),
-            new FirestackPackage(getApplicationContext())
+            new FirestackPackage()
       );
     }
   };

android/src/main/java/io/fullstack/firestack/FirestackPackage.java

@@ -15,8 +15,7 @@
 public class FirestackPackage implements ReactPackage {
     private Context mContext;
 
-    public FirestackPackage(Context ctx) {
-        mContext = ctx;
+    public FirestackPackage() {
     }
     /**
      * @param reactContext react application context that can be used to create modules
@@ -26,7 +25,7 @@ public FirestackPackage(Context ctx) {
     public List<NativeModule> createNativeModules(ReactApplicationContext reactContext) {
         List<NativeModule> modules = new ArrayList<>();
 
-        modules.add(new FirestackModule(reactContext, mContext));
+        modules.add(new FirestackModule(reactContext, reactContext.getBaseContext()));
         modules.add(new FirestackAuthModule(reactContext));
         modules.add(new FirestackDatabaseModule(reactContext));
         modules.add(new FirestackAnalyticsModule(reactContext));

package.json

@@ -47,7 +47,7 @@
       "project": "ios/Firestack.xcodeproj"
     },
     "android": {
-      "packageInstance": "new FirestackPackage(getApplicationContext())"
+      "packageInstance": "new FirestackPackage()"
     }
   },
   "devDependencies": {