fix(MultiIndex): handle if namespace isn't in search state (#139)

algolia · Jun 26, 2017 · 1aab324 · 1aab324
1 parent 124ffe6
commit 1aab324
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/packages/react-instantsearch/src/core/indexUtils.js b/packages/react-instantsearch/src/core/indexUtils.js
@@ -142,6 +142,7 @@ export function getCurrentRefinementValue(
     (hasMultipleIndex(context) &&
       searchState.indices &&
       namespace &&
+      searchState.indices[`${index}`] &&
       has(searchState.indices[`${index}`][namespace], `${attributeName}`)) ||
     (hasMultipleIndex(context) &&
       searchState.indices &&

diff --git a/packages/react-instantsearch/src/core/indexUtils.test.js b/packages/react-instantsearch/src/core/indexUtils.test.js
@@ -357,7 +357,7 @@ describe('utility method for manipulating the search state', () => {
         },
       };
 
-      expect.assertions(6); //async assertions
+      expect.assertions(7); //async assertions
 
       let expectedRefinement = value => expect(value).toEqual('refinement');
 
@@ -414,6 +414,17 @@ describe('utility method for manipulating the search state', () => {
 
       expect(value).toEqual('defaultValue');
 
+      value = getCurrentRefinementValue(
+        {},
+        searchState,
+        context,
+        'anotherNamespace.refinement.top',
+        'defaultValue',
+        () => {}
+      );
+
+      expect(value).toEqual('defaultValue');
+
       value = getCurrentRefinementValue(
         { defaultRefinement: 'defaultRefinement' },
         {},