Completed
Pull Request — master (#271)
by Marco
02:33
created
artifacts/phar/build.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,8 +1,8 @@  discard block
 block discarded – undo
1 1
 <?php
2
-chdir(__DIR__ . '/../../');
2
+chdir(__DIR__.'/../../');
3 3
 
4 4
 if (!file_exists('vendor/autoload.php')) {
5
-    echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!' . PHP_EOL;
5
+    echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!'.PHP_EOL;
6 6
     exit(1);
7 7
 }
8 8
 
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
     if (preg_match($exclude, $file)) {
22 22
         continue;
23 23
     }
24
-    $path = str_replace(__DIR__ . '/', '', $file);
24
+    $path = str_replace(__DIR__.'/', '', $file);
25 25
     $phar->addFromString($path, file_get_contents($file));
26 26
 }
27 27
 
@@ -52,8 +52,8 @@  discard block
 block discarded – undo
52 52
 
53 53
 function rglob($pattern = '*', $flags = 0, $path = '')
54 54
 {
55
-    $paths = glob($path . '*', GLOB_MARK | GLOB_ONLYDIR | GLOB_NOSORT);
56
-    $files = glob($path . $pattern, $flags);
55
+    $paths = glob($path.'*', GLOB_MARK|GLOB_ONLYDIR|GLOB_NOSORT);
56
+    $files = glob($path.$pattern, $flags);
57 57
     foreach ($paths as $path) {
58 58
         $files = array_merge($files, rglob($pattern, $flags, $path));
59 59
     }
Please login to merge, or discard this patch.
src/Hal/Application/Analyze.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
 
49 49
     /**
50 50
      * Analyze constructor.
51
-     * @param OutputInterface $output
51
+     * @param Output $output
52 52
      */
53 53
     public function __construct(Config $config, Output $output, Issuer $issuer)
54 54
     {
Please login to merge, or discard this patch.
src/Hal/Component/Issue/Issuer.php 1 patch
Doc Comments   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 
33 33
     /**
34 34
      * Issuer constructor.
35
-     * @param OutputInterface $output
35
+     * @param Output $output
36 36
      */
37 37
     public function __construct(Output $output)
38 38
     {
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
     }
131 131
 
132 132
     /**
133
-     * @param $status
133
+     * @param integer $status
134 134
      */
135 135
     protected function terminate($status)
136 136
     {
@@ -139,6 +139,7 @@  discard block
 block discarded – undo
139 139
 
140 140
     /**
141 141
      * @param $log
142
+     * @param string $logfile
142 143
      * @return $this
143 144
      */
144 145
     protected function log($logfile, $log)
@@ -152,7 +153,7 @@  discard block
 block discarded – undo
152 153
     }
153 154
 
154 155
     /**
155
-     * @param $debugKey
156
+     * @param string $debugKey
156 157
      * @param $value
157 158
      * @return $this
158 159
      */
@@ -163,7 +164,7 @@  discard block
 block discarded – undo
163 164
     }
164 165
 
165 166
     /**
166
-     * @param $debugKey
167
+     * @param string $debugKey
167 168
      * @return $this
168 169
      */
169 170
     public function clear($debugKey)
Please login to merge, or discard this patch.
src/Hal/Report/Cli/Reporter.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@
 block discarded – undo
22 22
     /**
23 23
      * Reporter constructor.
24 24
      * @param Config $config
25
-     * @param OutputInterface $output
25
+     * @param Output $output
26 26
      */
27 27
     public function __construct(Config $config, Output $output)
28 28
     {
Please login to merge, or discard this patch.
src/Hal/Report/Violations/Xml/Reporter.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 
4 4
 use Hal\Application\Config\Config;
5 5
 use Hal\Component\Output\Output;
6
-use Hal\Metric\Consolided;
7 6
 use Hal\Metric\Metrics;
8 7
 use Hal\Violation\Violation;
9 8
 
Please login to merge, or discard this patch.
src/Hal/Component/Output/ProgressBar.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,9 +18,9 @@
 block discarded – undo
18 18
      * @package Hal\Component\Issue
19 19
      */
20 20
 /**
21
- * Class ProgressBar
22
- * @package Hal\Component\Output
23
- */
21
+     * Class ProgressBar
22
+     * @package Hal\Component\Output
23
+     */
24 24
 class ProgressBar
25 25
 {
26 26
 
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@
 block discarded – undo
97 97
         if (DIRECTORY_SEPARATOR === '\\') {
98 98
             return
99 99
                 0 >= version_compare('10.0.10586',
100
-                    PHP_WINDOWS_VERSION_MAJOR . '.' . PHP_WINDOWS_VERSION_MINOR . '.' . PHP_WINDOWS_VERSION_BUILD)
100
+                    PHP_WINDOWS_VERSION_MAJOR.'.'.PHP_WINDOWS_VERSION_MINOR.'.'.PHP_WINDOWS_VERSION_BUILD)
101 101
                 || false !== getenv('ANSICON')
102 102
                 || 'ON' === getenv('ConEmuANSI')
103 103
                 || 'xterm' === getenv('TERM');
Please login to merge, or discard this patch.
src/Hal/Component/Output/TestOutput.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
      */
24 24
     public function writeln($message)
25 25
     {
26
-        $this->write(PHP_EOL . $message);
26
+        $this->write(PHP_EOL.$message);
27 27
         return $this;
28 28
     }
29 29
 
Please login to merge, or discard this patch.
src/Hal/Component/Output/CliOutput.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
      */
26 26
     public function writeln($message)
27 27
     {
28
-        $this->write(PHP_EOL . $message);
28
+        $this->write(PHP_EOL.$message);
29 29
         return $this;
30 30
     }
31 31
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
      */
35 35
     public function write($message)
36 36
     {
37
-        $this->quietMode||file_put_contents('php://stdout', $message);
37
+        $this->quietMode || file_put_contents('php://stdout', $message);
38 38
         return $this;
39 39
     }
40 40
 
Please login to merge, or discard this patch.
src/Hal/Report/Html/template/junit.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,9 +1,9 @@  discard block
 block discarded – undo
1 1
 <?php
2
-require __DIR__ . '/_header.php'; ?>
2
+require __DIR__.'/_header.php'; ?>
3 3
 
4 4
 <?php
5 5
 $unit = $project['unitTesting'];
6
-$getMetricForClass = function ($classname, $metric) use ($classes) {
6
+$getMetricForClass = function($classname, $metric) use ($classes) {
7 7
     foreach ($classes as $class) {
8 8
         if ($classname !== $class['name']) {
9 9
             continue;
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
         </div>
134 134
     </div>
135 135
 
136
-    <?php require __DIR__ . '/_footer.php'; ?>
136
+    <?php require __DIR__.'/_footer.php'; ?>
137 137
 
138 138
     <script type="text/javascript">
139 139
         var clusterize = new Clusterize({
@@ -188,12 +188,12 @@  discard block
 block discarded – undo
188 188
             foreach ($unit['tests'] as $test) {
189 189
                 array_push($unitTimeJson->children, [
190 190
                     'name' => $test->classname,
191
-                    'time' => (float)$test->time,
191
+                    'time' => (float) $test->time,
192 192
                 ]);
193 193
             }
194 194
             ?>
195 195
 
196
-            var json = <?php echo json_encode($unitTimeJson);?>;
196
+            var json = <?php echo json_encode($unitTimeJson); ?>;
197 197
             root = json;
198 198
             var nodes = treemap.nodes(json)
199 199
                 .filter(function (d) {
Please login to merge, or discard this patch.