Fix issue #190

Author: elfman

src/shared/containers/challenge-detail/index.jsx

@@ -20,6 +20,7 @@ import ChallengeCheckpoints from 'components/challenge-detail/Checkpoints';
 import React from 'react';
 import PT from 'prop-types';
 import { connect } from 'react-redux';
+import Error404 from 'components/Error404';
 import challengeActions from 'actions/challenge';
 import termsActions from 'actions/terms';
 import config from 'utils/config';
@@ -56,6 +57,10 @@ class ChallengeDetailPageContainer extends React.Component {
     const { challenge, loadChallengeDetails, loadTerms,
       openTermsModal, authTokens, challengeId } = this.props;
 
+    if (isNaN(challengeId)) {
+      return;
+    }
+
     if (challenge.id !== challengeId) {
       loadChallengeDetails(authTokens, challengeId);
     }
@@ -68,6 +73,10 @@ class ChallengeDetailPageContainer extends React.Component {
   }
 
   componentWillReceiveProps(nextProps) {
+    if (isNaN(nextProps.challengeId)) {
+      return;
+    }
+
     if (this.props.tokenV3 !== nextProps.tokenV3) {
       this.props.reloadChallengeDetails(nextProps.authTokens, this.props.challengeId);
     }
@@ -118,6 +127,10 @@ class ChallengeDetailPageContainer extends React.Component {
   }
 
   render() {
+    if (isNaN(this.props.challengeId)) {
+      return <Error404 />;
+    }
+
     const isEmpty = _.isEmpty(this.props.challenge);
 
     const hasRegistered = isRegistered(this.props.challenge.userDetails,
@@ -286,6 +299,9 @@ ChallengeDetailPageContainer.propTypes = {
 
 function extractChallengeDetail(v3, v2, challengeId) {
   let challenge = {};
+  if (isNaN(challengeId)) {
+    return challenge;
+  }
   if (!_.isEmpty(v3)) {
     challenge = _.clone(v3);
     if (!_.isEmpty(v2)) {