Completed
Pull Request — master (#283)
by Oskar
04:36
created
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/Metric/System/UnitTesting/UnitTesting.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 
73 73
         // JUNIT format
74 74
         foreach ($xpath->query('//testsuite[@file]') as $suite) {
75
-            array_push($testsuites, (object)[
75
+            array_push($testsuites, (object) [
76 76
                 'file' => $suite->getAttribute('file'),
77 77
                 'name' => $suite->getAttribute('name'),
78 78
                 'assertions' => $suite->getAttribute('assertions'),
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
                 $assertions = $case === $suite->firstChild->nextSibling ? $suite->getAttribute('assertions') : 0;
102 102
             }
103 103
 
104
-            $testsuites[$case->getAttribute('class')] = (object)[
104
+            $testsuites[$case->getAttribute('class')] = (object) [
105 105
                 'file' => $case->getAttribute('file'),
106 106
                 'name' => $case->getAttribute('class'),
107 107
                 'assertions' => $assertions,
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
             $traverser->addVisitor(new ExternalsVisitor($metricsOfUnitTest));
121 121
 
122 122
             if (!file_exists($suite->file) || !is_readable($suite->file)) {
123
-                throw new \LogicException('Cannot find source file referenced in testsuite: ' . $suite->file);
123
+                throw new \LogicException('Cannot find source file referenced in testsuite: '.$suite->file);
124 124
             }
125 125
 
126 126
             $code = file_get_contents($suite->file);
@@ -133,10 +133,10 @@  discard block
 block discarded – undo
133 133
 
134 134
             // list of externals sources of unit test
135 135
             $metric = $metricsOfUnitTest->get($suite->name);
136
-            $externals = (array)$metric->get('externals');
136
+            $externals = (array) $metric->get('externals');
137 137
 
138 138
             // global stats for each test
139
-            $infoAboutTests[$suite->name] = (object)[
139
+            $infoAboutTests[$suite->name] = (object) [
140 140
                 'nbExternals' => sizeof(array_unique($externals)),
141 141
                 'externals' => array_unique($externals),
142 142
                 'filename' => $suite->file,
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,5 +1,5 @@  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
 if (!isset($project['unitTesting'])) {
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 }
9 9
 
10 10
 $unit = $project['unitTesting'];
11
-$getMetricForClass = function ($classname, $metric) use ($classes) {
11
+$getMetricForClass = function($classname, $metric) use ($classes) {
12 12
     foreach ($classes as $class) {
13 13
         if ($classname !== $class['name']) {
14 14
             continue;
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
         </div>
137 137
     </div>
138 138
 
139
-    <?php require __DIR__ . '/_footer.php'; ?>
139
+    <?php require __DIR__.'/_footer.php'; ?>
140 140
 
141 141
     <script type="text/javascript">
142 142
         var clusterize = new Clusterize({
@@ -191,12 +191,12 @@  discard block
 block discarded – undo
191 191
             foreach ($unit['tests'] as $test) {
192 192
                 array_push($unitTimeJson->children, [
193 193
                     'name' => $test->classname,
194
-                    'time' => (float)$test->time,
194
+                    'time' => (float) $test->time,
195 195
                 ]);
196 196
             }
197 197
             ?>
198 198
 
199
-            var json = <?php echo json_encode($unitTimeJson);?>;
199
+            var json = <?php echo json_encode($unitTimeJson); ?>;
200 200
             root = json;
201 201
             var nodes = treemap.nodes(json)
202 202
                 .filter(function (d) {
Please login to merge, or discard this patch.
src/Hal/Metric/System/Packages/Composer/Packagist.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
 
37 37
         // get latest version
38 38
         $latest = '0.0.0';
39
-        foreach ((array)$json->package->versions as $version => $datas) {
39
+        foreach ((array) $json->package->versions as $version => $datas) {
40 40
             $version = preg_replace('([^\.\d])', '', $version);
41 41
             if (!preg_match('!\d+\.\d+\.\d+!', $version)) {
42 42
                 continue;
Please login to merge, or discard this patch.
src/Hal/Metric/System/Packages/Composer/Composer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -49,13 +49,13 @@  discard block
 block discarded – undo
49 49
             if (!preg_match('/composer\.json|composer-dist\.json/', $filename)) {
50 50
                 continue;
51 51
             }
52
-            $datas = (object)json_decode(file_get_contents($filename));
52
+            $datas = (object) json_decode(file_get_contents($filename));
53 53
 
54 54
             if (!isset($datas->require)) {
55 55
                 continue;
56 56
             }
57 57
 
58
-            $rawRequirements = array_merge($rawRequirements, (array)$datas->require);
58
+            $rawRequirements = array_merge($rawRequirements, (array) $datas->require);
59 59
         }
60 60
 
61 61
         $packagist = new Packagist();
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
             $package = $packagist->get($requirement);
65 65
 
66
-            $packages[$requirement] = (object)array(
66
+            $packages[$requirement] = (object) array(
67 67
                 'name' => $requirement,
68 68
                 'required' => $version,
69 69
                 'latest' => $package->latest,
Please login to merge, or discard this patch.
src/Hal/Report/Html/template/index.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php require __DIR__ . '/_header.php'; ?>
1
+<?php require __DIR__.'/_header.php'; ?>
2 2
     <div class="row">
3 3
         <div class="column">
4 4
             <div class="bloc bloc-number">
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
                             <tbody id="contentClassRank" class="clusterize-content">
96 96
                             <?php
97 97
                             $classesS = $classes;
98
-                            usort($classesS, function ($a, $b) {
98
+                            usort($classesS, function($a, $b) {
99 99
                                 return strcmp($b['pageRank'], $a['pageRank']);
100 100
                             });
101 101
                             //$classesS = array_slice($classesS, 0, 10);
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
                             <tbody id="contentPackages" class="clusterize-content">
134 134
                             <?php
135 135
                             $packages = isset($project['composer'], $project['composer']['packages']) ? $project['composer']['packages'] : [];
136
-                            usort($packages, function ($a, $b) {
136
+                            usort($packages, function($a, $b) {
137 137
                                 return strcmp($a->name, $b->name);
138 138
                             });
139 139
                             foreach ($packages as $package) { ?>
@@ -141,15 +141,15 @@  discard block
 block discarded – undo
141 141
                                     <td><?php echo $package->name; ?></td>
142 142
                                     <td><?php echo $package->required; ?></td>
143 143
                                     <td><?php echo $package->latest; ?></td>
144
-                                    <td><?php foreach($package->license as $license) { ?>
145
-                                            <a target="_blank" href="https://spdx.org/licenses/<?php echo $license;?>.html"><?php echo $license;?></a>
144
+                                    <td><?php foreach ($package->license as $license) { ?>
145
+                                            <a target="_blank" href="https://spdx.org/licenses/<?php echo $license; ?>.html"><?php echo $license; ?></a>
146 146
                                         <?php }; ?>
147 147
                                     </td>
148 148
                                 </tr>
149 149
                             <?php } ?>
150 150
                             </tbody>
151 151
                         </table>
152
-                        <?php if(0 === sizeof($packages)) { ?>
152
+                        <?php if (0 === sizeof($packages)) { ?>
153 153
                             <div>No composer.json file found</div>
154 154
                         <?php } ?>
155 155
                     </div>
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
         <div class="column">
161 161
             <div class="bloc bloc-number">
162 162
                 <div class="label">Licences of Composer dependencies</div>
163
-                <?php if(0 === sizeof($packages)) { ?>
163
+                <?php if (0 === sizeof($packages)) { ?>
164 164
                     <div>No composer.json file found</div>
165 165
                 <?php } ?>
166 166
                 <div id="svg-licenses"></div>
@@ -199,9 +199,9 @@  discard block
 block discarded – undo
199 199
                     }
200 200
                 }
201 201
                 ?>
202
-                chartLicenses(<?php echo json_encode(array_values($json));?>);
202
+                chartLicenses(<?php echo json_encode(array_values($json)); ?>);
203 203
             }
204 204
         };
205 205
     </script>
206 206
 
207
-<?php require __DIR__ . '/_footer.php'; ?>
208 207
\ No newline at end of file
208
+<?php require __DIR__.'/_footer.php'; ?>
209 209
\ No newline at end of file
Please login to merge, or discard this patch.