p15670423
/
monkey
1
0
Fork 5
Code Issues Pull Requests 3 Projects Releases Wiki Activity

Some unecessary set states removed to prevent refreshing of config page

This commit is contained in:
itay 2019-02-14 19:07:12 +02:00
parent 0513966c76
commit d67cb18cae
2 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
monkey/monkey_island/cc/ui/src/components/Main.js
View File

@ -29,9 +29,11 @@ class AppComponent extends AuthComponent {
updateStatus = () => {
this.auth.loggedIn()
.then(res => {
this.setState({
isLoggedIn: res
});
if (this.state.isLoggedIn !== res) {
this.setState({
isLoggedIn: res
});
}
if (res) {
this.authFetch('/api')

9
monkey/monkey_island/cc/ui/src/components/pages/ConfigurePage.js
View File

@ -141,9 +141,12 @@ class ConfigurePageComponent extends AuthComponent {
.then(res => res.json())
.then(res => {
// This check is used to prevent unnecessary re-rendering
this.setState({
allMonkeysAreDead: (!res['completed_steps']['run_monkey']) || (res['completed_steps']['infection_done'])
});
let allMonkeysAreDead = (!res['completed_steps']['run_monkey']) || (res['completed_steps']['infection_done']);
if (allMonkeysAreDead !== this.state.allMonkeysAreDead) {
this.setState({
allMonkeysAreDead: allMonkeysAreDead
});
}
});
};