update issue 212's fix logic as suggested

Author: 任庆伟

src/shared/components/challenge-listing/index.jsx

@@ -16,35 +16,14 @@ import Listing from './Listing';
 import ChallengeCardPlaceholder from './placeholders/ChallengeCard';
 import SRMCard from './SRMCard';
 
+import { updateQuery } from 'utils/url';
 import './style.scss';
 
 // Number of challenge placeholder card to display
 const CHALLENGE_PLACEHOLDER_COUNT = 8;
 
 export default function ChallengeListing(props) {
-  function CheckDateTime(str) {
-    const reg = /^(\d{4})-(\d{1,2})-(\d{1,2})T(\d{1,2}):(\d{1,2}):(\d{1,2})\.(\d{3})Z$/;
-    const r = str.match(reg);
-    if (r == null) return false;
-    r[2] -= 1;
-    const d = new Date(r[1], r[2], r[3], r[4], r[5], r[6]);
-    if (d.getFullYear() !== r[1]) return false;
-    if (d.getMonth() !== r[2]) return false;
-    if (d.getDate() !== r[3]) return false;
-    if (d.getHours() !== r[4]) return false;
-    if (d.getMinutes() !== r[5]) return false;
-    if (d.getSeconds() !== r[6]) return false;
-    return true;
-  }
-  const filterState = props.filterState;
-  if (filterState) {
-    if (!!filterState.startDate && CheckDateTime(filterState.startDate) === false) {
-      delete filterState.startDate;
-    }
-    if (!!filterState.endDate && CheckDateTime(filterState.endDate) === false) {
-      delete filterState.endDate;
-    }
-  }
+  updateQuery({ });
   let challenges = props.challenges;
 
   if (props.communityFilter) {

src/shared/reducers/challenge-listing/index.js

@@ -8,6 +8,7 @@ import logger from 'utils/logger';
 import { handleActions } from 'redux-actions';
 import { combine, resolveReducers } from 'utils/redux';
 import { updateQuery } from 'utils/url';
+import moment from 'moment';
 
 import filterPanel from '../challenge-listing/filter-panel';
 import sidebar, { factory as sidebarFactory } from '../challenge-listing/sidebar';
@@ -270,6 +271,12 @@ export function factory(req) {
 
   if (req) {
     state.filter = req.query.filter;
+    if (!!state.filter.startDate && moment(state.filter.startDate).isValid() === false) {
+      delete state.filter.startDate;
+    }
+    if (!!state.filter.endDate && moment(state.filter.endDate).isValid() === false) {
+      delete state.filter.endDate;
+    }
     state.selectedCommunityId = req.query.communityId;
   }
 

src/shared/utils/url.js

@@ -6,6 +6,7 @@
 
 import _ from 'lodash';
 import qs from 'qs';
+import moment from 'moment';
 
 /**
  * If executed client-side (determined in this case by the presence of global
@@ -28,6 +29,11 @@ export function updateQuery(update) {
     if (_.isUndefined(value)) delete query[key];
     else query[key] = value;
   });
+  if (!!query.filter.startDate && moment(query.filter.startDate).isValid() === false) {
+    delete query.filter.startDate; console.log('delete start date');
+  } else if (query.filter.endDate && moment(query.filter.endDate).isValid() === false) {
+    delete query.filter.endDate; console.log('delete end date');
+  }
 
   query = `?${qs.stringify(query, { encode: false })}`;
   window.history.replaceState(window.history.state, '', query);