This repository was archived by the owner on Nov 20, 2023. It is now read-only.

Commit 403bd89

author

Sanjana Desai

committed

bug fix in setState of componentWillMount - handling situation in which there are more than one product options

1 parent 8f249f4 commit 403bd89Copy full SHA for 403bd89

File tree

-4

lines changed

-4

lines changed

Lines changed: 9 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,9 @@ class Product extends Component {`
`5`	`5`	`constructor(props) {`
`6`	`6`	`super(props);`
`7`	`7`
`8`		`- this.state = {};`
	`8`	`+ this.state = {`
	`9`	`+ selectedOptions: {}`
	`10`	`+ };`
`9`	`11`
`10`	`12`	`this.handleOptionChange = this.handleOptionChange.bind(this);`
`11`	`13`	`this.handleQuantityChange = this.handleQuantityChange.bind(this);`
`@@ -14,9 +16,12 @@ class Product extends Component {`
`14`	`16`
`15`	`17`	`componentWillMount() {`
`16`	`18`	`this.props.product.options.forEach((selector) => {`
`17`		`- this.setState({`
`18`		`- selectedOptions: { [selector.name]: selector.values[0] }`
`19`		`- });`
	`19`	`+ this.setState((prevState) => ({`
	`20`	`+ selectedOptions:{`
	`21`	`+ ...prevState.selectedOptions,`
	`22`	`+ [selector.name]: selector.values[0]`
	`23`	`+ }`
	`24`	`+ }));`
`20`	`25`	`});`
`21`	`26`	`}`
`22`	`27`

Comments

(0)