victory-core 36.8.2 → 36.8.3

package/es/types/callbacks.d.ts

@@ -1,4 +1,4 @@
-import { D3Scale, Datum, ID } from "./prop-types";
+import { D3Scale, Datum, ID, ScalePropType } from "./prop-types";
 import { BlockProps, OrientationTypes } from "../victory-theme/types";
 /**
  * This is the first parameter of a callback when a callback is used to
@@ -15,9 +15,9 @@ export interface CallbackArgs {
     index?: ID;
     x?: number;
     y?: number;
-    scale?: {
-        x?: D3Scale;
-        y?: D3Scale;
+    scale?: ScalePropType | D3Scale | {
+        x?: ScalePropType | D3Scale;
+        y?: ScalePropType | D3Scale;
     };
     tick?: any;
     ticks?: any;

package/es/victory-animation/util.d.ts

@@ -34,11 +34,11 @@ export declare const interpolateFunction: (a: any, b: any) => (t: any) => any;
  * 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.
  */
-export declare const interpolateObject: (a: any, b: any) => (t: any) => {};
+export declare const interpolateObject: (startValue: any, endValue: any) => (t: any) => {};
 export declare const interpolateString: (a: any, b: any) => (t: number) => any;
 /**
  * By default, the list of interpolators used by `d3.interpolate` has a few

package/es/victory-animation/util.js

@@ -100,12 +100,12 @@ export 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.
  */
 
-export var interpolateObject = function (a, b) {
+export var interpolateObject = function (startValue, endValue) {
   var interpolateTypes = function (x, y) {
     if (x === y || !isInterpolatable(x) || !isInterpolatable(y)) {
       return interpolateImmediate(x, y);
@@ -130,6 +130,8 @@ export var interpolateObject = function (a, b) {
 
   var i = {};
   var c = {};
+  var a = startValue;
+  var b = endValue;
   var k;
 
   if (a === null || typeof a !== "object") {

package/es/victory-animation/victory-animation.js

@@ -61,8 +61,8 @@ export 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({

package/es/victory-label/victory-label.d.ts

@@ -17,7 +17,7 @@ export interface VictoryLabelProps {
     capHeight?: StringOrNumberOrCallback;
     children?: StringOrNumberOrCallback;
     className?: string;
-    datum?: object;
+    datum?: Record<string, any>;
     data?: any[];
     desc?: string;
     direction?: string;

package/es/victory-label/victory-label.js

@@ -91,8 +91,7 @@ var getStyles = function (style, props) {
   }
 
   var getSingleStyle = function (s) {
-    s = s ? _defaults({}, s, defaultStyles) : defaultStyles;
-    var baseStyles = Helpers.evaluateStyle(s, props);
+    var baseStyles = Helpers.evaluateStyle(s ? _defaults({}, s, defaultStyles) : defaultStyles, props);
     return _assign({}, baseStyles, {
       fontSize: getFontSize(baseStyles)
     });
@@ -296,22 +295,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);
   }
 };
@@ -513,8 +509,8 @@ var defaultProps = {
   // Magic number from d3.
   lineHeight: 1
 };
-export var VictoryLabel = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+export var VictoryLabel = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
 
   if (props.text === null || props.text === undefined) {
     return null;
@@ -535,7 +531,7 @@ export 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: TextSize.approximateTextSize(line, currentStyle),
       lineHeight: currentLineHeight,
       backgroundPadding: getSingleValue(backgroundPadding, i)

package/es/victory-primitives/arc.d.ts

@@ -12,7 +12,7 @@ export interface ArcProps extends VictoryCommonPrimitiveProps {
     startAngle?: number;
 }
 export declare const Arc: {
-    (props: ArcProps): React.ReactElement<any, string | React.JSXElementConstructor<any>>;
+    (initialProps: ArcProps): React.DetailedReactHTMLElement<any, HTMLElement>;
     propTypes: {
         closedPath: PropTypes.Requireable<boolean>;
         cx: PropTypes.Requireable<number>;

package/es/victory-primitives/arc.js

@@ -68,8 +68,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-export var Arc = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+export var Arc = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return /*#__PURE__*/React.cloneElement(props.pathComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,
     d: getArcPath(props),

package/es/victory-primitives/background.d.ts

@@ -12,7 +12,7 @@ export interface BackgroundProps extends VictoryCommonPrimitiveProps {
     y?: number;
 }
 export declare const Background: {
-    (props: BackgroundProps): React.ReactElement<any, string | React.JSXElementConstructor<any>>;
+    (initialProps: BackgroundProps): React.DetailedReactHTMLElement<any, HTMLElement>;
     propTypes: {
         circleComponent: PropTypes.Requireable<PropTypes.ReactElementLike>;
         height: PropTypes.Requireable<number>;

package/es/victory-primitives/background.js

@@ -30,8 +30,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-export var Background = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+export var Background = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return props.polar ? /*#__PURE__*/React.cloneElement(props.circleComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     style: props.style,
     role: props.role,

package/es/victory-primitives/border.d.ts

@@ -9,7 +9,7 @@ export interface BorderProps extends VictoryCommonPrimitiveProps {
     y?: number;
 }
 export declare const Border: {
-    (props: BorderProps): React.ReactElement<any, string | React.JSXElementConstructor<any>>;
+    (initialProps: BorderProps): React.DetailedReactHTMLElement<any, HTMLElement>;
     propTypes: {
         height: PropTypes.Requireable<number>;
         rectComponent: PropTypes.Requireable<PropTypes.ReactElementLike>;

package/es/victory-primitives/border.js

@@ -42,8 +42,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-export var Border = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+export var Border = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return /*#__PURE__*/React.cloneElement(props.rectComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,
     style: props.style,

package/es/victory-primitives/line-segment.d.ts

@@ -10,7 +10,7 @@ export interface LineSegmentProps extends VictoryCommonPrimitiveProps {
     y2?: number;
 }
 export declare const LineSegment: {
-    (props: LineSegmentProps): React.ReactElement<any, string | React.JSXElementConstructor<any>>;
+    (initialProps: LineSegmentProps): React.DetailedReactHTMLElement<any, HTMLElement>;
     propTypes: {
         datum: PropTypes.Requireable<any>;
         lineComponent: PropTypes.Requireable<PropTypes.ReactElementLike>;

package/es/victory-primitives/line-segment.js

@@ -42,8 +42,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-export var LineSegment = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+export var LineSegment = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   return /*#__PURE__*/React.cloneElement(props.lineComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,
     style: props.style,

package/es/victory-primitives/point.d.ts

@@ -12,7 +12,7 @@ export interface PointProps extends VictoryCommonPrimitiveProps {
     y?: number;
 }
 export declare const Point: {
-    (props: PointProps): React.ReactElement<any, string | React.JSXElementConstructor<any>>;
+    (initialProps: PointProps): React.DetailedReactHTMLElement<any, HTMLElement>;
     propTypes: {
         datum: PropTypes.Requireable<object>;
         getPath: PropTypes.Requireable<(...args: any[]) => any>;

package/es/victory-primitives/point.js

@@ -63,8 +63,8 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-export var Point = function (props) {
-  props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), props));
+export var Point = function (initialProps) {
+  var props = evaluateProps(_objectSpread(_objectSpread({}, defaultProps), initialProps));
   var userProps = UserProps.getSafeUserProps(props);
   return /*#__PURE__*/React.cloneElement(props.pathComponent, _objectSpread(_objectSpread({}, props.events), {}, {
     "aria-label": props.ariaLabel,

package/es/victory-primitives/whisker.d.ts

@@ -14,7 +14,7 @@ export interface WhiskerProps extends VictoryCommonPrimitiveProps {
     minorWhisker?: WhiskerAxes;
 }
 export declare const Whisker: {
-    (props: WhiskerProps): React.ReactElement<any, string | React.JSXElementConstructor<any>>;
+    (initialProps: WhiskerProps): React.DetailedReactHTMLElement<{}, HTMLElement>;
     propTypes: {
         groupComponent: PropTypes.Requireable<PropTypes.ReactElementLike>;
         lineComponent: PropTypes.Requireable<PropTypes.ReactElementLike>;

package/es/victory-primitives/whisker.js

@@ -41,23 +41,22 @@ var defaultProps = {
   role: "presentation",
   shapeRendering: "auto"
 };
-export 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;
+export 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,

package/es/victory-util/add-events.js

@@ -222,8 +222,7 @@ export function addEvents(WrappedComponent) {
       value: function applyExternalMutations(props, externalMutations) {
         if (!_isEmpty(externalMutations)) {
           var callbacks = props.externalEventMutations.reduce(function (memo, mutation) {
-            memo = _isFunction(mutation.callback) ? memo.concat(mutation.callback) : memo;
-            return memo;
+            return _isFunction(mutation.callback) ? memo.concat(mutation.callback) : memo;
           }, []);
           var compiledCallbacks = callbacks.length ? function () {
             callbacks.forEach(function (c) {
@@ -278,8 +277,8 @@ export 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 = _defaults({}, parentState, sharedParentState);
@@ -378,13 +377,15 @@ export function addEvents(WrappedComponent) {
         var dataKeys = _without(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.cloneElement(labelComponent, labelProps));
+            newMemo = newMemo.concat( /*#__PURE__*/React.cloneElement(labelComponent, labelProps));
           }
 
-          return memo;
+          return newMemo;
         }, []);
         var dataProps = this.getComponentProps(dataComponent, "data", "all");
         var children = [/*#__PURE__*/React.cloneElement(dataComponent, dataProps)].concat(_toConsumableArray(labelComponents));

package/es/victory-util/axis.js

@@ -46,11 +46,11 @@ export function getAxis(props) {
  */
 
 export function findAxisComponents(childComponents, predicate) {
-  predicate = predicate || _identity;
+  var predicateFunction = predicate || _identity;
 
   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.Children.toArray(child.props.children)));

package/es/victory-util/common-props.d.ts

@@ -20,6 +20,7 @@ export interface VictoryCommonThemeProps {
     animate?: boolean | AnimatePropTypeInterface;
     colorScale?: ColorScalePropType;
     containerComponent?: React.ReactElement;
+    disableInlineStyles?: boolean;
     domainPadding?: DomainPaddingPropType;
     externalEventMutations?: EventCallbackInterface<string | string[], StringOrNumberOrList>[];
     groupComponent?: React.ReactElement;

package/es/victory-util/data.js

@@ -66,12 +66,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 _orderBy(dataset, sortKey, order);
+  return _orderBy(dataset, formattedSortKey, order);
 } // This method will remove data points that break certain scales. (log scale only)
 
 
@@ -262,7 +264,10 @@ export 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) {
@@ -287,13 +292,13 @@ export 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) {
@@ -308,7 +313,7 @@ export function formatData(dataset, props, expectedKeys) {
       return memo;
     }, {});
 
-    var formattedDatum = _assign({}, processedValues, datum);
+    var formattedDatum = _assign({}, processedValues, parsedDatum);
 
     if (!_isEmpty(formattedDatum)) {
       dataArr.push(formattedDatum);
@@ -419,8 +424,8 @@ export 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";

package/es/victory-util/domain.js

@@ -164,18 +164,18 @@ function padDomain(domain, props, axis) {
 
 
 export function createDomainFunction(getDomainFromDataFunction, formatDomainFunction) {
-  getDomainFromDataFunction = _isFunction(getDomainFromDataFunction) ? getDomainFromDataFunction : getDomainFromData;
-  formatDomainFunction = _isFunction(formatDomainFunction) ? formatDomainFunction : formatDomain;
+  var getDomainFromDataFn = _isFunction(getDomainFromDataFunction) ? getDomainFromDataFunction : getDomainFromData;
+  var formatDomainFn = _isFunction(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.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;
   };
 }
 /**
@@ -208,27 +208,27 @@ export function getDomain(props, axis) {
  */
 
 export function getDomainFromCategories(props, axis, categories) {
-  categories = categories || Data.getCategories(props, axis);
+  var categoriesArray = categories || Data.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.containsStrings(categories) ? Data.getStringsFromCategories(props, axis) : [];
+  var stringArray = Collection.containsStrings(categoriesArray) ? Data.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.getMinValue(categoryValues);
   var max = maxDomain !== undefined ? maxDomain : Collection.getMaxValue(categoryValues);
   var categoryDomain = getDomainFromMinMax(min, max);
@@ -243,7 +243,7 @@ export function getDomainFromCategories(props, axis, categories) {
  */
 
 export function getDomainFromData(props, axis, dataset) {
-  dataset = dataset || Data.getData(props);
+  var datasetArray = dataset || Data.getData(props);
   var polar = props.polar,
       _props$startAngle2 = props.startAngle,
       startAngle = _props$startAngle2 === void 0 ? 0 : _props$startAngle2,
@@ -252,14 +252,14 @@ export 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.

package/es/victory-util/events.d.ts

@@ -16,7 +16,7 @@ interface ComponentWithEvents extends EventMixinCalculatedValues {
     state: any;
     setState: any;
 }
-export declare function getEvents(this: ComponentWithEvents, props: any, target: any, eventKey: any, getScopedEvents: any): any;
+export declare function getEvents(this: ComponentWithEvents, props: any, target?: any, eventKey?: any, getScopedEvents?: any): any;
 export declare function getScopedEvents(this: ComponentWithEvents, events: any, namespace: any, childType: any, baseProps: any): {};
 export declare function getPartialEvents(events: ComponentEventHandlers, eventKey: ComponentEventKey, childProps: unknown): PartialEvents;
 export interface PartialEvents {
@@ -33,7 +33,7 @@ export declare function getEventState(this: ComponentWithEvents, eventKey: Compo
  *
  * @return {Object} a object describing all mutations for VictorySharedEvents
  */
-export declare function getExternalMutationsWithChildren(mutations: any, baseProps: any, baseState: any, childNames: any): any;
+export declare function getExternalMutationsWithChildren(mutations: any, baseProps: {} | undefined, baseState: {} | undefined, childNames: any): any;
 /**
  * Returns a set of all mutations for a component
  *
@@ -44,7 +44,7 @@ export declare function getExternalMutationsWithChildren(mutations: any, basePro
  *
  * @return {Object} a object describing mutations for a given component
  */
-export declare function getExternalMutations(mutations: any, baseProps: any, baseState: any, childName?: any): {};
+export declare function getExternalMutations(mutations: any, baseProps?: {}, baseState?: {}, childName?: any): {};
 /**
  * Returns a set of mutations for a particular element given scoped baseProps and baseState
  *

package/es/victory-util/events.js

@@ -117,14 +117,14 @@ export 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
@@ -212,8 +212,7 @@ export function getScopedEvents(events, namespace, childType, baseProps) {
 
   var parseEventReturn = function (eventReturn, eventKey) {
     return Array.isArray(eventReturn) ? eventReturn.reduce(function (memo, props) {
-      memo = _assign({}, memo, parseEvent(props, eventKey));
-      return memo;
+      return _assign({}, memo, parseEvent(props, eventKey));
     }, {}) : parseEvent(eventReturn, eventKey);
   };
 
@@ -299,9 +298,10 @@ export function getEventState(eventKey, namespace, childType) {
  */
 // eslint-disable-next-line max-params
 
-export function getExternalMutationsWithChildren(mutations, baseProps, baseState, childNames) {
-  baseProps = baseProps || {};
-  baseState = baseState || {};
+export 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);
@@ -323,9 +323,10 @@ export function getExternalMutationsWithChildren(mutations, baseProps, baseState
  */
 // eslint-disable-next-line max-params
 
-export function getExternalMutations(mutations, baseProps, baseState, childName) {
-  baseProps = baseProps || {};
-  baseState = baseState || {};
+export 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 = _keys(baseProps);
 
@@ -391,8 +392,7 @@ export 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) {
@@ -431,13 +431,10 @@ export function getExternalMutation(mutations, baseProps, baseState, identifier)
 
 export 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 = _isFunction(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;
 }