Rename deprecated React APIs

dmt0 · dmt0 · commit defa252fc201 · 2020-10-21T21:13:12.000-04:00
diff --git a/dev/App.js b/dev/App.js
@@ -118,7 +118,7 @@ class App extends Component {
     this.updateState = this.updateState.bind(this);
   }
 
-  componentWillMount() {
+  UNSAFE_componentWillMount() {
     // curl https://api.github.com/repos/plotly/plotly.js/contents/test/image/mocks \
     // | jq '[.[] | .name ]' > mocks.json
     fetch('/mocks.json')
diff --git a/examples/demo/src/App.js b/examples/demo/src/App.js
@@ -27,7 +27,7 @@ class App extends Component {
     this.loadMock = this.loadMock.bind(this);
   }
 
-  componentWillMount() {
+  UNSAFE_componentWillMount() {
     fetch('https://api.github.com/repos/plotly/plotly.js/contents/test/image/mocks')
       .then(response => response.json())
       .then(mocks => this.setState({mocks}));
diff --git a/src/components/containers/PlotlyFold.js b/src/components/containers/PlotlyFold.js
@@ -177,7 +177,7 @@ class PlotlyFold extends Fold {
     this.determineVisibility(props, context);
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     this.determineVisibility(nextProps, nextContext);
   }
 
diff --git a/src/components/containers/PlotlySection.js b/src/components/containers/PlotlySection.js
@@ -43,7 +43,7 @@ export default class PlotlySection extends Section {
     this.determineVisibility(props, context);
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     this.determineVisibility(nextProps, nextContext);
   }
 
diff --git a/src/components/containers/TraceAccordion.js b/src/components/containers/TraceAccordion.js
@@ -17,7 +17,7 @@ class TraceAccordion extends Component {
     this.setLocals(props, context);
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     this.setLocals(nextProps, nextContext);
   }
 
diff --git a/src/components/containers/TraceMarkerSection.js b/src/components/containers/TraceMarkerSection.js
@@ -8,7 +8,7 @@ class TraceMarkerSection extends Component {
     this.setLocals(context);
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     this.setLocals(nextContext);
   }
 
diff --git a/src/components/fields/DataSelector.js b/src/components/fields/DataSelector.js
@@ -19,7 +19,7 @@ export class UnconnectedDataSelector extends Component {
     this.setLocals(props, context);
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     this.setLocals(nextProps, nextContext);
   }
 
diff --git a/src/components/fields/DropdownCustom.js b/src/components/fields/DropdownCustom.js
@@ -22,7 +22,7 @@ export class UnconnectedDropdownCustom extends Component {
     };
   }
 
-  componentWillReceiveProps(props) {
+  UNSAFE_componentWillReceiveProps(props) {
     this.setLocals(props);
   }
 
diff --git a/src/components/fields/HoverLabelNameLength.js b/src/components/fields/HoverLabelNameLength.js
@@ -18,7 +18,7 @@ export class UnconnectedHoverLabelNameLength extends Component {
     return props.fullValue > 0 ? 'clip' : props.fullValue === 0 ? 'hide' : 'no-clip';
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     if (nextProps.fullValue !== this.props.fullValue) {
       this.setState({
         currentOption: this.getCurrentOption(nextProps),
diff --git a/src/components/fields/LineSelectors.js b/src/components/fields/LineSelectors.js
@@ -67,7 +67,7 @@ class LineSelector extends Component {
     this.setLocals(props, context);
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     this.setLocals(nextProps, nextContext);
   }
 
diff --git a/src/components/fields/LocationSelector.js b/src/components/fields/LocationSelector.js
@@ -65,7 +65,7 @@ class UnconnectedLocationSelector extends Component {
     this.setMode = this.setMode.bind(this);
   }
 
-  componentWillMount() {
+  UNSAFE_componentWillMount() {
     this.setState({
       mode: this.props.container.locations ? 'location' : 'latlon',
     });
diff --git a/src/components/fields/SymbolSelector.js b/src/components/fields/SymbolSelector.js
@@ -351,7 +351,7 @@ class SymbolSelector extends Component {
     this.setLocals(props, context);
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     this.setLocals(nextProps, nextContext);
   }
 
diff --git a/src/components/fields/TraceSelector.js b/src/components/fields/TraceSelector.js
@@ -61,7 +61,7 @@ class TraceSelector extends Component {
     }
   }
 
-  componentWillReceiveProps(nextProps, nextContext) {
+  UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
     const {container, fullContainer, updateContainer} = nextProps;
     this.setTraceDefaults(container, fullContainer, updateContainer);
     this.setLocals(nextProps, nextContext);
diff --git a/src/components/fields/VisibilitySelect.js b/src/components/fields/VisibilitySelect.js
@@ -16,7 +16,7 @@ export class UnconnectedVisibilitySelect extends Component {
     this.setLocals(props);
   }
 
-  componentWillReceiveProps(props) {
+  UNSAFE_componentWillReceiveProps(props) {
     this.setLocals(props);
   }
 
diff --git a/src/components/widgets/CheckboxGroup.js b/src/components/widgets/CheckboxGroup.js
@@ -10,7 +10,7 @@ class CheckboxGroup extends Component {
     this.handleChange = this.handleChange.bind(this);
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     this.setState({options: nextProps.options});
   }
 
diff --git a/src/components/widgets/Dropzone.js b/src/components/widgets/Dropzone.js
@@ -40,7 +40,7 @@ class Dropzone extends Component {
     return <div className="dropzone-container__message">{_('File loaded!')}</div>;
   }
 
-  componentWillMount() {
+  UNSAFE_componentWillMount() {
     const _ = this.context.localize;
 
     if (this.props.value && this.props.value !== '') {
diff --git a/src/components/widgets/FlaglistCheckboxGroup.js b/src/components/widgets/FlaglistCheckboxGroup.js
@@ -37,7 +37,7 @@ class FlaglistCheckboxGroup extends Component {
   }
 
   // Sync local state to parent props.
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     this.setState({activeOption: this.parseFlags(nextProps.activeOption)});
   }
 
diff --git a/src/components/widgets/NumericInput.js b/src/components/widgets/NumericInput.js
@@ -31,7 +31,7 @@ export default class NumericInput extends Component {
         }`;
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     if (nextProps.value !== this.state.value) {
       this.setState({value: nextProps.value});
     }
diff --git a/src/components/widgets/RadioBlocks.js b/src/components/widgets/RadioBlocks.js
@@ -10,7 +10,7 @@ class RadioBlocks extends Component {
     this.renderOption = this.renderOption.bind(this);
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     // Reset the value to the graph's actual value
     if (nextProps.activeOption !== this.state.activeOption) {
       this.setState({
diff --git a/src/components/widgets/TextArea.js b/src/components/widgets/TextArea.js
@@ -12,7 +12,7 @@ export default class TextArea extends Component {
     this.onChange = this.onChange.bind(this);
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     // Reset the value to the graph's actual value
     if (nextProps.value !== this.state.value) {
       this.setState({
diff --git a/src/components/widgets/TextInput.js b/src/components/widgets/TextInput.js
@@ -9,7 +9,7 @@ export default class TextInput extends Component {
     this.state = {value: props.value};
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     if (nextProps.value !== this.state.value) {
       this.setState({value: nextProps.value});
     }
diff --git a/src/components/widgets/text_editors/LaTeX.js b/src/components/widgets/text_editors/LaTeX.js
@@ -19,7 +19,7 @@ export default class LaTeX extends TextArea {
     this.onBlur = this.onBlur.bind(this);
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     const unwrappedNextValue = this.unwrap(nextProps.value);
 
     if (unwrappedNextValue !== this.state.value) {
diff --git a/src/components/widgets/text_editors/RichText/LinkEditor.js b/src/components/widgets/text_editors/RichText/LinkEditor.js
@@ -26,7 +26,7 @@ class LinkEditor extends Component {
     }
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     const {props} = this;
 
     // Update position if we are editing a new link
diff --git a/src/components/widgets/text_editors/RichText/index.js b/src/components/widgets/text_editors/RichText/index.js
@@ -41,7 +41,7 @@ class RichText extends Component {
      * After this, as long as this component is mounted, it owns the source
      * of truth for the annotation value via `this.state.editorState`.
      * This state may be updated externally via a prop update.
-     * See `componentWillReceiveProps`.
+     * See `UNSAFE_componentWillReceiveProps`.
      */
     this.state = {
       editorState: props.value.toString().trim().length
@@ -67,7 +67,7 @@ class RichText extends Component {
     this.renderLinkEditor = this.renderLinkEditor.bind(this);
   }
 
-  componentWillReceiveProps(nextProps) {
+  UNSAFE_componentWillReceiveProps(nextProps) {
     const {linkEditorFocus, editorFocus} = this.state;
 
     /*
diff --git a/src/lib/connectAggregationToTransform.js b/src/lib/connectAggregationToTransform.js
@@ -11,7 +11,7 @@ export default function connectAggregationToTransform(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectAnnotationToLayout.js b/src/lib/connectAnnotationToLayout.js
@@ -14,7 +14,7 @@ export default function connectAnnotationToLayout(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectCartesianSubplotToLayout.js b/src/lib/connectCartesianSubplotToLayout.js
@@ -11,7 +11,7 @@ export default function connectCartesianSubplotToLayout(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectImageToLayout.js b/src/lib/connectImageToLayout.js
@@ -14,7 +14,7 @@ export default function connectImageToLayout(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectLayersToMapbox.js b/src/lib/connectLayersToMapbox.js
@@ -14,7 +14,7 @@ export default function connectLayersToMapbox(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectNonCartesianSubplotToLayout.js b/src/lib/connectNonCartesianSubplotToLayout.js
@@ -11,7 +11,7 @@ export default function connectNonCartesianSubplotToLayout(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectRangeSelectorToAxis.js b/src/lib/connectRangeSelectorToAxis.js
@@ -13,7 +13,7 @@ export default function connectRangeSelectorToAxis(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectShapeToLayout.js b/src/lib/connectShapeToLayout.js
@@ -14,7 +14,7 @@ export default function connectShapeToLayout(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectSliderToLayout.js b/src/lib/connectSliderToLayout.js
@@ -10,7 +10,7 @@ export default function connectSliderToLayout(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectToContainer.js b/src/lib/connectToContainer.js
@@ -41,7 +41,7 @@ export default function connectToContainer(WrappedComponent, config = {}) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectTraceToPlot.js b/src/lib/connectTraceToPlot.js
@@ -23,7 +23,7 @@ export default function connectTraceToPlot(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectTransformToTrace.js b/src/lib/connectTransformToTrace.js
@@ -13,7 +13,7 @@ export default function connectTransformToTrace(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 
diff --git a/src/lib/connectUpdateMenuToLayout.js b/src/lib/connectUpdateMenuToLayout.js
@@ -10,7 +10,7 @@ export default function connectUpdateMenuToLayout(WrappedComponent) {
       this.setLocals(props, context);
     }
 
-    componentWillReceiveProps(nextProps, nextContext) {
+    UNSAFE_componentWillReceiveProps(nextProps, nextContext) {
       this.setLocals(nextProps, nextContext);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ class App extends Component {`
`118`	`118`	`this.updateState = this.updateState.bind(this);`
`119`	`119`	`}`
`120`	`120`
`121`		`- componentWillMount() {`
	`121`	`+ UNSAFE_componentWillMount() {`
`122`	`122`	`// curl https://api.github.com/repos/plotly/plotly.js/contents/test/image/mocks \`
`123`	`123`	`// \| jq '[.[] \| .name ]' > mocks.json`
`124`	`124`	`fetch('/mocks.json')`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class App extends Component {`
`27`	`27`	`this.loadMock = this.loadMock.bind(this);`
`28`	`28`	`}`
`29`	`29`
`30`		`- componentWillMount() {`
	`30`	`+ UNSAFE_componentWillMount() {`
`31`	`31`	`fetch('https://api.github.com/repos/plotly/plotly.js/contents/test/image/mocks')`
`32`	`32`	`.then(response => response.json())`
`33`	`33`	`.then(mocks => this.setState({mocks}));`
Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ class PlotlyFold extends Fold {`
`177`	`177`	`this.determineVisibility(props, context);`
`178`	`178`	`}`
`179`	`179`
`180`		`- componentWillReceiveProps(nextProps, nextContext) {`
	`180`	`+ UNSAFE_componentWillReceiveProps(nextProps, nextContext) {`
`181`	`181`	`this.determineVisibility(nextProps, nextContext);`
`182`	`182`	`}`
`183`	`183`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ export default class PlotlySection extends Section {`
`43`	`43`	`this.determineVisibility(props, context);`
`44`	`44`	`}`
`45`	`45`
`46`		`- componentWillReceiveProps(nextProps, nextContext) {`
	`46`	`+ UNSAFE_componentWillReceiveProps(nextProps, nextContext) {`
`47`	`47`	`this.determineVisibility(nextProps, nextContext);`
`48`	`48`	`}`
`49`	`49`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ class TraceAccordion extends Component {`
`17`	`17`	`this.setLocals(props, context);`
`18`	`18`	`}`
`19`	`19`
`20`		`- componentWillReceiveProps(nextProps, nextContext) {`
	`20`	`+ UNSAFE_componentWillReceiveProps(nextProps, nextContext) {`
`21`	`21`	`this.setLocals(nextProps, nextContext);`
`22`	`22`	`}`
`23`	`23`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ class TraceMarkerSection extends Component {`
`8`	`8`	`this.setLocals(context);`
`9`	`9`	`}`
`10`	`10`
`11`		`- componentWillReceiveProps(nextProps, nextContext) {`
	`11`	`+ UNSAFE_componentWillReceiveProps(nextProps, nextContext) {`
`12`	`12`	`this.setLocals(nextContext);`
`13`	`13`	`}`
`14`	`14`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ export class UnconnectedDataSelector extends Component {`
`19`	`19`	`this.setLocals(props, context);`
`20`	`20`	`}`
`21`	`21`
`22`		`- componentWillReceiveProps(nextProps, nextContext) {`
	`22`	`+ UNSAFE_componentWillReceiveProps(nextProps, nextContext) {`
`23`	`23`	`this.setLocals(nextProps, nextContext);`
`24`	`24`	`}`
`25`	`25`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ export class UnconnectedDropdownCustom extends Component {`
`22`	`22`	`};`
`23`	`23`	`}`
`24`	`24`
`25`		`- componentWillReceiveProps(props) {`
	`25`	`+ UNSAFE_componentWillReceiveProps(props) {`
`26`	`26`	`this.setLocals(props);`
`27`	`27`	`}`
`28`	`28`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ export class UnconnectedHoverLabelNameLength extends Component {`
`18`	`18`	`return props.fullValue > 0 ? 'clip' : props.fullValue === 0 ? 'hide' : 'no-clip';`
`19`	`19`	`}`
`20`	`20`
`21`		`- componentWillReceiveProps(nextProps) {`
	`21`	`+ UNSAFE_componentWillReceiveProps(nextProps) {`
`22`	`22`	`if (nextProps.fullValue !== this.props.fullValue) {`
`23`	`23`	`this.setState({`
`24`	`24`	`currentOption: this.getCurrentOption(nextProps),`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ class LineSelector extends Component {`
`67`	`67`	`this.setLocals(props, context);`
`68`	`68`	`}`
`69`	`69`
`70`		`- componentWillReceiveProps(nextProps, nextContext) {`
	`70`	`+ UNSAFE_componentWillReceiveProps(nextProps, nextContext) {`
`71`	`71`	`this.setLocals(nextProps, nextContext);`
`72`	`72`	`}`
`73`	`73`