Fix: fix after merge

Author: salimtb

ui/components/app/assets/asset-list/asset-list.tsx

@@ -7,6 +7,7 @@ import {
   getAllDetectedTokensForSelectedAddress,
   getDetectedTokensInCurrentNetwork,
   getIstokenDetectionInactiveOnNonMainnetSupportedNetwork,
+  getNetworkConfigurationsByChainId,
   getPreferences,
   getSelectedAccount,
 } from '../../../../selectors';
@@ -77,8 +78,17 @@ const AssetList = ({ onClickAsset, showTokensLinks }: AssetListProps) => {
   const isTokenDetectionInactiveOnNonMainnetSupportedNetwork = useSelector(
     getIstokenDetectionInactiveOnNonMainnetSupportedNetwork,
   );
+
+  const allNetworks = useSelector(getNetworkConfigurationsByChainId);
   const { tokenNetworkFilter } = useSelector(getPreferences);
-  const allNetworksFilterShown = Object.keys(tokenNetworkFilter ?? {}).length;
+  const allOpts: Record<string, boolean> = {};
+  Object.keys(allNetworks || {}).forEach((chainId) => {
+    allOpts[chainId] = true;
+  });
+
+  const allNetworksFilterShown =
+    Object.keys(tokenNetworkFilter || {}).length !==
+    Object.keys(allOpts || {}).length;
 
   const [showFundingMethodModal, setShowFundingMethodModal] = useState(false);
   const [showReceiveModal, setShowReceiveModal] = useState(false);

ui/components/app/detected-token/detected-token-selection-popover/detected-token-selection-popover.js

@@ -14,6 +14,7 @@ import {
   getCurrentChainId,
   getCurrentNetwork,
   getDetectedTokensInCurrentNetwork,
+  getNetworkConfigurationsByChainId,
   getPreferences,
 } from '../../../../selectors';
 
@@ -37,8 +38,16 @@ const DetectedTokenSelectionPopover = ({
   const chainId = useSelector(getCurrentChainId);
 
   const detectedTokens = useSelector(getDetectedTokensInCurrentNetwork);
+  const allNetworks = useSelector(getNetworkConfigurationsByChainId);
   const { tokenNetworkFilter } = useSelector(getPreferences);
-  const allNetworksFilterShown = Object.keys(tokenNetworkFilter ?? {}).length;
+  const allOpts = {};
+  Object.keys(allNetworks || {}).forEach((networkId) => {
+    allOpts[networkId] = true;
+  });
+
+  const allNetworksFilterShown =
+    Object.keys(tokenNetworkFilter || {}).length !==
+    Object.keys(allOpts || {}).length;
 
   const currentNetwork = useSelector(getCurrentNetwork);
 

ui/components/app/detected-token/detected-token.js

@@ -60,10 +60,16 @@ const DetectedToken = ({ setShowDetectedTokens }) => {
     getAllDetectedTokensForSelectedAddress,
   );
   const currentChainId = useSelector(getCurrentChainId);
+  const allNetworks = useSelector(getNetworkConfigurationsByChainId);
   const { tokenNetworkFilter } = useSelector(getPreferences);
-  const allNetworksFilterShown = Object.keys(tokenNetworkFilter ?? {}).length;
+  const allOpts = {};
+  Object.keys(allNetworks || {}).forEach((chainId) => {
+    allOpts[chainId] = true;
+  });
 
-  const configuration = useSelector(getNetworkConfigurationsByChainId);
+  const allNetworksFilterShown =
+    Object.keys(tokenNetworkFilter || {}).length !==
+    Object.keys(allOpts || {}).length;
 
   const totalDetectedTokens = useMemo(() => {
     return process.env.PORTFOLIO_VIEW && !allNetworksFilterShown
@@ -148,7 +154,7 @@ const DetectedToken = ({ setShowDetectedTokens }) => {
 
       const importPromises = Object.entries(tokensByChainId).map(
         async ([networkId, { tokens }]) => {
-          const chainConfig = configuration[networkId];
+          const chainConfig = allNetworks[networkId];
           const { defaultRpcEndpointIndex } = chainConfig;
           const { networkClientId: networkInstanceId } =
             chainConfig.rpcEndpoints[defaultRpcEndpointIndex];
@@ -202,7 +208,7 @@ const DetectedToken = ({ setShowDetectedTokens }) => {
 
       const promises = Object.entries(groupedByChainId).map(
         async ([chainId, tokens]) => {
-          const chainConfig = configuration[chainId];
+          const chainConfig = allNetworks[chainId];
           const { defaultRpcEndpointIndex } = chainConfig;
           const { networkClientId: networkInstanceId } =
             chainConfig.rpcEndpoints[defaultRpcEndpointIndex];

ui/components/multichain/detected-token-banner/detected-token-banner.js

@@ -10,6 +10,7 @@ import {
   getPreferences,
   getSelectedInternalAccount,
   getAllDetectedTokens,
+  getNetworkConfigurationsByChainId,
 } from '../../../selectors';
 import { MetaMetricsContext } from '../../../contexts/metametrics';
 import {
@@ -26,9 +27,16 @@ export const DetectedTokensBanner = ({
 }) => {
   const t = useI18nContext();
   const trackEvent = useContext(MetaMetricsContext);
+  const allNetworks = useSelector(getNetworkConfigurationsByChainId);
   const { tokenNetworkFilter } = useSelector(getPreferences);
+  const allOpts = {};
+  Object.keys(allNetworks || {}).forEach((chainId) => {
+    allOpts[chainId] = true;
+  });
 
-  const allNetworksFilterShown = Object.keys(tokenNetworkFilter ?? {}).length;
+  const allNetworksFilterShown =
+    Object.keys(tokenNetworkFilter || {}).length !==
+    Object.keys(allOpts || {}).length;
 
   const detectedTokens = useSelector(getDetectedTokensInCurrentNetwork);
   const { address: selectedAddress } = useSelector(getSelectedInternalAccount);
@@ -58,7 +66,7 @@ export const DetectedTokensBanner = ({
       ? Object.values(detectedTokensMultichain)
           .flat()
           .map(({ address, symbol }) => `${symbol} - ${address}`)
-      : detectedTokens.map(({ address, symbol }) => `${symbol} - ${address}`);
+      : detectedTokens?.map(({ address, symbol }) => `${symbol} - ${address}`);
 
   const totalTokens =
     process.env.PORTFOLIO_VIEW && !allNetworksFilterShown

ui/selectors/selectors.js

@@ -734,10 +734,6 @@ export const getMarketData = (state) => {
   return state.metamask.marketData;
 };
 
-export function getCurrencyRates(state) {
-  return state.metamask.currencyRates;
-}
-
 export function getAddressBook(state) {
   const chainId = getCurrentChainId(state);
   if (!state.metamask.addressBook[chainId]) {