Drop transferPropsTo usage in React core
These were completely unnecessary uses of transferPropsTo.
Sebastian Markbage committed
Jul 8, 2014 at 14:27 UTC
225d76f77219dc9ae1d394893f31fde285961fbe
4 files changed
+7
-6
src/addons/transitions/ReactCSSTransitionGroup.js
+4
-3
@@ -24,6 +24,8 @@ var React = require('React');
24
var ReactTransitionGroup = require('ReactTransitionGroup');
25
var ReactCSSTransitionGroupChild = require('ReactCSSTransitionGroupChild');
26
27
+var merge = require('merge');
28
+
29
var ReactCSSTransitionGroup = React.createClass({
30
displayName: 'ReactCSSTransitionGroup',
31
@@ -55,10 +57,9 @@ var ReactCSSTransitionGroup = React.createClass({
57
},
58
59
render: function() {
58
- return this.transferPropsTo(
60
+ return (
61
ReactTransitionGroup(
60
- {childFactory: this._wrapChild},
61
- this.props.children
62
+ merge(this.props, {childFactory: this._wrapChild})
63
)
64
);
65
}
src/addons/transitions/ReactTransitionGroup.js
+1
-1
@@ -183,7 +183,7 @@ var ReactTransitionGroup = React.createClass({
183
);
184
}
185
}
186
- return this.transferPropsTo(this.props.component(null, childrenToRender));
186
+ return this.props.component(this.props, childrenToRender);
187
}
188
});
189
src/browser/ui/dom/components/ReactDOMForm.js
+1
-1
@@ -42,7 +42,7 @@ var ReactDOMForm = ReactCompositeComponent.createClass({
42
// TODO: Instead of using `ReactDOM` directly, we should use JSX. However,
43
// `jshint` fails to parse JSX so in order for linting to work in the open
44
// source repo, we need to just use `ReactDOM.form`.
45
- return this.transferPropsTo(form(null, this.props.children));
45
+ return form(this.props);
46
},
47
48
componentDidMount: function() {
src/browser/ui/dom/components/createFullPageComponent.js
+1
-1
@@ -53,7 +53,7 @@ function createFullPageComponent(componentClass) {
53
},
54
55
render: function() {
56
- return this.transferPropsTo(componentClass(null, this.props.children));
56
+ return componentClass(this.props);
57
}
58
});
59