victory-core 36.8.1 → 36.8.3

package/dist/victory-core.js

@@ -224,12 +224,12 @@ var interpolateFunction = function (a, b) {
  * differs in that it uses our custom interpolators when interpolating the value of each property in
  * an object. This allows the correct interpolation of nested objects, including styles
  *
- * @param {any} a - Start value.
- * @param {any} b - End value.
+ * @param {any} startValue - Start value.
+ * @param {any} endValue - End value.
  * @returns {Function} An interpolation function.
  */
 
-var interpolateObject = function (a, b) {
+var interpolateObject = function (startValue, endValue) {
   var interpolateTypes = function (x, y) {
     if (x === y || !isInterpolatable(x) || !isInterpolatable(y)) {
       return interpolateImmediate(x, y);
@@ -254,6 +254,8 @@ var interpolateObject = function (a, b) {
 
   var i = {};
   var c = {};
+  var a = startValue;
+  var b = endValue;
   var k;
 
   if (a === null || typeof a !== "object") {
@@ -421,8 +423,8 @@ var VictoryAnimation = /*#__PURE__*/function (_React$Component) {
         step can generate imprecise values, sometimes greater than 1
         if this happens set the state to 1 and return, cancelling the timer
       */
-      duration = duration !== undefined ? duration : _this.props.duration;
-      var step = duration ? elapsed / duration : 1;
+      var animationDuration = duration !== undefined ? duration : _this.props.duration;
+      var step = animationDuration ? elapsed / animationDuration : 1;
 
       if (step >= 1) {
         _this.setState({
@@ -1348,8 +1350,7 @@ var getStyles = function (style, props) {
   }
 
   var getSingleStyle = function (s) {
-    s = s ? lodash_defaults__WEBPACK_IMPORTED_MODULE_1___default()({}, s, defaultStyles) : defaultStyles;
-    var baseStyles = _victory_util_helpers__WEBPACK_IMPORTED_MODULE_4__.evaluateStyle(s, props);
+    var baseStyles = _victory_util_helpers__WEBPACK_IMPORTED_MODULE_4__.evaluateStyle(s ? lodash_defaults__WEBPACK_IMPORTED_MODULE_1___default()({}, s, defaultStyles) : defaultStyles, props);
     return lodash_assign__WEBPACK_IMPORTED_MODULE_2___default()({}, baseStyles, {
       fontSize: getFontSize(baseStyles)
     });
@@ -1553,22 +1554,19 @@ var getInlineXOffset = function (calculatedProps, textElements, index) {
   switch (textAnchor) {
     case "start":
       return widths.reduce(function (memo, width, i) {
-        memo = i < index ? memo + width : memo;
-        return memo;
+        return i < index ? memo + width : memo;
       }, 0);
 
     case "end":
       return widths.reduce(function (memo, width, i) {
-        memo = i > index ? memo - width : memo;
-        return memo;
+        return i > index ? memo - width : memo;
       }, 0);
 
     default:
       // middle
       return widths.reduce(function (memo, width, i) {
         var offsetWidth = i < index ? width : 0;
-        memo = i === index ? memo + width / 2 : memo + offsetWidth;
-        return memo;
+        return i === index ? memo + width / 2 : memo + offsetWidth;
       }, centerOffset);
   }
 };
@@ -1700,7 +1698,7 @@ var getCalculatedProps = function (props) {
     verticalAnchor: verticalAnchor,
     dx: dx,
     dy: dy,
-    originalDy: props.dy,
+    originalDy: _victory_util_helpers__WEBPACK_IMPORTED_MODULE_4__.evaluateProp(props.dy, props),
     transform: transform,
     x: x,
     y: y
@@ -1770,8 +1768,8 @@ var defaultProps = {
   // Magic number from d3.
   lineHeight: 1
 };
-var VictoryLabel = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+var VictoryLabel = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
 
   if (props.text === null || props.text === undefined) {
     return null;
@@ -1792,7 +1790,7 @@ var VictoryLabel = function (props) {
       fontSize: currentStyle.fontSize || defaultStyles.fontSize,
       capHeight: capHeightPx,
       text: line,
-      // @ts-expect-error TODO: This looks like a bug:
+      // TODO: This looks like a bug:
       textSize: _victory_util_textsize__WEBPACK_IMPORTED_MODULE_12__.approximateTextSize(line, currentStyle),
       lineHeight: currentLineHeight,
       backgroundPadding: getSingleValue(backgroundPadding, i)
@@ -2232,8 +2230,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-var Arc = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+var Arc = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default().cloneElement(props.pathComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,
     d: getArcPath(props),
@@ -2313,8 +2311,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-var Background = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+var Background = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return props.polar ? /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default().cloneElement(props.circleComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     style: props.style,
     role: props.role,
@@ -2413,8 +2411,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-var Border = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+var Border = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default().cloneElement(props.rectComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,
     style: props.style,
@@ -2639,8 +2637,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-var LineSegment = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+var LineSegment = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default().cloneElement(props.lineComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,
     style: props.style,
@@ -2868,8 +2866,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-var Point = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+var Point = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   var userProps = _victory_util_user_props__WEBPACK_IMPORTED_MODULE_5__.getSafeUserProps(props);
   return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default().cloneElement(props.pathComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,
@@ -3118,23 +3116,22 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-var Whisker = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
-  var _props = props,
-      ariaLabel = _props.ariaLabel,
-      groupComponent = _props.groupComponent,
-      lineComponent = _props.lineComponent,
-      events = _props.events,
-      className = _props.className,
-      majorWhisker = _props.majorWhisker,
-      minorWhisker = _props.minorWhisker,
-      transform = _props.transform,
-      clipPath = _props.clipPath,
-      role = _props.role,
-      shapeRendering = _props.shapeRendering,
-      style = _props.style,
-      desc = _props.desc,
-      tabIndex = _props.tabIndex;
+var Whisker = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
+  var ariaLabel = props.ariaLabel,
+      groupComponent = props.groupComponent,
+      lineComponent = props.lineComponent,
+      events = props.events,
+      className = props.className,
+      majorWhisker = props.majorWhisker,
+      minorWhisker = props.minorWhisker,
+      transform = props.transform,
+      clipPath = props.clipPath,
+      role = props.role,
+      shapeRendering = props.shapeRendering,
+      style = props.style,
+      desc = props.desc,
+      tabIndex = props.tabIndex;
 
   var baseProps = _objectSpread(_objectSpread({}, events), {}, {
     style: style,
@@ -4301,8 +4298,7 @@ function addEvents(WrappedComponent) {
       value: function applyExternalMutations(props, externalMutations) {
         if (!lodash_isEmpty__WEBPACK_IMPORTED_MODULE_5___default()(externalMutations)) {
           var callbacks = props.externalEventMutations.reduce(function (memo, mutation) {
-            memo = lodash_isFunction__WEBPACK_IMPORTED_MODULE_4___default()(mutation.callback) ? memo.concat(mutation.callback) : memo;
-            return memo;
+            return lodash_isFunction__WEBPACK_IMPORTED_MODULE_4___default()(mutation.callback) ? memo.concat(mutation.callback) : memo;
           }, []);
           var compiledCallbacks = callbacks.length ? function () {
             callbacks.forEach(function (c) {
@@ -4357,8 +4353,8 @@ function addEvents(WrappedComponent) {
     }, {
       key: "getBaseProps",
       value: function getBaseProps(props, getSharedEventState) {
-        getSharedEventState = getSharedEventState || this.getSharedEventState.bind(this);
-        var sharedParentState = getSharedEventState("parent", "parent");
+        var getSharedEventStateFunction = getSharedEventState || this.getSharedEventState.bind(this);
+        var sharedParentState = getSharedEventStateFunction("parent", "parent");
         var parentState = this.getEventState("parent", "parent");
 
         var baseParentProps = lodash_defaults__WEBPACK_IMPORTED_MODULE_7___default()({}, parentState, sharedParentState);
@@ -4393,13 +4389,15 @@ function addEvents(WrappedComponent) {
           return undefined;
         }
 
+        var currentProps = component && typeof component === "object" && "props" in component ? component.props : undefined;
+
         if (this.hasEvents) {
           var baseEvents = this.getEvents(this.props, type, key);
 
           var componentProps = lodash_defaults__WEBPACK_IMPORTED_MODULE_7___default()({
             index: index,
             key: id
-          }, this.getEventState(key, type), this.getSharedEventState(key, type), component.props, baseProps, {
+          }, this.getEventState(key, type), this.getSharedEventState(key, type), currentProps, baseProps, {
             id: id
           });
 
@@ -4413,7 +4411,7 @@ function addEvents(WrappedComponent) {
         return lodash_defaults__WEBPACK_IMPORTED_MODULE_7___default()({
           index: index,
           key: id
-        }, component.props, baseProps, {
+        }, currentProps, baseProps, {
           id: id
         });
       }
@@ -4455,13 +4453,15 @@ function addEvents(WrappedComponent) {
         var dataKeys = lodash_without__WEBPACK_IMPORTED_MODULE_0___default()(this.dataKeys, "all");
 
         var labelComponents = dataKeys.reduce(function (memo, key) {
+          var newMemo = memo;
+
           var labelProps = _this8.getComponentProps(labelComponent, "labels", key);
 
           if (labelProps && labelProps.text !== undefined && labelProps.text !== null) {
-            memo = memo.concat( /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_9___default().cloneElement(labelComponent, labelProps));
+            newMemo = newMemo.concat( /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_9___default().cloneElement(labelComponent, labelProps));
           }
 
-          return memo;
+          return newMemo;
         }, []);
         var dataProps = this.getComponentProps(dataComponent, "data", "all");
         var children = [/*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_9___default().cloneElement(dataComponent, dataProps)].concat(_toConsumableArray(labelComponents));
@@ -4607,11 +4607,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)));
@@ -5355,12 +5355,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)
 
 
@@ -5551,7 +5553,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) {
@@ -5576,13 +5581,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) {
@@ -5597,7 +5602,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);
@@ -5708,8 +5713,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";
@@ -6075,18 +6080,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;
   };
 }
 /**
@@ -6119,27 +6124,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);
@@ -6154,7 +6159,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,
@@ -6163,14 +6168,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.
@@ -6483,14 +6488,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
@@ -6578,8 +6583,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);
   };
 
@@ -6665,9 +6669,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);
@@ -6689,9 +6694,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);
 
@@ -6757,8 +6763,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) {
@@ -6797,13 +6802,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;
 }
@@ -7161,6 +7163,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]);
 
@@ -7178,16 +7181,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);
   };
 
@@ -7213,8 +7216,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));
@@ -7468,11 +7470,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;
 }
@@ -7609,12 +7613,10 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var _helpers__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ./helpers */ "./victory-util/helpers.ts");
 
 
-/* eslint-disable no-use-before-define */
-
 
 // Private Functions
-function getVerticalAnchor(props, datum) {
-  datum = datum || {};
+function getVerticalAnchor(props) {
+  var datum = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
   var sign = datum._y >= 0 ? 1 : -1;
   var labelStyle = props.style && props.style.labels || {};
 
@@ -7627,8 +7629,8 @@ function getVerticalAnchor(props, datum) {
   return "middle";
 }
 
-function getTextAnchor(props, datum) {
-  datum = datum || {};
+function getTextAnchor(props) {
+  var datum = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
   var style = props.style,
       horizontal = props.horizontal;
   var sign = datum._y >= 0 ? 1 : -1;
@@ -7643,14 +7645,14 @@ function getTextAnchor(props, datum) {
   return sign >= 0 ? "start" : "end";
 }
 
-function getAngle(props, datum) {
-  datum = datum || {};
+function getAngle(props) {
+  var datum = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
   var labelStyle = props.style && props.style.labels || {};
   return datum.angle === undefined ? labelStyle.angle : datum.angle;
 }
 
-function getPadding(props, datum) {
-  datum = datum || {};
+function getPadding(props) {
+  var datum = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
   var horizontal = props.horizontal,
       style = props.style;
   var labelStyle = style.labels || {};
@@ -7729,8 +7731,9 @@ function getPolarOrientation(degrees) {
 } // Exported Functions
 
 
-function getText(props, datum, index) {
-  datum = datum || {};
+function getText(props) {
+  var datum = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
+  var index = arguments.length > 2 ? arguments[2] : undefined;
 
   if (datum.label !== undefined) {
     return datum.label;
@@ -8553,8 +8556,7 @@ function getSVGEventCoordinates(evt, svg) {
   }
 
   var location = isReactTouchEvent(evt) ? evt.changedTouches[0] : evt;
-  svg = svg || getParentSVG(location);
-  var matrix = getTransformationMatrix(svg);
+  var matrix = getTransformationMatrix(svg || getParentSVG(location));
   return {
     x: transformTarget(location.clientX, matrix, "x"),
     y: transformTarget(location.clientY, matrix, "y")
@@ -9196,11 +9198,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();
@@ -9387,36 +9388,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
@@ -9425,12 +9428,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
 
@@ -9439,7 +9442,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
@@ -9447,39 +9453,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
 
@@ -9488,23 +9497,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
   };
 }
 /**
@@ -9670,6 +9682,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 */ });
@@ -9750,6 +9763,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
@@ -9759,7 +9784,6 @@ var testIfSafeProp = function (key) {
  * @returns {Object}: object containing remaining acceptable props
  */
 
-
 var getSafeUserProps = function (props) {
   var propsToFilter = _objectSpread({}, props);
 
@@ -9962,12 +9986,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);
     }
@@ -9998,17 +10023,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);
 }
@@ -10100,9 +10124,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) {
@@ -10112,7 +10136,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);
@@ -10274,10 +10298,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))))