instantsearch.js 4.44.0 → 4.44.1

package/CHANGELOG.md

@@ -1,3 +1,12 @@
+## [4.44.1](https://github.com/algolia/instantsearch.js/compare/v4.44.0...v4.44.1) (2022-08-25)
+
+
+### Bug Fixes
+
+* **connectNumericMenu + connectRange:** stop sending invalid clickedFilters event ([#5085](https://github.com/algolia/instantsearch.js/issues/5085)) ([20996c7](https://github.com/algolia/instantsearch.js/commit/20996c7a159988c58e00ff24d2d2dc98af8b980f))
+
+
+
 # [4.44.0](https://github.com/algolia/instantsearch.js/compare/v4.43.1...v4.44.0) (2022-08-08)
 
 

package/cjs/connectors/numeric-menu/connectNumericMenu.js

@@ -32,47 +32,12 @@ var withUsage = (0, _index.createDocumentationMessageGenerator)({
 var $$type = 'ais.numericMenu';
 
 var createSendEvent = function createSendEvent(_ref) {
-  var instantSearchInstance = _ref.instantSearchInstance,
-      helper = _ref.helper,
-      attribute = _ref.attribute;
+  var instantSearchInstance = _ref.instantSearchInstance;
   return function () {
-    for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
-      args[_key] = arguments[_key];
-    }
-
-    if (args.length === 1) {
-      instantSearchInstance.sendEventToInsights(args[0]);
+    if (arguments.length === 1) {
+      instantSearchInstance.sendEventToInsights(arguments.length <= 0 ? undefined : arguments[0]);
       return;
     }
-
-    var eventType = args[0],
-        facetValue = args[1],
-        _args$ = args[2],
-        eventName = _args$ === void 0 ? 'Filter Applied' : _args$;
-
-    if (eventType !== 'click') {
-      return;
-    } // facetValue === "%7B%22start%22:5,%22end%22:10%7D"
-
-
-    var filters = (0, _index.convertNumericRefinementsToFilters)(getRefinedState(helper.state, attribute, facetValue), attribute);
-
-    if (filters && filters.length > 0) {
-      /*
-        filters === ["price<=10", "price>=5"]
-      */
-      instantSearchInstance.sendEventToInsights({
-        insightsMethod: 'clickedFilters',
-        widgetType: $$type,
-        eventType: eventType,
-        payload: {
-          eventName: eventName,
-          index: helper.getIndex(),
-          filters: filters
-        },
-        attribute: attribute
-      });
-    }
   };
 };
 
@@ -216,9 +181,7 @@ var connectNumericMenu = function connectNumericMenu(renderFn) {
 
         if (!connectorState.sendEvent) {
           connectorState.sendEvent = createSendEvent({
-            instantSearchInstance: instantSearchInstance,
-            helper: helper,
-            attribute: attribute
+            instantSearchInstance: instantSearchInstance
           });
         }
 

package/cjs/connectors/range/connectRange.js

@@ -156,50 +156,12 @@ var connectRange = function connectRange(renderFn) {
       return null;
     };
 
-    var sendEventWithRefinedState = function sendEventWithRefinedState(refinedState, instantSearchInstance, helper) {
-      var eventName = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : 'Filter Applied';
-      var filters = (0, _index.convertNumericRefinementsToFilters)(refinedState, attribute);
-
-      if (filters && filters.length > 0) {
-        instantSearchInstance.sendEventToInsights({
-          insightsMethod: 'clickedFilters',
-          widgetType: $$type,
-          eventType: 'click',
-          payload: {
-            eventName: eventName,
-            index: helper.getIndex(),
-            filters: filters
-          },
-          attribute: attribute
-        });
-      }
-    };
-
-    var createSendEvent = function createSendEvent(instantSearchInstance, helper, currentRange) {
+    var createSendEvent = function createSendEvent(instantSearchInstance) {
       return function () {
-        for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
-          args[_key] = arguments[_key];
-        }
-
-        if (args.length === 1) {
-          instantSearchInstance.sendEventToInsights(args[0]);
+        if (arguments.length === 1) {
+          instantSearchInstance.sendEventToInsights(arguments.length <= 0 ? undefined : arguments[0]);
           return;
         }
-
-        var eventType = args[0],
-            facetValue = args[1],
-            eventName = args[2];
-
-        if (eventType !== 'click') {
-          return;
-        }
-
-        var _facetValue = _slicedToArray(facetValue, 2),
-            nextMin = _facetValue[0],
-            nextMax = _facetValue[1];
-
-        var refinedState = getRefinedState(helper, currentRange, nextMin, nextMax);
-        sendEventWithRefinedState(refinedState, instantSearchInstance, helper, eventName);
       };
     };
 
@@ -245,7 +207,7 @@ var connectRange = function connectRange(renderFn) {
       return [min, max];
     }
 
-    function _refine(instantSearchInstance, helper, currentRange) {
+    function _refine(helper, currentRange) {
       return function () {
         var _ref11 = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : [undefined, undefined],
             _ref12 = _slicedToArray(_ref11, 2),
@@ -255,7 +217,6 @@ var connectRange = function connectRange(renderFn) {
         var refinedState = getRefinedState(helper, currentRange, nextMin, nextMax);
 
         if (refinedState) {
-          sendEventWithRefinedState(refinedState, instantSearchInstance, helper);
           helper.setState(refinedState).search();
         }
       };
@@ -301,12 +262,12 @@ var connectRange = function connectRange(renderFn) {
           // On first render pass an empty range
           // to be able to bypass the validation
           // related to it
-          refine = _refine(instantSearchInstance, helper, {
+          refine = _refine(helper, {
             min: undefined,
             max: undefined
           });
         } else {
-          refine = _refine(instantSearchInstance, helper, currentRange);
+          refine = _refine(helper, currentRange);
         }
 
         return {
@@ -314,7 +275,7 @@ var connectRange = function connectRange(renderFn) {
           canRefine: currentRange.min !== currentRange.max,
           format: rangeFormatter,
           range: currentRange,
-          sendEvent: createSendEvent(instantSearchInstance, helper, currentRange),
+          sendEvent: createSendEvent(instantSearchInstance),
           widgetParams: _objectSpread(_objectSpread({}, widgetParams), {}, {
             precision: precision
           }),

package/cjs/lib/version.js

@@ -4,5 +4,5 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-var _default = '4.44.0';
+var _default = '4.44.1';
 exports.default = _default;

package/dist/instantsearch.development.js

@@ -1,4 +1,4 @@
-/*! InstantSearch.js 4.44.0 | © Algolia, Inc. and contributors; MIT License | https://github.com/algolia/instantsearch.js */
+/*! InstantSearch.js 4.44.1 | © Algolia, Inc. and contributors; MIT License | https://github.com/algolia/instantsearch.js */
 (function (global, factory) {
   typeof exports === 'object' && typeof module !== 'undefined' ? module.exports = factory() :
   typeof define === 'function' && define.amd ? define(factory) :
@@ -7918,31 +7918,6 @@
     }
   }
 
-  function convertNumericRefinementsToFilters(state, attribute) {
-    if (!state) {
-      return null;
-    }
-
-    var filtersObj = state.numericRefinements[attribute];
-    /*
-      filtersObj === {
-        "<=": [10],
-        "=": [],
-        ">=": [5]
-      }
-    */
-
-    var filters = [];
-    Object.keys(filtersObj).filter(function (operator) {
-      return Array.isArray(filtersObj[operator]) && filtersObj[operator].length > 0;
-    }).forEach(function (operator) {
-      filtersObj[operator].forEach(function (value) {
-        filters.push("".concat(attribute).concat(operator).concat(value));
-      });
-    });
-    return filters;
-  }
-
   // copied from
   // https://github.com/algolia/autocomplete.js/blob/307a7acc4283e10a19cb7d067f04f1bea79dc56f/packages/autocomplete-core/src/utils/createConcurrentSafePromise.ts#L1:L1
 
@@ -8603,7 +8578,7 @@
     instantSearchInstance.renderState = _objectSpread2(_objectSpread2({}, instantSearchInstance.renderState), {}, _defineProperty({}, parentIndexName, _objectSpread2(_objectSpread2({}, instantSearchInstance.renderState[parentIndexName]), renderState)));
   }
 
-  var version$1 = '4.44.0';
+  var version$1 = '4.44.1';
 
   var NAMESPACE = 'ais';
   var component = function component(componentName) {
@@ -12117,47 +12092,12 @@
   var $$type = 'ais.numericMenu';
 
   var createSendEvent = function createSendEvent(_ref) {
-    var instantSearchInstance = _ref.instantSearchInstance,
-        helper = _ref.helper,
-        attribute = _ref.attribute;
+    var instantSearchInstance = _ref.instantSearchInstance;
     return function () {
-      for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
-        args[_key] = arguments[_key];
-      }
-
-      if (args.length === 1) {
-        instantSearchInstance.sendEventToInsights(args[0]);
+      if (arguments.length === 1) {
+        instantSearchInstance.sendEventToInsights(arguments.length <= 0 ? undefined : arguments[0]);
         return;
       }
-
-      var eventType = args[0],
-          facetValue = args[1],
-          _args$ = args[2],
-          eventName = _args$ === void 0 ? 'Filter Applied' : _args$;
-
-      if (eventType !== 'click') {
-        return;
-      } // facetValue === "%7B%22start%22:5,%22end%22:10%7D"
-
-
-      var filters = convertNumericRefinementsToFilters(getRefinedState(helper.state, attribute, facetValue), attribute);
-
-      if (filters && filters.length > 0) {
-        /*
-          filters === ["price<=10", "price>=5"]
-        */
-        instantSearchInstance.sendEventToInsights({
-          insightsMethod: 'clickedFilters',
-          widgetType: $$type,
-          eventType: eventType,
-          payload: {
-            eventName: eventName,
-            index: helper.getIndex(),
-            filters: filters
-          },
-          attribute: attribute
-        });
-      }
     };
   };
 
@@ -12301,9 +12241,7 @@
 
           if (!connectorState.sendEvent) {
             connectorState.sendEvent = createSendEvent({
-              instantSearchInstance: instantSearchInstance,
-              helper: helper,
-              attribute: attribute
+              instantSearchInstance: instantSearchInstance
             });
           }
 
@@ -12729,50 +12667,12 @@
         return null;
       };
 
-      var sendEventWithRefinedState = function sendEventWithRefinedState(refinedState, instantSearchInstance, helper) {
-        var eventName = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : 'Filter Applied';
-        var filters = convertNumericRefinementsToFilters(refinedState, attribute);
-
-        if (filters && filters.length > 0) {
-          instantSearchInstance.sendEventToInsights({
-            insightsMethod: 'clickedFilters',
-            widgetType: $$type$1,
-            eventType: 'click',
-            payload: {
-              eventName: eventName,
-              index: helper.getIndex(),
-              filters: filters
-            },
-            attribute: attribute
-          });
-        }
-      };
-
-      var createSendEvent = function createSendEvent(instantSearchInstance, helper, currentRange) {
+      var createSendEvent = function createSendEvent(instantSearchInstance) {
         return function () {
-          for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
-            args[_key] = arguments[_key];
-          }
-
-          if (args.length === 1) {
-            instantSearchInstance.sendEventToInsights(args[0]);
+          if (arguments.length === 1) {
+            instantSearchInstance.sendEventToInsights(arguments.length <= 0 ? undefined : arguments[0]);
             return;
           }
-
-          var eventType = args[0],
-              facetValue = args[1],
-              eventName = args[2];
-
-          if (eventType !== 'click') {
-            return;
-          }
-
-          var _facetValue = _slicedToArray(facetValue, 2),
-              nextMin = _facetValue[0],
-              nextMax = _facetValue[1];
-
-          var refinedState = getRefinedState(helper, currentRange, nextMin, nextMax);
-          sendEventWithRefinedState(refinedState, instantSearchInstance, helper, eventName);
         };
       };
 
@@ -12818,7 +12718,7 @@
         return [min, max];
       }
 
-      function _refine(instantSearchInstance, helper, currentRange) {
+      function _refine(helper, currentRange) {
         return function () {
           var _ref11 = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : [undefined, undefined],
               _ref12 = _slicedToArray(_ref11, 2),
@@ -12828,7 +12728,6 @@
           var refinedState = getRefinedState(helper, currentRange, nextMin, nextMax);
 
           if (refinedState) {
-            sendEventWithRefinedState(refinedState, instantSearchInstance, helper);
             helper.setState(refinedState).search();
           }
         };
@@ -12874,12 +12773,12 @@
             // On first render pass an empty range
             // to be able to bypass the validation
             // related to it
-            refine = _refine(instantSearchInstance, helper, {
+            refine = _refine(helper, {
               min: undefined,
               max: undefined
             });
           } else {
-            refine = _refine(instantSearchInstance, helper, currentRange);
+            refine = _refine(helper, currentRange);
           }
 
           return {
@@ -12887,7 +12786,7 @@
             canRefine: currentRange.min !== currentRange.max,
             format: rangeFormatter,
             range: currentRange,
-            sendEvent: createSendEvent(instantSearchInstance, helper, currentRange),
+            sendEvent: createSendEvent(instantSearchInstance),
             widgetParams: _objectSpread2(_objectSpread2({}, widgetParams), {}, {
               precision: precision
             }),