fix: rollback to componentWillReceiveProps, fix cash in react@15

recharts · Oct 17, 2019 · 8ae2390 · 8ae2390
1 parent c252d78
commit 8ae2390
Show file tree

Hide file tree

Showing 13 changed files with 13 additions and 13 deletions.
diff --git a/src/cartesian/Area.js b/src/cartesian/Area.js
@@ -198,7 +198,7 @@ class Area extends PureComponent {
   state = { isAnimationFinished: true };
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, points, baseLine } = this.props;
 
     if (nextProps.animationId !== animationId) {

diff --git a/src/cartesian/Bar.js b/src/cartesian/Bar.js
@@ -172,7 +172,7 @@ class Bar extends PureComponent {
   state = { isAnimationFinished: false };
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, data } = this.props;
 
     if (nextProps.animationId !== animationId) {

diff --git a/src/cartesian/Brush.js b/src/cartesian/Brush.js
@@ -71,7 +71,7 @@ class Brush extends PureComponent {
   }
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { data, width, x, travellerWidth, updateId } = this.props;
 
     if (

diff --git a/src/cartesian/Line.js b/src/cartesian/Line.js
@@ -156,7 +156,7 @@ class Line extends PureComponent {
   }
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, points } = this.props;
 
     if (nextProps.animationId !== animationId) {

diff --git a/src/cartesian/Scatter.js b/src/cartesian/Scatter.js
@@ -156,7 +156,7 @@ class Scatter extends PureComponent {
   state = { isAnimationFinished: false };
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, points } = this.props;
 
     if (nextProps.animationId !== animationId) {

diff --git a/src/chart/Sankey.js b/src/chart/Sankey.js
@@ -369,7 +369,7 @@ class Sankey extends PureComponent {
   }
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { data, width, height, margin, iterations, nodeWidth, nodePadding, nameKey } = this.props;
     if (nextProps.data !== data || nextProps.width !== width ||
       nextProps.height !== height || !shallowEqual(nextProps.margin, margin) ||

diff --git a/src/chart/Treemap.js b/src/chart/Treemap.js
@@ -282,7 +282,7 @@ class Treemap extends PureComponent {
   }
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { type, width, height, data, dataKey, aspectRatio } = nextProps;
 
     if (data !== this.props.data ||

diff --git a/src/chart/generateCategoricalChart.js b/src/chart/generateCategoricalChart.js
@@ -176,7 +176,7 @@ const generateCategoricalChart = ({
     }
 
     // eslint-disable-next-line camelcase
-    UNSAFE_componentWillReceiveProps(nextProps) {
+    componentWillReceiveProps(nextProps) {
       const { data, children, width, height, layout, stackOffset, margin } = this.props;
       const { updateId } = this.state;
 

diff --git a/src/numberAxis/Funnel.js b/src/numberAxis/Funnel.js
@@ -161,7 +161,7 @@ class Funnel extends PureComponent {
   state = { isAnimationFinished: false };
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, trapezoids } = this.props;
 
     if (nextProps.isAnimationActive !== this.props.isAnimationActive) {

diff --git a/src/polar/Pie.js b/src/polar/Pie.js
@@ -237,7 +237,7 @@ class Pie extends PureComponent {
   state = { isAnimationFinished: false };
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, sectors } = this.props;
 
     if (nextProps.isAnimationActive !== this.props.isAnimationActive) {

diff --git a/src/polar/Radar.js b/src/polar/Radar.js
@@ -97,7 +97,7 @@ class Radar extends PureComponent {
   state = { isAnimationFinished: false };
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, points } = this.props;
 
     if (nextProps.animationId !== animationId) {

diff --git a/src/polar/RadialBar.js b/src/polar/RadialBar.js
@@ -170,7 +170,7 @@ class RadialBar extends PureComponent {
   };
 
   // eslint-disable-next-line camelcase
-  UNSAFE_componentWillReceiveProps(nextProps) {
+  componentWillReceiveProps(nextProps) {
     const { animationId, data } = this.props;
 
     if (nextProps.animationId !== animationId) {

diff --git a/src/util/AnimationDecorator.js b/src/util/AnimationDecorator.js
@@ -20,7 +20,7 @@ export default function (WrappedComponent) {
     };
 
     // eslint-disable-next-line camelcase
-    UNSAFE_componentWillReceiveProps(nextProps) {
+    componentWillReceiveProps(nextProps) {
       const { animationId } = this.state;
 
       if (this.props.data !== nextProps.data) {