victory-group 36.8.2 → 36.8.3

package/dist/victory-group.js

@@ -72,16 +72,15 @@ var fallbackProps = {
   offset: 0
 }; // eslint-disable-next-line max-statements
 
-function getCalculatedProps(props, childComponents) {
+function getCalculatedProps(initialProps, childComponents) {
   var role = "group";
-  props = victory_core__WEBPACK_IMPORTED_MODULE_3__.modifyProps(props, fallbackProps, role);
+  var props = victory_core__WEBPACK_IMPORTED_MODULE_3__.modifyProps(initialProps, fallbackProps, role);
   var style = victory_core__WEBPACK_IMPORTED_MODULE_4__.getStyle(props.theme, props.style, role);
-  var _props = props,
-      offset = _props.offset,
-      colorScale = _props.colorScale,
-      color = _props.color,
-      polar = _props.polar,
-      horizontal = _props.horizontal;
+  var offset = props.offset,
+      colorScale = props.colorScale,
+      color = props.color,
+      polar = props.polar,
+      horizontal = props.horizontal;
   var categories = props.categories || victory_core__WEBPACK_IMPORTED_MODULE_4__.getCategories(props, childComponents, null);
   var datasets = props.datasets || victory_core__WEBPACK_IMPORTED_MODULE_4__.getDataFromChildren(props, null);
   var domain = {
@@ -251,21 +250,19 @@ function getDataWithOffset(props) {
   });
 }
 
-function getChildren(props, childComponents, calculatedProps) {
-  props = victory_core__WEBPACK_IMPORTED_MODULE_3__.modifyProps(props, fallbackProps, "stack");
-  childComponents = childComponents || react__WEBPACK_IMPORTED_MODULE_1___default().Children.toArray(props.children);
-  calculatedProps = calculatedProps || getCalculatedProps(props, childComponents);
-  var _calculatedProps = calculatedProps,
-      datasets = _calculatedProps.datasets;
-  var _props2 = props,
-      labelComponent = _props2.labelComponent,
-      polar = _props2.polar;
-  var childProps = getChildProps(props, calculatedProps);
+function getChildren(initialProps, childComponents, calculatedProps) {
+  var props = victory_core__WEBPACK_IMPORTED_MODULE_3__.modifyProps(initialProps, fallbackProps, "stack");
+  var children = childComponents || react__WEBPACK_IMPORTED_MODULE_1___default().Children.toArray(props.children);
+  var newCalculatedProps = calculatedProps || getCalculatedProps(props, children);
+  var datasets = newCalculatedProps.datasets;
+  var labelComponent = props.labelComponent,
+      polar = props.polar;
+  var childProps = getChildProps(props, newCalculatedProps);
   var parentName = props.name || "group";
-  return childComponents.map(function (child, index) {
+  return children.map(function (child, index) {
     var role = child.type && child.type.role;
-    var xOffset = polar ? getPolarX0(props, calculatedProps, index, role) : getX0(props, calculatedProps, index, role);
-    var style = role === "voronoi" || role === "tooltip" || role === "label" ? child.props.style : victory_core__WEBPACK_IMPORTED_MODULE_4__.getChildStyle(child, index, calculatedProps);
+    var xOffset = polar ? getPolarX0(props, newCalculatedProps, index, role) : getX0(props, newCalculatedProps, index, role);
+    var style = role === "voronoi" || role === "tooltip" || role === "label" ? child.props.style : victory_core__WEBPACK_IMPORTED_MODULE_4__.getChildStyle(child, index, newCalculatedProps);
     var labels = props.labels ? getLabels(props, datasets, index) : child.props.labels;
     var name = child.props.name || "".concat(parentName, "-").concat(role, "-").concat(index);
     return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default().cloneElement(child, lodash_assign__WEBPACK_IMPORTED_MODULE_0___default()({
@@ -432,7 +429,7 @@ var VictoryGroupBase = function (initialProps) {
   }, [setAnimationState, previousProps, propsWithDefaults, props]);
 
   if (!lodash_isEmpty__WEBPACK_IMPORTED_MODULE_0___default()(events)) {
-    return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement(victory_shared_events__WEBPACK_IMPORTED_MODULE_13__["default"], {
+    return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement(victory_shared_events__WEBPACK_IMPORTED_MODULE_13__.VictorySharedEvents, {
       container: container,
       eventKey: eventKey,
       events: events,
@@ -8679,11 +8676,11 @@ function getAxis(props) {
  */
 
 function findAxisComponents(childComponents, predicate) {
-  predicate = predicate || (lodash_identity__WEBPACK_IMPORTED_MODULE_9___default());
+  var predicateFunction = predicate || (lodash_identity__WEBPACK_IMPORTED_MODULE_9___default());
 
   var findAxes = function (children) {
     return children.reduce(function (memo, child) {
-      if (child.type && child.type.role === "axis" && predicate(child)) {
+      if (child.type && child.type.role === "axis" && predicateFunction(child)) {
         return memo.concat(child);
       } else if (child.props && child.props.children) {
         return memo.concat(findAxes(react__WEBPACK_IMPORTED_MODULE_12___default().Children.toArray(child.props.children)));
@@ -9427,12 +9424,14 @@ function sortData(dataset, sortKey) {
   } // Ensures previous VictoryLine api for sortKey prop stays consistent
 
 
+  var formattedSortKey = sortKey;
+
   if (sortKey === "x" || sortKey === "y") {
-    sortKey = "_".concat(sortKey);
+    formattedSortKey = "_".concat(sortKey);
   }
 
   var order = sortOrder === "ascending" ? "asc" : "desc";
-  return lodash_orderBy__WEBPACK_IMPORTED_MODULE_5___default()(dataset, sortKey, order);
+  return lodash_orderBy__WEBPACK_IMPORTED_MODULE_5___default()(dataset, formattedSortKey, order);
 } // This method will remove data points that break certain scales. (log scale only)
 
 
@@ -9623,7 +9622,10 @@ function formatData(dataset, props, expectedKeys) {
     return [];
   }
 
-  var defaultKeys = ["x", "y", "y0"];
+  var defaultKeys = ["x", "y", "y0"]; // TODO: We shouldn’t mutate the expectedKeys param here,
+  // but we need to figure out why changing it causes regressions in tests.
+  // eslint-disable-next-line no-param-reassign
+
   expectedKeys = Array.isArray(expectedKeys) ? expectedKeys : defaultKeys;
 
   var createAccessor = function (name) {
@@ -9648,13 +9650,13 @@ function formatData(dataset, props, expectedKeys) {
 
   var data = preformattedData ? dataset : dataset.reduce(function (dataArr, datum, index) {
     // eslint-disable-line complexity
-    datum = parseDatum(datum);
+    var parsedDatum = parseDatum(datum);
     var fallbackValues = {
       x: index,
-      y: datum
+      y: parsedDatum
     };
     var processedValues = expectedKeys.reduce(function (memo, type) {
-      var processedValue = accessor[type](datum);
+      var processedValue = accessor[type](parsedDatum);
       var value = processedValue !== undefined ? processedValue : fallbackValues[type];
 
       if (value !== undefined) {
@@ -9669,7 +9671,7 @@ function formatData(dataset, props, expectedKeys) {
       return memo;
     }, {});
 
-    var formattedDatum = lodash_assign__WEBPACK_IMPORTED_MODULE_12___default()({}, processedValues, datum);
+    var formattedDatum = lodash_assign__WEBPACK_IMPORTED_MODULE_12___default()({}, processedValues, parsedDatum);
 
     if (!lodash_isEmpty__WEBPACK_IMPORTED_MODULE_4___default()(formattedDatum)) {
       dataArr.push(formattedDatum);
@@ -9780,8 +9782,8 @@ function getStringsFromData(props, axis) {
   }, []);
   var sortedData = sortData(data, props.sortKey, props.sortOrder);
   var dataStrings = sortedData.reduce(function (dataArr, datum) {
-    datum = parseDatum(datum);
-    dataArr.push(accessor(datum));
+    var parsedDatum = parseDatum(datum);
+    dataArr.push(accessor(parsedDatum));
     return dataArr;
   }, []).filter(function (datum) {
     return typeof datum === "string";
@@ -10025,18 +10027,18 @@ function padDomain(domain, props, axis) {
 
 
 function createDomainFunction(getDomainFromDataFunction, formatDomainFunction) {
-  getDomainFromDataFunction = lodash_isFunction__WEBPACK_IMPORTED_MODULE_2___default()(getDomainFromDataFunction) ? getDomainFromDataFunction : getDomainFromData;
-  formatDomainFunction = lodash_isFunction__WEBPACK_IMPORTED_MODULE_2___default()(formatDomainFunction) ? formatDomainFunction : formatDomain;
+  var getDomainFromDataFn = lodash_isFunction__WEBPACK_IMPORTED_MODULE_2___default()(getDomainFromDataFunction) ? getDomainFromDataFunction : getDomainFromData;
+  var formatDomainFn = lodash_isFunction__WEBPACK_IMPORTED_MODULE_2___default()(formatDomainFunction) ? formatDomainFunction : formatDomain;
   return function (props, axis) {
     var propsDomain = getDomainFromProps(props, axis);
 
     if (propsDomain) {
-      return formatDomainFunction(propsDomain, props, axis);
+      return formatDomainFn(propsDomain, props, axis);
     }
 
     var categories = _data__WEBPACK_IMPORTED_MODULE_10__.getCategories(props, axis);
-    var domain = categories ? getDomainFromCategories(props, axis, categories) : getDomainFromDataFunction(props, axis);
-    return domain ? formatDomainFunction(domain, props, axis) : undefined;
+    var domain = categories ? getDomainFromCategories(props, axis, categories) : getDomainFromDataFn(props, axis);
+    return domain ? formatDomainFn(domain, props, axis) : undefined;
   };
 }
 /**
@@ -10069,27 +10071,27 @@ function getDomain(props, axis) {
  */
 
 function getDomainFromCategories(props, axis, categories) {
-  categories = categories || _data__WEBPACK_IMPORTED_MODULE_10__.getCategories(props, axis);
+  var categoriesArray = categories || _data__WEBPACK_IMPORTED_MODULE_10__.getCategories(props, axis);
   var polar = props.polar,
       _props$startAngle = props.startAngle,
       startAngle = _props$startAngle === void 0 ? 0 : _props$startAngle,
       _props$endAngle = props.endAngle,
       endAngle = _props$endAngle === void 0 ? 360 : _props$endAngle;
 
-  if (!categories) {
+  if (!categoriesArray) {
     return undefined;
   }
 
   var minDomain = getMinFromProps(props, axis);
   var maxDomain = getMaxFromProps(props, axis);
-  var stringArray = _collection__WEBPACK_IMPORTED_MODULE_8__.containsStrings(categories) ? _data__WEBPACK_IMPORTED_MODULE_10__.getStringsFromCategories(props, axis) : [];
+  var stringArray = _collection__WEBPACK_IMPORTED_MODULE_8__.containsStrings(categoriesArray) ? _data__WEBPACK_IMPORTED_MODULE_10__.getStringsFromCategories(props, axis) : [];
   var stringMap = stringArray.length === 0 ? null : stringArray.reduce(function (memo, string, index) {
     memo[string] = index + 1;
     return memo;
   }, {});
-  var categoryValues = stringMap ? categories.map(function (value) {
+  var categoryValues = stringMap ? categoriesArray.map(function (value) {
     return stringMap[value];
-  }) : categories;
+  }) : categoriesArray;
   var min = minDomain !== undefined ? minDomain : _collection__WEBPACK_IMPORTED_MODULE_8__.getMinValue(categoryValues);
   var max = maxDomain !== undefined ? maxDomain : _collection__WEBPACK_IMPORTED_MODULE_8__.getMaxValue(categoryValues);
   var categoryDomain = getDomainFromMinMax(min, max);
@@ -10104,7 +10106,7 @@ function getDomainFromCategories(props, axis, categories) {
  */
 
 function getDomainFromData(props, axis, dataset) {
-  dataset = dataset || _data__WEBPACK_IMPORTED_MODULE_10__.getData(props);
+  var datasetArray = dataset || _data__WEBPACK_IMPORTED_MODULE_10__.getData(props);
   var polar = props.polar,
       _props$startAngle2 = props.startAngle,
       startAngle = _props$startAngle2 === void 0 ? 0 : _props$startAngle2,
@@ -10113,14 +10115,14 @@ function getDomainFromData(props, axis, dataset) {
   var minDomain = getMinFromProps(props, axis);
   var maxDomain = getMaxFromProps(props, axis);
 
-  if (dataset.length < 1) {
+  if (datasetArray.length < 1) {
     return minDomain !== undefined && maxDomain !== undefined ? getDomainFromMinMax(minDomain, maxDomain) : undefined;
   }
 
-  var min = minDomain !== undefined ? minDomain : getExtremeFromData(dataset, axis, "min");
-  var max = maxDomain !== undefined ? maxDomain : getExtremeFromData(dataset, axis, "max");
+  var min = minDomain !== undefined ? minDomain : getExtremeFromData(datasetArray, axis, "min");
+  var max = maxDomain !== undefined ? maxDomain : getExtremeFromData(datasetArray, axis, "max");
   var domain = getDomainFromMinMax(min, max);
-  return polar && axis === "x" && Math.abs(startAngle - endAngle) === 360 ? getSymmetricDomain(domain, getFlatData(dataset, axis)) : domain;
+  return polar && axis === "x" && Math.abs(startAngle - endAngle) === 360 ? getSymmetricDomain(domain, getFlatData(datasetArray, axis)) : domain;
 }
 /**
  * Returns a domain in the form of a two element array given a min and max value.
@@ -10433,14 +10435,14 @@ function getScopedEvents(events, namespace, childType, baseProps) {
   // eslint-disable-next-line no-invalid-this
 
 
-  baseProps = baseProps || this.baseProps; // returns the original base props or base state of a given target element
+  var newBaseProps = baseProps || this.baseProps; // returns the original base props or base state of a given target element
 
   var getTargetProps = function (identifier, type) {
     var childName = identifier.childName,
         target = identifier.target,
         key = identifier.key; // eslint-disable-next-line no-invalid-this
 
-    var baseType = type === "props" ? baseProps : _this2.state || {};
+    var baseType = type === "props" ? newBaseProps : _this2.state || {};
     var base = childName === undefined || childName === null || !baseType[childName] ? baseType : baseType[childName];
     return key === "parent" ? base.parent : base[key] && base[key][target];
   }; // Returns the state object with the mutation caused by a given eventReturn
@@ -10528,8 +10530,7 @@ function getScopedEvents(events, namespace, childType, baseProps) {
 
   var parseEventReturn = function (eventReturn, eventKey) {
     return Array.isArray(eventReturn) ? eventReturn.reduce(function (memo, props) {
-      memo = lodash_assign__WEBPACK_IMPORTED_MODULE_8___default()({}, memo, parseEvent(props, eventKey));
-      return memo;
+      return lodash_assign__WEBPACK_IMPORTED_MODULE_8___default()({}, memo, parseEvent(props, eventKey));
     }, {}) : parseEvent(eventReturn, eventKey);
   };
 
@@ -10615,9 +10616,10 @@ function getEventState(eventKey, namespace, childType) {
  */
 // eslint-disable-next-line max-params
 
-function getExternalMutationsWithChildren(mutations, baseProps, baseState, childNames) {
-  baseProps = baseProps || {};
-  baseState = baseState || {};
+function getExternalMutationsWithChildren(mutations) {
+  var baseProps = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
+  var baseState = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
+  var childNames = arguments.length > 3 ? arguments[3] : undefined;
   return childNames.reduce(function (memo, childName) {
     var childState = baseState[childName];
     var mutation = getExternalMutations(mutations, baseProps[childName], baseState[childName], childName);
@@ -10639,9 +10641,10 @@ function getExternalMutationsWithChildren(mutations, baseProps, baseState, child
  */
 // eslint-disable-next-line max-params
 
-function getExternalMutations(mutations, baseProps, baseState, childName) {
-  baseProps = baseProps || {};
-  baseState = baseState || {};
+function getExternalMutations(mutations) {
+  var baseProps = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
+  var baseState = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
+  var childName = arguments.length > 3 ? arguments[3] : undefined;
 
   var eventKeys = lodash_keys__WEBPACK_IMPORTED_MODULE_0___default()(baseProps);
 
@@ -10707,8 +10710,7 @@ function getExternalMutation(mutations, baseProps, baseState, identifier) {
     return false;
   };
 
-  mutations = Array.isArray(mutations) ? mutations : [mutations];
-  var scopedMutations = mutations;
+  var scopedMutations = Array.isArray(mutations) ? mutations : [mutations];
 
   if (identifier.childName) {
     scopedMutations = mutations.filter(function (m) {
@@ -10747,13 +10749,10 @@ function getExternalMutation(mutations, baseProps, baseState, identifier) {
 
 function getComponentEvents(props, components) {
   var events = Array.isArray(components) && components.reduce(function (memo, componentName) {
-    var _memo;
-
     var component = props[componentName];
     var defaultEvents = component && component.type && component.type.defaultEvents;
     var componentEvents = lodash_isFunction__WEBPACK_IMPORTED_MODULE_6___default()(defaultEvents) ? defaultEvents(component.props) : defaultEvents;
-    memo = Array.isArray(componentEvents) ? (_memo = memo).concat.apply(_memo, _toConsumableArray(componentEvents)) : memo;
-    return memo;
+    return Array.isArray(componentEvents) ? memo.concat.apply(memo, _toConsumableArray(componentEvents)) : memo;
   }, []);
   return events && events.length ? events : undefined;
 }
@@ -11111,6 +11110,7 @@ function reduceChildren(children, iteratee) {
 
   var traverseChildren = function (childArray, names, parent) {
     return childArray.reduce(function (memo, child, index) {
+      var newMemo = memo;
       var childRole = child.type && child.type.role;
       var childName = child.props.name || "".concat(childRole, "-").concat(names[index]);
 
@@ -11128,16 +11128,16 @@ function reduceChildren(children, iteratee) {
         });
 
         var nestedResults = traverseChildren(nestedChildren, _childNames, child);
-        memo = combine(memo, nestedResults);
+        newMemo = combine(newMemo, nestedResults);
       } else {
         var result = iteratee(child, childName, parent);
 
         if (result) {
-          memo = combine(memo, result);
+          newMemo = combine(newMemo, result);
         }
       }
 
-      return memo;
+      return newMemo;
     }, initialMemo);
   };
 
@@ -11163,8 +11163,7 @@ function isHorizontal(props) {
       var childProps = child.props || {};
 
       if (memo || childProps.horizontal || !childProps.children) {
-        memo = memo || childProps.horizontal;
-        return memo;
+        return memo || childProps.horizontal;
       }
 
       return traverseChildren(react__WEBPACK_IMPORTED_MODULE_6___default().Children.toArray(childProps.children));
@@ -11399,11 +11398,13 @@ function isMap(x) {
 }
 function shallowToJS(x, whitelist) {
   return isIterable(x) ? x.reduce(function (result, curr, key) {
+    var newCurr = curr;
+
     if (whitelist && whitelist[key]) {
-      curr = shallowToJS(curr);
+      newCurr = shallowToJS(curr);
     }
 
-    result[key] = curr;
+    result[key] = newCurr;
     return result;
   }, isList(x) ? [] : {}) : x;
 }
@@ -12050,11 +12051,10 @@ var Timer = /*#__PURE__*/function () {
   }, {
     key: "subscribe",
     value: function subscribe(callback, duration) {
-      duration = this.shouldAnimate ? duration : 0;
       var subscriptionID = this.subscribers.push({
         startTime: (0,victory_vendor_d3_timer__WEBPACK_IMPORTED_MODULE_0__.now)(),
         callback: callback,
-        duration: duration
+        duration: this.shouldAnimate ? duration : 0
       });
       this.activeSubscriptions++;
       this.start();
@@ -12241,36 +12241,38 @@ function getInitialChildProps(animate, data) {
 
 
 function getChildBeforeLoad(animate, child, data, cb) {
-  animate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, {
+  var newAnimate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, {
     onEnd: cb
   });
 
-  if (animate && animate.onLoad && !animate.onLoad.duration) {
+  if (newAnimate && newAnimate.onLoad && !newAnimate.onLoad.duration) {
     return {
-      animate: animate,
+      animate: newAnimate,
       data: data
     };
   }
 
-  var before = animate.onLoad && animate.onLoad.before ? animate.onLoad.before : (lodash_identity__WEBPACK_IMPORTED_MODULE_1___default()); // If nodes need to exit, transform them with the provided onLoad.before function.
+  var before = newAnimate.onLoad && newAnimate.onLoad.before ? newAnimate.onLoad.before : (lodash_identity__WEBPACK_IMPORTED_MODULE_1___default()); // If nodes need to exit, transform them with the provided onLoad.before function.
 
-  data = data.map(function (datum, idx) {
+  var newData = data.map(function (datum, idx) {
     return lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, datum, before(datum, idx, data));
   });
   return {
-    animate: animate,
-    data: data,
+    animate: newAnimate,
+    data: newData,
     clipWidth: 0
   };
 } // eslint-disable-next-line max-params
 
 
 function getChildOnLoad(animate, data, cb) {
-  animate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, {
+  var newAnimate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, {
     onEnd: cb
   });
 
-  if (animate && animate.onLoad && !animate.onLoad.duration) {
+  var newData = data;
+
+  if (newAnimate && newAnimate.onLoad && !newAnimate.onLoad.duration) {
     return {
       animate: animate,
       data: data
@@ -12279,12 +12281,12 @@ function getChildOnLoad(animate, data, cb) {
 
   var after = animate.onLoad && animate.onLoad.after ? animate.onLoad.after : (lodash_identity__WEBPACK_IMPORTED_MODULE_1___default()); // If nodes need to exit, transform them with the provided onLoad.after function.
 
-  data = data.map(function (datum, idx) {
+  newData = data.map(function (datum, idx) {
     return lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, datum, after(datum, idx, data));
   });
   return {
-    animate: animate,
-    data: data
+    animate: newAnimate,
+    data: newData
   };
 } // eslint-disable-next-line max-params, max-len
 
@@ -12293,7 +12295,10 @@ function getChildPropsOnExit(animate, child, data, exitingNodes, cb) {
   // Whether or not _this_ child has exiting nodes, we want the exit-
   // transition for all children to have the same duration, delay, etc.
   var onExit = animate && animate.onExit;
-  animate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, onExit);
+
+  var newAnimate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, onExit);
+
+  var newData = data;
 
   if (exitingNodes) {
     // After the exit transition occurs, trigger the animations for
@@ -12301,39 +12306,42 @@ function getChildPropsOnExit(animate, child, data, exitingNodes, cb) {
     animate.onEnd = cb;
     var before = animate.onExit && animate.onExit.before ? animate.onExit.before : (lodash_identity__WEBPACK_IMPORTED_MODULE_1___default()); // If nodes need to exit, transform them with the provided onExit.before function.
 
-    data = data.map(function (datum, idx) {
+    newData = data.map(function (datum, idx) {
       var key = (datum.key || idx).toString();
       return exitingNodes[key] ? lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, datum, before(datum, idx, data)) : datum;
     });
   }
 
   return {
-    animate: animate,
-    data: data
+    animate: newAnimate,
+    data: newData
   };
 } // eslint-disable-next-line max-params,max-len
 
 
 function getChildPropsBeforeEnter(animate, child, data, enteringNodes, cb) {
+  var newAnimate = animate;
+  var newData = data;
+
   if (enteringNodes) {
     // Perform a normal animation here, except - when it finishes - trigger
     // the transition for entering nodes.
-    animate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, {
+    newAnimate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, {
       onEnd: cb
     });
     var before = animate.onEnter && animate.onEnter.before ? animate.onEnter.before : (lodash_identity__WEBPACK_IMPORTED_MODULE_1___default()); // We want the entering nodes to be included in the transition target
     // domain.  However, we may not want these nodes to be displayed initially,
     // so perform the `onEnter.before` transformation on each node.
 
-    data = data.map(function (datum, idx) {
+    newData = data.map(function (datum, idx) {
       var key = (datum.key || idx).toString();
       return enteringNodes[key] ? lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, datum, before(datum, idx, data)) : datum;
     });
   }
 
   return {
-    animate: animate,
-    data: data
+    animate: newAnimate,
+    data: newData
   };
 } // eslint-disable-next-line max-params, max-len
 
@@ -12342,23 +12350,26 @@ function getChildPropsOnEnter(animate, data, enteringNodes, cb) {
   // Whether or not _this_ child has entering nodes, we want the entering-
   // transition for all children to have the same duration, delay, etc.
   var onEnter = animate && animate.onEnter;
-  animate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, onEnter);
+
+  var newAnimate = lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, animate, onEnter);
+
+  var newData = data;
 
   if (enteringNodes) {
     // Old nodes have been transitioned to their new values, and the
     // domain should encompass the nodes that will now enter. So perform
     // the `onEnter.after` transformation on each node.
-    animate.onEnd = cb;
-    var after = animate.onEnter && animate.onEnter.after ? animate.onEnter.after : (lodash_identity__WEBPACK_IMPORTED_MODULE_1___default());
-    data = data.map(function (datum, idx) {
+    newAnimate.onEnd = cb;
+    var after = newAnimate.onEnter && newAnimate.onEnter.after ? newAnimate.onEnter.after : (lodash_identity__WEBPACK_IMPORTED_MODULE_1___default());
+    newData = data.map(function (datum, idx) {
       var key = getDatumKey(datum, idx);
       return enteringNodes[key] ? lodash_assign__WEBPACK_IMPORTED_MODULE_3___default()({}, datum, after(datum, idx, data)) : datum;
     });
   }
 
   return {
-    animate: animate,
-    data: data
+    animate: newAnimate,
+    data: newData
   };
 }
 /**
@@ -12524,6 +12535,7 @@ function getTransitionPropsFactory(props, state, setState) {
 "use strict";
 __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
+/* harmony export */   "assert": () => (/* binding */ assert),
 /* harmony export */   "getSafeUserProps": () => (/* binding */ getSafeUserProps),
 /* harmony export */   "withSafeUserProps": () => (/* binding */ withSafeUserProps)
 /* harmony export */ });
@@ -12604,6 +12616,18 @@ var testIfSafeProp = function (key) {
   if (doesPropStartWith(key) || isExactMatch(key)) return true;
   return false;
 };
+/**
+ * Asserts that value is not null or undefined, throwing an error if it is.
+ * @param value The value to assert
+ * @param message The error message to throw
+ */
+
+
+function assert(value, message) {
+  if (value === undefined || value === null) {
+    throw new Error(message);
+  }
+}
 /**
  * getSafeUserProps - function that takes in a props object and removes any
  * key-value entries that do not match filter strings in the USER_PROPS_SAFELIST
@@ -12613,7 +12637,6 @@ var testIfSafeProp = function (key) {
  * @returns {Object}: object containing remaining acceptable props
  */
 
-
 var getSafeUserProps = function (props) {
   var propsToFilter = _objectSpread({}, props);
 
@@ -12816,12 +12839,13 @@ function getDataFromChildren(props, childComponents) {
     var childProps = lodash_assign__WEBPACK_IMPORTED_MODULE_8___default()({}, child.props, parentProps);
 
     var childData;
+    var childElement = child;
 
     if (!_data__WEBPACK_IMPORTED_MODULE_12__.isDataComponent(child)) {
       return null;
     } else if (child.type && lodash_isFunction__WEBPACK_IMPORTED_MODULE_5___default()(child.type.getData)) {
-      child = parent ? /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_9___default().cloneElement(child, parent.props) : child;
-      childData = child.type.getData(childProps);
+      childElement = parent ? /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_9___default().cloneElement(child, parent.props) : child;
+      childData = childElement.type.getData(childProps);
     } else {
       childData = _data__WEBPACK_IMPORTED_MODULE_12__.getData(childProps);
     }
@@ -12852,17 +12876,16 @@ function getData(props, childComponents) {
     return _data__WEBPACK_IMPORTED_MODULE_12__.getData(props);
   }
 
-  childComponents = childComponents || react__WEBPACK_IMPORTED_MODULE_9___default().Children.toArray(props.children);
-  return getDataFromChildren(props, childComponents);
+  return getDataFromChildren(props, childComponents || react__WEBPACK_IMPORTED_MODULE_9___default().Children.toArray(props.children));
 }
 function getWidth(props, groupLength, seriesLength) {
   var datasets = props.datasets,
       horizontal = props.horizontal;
   var range = horizontal ? _helpers__WEBPACK_IMPORTED_MODULE_11__.getRange(props, "y") : _helpers__WEBPACK_IMPORTED_MODULE_11__.getRange(props, "x");
   var extent = Math.abs(range[1] - range[0]);
-  seriesLength = seriesLength !== undefined ? seriesLength : Array.isArray(datasets[0]) && datasets[0].length || 1;
-  groupLength = groupLength || datasets.length;
-  var bars = groupLength * seriesLength;
+  var seriesLengthValue = seriesLength !== undefined ? seriesLength : Array.isArray(datasets[0]) && datasets[0].length || 1;
+  var groupLengthValue = groupLength || datasets.length;
+  var bars = groupLengthValue * seriesLengthValue;
   var barRatio = 0.5;
   return Math.round(barRatio * extent / bars);
 }
@@ -12954,9 +12977,9 @@ function getDomainFromChildren(props, axis, childComponents) {
   return [min, max];
 }
 function getDomain(props, axis, childComponents) {
-  childComponents = childComponents || react__WEBPACK_IMPORTED_MODULE_9___default().Children.toArray(props.children);
+  var children = childComponents || react__WEBPACK_IMPORTED_MODULE_9___default().Children.toArray(props.children);
   var propsDomain = _domain__WEBPACK_IMPORTED_MODULE_13__.getDomainFromProps(props, axis);
-  var domainPadding = getDefaultDomainPadding(props, axis, childComponents);
+  var domainPadding = getDefaultDomainPadding(props, axis, children);
   var domain;
 
   if (propsDomain) {
@@ -12966,7 +12989,7 @@ function getDomain(props, axis, childComponents) {
     var maxDomain = _domain__WEBPACK_IMPORTED_MODULE_13__.getMaxFromProps(props, axis);
     var dataset = (props.data || props.y) && _data__WEBPACK_IMPORTED_MODULE_12__.getData(props);
     var dataDomain = dataset ? _domain__WEBPACK_IMPORTED_MODULE_13__.getDomainFromData(props, axis, dataset) : [];
-    var childDomain = getDomainFromChildren(props, axis, childComponents);
+    var childDomain = getDomainFromChildren(props, axis, children);
     var min = minDomain || _collection__WEBPACK_IMPORTED_MODULE_14__.getMinValue([].concat(_toConsumableArray(dataDomain), _toConsumableArray(childDomain)));
     var max = maxDomain || _collection__WEBPACK_IMPORTED_MODULE_14__.getMaxValue([].concat(_toConsumableArray(dataDomain), _toConsumableArray(childDomain)));
     domain = _domain__WEBPACK_IMPORTED_MODULE_13__.getDomainFromMinMax(min, max);
@@ -13128,10 +13151,10 @@ function getCategoryAndAxisStringsFromChildren(props, axis, childComponents) {
   return lodash_uniq__WEBPACK_IMPORTED_MODULE_4___default()(lodash_flatten__WEBPACK_IMPORTED_MODULE_6___default()([].concat(_toConsumableArray(categoryStrings), _toConsumableArray(axisStrings))));
 }
 function getStringsFromChildren(props, childComponents) {
-  childComponents = childComponents || react__WEBPACK_IMPORTED_MODULE_9___default().Children.toArray(props.children);
-  var xStrings = getCategoryAndAxisStringsFromChildren(props, "x", childComponents);
-  var yStrings = getCategoryAndAxisStringsFromChildren(props, "y", childComponents);
-  var dataStrings = getStringsFromData(childComponents);
+  var children = childComponents || react__WEBPACK_IMPORTED_MODULE_9___default().Children.toArray(props.children);
+  var xStrings = getCategoryAndAxisStringsFromChildren(props, "x", children);
+  var yStrings = getCategoryAndAxisStringsFromChildren(props, "y", children);
+  var dataStrings = getStringsFromData(children);
   return {
     x: lodash_uniq__WEBPACK_IMPORTED_MODULE_4___default()(lodash_flatten__WEBPACK_IMPORTED_MODULE_6___default()([].concat(_toConsumableArray(xStrings), _toConsumableArray(dataStrings.x)))),
     y: lodash_uniq__WEBPACK_IMPORTED_MODULE_4___default()(lodash_flatten__WEBPACK_IMPORTED_MODULE_6___default()([].concat(_toConsumableArray(yStrings), _toConsumableArray(dataStrings.y))))
@@ -13161,7 +13184,7 @@ function getCategories(props, childComponents, allStrings) {
 "use strict";
 __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
-/* harmony export */   "default": () => (/* binding */ VictorySharedEvents)
+/* harmony export */   "VictorySharedEvents": () => (/* binding */ VictorySharedEvents)
 /* harmony export */ });
 /* harmony import */ var lodash_difference__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! lodash/difference */ "../../../node_modules/.pnpm/lodash@4.17.21/node_modules/lodash/difference.js");
 /* harmony import */ var lodash_difference__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(lodash_difference__WEBPACK_IMPORTED_MODULE_0__);
@@ -13179,12 +13202,9 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var lodash_assign__WEBPACK_IMPORTED_MODULE_6___default = /*#__PURE__*/__webpack_require__.n(lodash_assign__WEBPACK_IMPORTED_MODULE_6__);
 /* harmony import */ var react__WEBPACK_IMPORTED_MODULE_7__ = __webpack_require__(/*! react */ "react");
 /* harmony import */ var react__WEBPACK_IMPORTED_MODULE_7___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_7__);
-/* harmony import */ var prop_types__WEBPACK_IMPORTED_MODULE_12__ = __webpack_require__(/*! prop-types */ "../../../node_modules/.pnpm/prop-types@15.8.1/node_modules/prop-types/index.js");
-/* harmony import */ var prop_types__WEBPACK_IMPORTED_MODULE_12___default = /*#__PURE__*/__webpack_require__.n(prop_types__WEBPACK_IMPORTED_MODULE_12__);
 /* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_10__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/events.js");
 /* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_11__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/helpers.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_13__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/prop-types.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_14__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/timer-context.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_12__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/timer-context.js");
 /* harmony import */ var react_fast_compare__WEBPACK_IMPORTED_MODULE_8__ = __webpack_require__(/*! react-fast-compare */ "../../../node_modules/.pnpm/react-fast-compare@3.2.0/node_modules/react-fast-compare/index.js");
 /* harmony import */ var react_fast_compare__WEBPACK_IMPORTED_MODULE_8___default = /*#__PURE__*/__webpack_require__.n(react_fast_compare__WEBPACK_IMPORTED_MODULE_8__);
 /* harmony import */ var json_stringify_safe__WEBPACK_IMPORTED_MODULE_9__ = __webpack_require__(/*! json-stringify-safe */ "../../../node_modules/.pnpm/json-stringify-safe@5.0.1/node_modules/json-stringify-safe/stringify.js");
@@ -13241,11 +13261,6 @@ function _getPrototypeOf(o) { _getPrototypeOf = Object.setPrototypeOf ? Object.g
 
 
 
- // DISCLAIMER:
-// This file is not currently tested, and it is first on the list of files
-// to refactor in our current refactoring effort. Please do not make changes
-// to this file without manual testing and/or refactoring and adding tests.
-
 var VictorySharedEvents = /*#__PURE__*/function (_React$Component) {
   _inherits(VictorySharedEvents, _React$Component);
 
@@ -13257,14 +13272,21 @@ var VictorySharedEvents = /*#__PURE__*/function (_React$Component) {
     _classCallCheck(this, VictorySharedEvents);
 
     _this = _super.call(this, props);
-    _this.state = _this.state || {};
+    _this.getScopedEvents = void 0;
+    _this.getEventState = void 0;
+    _this.baseProps = void 0;
+    _this.sharedEventsCache = void 0;
+    _this.globalEvents = void 0;
+    _this.prevGlobalEventKeys = void 0;
+    _this.boundGlobalEvents = void 0;
     _this.getScopedEvents = victory_core__WEBPACK_IMPORTED_MODULE_10__.getScopedEvents.bind(_assertThisInitialized(_this));
     _this.getEventState = victory_core__WEBPACK_IMPORTED_MODULE_10__.getEventState.bind(_assertThisInitialized(_this));
-    _this.baseProps = _this.getBaseProps(props);
+    _this.state = _this.state || {};
     _this.sharedEventsCache = {};
     _this.globalEvents = {};
     _this.prevGlobalEventKeys = [];
     _this.boundGlobalEvents = {};
+    _this.baseProps = _this.getBaseProps(props);
     return _this;
   }
 
@@ -13355,8 +13377,7 @@ var VictorySharedEvents = /*#__PURE__*/function (_React$Component) {
     value: function applyExternalMutations(props, externalMutations) {
       if (!lodash_isEmpty__WEBPACK_IMPORTED_MODULE_3___default()(externalMutations)) {
         var callbacks = props.externalEventMutations.reduce(function (memo, mutation) {
-          memo = lodash_isFunction__WEBPACK_IMPORTED_MODULE_5___default()(mutation.callback) ? memo.concat(mutation.callback) : memo;
-          return memo;
+          return lodash_isFunction__WEBPACK_IMPORTED_MODULE_5___default()(mutation.callback) ? memo.concat(mutation.callback) : memo;
         }, []);
         var compiledCallbacks = callbacks.length ? function () {
           callbacks.forEach(function (c) {
@@ -13479,9 +13500,9 @@ var VictorySharedEvents = /*#__PURE__*/function (_React$Component) {
       var _this7 = this;
 
       var children = this.getNewChildren(props, baseProps);
-      var parents = Array.isArray(events) && events.filter(function (event) {
+      var parents = Array.isArray(events) ? events.filter(function (event) {
         return event.target === "parent";
-      });
+      }) : [];
       var sharedEvents = parents.length > 0 ? {
         events: parents,
         // partially apply childName (null) and baseProps,
@@ -13527,33 +13548,9 @@ var VictorySharedEvents = /*#__PURE__*/function (_React$Component) {
 
   return VictorySharedEvents;
 }((react__WEBPACK_IMPORTED_MODULE_7___default().Component));
-
 VictorySharedEvents.displayName = "VictorySharedEvents";
 VictorySharedEvents.role = "shared-event-wrapper";
-VictorySharedEvents.propTypes = {
-  children: prop_types__WEBPACK_IMPORTED_MODULE_12___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_12___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_12___default().node)), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().node)]),
-  container: (prop_types__WEBPACK_IMPORTED_MODULE_12___default().node),
-  eventKey: prop_types__WEBPACK_IMPORTED_MODULE_12___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_12___default().array), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().func), victory_core__WEBPACK_IMPORTED_MODULE_13__.allOfType([victory_core__WEBPACK_IMPORTED_MODULE_13__.integer, victory_core__WEBPACK_IMPORTED_MODULE_13__.nonNegative]), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().string)]),
-  events: prop_types__WEBPACK_IMPORTED_MODULE_12___default().arrayOf(prop_types__WEBPACK_IMPORTED_MODULE_12___default().shape({
-    childName: prop_types__WEBPACK_IMPORTED_MODULE_12___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_12___default().string), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().array)]),
-    eventHandlers: (prop_types__WEBPACK_IMPORTED_MODULE_12___default().object),
-    eventKey: prop_types__WEBPACK_IMPORTED_MODULE_12___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_12___default().array), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().func), victory_core__WEBPACK_IMPORTED_MODULE_13__.allOfType([victory_core__WEBPACK_IMPORTED_MODULE_13__.integer, victory_core__WEBPACK_IMPORTED_MODULE_13__.nonNegative]), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().string)]),
-    target: (prop_types__WEBPACK_IMPORTED_MODULE_12___default().string)
-  })),
-  externalEventMutations: prop_types__WEBPACK_IMPORTED_MODULE_12___default().arrayOf(prop_types__WEBPACK_IMPORTED_MODULE_12___default().shape({
-    callback: (prop_types__WEBPACK_IMPORTED_MODULE_12___default().func),
-    childName: prop_types__WEBPACK_IMPORTED_MODULE_12___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_12___default().string), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().array)]),
-    eventKey: prop_types__WEBPACK_IMPORTED_MODULE_12___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_12___default().array), victory_core__WEBPACK_IMPORTED_MODULE_13__.allOfType([victory_core__WEBPACK_IMPORTED_MODULE_13__.integer, victory_core__WEBPACK_IMPORTED_MODULE_13__.nonNegative]), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().string)]),
-    mutation: (prop_types__WEBPACK_IMPORTED_MODULE_12___default().func),
-    target: prop_types__WEBPACK_IMPORTED_MODULE_12___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_12___default().string), (prop_types__WEBPACK_IMPORTED_MODULE_12___default().array)])
-  })),
-  groupComponent: (prop_types__WEBPACK_IMPORTED_MODULE_12___default().node)
-};
-VictorySharedEvents.defaultProps = {
-  groupComponent: /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_7___default().createElement("g", null)
-};
-VictorySharedEvents.contextType = victory_core__WEBPACK_IMPORTED_MODULE_14__["default"];
-
+VictorySharedEvents.contextType = victory_core__WEBPACK_IMPORTED_MODULE_12__["default"];
 
 /***/ }),