diff --git a/ui/components/ui/update-nickname-popover/update-nickname-popover.js b/ui/components/ui/update-nickname-popover/update-nickname-popover.js index 0c10b988c..e63bc41fe 100644 --- a/ui/components/ui/update-nickname-popover/update-nickname-popover.js +++ b/ui/components/ui/update-nickname-popover/update-nickname-popover.js @@ -20,8 +20,10 @@ export default function UpdateNicknamePopover({ }) { const t = useContext(I18nContext); - const [nicknameInput, setNicknameInput] = useState(nickname === null ? "" : nickname); - const [memoInput, setMemoInput] = useState(memo === null ? "" : memo); + const [nicknameInput, setNicknameInput] = useState( + nickname === null ? '' : nickname, + ); + const [memoInput, setMemoInput] = useState(memo === null ? '' : memo); const handleNicknameChange = (event) => { setNicknameInput(event.target.value); diff --git a/ui/pages/asset/components/asset-options.js b/ui/pages/asset/components/asset-options.js index c29efe099..f1ae4ec2f 100644 --- a/ui/pages/asset/components/asset-options.js +++ b/ui/pages/asset/components/asset-options.js @@ -85,26 +85,39 @@ const AssetOptions = ({ ); }; +const isntFunc = (p) => { + return typeof p !== 'function'; +}; + AssetOptions.propTypes = { isEthNetwork: PropTypes.bool, isNativeAsset: PropTypes.bool, onClickBlockExplorer: PropTypes.func.isRequired, onViewAccountDetails: PropTypes.func.isRequired, - onRemove: (props, propName, componentName) => { - if (props.isNativeAsset === false && typeof(props.onRemove) !== "function") { - throw new Error("When isNativeAsset is true, onRemove is a required prop"); + onRemove: (props) => { + if (props.isNativeAsset === false && isntFunc(props.onRemove)) { + throw new Error( + 'When isNativeAsset is true, onRemove is a required prop', + ); } }, - onViewTokenDetails: (props, propName, componentName) => { - if (props.isNativeAsset === false && typeof(props.onViewTokenDetails) !== "function") { - throw new Error("When isNativeAsset is true, onViewTokenDetails is a required prop"); + onViewTokenDetails: (props) => { + if (props.isNativeAsset === false && isntFunc(props.onViewTokenDetails)) { + throw new Error( + 'When isNativeAsset is true, onViewTokenDetails is a required prop', + ); } }, - tokenSymbol: (props, propName, componentName) => { - if (props.isNativeAsset === false && typeof(props.tokenSymbol) !== "string") { - throw new Error("When isNativeAsset is true, tokenSymbol is a required prop"); + tokenSymbol: (props) => { + if ( + props.isNativeAsset === false && + typeof props.tokenSymbol !== 'string' + ) { + throw new Error( + 'When isNativeAsset is true, tokenSymbol is a required prop', + ); } - } + }, }; export default AssetOptions; diff --git a/ui/pages/home/home.component.js b/ui/pages/home/home.component.js index d87df82aa..cd728b85b 100644 --- a/ui/pages/home/home.component.js +++ b/ui/pages/home/home.component.js @@ -123,10 +123,13 @@ export default class Home extends PureComponent { showRecoveryPhraseReminder: PropTypes.bool.isRequired, setRecoveryPhraseReminderHasBeenShown: PropTypes.func.isRequired, setRecoveryPhraseReminderLastShown: PropTypes.func.isRequired, - seedPhraseBackedUp: (props, propName, componentName) => { - if (props.seedPhraseBackedUp !== null && typeof(props.seedPhraseBackedUp) !== "boolean") { + seedPhraseBackedUp: (props) => { + if ( + props.seedPhraseBackedUp !== null && + typeof props.seedPhraseBackedUp !== 'boolean' + ) { throw new Error( - `seedPhraseBackedUp is required to be null or boolean. Received ${props.seedPhraseBackedUp}` + `seedPhraseBackedUp is required to be null or boolean. Received ${props.seedPhraseBackedUp}`, ); } },