From 192eacc54e6465e1d4e16f593a9e79de91d76cf3 Mon Sep 17 00:00:00 2001 From: Judith Lao Date: Mon, 20 Jun 2022 11:05:22 -0400 Subject: [PATCH] fix: do not break page when mediaUrl is undefined --- packages/frontend/src/components/nft/NFTMedia.js | 10 +++++----- .../frontend/src/hooks/fungibleTokensIncludingNEAR.js | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/frontend/src/components/nft/NFTMedia.js b/packages/frontend/src/components/nft/NFTMedia.js index 92e328e6e1..f953aa8a61 100644 --- a/packages/frontend/src/components/nft/NFTMedia.js +++ b/packages/frontend/src/components/nft/NFTMedia.js @@ -1,13 +1,13 @@ -import React, { useMemo } from "react"; +import React, { useMemo } from 'react'; -import FailedToLoad from "../../images/failed_to_load.svg"; +import FailedToLoad from '../../images/failed_to_load.svg'; export function NFTMedia({ mediaUrl, autoPlay = false }) { const [isVideo, mimeType] = useMemo(() => { let mimeType; // check mediaUrl string for .webm or .mp4 endings (case-insensitive) - if (mediaUrl && mediaUrl.match(/\.webm$/i)) mimeType = "webm"; - else if (mediaUrl && mediaUrl.match(/\.mp4$/i)) mimeType = "mp4"; + if (mediaUrl && mediaUrl.match(/\.webm$/i)) mimeType = 'webm'; + else if (mediaUrl && mediaUrl.match(/\.mp4$/i)) mimeType = 'mp4'; // if there is a mediaUrl and a truthy mimeType (webm or mp4), we have a video const isVideo = !!mediaUrl && mimeType; return [isVideo, mimeType]; @@ -23,7 +23,7 @@ export function NFTMedia({ mediaUrl, autoPlay = false }) { onError={(e) => { e.target.onerror = null; e.target.parentElement.setAttribute( - "poster", + 'poster', FailedToLoad ); }} diff --git a/packages/frontend/src/hooks/fungibleTokensIncludingNEAR.js b/packages/frontend/src/hooks/fungibleTokensIncludingNEAR.js index 7ab9145cb9..b05fc6613b 100644 --- a/packages/frontend/src/hooks/fungibleTokensIncludingNEAR.js +++ b/packages/frontend/src/hooks/fungibleTokensIncludingNEAR.js @@ -9,7 +9,7 @@ import compare from '../utils/compare'; export const useFungibleTokensIncludingNEAR = function ({ showTokensWithZeroBalance = false, includeNearContractName = false } = {}) { - const NEARAsTokenWithMetadata = useSelector(state => selectNEARAsTokenWithMetadata(state, {includeNearContractName})); + const NEARAsTokenWithMetadata = useSelector((state) => selectNEARAsTokenWithMetadata(state, {includeNearContractName})); const accountId = useSelector(selectActiveAccountId); const fungibleTokens = useSelector((state) => selectTokensWithMetadataForAccountId(state, { accountId, showTokensWithZeroBalance })