From 467c1d7b0260ffc7ff809f4a630fa873a05f66dc Mon Sep 17 00:00:00 2001
From: esikkala <esko.ikkala@aalto.fi>
Date: Tue, 11 Dec 2018 14:45:56 +0200
Subject: [PATCH] Fetch new facet values values when filters have changed

---
 src/client/components/FacetDialog.js |  7 ++++++-
 src/client/components/ResultTable.js | 14 ++------------
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/src/client/components/FacetDialog.js b/src/client/components/FacetDialog.js
index cec6445a..232c6dc8 100644
--- a/src/client/components/FacetDialog.js
+++ b/src/client/components/FacetDialog.js
@@ -23,10 +23,15 @@ const styles = () => ({
 class FacetDialog extends React.Component {
 
   componentDidMount = () => {
-    // console.log('facet dialog mounted, fetch facet');
     this.props.fetchFacet();
   }
 
+  componentDidUpdate = prevProps => {
+    if (prevProps.facet.facetFilters != this.props.facet.facetFilters) {
+      this.props.fetchFacet();
+    }
+  }
+
   handleClose = () => this.props.closeFacetDialog();
 
   facetRenderer = facetValues => {
diff --git a/src/client/components/ResultTable.js b/src/client/components/ResultTable.js
index c21b4791..cd8eef36 100644
--- a/src/client/components/ResultTable.js
+++ b/src/client/components/ResultTable.js
@@ -82,22 +82,12 @@ class ResultTable extends React.Component {
       });
     }
     if (prevProps.facetFilters != this.props.facetFilters) {
-      console.log('filters updated')
+      // console.log('filters updated')
       this.props.updatePage(0);
       this.props.fetchManuscripts();
     }
-
   }
-
-  // idRenderer = id => {
-  //   const plainId = id.substring(id.lastIndexOf('/') + 1);
-  //   return (
-  //     <div className={this.props.classes.tableColumn}>
-  //       <a target='_blank' rel='noopener noreferrer' href={id}>{plainId}</a>
-  //     </div>
-  //   );
-  // };
-
+  
   stringListRenderer = cell => {
     if (cell == null || cell === '-'){
       return '-';
-- 
GitLab