Consolidate loops in address book migration

MetaMask · Sep 21, 2023 · 3570cfe · 3570cfe
1 parent 42ba925
commit 3570cfe
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/app/store/migrations.js b/app/store/migrations.js
@@ -671,13 +671,10 @@ export const migrations = {
           (chainId) => localChainIds.has(chainId),
         );
 
-        if (recognizedChainIdCandidates.length > 1) {
-          for (const chainId of recognizedChainIdCandidates) {
+        for (const chainId of recognizedChainIdCandidates) {
+          if (recognizedChainIdCandidates.length > 1) {
             ambiguousAddressEntries[chainId] = Object.keys(addressEntries);
           }
-        }
-
-        for (const chainId of recognizedChainIdCandidates) {
           migratedAddressBook[chainId] = mapValues(addressEntries, (entry) => ({
             ...entry,
             chainId,