victory-errorbar 36.8.2 → 36.8.3

package/dist/victory-errorbar.js

@@ -143,14 +143,12 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-var ErrorBar = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
-  var _props = props,
-      groupComponent = _props.groupComponent;
+var ErrorBar = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
+  var groupComponent = props.groupComponent;
   var userProps = victory_core__WEBPACK_IMPORTED_MODULE_4__.getSafeUserProps(props);
-  var _props2 = props,
-      tabIndex = _props2.tabIndex,
-      ariaLabel = _props2.ariaLabel;
+  var tabIndex = props.tabIndex,
+      ariaLabel = props.ariaLabel;
   var error = calculateError(props);
   var children = [error.right ? renderBorder(props, error, "right") : null, error.left ? renderBorder(props, error, "left") : null, error.bottom ? renderBorder(props, error, "bottom") : null, error.top ? renderBorder(props, error, "top") : null, error.right ? renderCross(props, error, "right") : null, error.left ? renderCross(props, error, "left") : null, error.bottom ? renderCross(props, error, "bottom") : null, error.top ? renderCross(props, error, "top") : null].filter(Boolean);
   return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default().cloneElement(groupComponent, _objectSpread({
@@ -354,29 +352,30 @@ var getLabelProps = function (dataProps, text, style) {
   return lodash_defaults__WEBPACK_IMPORTED_MODULE_2___default()({}, labelProps, victory_core__WEBPACK_IMPORTED_MODULE_6__.omit(tooltipTheme, ["style"]));
 };
 
-var getBaseProps = function (props, fallbackProps) {
-  var modifiedProps = victory_core__WEBPACK_IMPORTED_MODULE_6__.modifyProps(props, fallbackProps, "errorbar");
-  props = lodash_assign__WEBPACK_IMPORTED_MODULE_1___default()({}, modifiedProps, getCalculatedValues(modifiedProps));
-  var _props = props,
-      borderWidth = _props.borderWidth,
-      data = _props.data,
-      domain = _props.domain,
-      events = _props.events,
-      groupComponent = _props.groupComponent,
-      height = _props.height,
-      horizontal = _props.horizontal,
-      labels = _props.labels,
-      name = _props.name,
-      origin = _props.origin,
-      padding = _props.padding,
-      polar = _props.polar,
-      scale = _props.scale,
-      sharedEvents = _props.sharedEvents,
-      standalone = _props.standalone,
-      style = _props.style,
-      theme = _props.theme,
-      width = _props.width,
-      disableInlineStyles = _props.disableInlineStyles;
+var getBaseProps = function (initialProps, fallbackProps) {
+  var modifiedProps = victory_core__WEBPACK_IMPORTED_MODULE_6__.modifyProps(initialProps, fallbackProps, "errorbar");
+
+  var props = lodash_assign__WEBPACK_IMPORTED_MODULE_1___default()({}, modifiedProps, getCalculatedValues(modifiedProps));
+
+  var borderWidth = props.borderWidth,
+      data = props.data,
+      domain = props.domain,
+      events = props.events,
+      groupComponent = props.groupComponent,
+      height = props.height,
+      horizontal = props.horizontal,
+      labels = props.labels,
+      name = props.name,
+      origin = props.origin,
+      padding = props.padding,
+      polar = props.polar,
+      scale = props.scale,
+      sharedEvents = props.sharedEvents,
+      standalone = props.standalone,
+      style = props.style,
+      theme = props.theme,
+      width = props.width,
+      disableInlineStyles = props.disableInlineStyles;
   var initialChildProps = {
     parent: {
       data: data,
@@ -403,13 +402,13 @@ var getBaseProps = function (props, fallbackProps) {
         x = _Helpers$scalePoint.x,
         y = _Helpers$scalePoint.y;
 
-    datum = formatDataFromDomain(datum, domain);
-    var errorX = getErrors(props, datum, "x");
-    var errorY = getErrors(props, datum, "y");
+    var formattedDatum = formatDataFromDomain(datum, domain);
+    var errorX = getErrors(props, formattedDatum, "x");
+    var errorY = getErrors(props, formattedDatum, "y");
     var dataProps = {
       borderWidth: borderWidth,
       data: data,
-      datum: datum,
+      datum: formattedDatum,
       errorX: horizontal ? errorY : errorX,
       errorY: horizontal ? errorX : errorY,
       groupComponent: groupComponent,
@@ -7345,12 +7344,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);
@@ -7375,6 +7374,8 @@ var interpolateObject = function (a, b) {
 
   var i = {};
   var c = {};
+  var a = startValue;
+  var b = endValue;
   var k;
 
   if (a === null || typeof a !== "object") {
@@ -7542,8 +7543,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({
@@ -8159,8 +8160,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)
     });
@@ -8364,22 +8364,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);
   }
 };
@@ -8581,8 +8578,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;
@@ -8603,7 +8600,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)
@@ -10296,8 +10293,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) {
@@ -10352,8 +10348,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);
@@ -10452,13 +10448,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));
@@ -10857,12 +10855,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)
 
 
@@ -11053,7 +11053,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) {
@@ -11078,13 +11081,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) {
@@ -11099,7 +11102,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);
@@ -11210,8 +11213,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";
@@ -11577,18 +11580,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;
   };
 }
 /**
@@ -11621,27 +11624,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);
@@ -11656,7 +11659,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,
@@ -11665,14 +11668,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.
@@ -11985,14 +11988,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
@@ -12080,8 +12083,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);
   };
 
@@ -12167,9 +12169,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);
@@ -12191,9 +12194,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);
 
@@ -12259,8 +12263,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) {
@@ -12299,13 +12302,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;
 }
@@ -12663,6 +12663,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]);
 
@@ -12680,16 +12681,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);
   };
 
@@ -12715,8 +12716,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));
@@ -12769,11 +12769,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;
 }
@@ -12801,12 +12803,10 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var _helpers__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ./helpers */ "../../victory-core/es/victory-util/helpers.js");
 
 
-/* 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 || {};
 
@@ -12819,8 +12819,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;
@@ -12835,14 +12835,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 || {};
@@ -12921,8 +12921,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;
@@ -14082,11 +14083,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();
@@ -14273,36 +14273,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
@@ -14311,12 +14313,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
 
@@ -14325,7 +14327,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
@@ -14333,39 +14338,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
 
@@ -14374,23 +14382,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
   };
 }
 /**
@@ -14556,6 +14567,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 */ });
@@ -14636,6 +14648,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
@@ -14645,7 +14669,6 @@ var testIfSafeProp = function (key) {
  * @returns {Object}: object containing remaining acceptable props
  */
 
-
 var getSafeUserProps = function (props) {
   var propsToFilter = _objectSpread({}, props);