fix(geolocation): fix location update bug

Author: akhenda

package-lock.json

@@ -1,21 +1,24 @@
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
-import { View, PermissionsAndroid, Platform } from 'react-native';
 import { connect } from 'react-redux';
 import { Container } from 'native-base';
 import Radar from 'react-native-radar';
 import { Actions } from 'react-native-router-flux';
 import RNAndroidLocationEnabler from 'react-native-android-location-enabler';
+import {
+  View, PermissionsAndroid, Platform, StatusBar,
+} from 'react-native';
 
-import shallowCompare from 'src/utils/shallowCompare';
-import { updateLocation } from 'src/state/actions/app';
-import { getCategories, selectCategory, clearListings } from 'src/state/actions/listings';
 import SearchBar from 'src/components/SearchBar';
 import Salutation from 'src/components/Salutation';
+import shallowCompare from 'src/utils/shallowCompare';
+import { updateLocation } from 'src/state/actions/app';
 import CategoriesList from 'src/components/CategoriesList';
 import LoadingIndicator from 'src/components/LoadingIndicator';
 import AnimatedContentWrapper from 'src/components/AnimatedContentWrapper';
+import { getCategories, selectCategory, clearListings } from 'src/state/actions/listings';
 
+import { colors } from 'src/theme';
 import styles from './styles/HomeScreenStyles';
 
 
@@ -32,47 +35,50 @@ class HomeScreen extends Component {
 
   componentDidMount() {
     let granted;
-    this.props.getCategories(this.props.token);
+    const { user, loading, catLoading } = this.props;
 
-    const locationSetup = () => {
-      // identify the user and request permissions
-      const { user } = this.props;
-      const description = user.name || user.username;
-
-      Radar.setUserId(String(user.id));
-      Radar.setDescription(description);
-      Radar.requestPermissions(false);
-
-      // track the user's location once in the foreground
-      Radar.trackOnce().then((result) => {
-        // do something with result.events, result.user.geofences
-        this.props.updateLocation(result.location);
-      }).catch(() => {
-        // optionally, do something with err
-      });
-
-      Radar.on('location', (result) => {
-        this.props.updateLocation(result.location);
-      });
-    };
+    this.props.getCategories(this.props.token);
 
-    if (Platform.OS === 'android') {
-      RNAndroidLocationEnabler
-        .promptForEnableLocationIfNeeded({ interval: 7000, fastInterval: 3000 })
-        .then(async () => {
-          // success => {alreadyEnabled: true, enabled: true, status: 'enabled'}
-          granted = await PermissionsAndroid
-            .request(PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION, {
-              title: 'WAPI? App needs to access your location 📌',
-              message: 'We need to access to your location '
-              + 'so that we can personalize your experience. 🙂',
-            });
-          if (granted && Object.keys(this.props.user) > 0) locationSetup();
-        }).catch(() => {});
+    if (!loading && !catLoading) {
+      const locationSetup = () => {
+        // identify the user and request permissions
+        const description = user.name || user.username;
+
+        Radar.setUserId(String(user.id));
+        Radar.setDescription(description);
+        Radar.requestPermissions(false);
+
+        // track the user's location once in the foreground
+        Radar.trackOnce().then((result) => {
+          // do something with result.events, result.user.geofences
+          this.props.updateLocation(result.location);
+        }).catch(() => {
+          // optionally, do something with err
+        });
+
+        Radar.on('location', (result) => {
+          this.props.updateLocation(result.location);
+        });
+      };
+
+      if (Platform.OS === 'android') {
+        RNAndroidLocationEnabler
+          .promptForEnableLocationIfNeeded({ interval: 7000, fastInterval: 3000 })
+          .then(async () => {
+            // success => {alreadyEnabled: true, enabled: true, status: 'enabled'}
+            granted = await PermissionsAndroid
+              .request(PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION, {
+                title: 'WAPI? App needs to access your location 📌',
+                message: 'We need to access to your location '
+                + 'so that we can personalize your experience. 🙂',
+              });
+            if (granted && Object.keys(this.props.user).length > 0) locationSetup();
+          }).catch(() => {});
+      }
+
+      if (Platform.OS === 'ios' && Object.keys(this.props.user).length > 0) locationSetup();
     }
 
-    if (Platform.OS === 'ios' && Object.keys(this.props.user) > 0) locationSetup();
-
     this.props.clearListings();
   }
 
@@ -116,6 +122,7 @@ class HomeScreen extends Component {
 
     return (
       <Container style={styles.container}>
+        <StatusBar translucent barStyle="light-content" backgroundColor={colors.statusBarTranslucent} />
         <AnimatedContentWrapper
           headerTitle="Discover"
           showToolbarRightButton={false}

src/containers/HomeScreen.js

@@ -25,7 +25,10 @@ const INITIAL_STATE = {
     shopping: false,
     volunteering: false,
   },
-  currentLocation: null,
+  currentLocation: {
+    latitude: 0,
+    longitude: 0,
+  },
   favourites: [],
 };