fix line looks selected, but it's not #1027

metasfresh · Jul 19, 2017 · 041388d · 041388d
1 parent 1d0893f
commit 041388d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/src/components/app/DocumentList.js b/src/components/app/DocumentList.js
@@ -161,8 +161,7 @@ class DocumentList extends Component {
                 cachedSelection && !disconnectFromState &&
                     dispatch(selectTableItems(cachedSelection, windowType))
                 this.setState({
-                    refreshSelection: true,
-                    cachedSelection: null
+                    refreshSelection: true
                 }, () => this.setState({
                     refreshSelection: false
                 }))
@@ -482,7 +481,7 @@ class DocumentList extends Component {
     render() {
         const {
             layout, data, viewId, clickOutsideLock, refresh, page, filters,
-            cachedSelection, isShowIncluded, hasShowIncluded
+            cachedSelection, isShowIncluded, hasShowIncluded, refreshSelection
         } = this.state;
 
         const {
@@ -582,7 +581,7 @@ class DocumentList extends Component {
                                 disableOnClickOutside={clickOutsideLock}
                                 defaultSelected={cachedSelection ?
                                     cachedSelection : selected}
-                                refreshSelection={this.state.refreshSelection}
+                                refreshSelection={refreshSelection}
                                 queryLimitHit={data.queryLimitHit}
                                 doesSelectionExist={this.doesSelectionExist}
                                 showIncludedViewOnSelect={this.showIncludedViewOnSelect}

diff --git a/src/components/table/Table.js b/src/components/table/Table.js
@@ -70,7 +70,7 @@ class Table extends Component {
         const {
             mainTable, open, rowData, defaultSelected, disconnectFromState,
             dispatch, type, refreshSelection, supportIncludedViewOnSelect,
-            showIncludedViewOnSelect
+            showIncludedViewOnSelect, viewId
         } = this.props;
 
         const {
@@ -112,6 +112,15 @@ class Table extends Component {
                 selected: defaultSelected
             })
         }
+
+        if(
+            JSON.stringify(prevProps.viewId) !==
+            JSON.stringify(viewId)
+        ){
+            this.setState({
+                selected: []
+            })
+        }
     }
 
     showSelectedIncludedView = (selected) => {