feat: hotfix for not populating surveyor names

hopetambala · Dec 10, 2020 · ae0103b · ae0103b
1 parent 907a6ec
commit ae0103b
Show file tree

Hide file tree

Showing 6 changed files with 25 additions and 23 deletions.
diff --git a/domains/Auth/SignIn/index.js b/domains/Auth/SignIn/index.js
@@ -130,12 +130,9 @@ const SignIn = ({ navigation }) => {
   };
 
   const storeUserInformation = async () => {
-    const currentUser = await retrieveCurrentUserAsyncFunction();
-    getData('organization').then((asyncOrg) => {
-      if (asyncOrg !== currentUser.organization) {
-        storeData(currentUser.organization, 'organization');
-        storeData(currentUser, 'currentUser');
-      }
+    await retrieveCurrentUserAsyncFunction().then((currentUser) => {
+      storeData(currentUser.organization, 'organization');
+      storeData(currentUser, 'currentUser');
     });
   };
 

diff --git a/domains/DataCollection/Forms/IdentificationForm/index.js b/domains/DataCollection/Forms/IdentificationForm/index.js
@@ -9,20 +9,20 @@ import { postObjectsToClass } from '../../../../services/parse/crud';
 
 import { storeData } from '../../../../modules/async-storage';
 import checkOnlineStatus from '../../../../modules/offline';
-import generateRandomID from '../../../../modules/utils';
+import { generateRandomID } from '../../../../modules/utils';
 import { layout } from '../../../../modules/theme';
 import I18n from '../../../../modules/i18n';
 
 import PaperButton from '../../../../components/Button';
 import PaperInputPicker from '../../../../components/FormikFields/PaperInputPicker';
 import yupValidationPicker from '../../../../components/FormikFields/YupValidation';
 
-import backgroundPostPatient from './utils';
+import { backgroundPostPatient, surveyingUserFailsafe } from './utils';
 import configArray from './config/config';
 
 const IdentificationForm = ({
   scrollViewScroll, setScrollViewScroll,
-  setSelectedForm, setSurveyee, surveyingOrganization, surveyingUser
+  setSelectedForm, setSurveyee, surveyingOrganization
 }) => {
   useEffect(() => {
     const interval = setInterval(() => {
@@ -49,12 +49,13 @@ const IdentificationForm = ({
       <TouchableWithoutFeedback onPress={Keyboard.dismiss()} accessible={false}>
         <Formik
           initialValues={{}}
-          onSubmit={(values, actions) => {
+          onSubmit={async (values, actions) => {
             setPhotoFile('Submitted Photo String');
 
             const formObject = values;
             formObject.surveyingOrganization = surveyingOrganization;
-            formObject.surveyingUser = surveyingUser;
+            formObject.surveyingUser = await surveyingUserFailsafe();
+
             formObject.latitude = values.location?.latitude || 0;
             formObject.longitude = values.location?.longitude || 0;
             formObject.altitude = values.location?.altitude || 0;

diff --git a/domains/DataCollection/Forms/IdentificationForm/utils/index.js b/domains/DataCollection/Forms/IdentificationForm/utils/index.js
@@ -30,4 +30,9 @@ const backgroundPostPatient = () => {
     });
 };
 
-export default backgroundPostPatient;
+const surveyingUserFailsafe = async () => {
+  const user = await getData('currentUser');
+  return user.username || user.email || user.id || user.firstname || user.lastname;
+};
+
+export { backgroundPostPatient, surveyingUserFailsafe };
diff --git a/domains/DataCollection/Forms/SupplementaryForm/index.js b/domains/DataCollection/Forms/SupplementaryForm/index.js
@@ -112,14 +112,14 @@ const SupplementaryForm = ({
           {formikProps.isSubmitting ? (
             <ActivityIndicator />
           ) : (
-              <Button
-                disabled={!surveyee.objectId}
-                onPress={formikProps.handleSubmit}
-              >
-                {surveyee.objectId && <Text>{I18n.t('global.submit')}</Text>}
-                {!surveyee.objectId && <Text>{I18n.t('supplementaryForms.attachResident')}</Text>}
-              </Button>
-            )}
+            <Button
+              disabled={!surveyee.objectId}
+              onPress={formikProps.handleSubmit}
+            >
+              {surveyee.objectId && <Text>{I18n.t('global.submit')}</Text>}
+              {!surveyee.objectId && <Text>{I18n.t('supplementaryForms.attachResident')}</Text>}
+            </Button>
+          )}
         </View>
       )}
     </Formik>

diff --git a/domains/DataCollection/index.js b/domains/DataCollection/index.js
@@ -52,8 +52,6 @@ const DataCollection = ({ navigation }) => {
   useEffect(() => {
     getData('currentUser').then((user) => {
       setSurveyingUser(`${user.firstname || ''} ${user.lastname || ''}`);
-    }).catch(() => {
-      setSurveyingUser(`${''} ${''}`);
     });
 
     getData('organization').then((org) => {

diff --git a/modules/utils/index.js b/modules/utils/index.js
@@ -1,3 +1,4 @@
 const generateRandomID = () => Math.random().toString(20).substr(2, 12);
+const isEmpty = (str) => (!str || str.length === 0);
 
-export default generateRandomID;
+export { isEmpty, generateRandomID };