Merge pull request #371 from smacker/fix_mode_switcher

smacker · web-flow · commit 757a97a5660b · 2019-03-12T20:08:51.000+01:00
fix: switching between uast modes returns inconsistent results
diff --git a/frontend/src/components/CodeViewer.js b/frontend/src/components/CodeViewer.js
@@ -190,10 +190,11 @@ class CodeViewer extends Component {
   }
 
   handleModeChange(mode) {
-    this.setState({ mode });
-    if (this.state.showUast) {
-      this.parseCode();
-    }
+    this.setState({ mode }, () => {
+      if (this.state.showUast) {
+        this.parseCode();
+      }
+    });
   }
 
   render() {

Original file line number	Diff line number	Diff line change
`@@ -190,10 +190,11 @@ class CodeViewer extends Component {`
`190`	`190`	`}`
`191`	`191`
`192`	`192`	`handleModeChange(mode) {`
`193`		`- this.setState({ mode });`
`194`		`- if (this.state.showUast) {`
`195`		`- this.parseCode();`
`196`		`- }`
	`193`	`+ this.setState({ mode }, () => {`
	`194`	`+ if (this.state.showUast) {`
	`195`	`+ this.parseCode();`
	`196`	`+ }`
	`197`	`+ });`
`197`	`198`	`}`
`198`	`199`
`199`	`200`	`render() {`