Test Failed
Push — CI ( 785a66...02428e )
by Adam
54:23
created
src/SuiteCrm/Install/InstallCommand.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
     /**
33 33
      * @param InputInterface $input
34 34
      * @param OutputInterface $output
35
-     * @return bool
35
+     * @return boolean|null
36 36
      */
37 37
     protected function execute(InputInterface $input, OutputInterface $output) {
38 38
         parent::_execute($input, $output);
Please login to merge, or discard this patch.
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
      */
43 43
     protected function execute(InputInterface $input, OutputInterface $output) {
44 44
         parent::_execute($input, $output);
45
-        $this->log("Starting command " . static::COMMAND_NAME . "...");
45
+        $this->log("Starting command ".static::COMMAND_NAME."...");
46 46
         $this->doPhpVersionCheck();
47 47
         $this->setupSugarGlobals();
48 48
         $this->setupSugarLogger();
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 
51 51
 
52 52
 
53
-        $this->log("Command " . static::COMMAND_NAME . " done.");
53
+        $this->log("Command ".static::COMMAND_NAME." done.");
54 54
     }
55 55
 
56 56
 
@@ -60,17 +60,17 @@  discard block
 block discarded – undo
60 60
     protected function executeInstallation() {
61 61
         define('sugarEntry', true);
62 62
         //require_once(PROJECT_ROOT . '/include/SugarLogger/LoggerManager.php');
63
-        require_once(PROJECT_ROOT . '/sugar_version.php');
64
-        require_once(PROJECT_ROOT . '/suitecrm_version.php');
65
-        require_once(PROJECT_ROOT . '/include/utils.php');
66
-        require_once(PROJECT_ROOT . '/install/install_utils.php');
67
-        require_once(PROJECT_ROOT . '/install/install_defaults.php');
68
-        require_once(PROJECT_ROOT . '/include/TimeDate.php');
69
-        require_once(PROJECT_ROOT . '/include/Localization/Localization.php');
70
-        require_once(PROJECT_ROOT . '/include/SugarTheme/SugarTheme.php');
71
-        require_once(PROJECT_ROOT . '/include/utils/LogicHook.php');
72
-        require_once(PROJECT_ROOT . '/data/SugarBean.php');
73
-        require_once(PROJECT_ROOT . '/include/entryPoint.php');
63
+        require_once(PROJECT_ROOT.'/sugar_version.php');
64
+        require_once(PROJECT_ROOT.'/suitecrm_version.php');
65
+        require_once(PROJECT_ROOT.'/include/utils.php');
66
+        require_once(PROJECT_ROOT.'/install/install_utils.php');
67
+        require_once(PROJECT_ROOT.'/install/install_defaults.php');
68
+        require_once(PROJECT_ROOT.'/include/TimeDate.php');
69
+        require_once(PROJECT_ROOT.'/include/Localization/Localization.php');
70
+        require_once(PROJECT_ROOT.'/include/SugarTheme/SugarTheme.php');
71
+        require_once(PROJECT_ROOT.'/include/utils/LogicHook.php');
72
+        require_once(PROJECT_ROOT.'/data/SugarBean.php');
73
+        require_once(PROJECT_ROOT.'/include/entryPoint.php');
74 74
         //
75 75
         $timedate = \TimeDate::getInstance();
76 76
         setPhpIniSettings();
@@ -82,8 +82,8 @@  discard block
 block discarded – undo
82 82
 
83 83
         try {
84 84
             SystemChecker::runChecks();
85
-        } catch(\Exception $e) {
86
-            $this->log("SYSTEM CHECK ERROR! " . $e->getMessage());
85
+        }catch (\Exception $e) {
86
+            $this->log("SYSTEM CHECK ERROR! ".$e->getMessage());
87 87
             die();
88 88
         }
89 89
 
@@ -111,8 +111,8 @@  discard block
 block discarded – undo
111 111
      * @throws \Exception
112 112
      */
113 113
     protected function doPhpVersionCheck() {
114
-        if (version_compare(phpversion(),'5.2.0') < 0) {
115
-            throw new \Exception('Minimum PHP version required is 5.2.0.  You are using PHP version  '. phpversion());
114
+        if (version_compare(phpversion(), '5.2.0') < 0) {
115
+            throw new \Exception('Minimum PHP version required is 5.2.0.  You are using PHP version  '.phpversion());
116 116
         }
117 117
     }
118 118
 }
119 119
\ No newline at end of file
Please login to merge, or discard this patch.
src/SuiteCrm/Install/LoggerManager.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
     protected function log($msg, $level) {
65 65
         if ($this->logToConsole) {
66 66
             $msg = is_array($msg) ? implode(" - ", $msg) : $msg;
67
-            $this->cmdOutput->writeln("[$level]: " . $msg);
67
+            $this->cmdOutput->writeln("[$level]: ".$msg);
68 68
         }
69 69
     }
70 70
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
      * @param string $logLevel
92 92
      */
93 93
     public function setLevel($logLevel) {
94
-        if(in_array($logLevel, array_keys(self::$logLevelMapping))) {
94
+        if (in_array($logLevel, array_keys(self::$logLevelMapping))) {
95 95
             $this->defaultLogLevel = $logLevel;
96 96
         }
97 97
     }
Please login to merge, or discard this patch.
src/SuiteCrm/Install/SystemChecker.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,9 +1,9 @@
 block discarded – undo
1 1
 <?php
2 2
 /**
3
- * Created by Adam Jakab.
4
- * Date: 08/03/16
5
- * Time: 16.07
6
- */
3
+     * Created by Adam Jakab.
4
+     * Date: 08/03/16
5
+     * Time: 16.07
6
+     */
7 7
 
8 8
 namespace SuiteCrm\Install;
9 9
 
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
             'include/javascript'
40 40
         ];
41 41
         foreach ($cache_files as $cache_file) {
42
-            $dirname = PROJECT_ROOT . '/' . sugar_cached( $cache_file);
42
+            $dirname = PROJECT_ROOT.'/'.sugar_cached($cache_file);
43 43
             $ok = FALSE;
44 44
             if ((is_dir($dirname)) || @sugar_mkdir($dirname, 0755, TRUE)) {
45 45
                 $ok = make_writable($dirname);
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
             if (!$ok) {
48 48
                 throw new \Exception("The Cache Directory($dirname) is not writable.");
49 49
             } else {
50
-                echo "\n" . $dirname;
50
+                echo "\n".$dirname;
51 51
             }
52 52
         }
53 53
     }
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
      * @throws \Exception
57 57
      */
58 58
     protected static function checkCustomDirectory() {
59
-        if (!make_writable(PROJECT_ROOT . '/custom')) {
59
+        if (!make_writable(PROJECT_ROOT.'/custom')) {
60 60
             throw new \Exception("The Custom Directory exists but is not writable.");
61 61
         }
62 62
     }
@@ -65,10 +65,10 @@  discard block
 block discarded – undo
65 65
      * @throws \Exception
66 66
      */
67 67
     protected static function checkConfigOverrideFile() {
68
-        if (file_exists(PROJECT_ROOT . '/config_override.php')
69
-            && (!(make_writable(PROJECT_ROOT . '/config_override.php'))
68
+        if (file_exists(PROJECT_ROOT.'/config_override.php')
69
+            && (!(make_writable(PROJECT_ROOT.'/config_override.php'))
70 70
                 || !(is_writable(
71
-                    PROJECT_ROOT . '/config_override.php'
71
+                    PROJECT_ROOT.'/config_override.php'
72 72
                 )))
73 73
         ) {
74 74
             throw new \Exception("The config_override file is not writable!");
@@ -79,8 +79,8 @@  discard block
 block discarded – undo
79 79
      * @throws \Exception
80 80
      */
81 81
     protected static function checkConfigFile() {
82
-        if (file_exists(PROJECT_ROOT . '/config.php')
83
-            && (!(make_writable(PROJECT_ROOT . '/config.php')) || !(is_writable(PROJECT_ROOT . '/config.php')))
82
+        if (file_exists(PROJECT_ROOT.'/config.php')
83
+            && (!(make_writable(PROJECT_ROOT.'/config.php')) || !(is_writable(PROJECT_ROOT.'/config.php')))
84 84
         ) {
85 85
             throw new \Exception("The config file is not writable!");
86 86
         }
Please login to merge, or discard this patch.