Skip to content

Commit

Permalink
Merge pull request #2250 from Expensify/marcaaron-fixPersonalDetailsC…
Browse files Browse the repository at this point in the history
…rash

Initialize current user's personal details when personalDetailsList is non empty and they can't be found
  • Loading branch information
Jag96 authored Apr 7, 2021
2 parents c9f106a + 9c508d8 commit 3a83195
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions src/libs/actions/PersonalDetails.js
Original file line number Diff line number Diff line change
Expand Up @@ -110,10 +110,13 @@ function fetch() {
.then((data) => {
let myPersonalDetails = {};

// If personalDetailsList is empty, ensure we set the personal details for the current user
const personalDetailsList = _.isEmpty(data.personalDetailsList)
? {[currentUserEmail]: myPersonalDetails}
: data.personalDetailsList;
// If personalDetailsList does not have the current user ensure we initialize their details with an empty
// object at least
const personalDetailsList = _.isEmpty(data.personalDetailsList) ? {} : data.personalDetailsList;
if (!personalDetailsList[currentUserEmail]) {
personalDetailsList[currentUserEmail] = {};
}

const allPersonalDetails = formatPersonalDetails(personalDetailsList);
Onyx.merge(ONYXKEYS.PERSONAL_DETAILS, allPersonalDetails);

Expand Down

0 comments on commit 3a83195

Please sign in to comment.