From cd64bcfadc496a964ec9d9702ef6280fd1f3eb23 Mon Sep 17 00:00:00 2001 From: Youcef Mammar Date: Thu, 11 Apr 2019 10:42:29 +0200 Subject: [PATCH] fix(currentRefinements): don't rely on `_objectSpread` (#3672) The `_objectSpread` polyfill calls `Object.keys()` under the hood which does not support anything other than objects on IE11. --- .../connectCurrentRefinements.js | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/src/connectors/current-refinements/connectCurrentRefinements.js b/src/connectors/current-refinements/connectCurrentRefinements.js index 60f0e860be..9253cffc8e 100644 --- a/src/connectors/current-refinements/connectCurrentRefinements.js +++ b/src/connectors/current-refinements/connectCurrentRefinements.js @@ -254,17 +254,23 @@ function normalizeRefinement(refinement) { ? `${getOperatorSymbol(refinement.operator)} ${refinement.name}` : refinement.name; - return { + const normalizedRefinement = { attribute: refinement.attributeName, type: refinement.type, value, label, - ...(refinement.operator !== undefined && { operator: refinement.operator }), - ...(refinement.count !== undefined && { count: refinement.count }), - ...(refinement.exhaustive !== undefined && { - exhaustive: refinement.exhaustive, - }), }; + + if (refinement.operator !== undefined) { + normalizedRefinement.operator = refinement.operator; + } + if (refinement.count !== undefined) { + normalizedRefinement.count = refinement.count; + } + if (refinement.exhaustive !== undefined) { + normalizedRefinement.exhaustive = refinement.exhaustive; + } + return normalizedRefinement; } function groupItemsByRefinements(items, helper) {