fix: avoid query when components are destroyed

Closes: #357
algolia · Nov 11, 2017 · c41c606 · c41c606
1 parent 0192ad9
commit c41c606
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 0 deletions.
diff --git a/src/components/Menu.vue b/src/components/Menu.vue
@@ -84,7 +84,9 @@
     },
 
     destroyed() {
+      this.searchStore.stop();
       this.searchStore.removeFacet(this.attribute);
+      this.searchStore.start();
     },
   };
 </script>
diff --git a/src/components/RangeInput.vue b/src/components/RangeInput.vue
@@ -111,7 +111,9 @@ export default {
   },
 
   destroyed() {
+    this.searchStore.stop();
     this.searchStore.removeFacet(this.attributeName);
+    this.searchStore.start();
   },
 
   computed: {

diff --git a/src/components/Rating.vue b/src/components/Rating.vue
@@ -55,7 +55,9 @@ export default {
     this.searchStore.addFacet(this.attributeName, FACET_OR);
   },
   destroyed() {
+    this.searchStore.stop();
     this.searchStore.removeFacet(this.attributeName);
+    this.searchStore.start();
   },
   computed: {
     show() {

diff --git a/src/components/RefinementList.vue b/src/components/RefinementList.vue
@@ -62,7 +62,9 @@ export default {
     this.searchStore.addFacet(this.attributeName, this.operator);
   },
   destroyed() {
+    this.searchStore.stop();
     this.searchStore.removeFacet(this.attributeName);
+    this.searchStore.start();
   },
   computed: {
     facetValues() {

diff --git a/src/components/TreeMenu.vue b/src/components/TreeMenu.vue
@@ -42,7 +42,9 @@ export default {
     );
   },
   destroyed() {
+    this.searchStore.stop();
     this.searchStore.removeFacet(this.attribute);
+    this.searchStore.start();
   },
   computed: {
     facetValues() {

diff --git a/src/components/__tests__/menu-tree.js b/src/components/__tests__/menu-tree.js
@@ -36,6 +36,8 @@ const addFacet = jest.fn();
 const searchStore = {
   getFacetValues,
   addFacet,
+  stop: () => {},
+  start: () => {},
 };
 
 test('renders proper HTML', () => {

diff --git a/src/components/__tests__/refinement-list.js b/src/components/__tests__/refinement-list.js
@@ -17,6 +17,8 @@ const addFacet = jest.fn();
 const searchStore = {
   getFacetValues,
   addFacet,
+  stop: () => {},
+  start: () => {},
 };
 
 test('renders proper HTML', () => {