[Hotfix] Fix empty row bug in balances (#1646)

* add filterAndFillTokenDetails fn to fill tokens

1. fixes empty row bug

* PR fixes + fill in addressMainnet when on chain 1

src/services/factories/tokenList.ts

@@ -266,8 +266,12 @@ export function getTokensFactory(factoryParams: {
     }
   }
 
-  async function updateTokenDetails(networkId: number, numTokens: number): Promise<void> {
-    const tokensConfig = tokenListApi.getTokens(networkId)
+  async function updateTokenDetails(
+    networkId: number,
+    numTokens: number,
+    tokensConfig: TokenDetails[] = [],
+  ): Promise<void> {
+    if (tokensConfig.length === 0) tokensConfig = tokenListApi.getTokens(networkId)
 
     // Get filtered ids and addresses
     const filteredAddressesAndIds = await _getFilteredIdsMap(networkId, numTokens, tokensConfig)
@@ -284,6 +288,36 @@ export function getTokensFactory(factoryParams: {
     tokenListApi.persistTokens({ networkId, tokenList: tokenDetails })
   }
 
+  async function filterAndFillTokenList(networkId: Network, tokensConfig: TokenDetails[]): Promise<TokenDetails[]> {
+    const tokensWithNoDetails: TokenDetails[] = []
+    const tokensWithDetails = tokensConfig.reduce<TokenDetails[]>((acc, token) => {
+      // fill in addressMainnet if not present and on Mainnet
+      if (networkId === Network.Mainnet && !token.addressMainnet) {
+        token.addressMainnet = token.address
+      }
+
+      if (!token.name || !token.symbol) {
+        tokensWithNoDetails.push(token)
+        return acc
+      }
+
+      acc.push(token)
+      return acc
+    }, [])
+
+    // bail out if no non-detailed tokens
+    if (tokensWithNoDetails.length === 0) return tokensWithDetails
+
+    const updatedTokens = await Promise.all(
+      tokensWithNoDetails.map(async (token) => ({
+        ...token,
+        ...(await getErc20DetailsOrAddress(networkId, token.address)),
+      })),
+    )
+
+    return tokensWithDetails.concat(updatedTokens)
+  }
+
   async function updateTokens(networkId: number): Promise<void> {
     areTokensUpdated.add(networkId)
 
@@ -293,16 +327,19 @@ export function getTokensFactory(factoryParams: {
       tokenListApi.setListReady(false)
 
       const numTokens = await retry(() => exchangeApi.getNumTokens(networkId))
-      const tokens = tokenListApi.getTokens(networkId)
+      const preTokens = tokenListApi.getTokens(networkId)
+
+      logDebug(`[tokenListFactory][${networkId}] Contract has ${numTokens}; local list has ${preTokens.length}`)
 
-      logDebug(`[tokenListFactory][${networkId}] Contract has ${numTokens}; local list has ${tokens.length}`)
+      // Check token list for any tokens with missing details and fill if so
+      const tokens = await filterAndFillTokenList(networkId, preTokens)
 
       // TODO: review this logic
       // why update tokenDetails when token list lengths don't match?
       // why would token IDs not also need to be updated?
       if (numTokens > tokens.length) {
         // When there are more tokens in the contract than locally, fetch the new tokens
-        await updateTokenDetails(networkId, numTokens)
+        await updateTokenDetails(networkId, numTokens, tokens)
       } else {
         // Otherwise, only update the ids
         await updateTokenIds(networkId, tokens)