Add checks for location and facetState props when checking for url parameter constraints

Author: annasahola

src/client/components/facet_results/ApexCharts.js

@@ -45,7 +45,7 @@ class ApexChart extends React.Component {
     let constraints = []
 
     // first check if page or constraints were given as url parameter
-    if (this.props.location.search !== '') {
+    if (this.props.location && this.props.location.search !== '') {
       const qs = this.props.location.search.replace('?', '')
       const parsedConstraints = querystring.parse(qs).constraints
       // do not try to import constraints twice for ApexChartsDouble components
@@ -108,7 +108,7 @@ class ApexChart extends React.Component {
 
     // check if facets are still fetching
     let someFacetIsFetching = false
-    if (pageType === 'facetResults') Object.values(this.props.facetState.facets).forEach(facet => { if (facet.isFetching) { someFacetIsFetching = true } })
+    if (pageType === 'facetResults' && this.props.facetState) Object.values(this.props.facetState.facets).forEach(facet => { if (facet.isFetching) { someFacetIsFetching = true } })
 
     // refetch default facets (excl. text facets) when facets have been updated
     if (this.state.defaultFacetFetchingRequired && this.props.facetUpdateID > 0 && !someFacetIsFetching) {

src/client/components/facet_results/Deck.js

@@ -76,7 +76,7 @@ class Deck extends React.Component {
     let constraints = []
 
     // first check if page or constraints were given as url parameter
-    if (this.props.location.search !== '') {
+    if (this.props.location && this.props.location.search !== '') {
       const qs = this.props.location.search.replace('?', '')
       const parsedConstraints = querystring.parse(qs).constraints
       constraints = parsedConstraints ? JSON.parse(decodeURIComponent(parsedConstraints)) : []

src/client/components/facet_results/LeafletMap.js

@@ -78,7 +78,7 @@ class LeafletMap extends React.Component {
     let constraints = []
 
     // first check if page or constraints were given as url parameter
-    if (this.props.location.search !== '') {
+    if (this.props.location && this.props.location.search !== '') {
       const qs = this.props.location.search.replace('?', '')
       const parsedConstraints = querystring.parse(qs).constraints
       constraints = parsedConstraints ? JSON.parse(decodeURIComponent(parsedConstraints)) : []

src/client/components/facet_results/ResultTable.js

@@ -87,7 +87,7 @@ class ResultTable extends React.Component {
     let constraints = []
 
     // first check if page or constraints were given as url parameter
-    if (this.props.location.search === '') {
+    if (this.props.location && this.props.location.search === '') {
       page = this.props.data.page === -1 ? 0 : this.props.data.page
     } else {
       const qs = this.props.location.search.replace('?', '')
@@ -136,7 +136,7 @@ class ResultTable extends React.Component {
 
     // check if facets are still fetching
     let someFacetIsFetching = false
-    Object.values(this.props.facetState.facets).forEach(facet => { if (facet.isFetching) { someFacetIsFetching = true } })
+    if (this.props.facetState) Object.values(this.props.facetState.facets).forEach(facet => { if (facet.isFetching) { someFacetIsFetching = true } })
     // refetch default facets (excl. text facets) when facets have been updated
     if (this.state.defaultFacetFetchingRequired && this.props.facetUpdateID > 0 && !someFacetIsFetching) {
       const defaultFacets = this.props.perspectiveConfig.defaultActiveFacets