JBDS-4024 Components installation sometimes called twice

Fix updates condition for install method to avoid the problem.

Author: dgolovin

browser/model/cdk.js

@@ -94,7 +94,7 @@ class CDKInstall extends InstallableItem {
       let name = this.getInstallAfter().productName;
       progress.setStatus(`Waiting for ${name} to finish installation`);
       ipcRenderer.on('installComplete', (event, arg) => {
-        if (!this.isInstalled() && this.getInstallAfter().isInstalled()) {
+        if (!this.isInstalled() &&arg === this.getInstallAfter().keyName) {
           this.postVagrantInstall(progress, success, failure);
         }
       });

browser/model/cygwin.js

@@ -80,7 +80,7 @@ class CygwinInstall extends InstallableItem {
       let name = this.getInstallAfter().productName;
       progress.setStatus(`Waiting for ${name} to finish installation`);
       ipcRenderer.on('installComplete', (event, arg) => {
-        if (!this.isInstalled() && this.getInstallAfter().isInstalled() ) {
+        if (!this.isInstalled() && arg === this.getInstallAfter().keyName) {
           this.postVirtualboxInstall(progress, success, failure);
         }
       });
@@ -112,7 +112,6 @@ class CygwinInstall extends InstallableItem {
       '[Environment]::SetEnvironmentVariable("Path", "$cygwinPath;$oldPath", "User");',
       '[Environment]::Exit(0)'
     ].join('\r\n');
-
     let originalExecFile = path.join(this.installerDataSvc.cygwinDir(),'setup-x86_64.exe');
     installer.execFile(
       this.downloadedFile, opts

browser/model/jbds.js

@@ -137,7 +137,7 @@ class JbdsInstall extends InstallableItem {
       let name = this.getInstallAfter().productName;
       progress.setStatus(`Waiting for ${name} to finish installation`);
       ipcRenderer.on('installComplete', (event, arg) => {
-        if(!this.isInstalled() && this.getInstallAfter().isInstalled()) {
+        if(!this.isInstalled() && arg === this.getInstallAfter().keyName) {
           this.postInstall(progress, success, failure);
         }
       });

browser/model/vagrant.js

@@ -130,7 +130,7 @@ class VagrantInstall extends InstallableItem {
       let name = this.getInstallAfter().productName;
       progress.setStatus(`Waiting for ${name} to finish installation`);
       ipcRenderer.on('installComplete', (event, arg) => {
-        if (!this.isInstalled() && this.getInstallAfter().isInstalled()) {
+        if (!this.isInstalled() && arg === this.getInstallAfter().keyName) {
           this.postCygwinInstall(progress, success, failure);
         }
       });

browser/model/virtualbox.js

@@ -154,7 +154,7 @@ class VirtualBoxInstall extends InstallableItem {
       let name = this.getInstallAfter().productName;
       progress.setStatus(`Waiting for ${name} to finish installation`);
       ipcRenderer.on('installComplete', (event, arg) => {
-        if (!this.isInstalled() && this.getInstallAfter().isInstalled()) {
+        if (!this.isInstalled() && arg === this.getInstallAfter().keyName) {
           this.postOpenJdk(progress, success, failure);
         }
       });