fix(autoload): call this.autoloadAsyncOptions() into `componentDidM…

…ount()` see: * JedWatson#279 * facebook/react#1740
iam4x · Jun 29, 2015 · f1a558d · f1a558d
1 parent bbefe62
commit f1a558d
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/lib/Select.js b/lib/Select.js
@@ -133,12 +133,7 @@ var Select = React.createClass({
 			}
 		};
 
-		this.setState(this.getStateFromValue(this.props.value), function () {
-			//Executes after state change is done. Fixes issue #201
-			if (this.props.asyncOptions && this.props.autoload) {
-				this.autoloadAsyncOptions();
-			}
-		});
+		this.setState(this.getStateFromValue(this.props.value));
 	},
 
 	componentWillUnmount: function componentWillUnmount() {
@@ -150,6 +145,12 @@ var Select = React.createClass({
 		}
 	},
 
+	componentDidMount: function componentdDidMount() {
+		if (this.props.asyncOptions && this.props.autoload) {
+			this.autoloadAsyncOptions();
+		}
+	},
+
 	componentWillReceiveProps: function componentWillReceiveProps(newProps) {
 		if (JSON.stringify(newProps.options) !== JSON.stringify(this.props.options)) {
 			this.setState({
@@ -814,4 +815,4 @@ var Select = React.createClass({
 
 });
 
-module.exports = Select;
+module.exports = Select;