Code cleanup from redundant falsy checks in installer.js

Author: dgolovin

browser/model/helpers/installer.js

@@ -19,12 +19,12 @@ class Installer {
     return new Promise((resolve, reject) => {
       Logger.info(this.key + ' - Execute command ' + command);
       child_process.exec(command, options, (error, stdout, stderr) => {
-        if (error && error !== '') {
+        if (error) {
           Logger.error(this.key + ' - ' + error);
           Logger.error(this.key + ' - ' + stderr);
           reject(error);
         } else {
-          if (stdout && stdout != '') {
+          if (stdout) {
             Logger.info(this.key + ' - ' + stdout);
           }
           Logger.info(this.key + ' - Execute ' + command + ' SUCCESS');
@@ -39,12 +39,12 @@ class Installer {
       Logger.info(this.key + ' - Execute ' + file + ' ' + args);
       child_process.execFile(file, args, {'maxBuffer': 1024*1024} , (error, stdout, stderr) => {
         // vagrant exits with code 3010
-        if (error && error !== '' && error.code !== 3010) {
+        if (error && error.code !== 3010) {
           Logger.error(this.key + ' - ' + error);
           Logger.error(this.key + ' - ' + stderr);
           reject(error);
         } else {
-          if (stdout && stdout != '') {
+          if (stdout) {
             Logger.info(this.key + ' - ' + stdout);
           }
           Logger.info(this.key + ' - Execute ' + file + ' ' + args + ' SUCCESS');