Completed
Pull Request — master (#448)
by
unknown
08:31
created
phpmetrics/src/Hal/Report/Violations/Xml/Reporter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@
 block discarded – undo
67 67
                 $item->setAttribute('rule', $violation->getName());
68 68
                 $item->setAttribute('ruleset', $violation->getName());
69 69
                 $item->setAttribute('externalInfoUrl', 'http://www.phpmetrics.org/documentation/index.html');
70
-                $item->setAttribute('priority', (string)$map[$violation->getLevel()]);
70
+                $item->setAttribute('priority', (string) $map[$violation->getLevel()]);
71 71
                 $item->nodeValue = $violation->getDescription();
72 72
                 $node->appendChild($item);
73 73
             }
Please login to merge, or discard this patch.
phpmetrics/src/Hal/Report/Html/Reporter.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
     {
58 58
         $this->config = $config;
59 59
         $this->output = $output;
60
-        $this->templateDir = __DIR__ . '/../../../../templates';
60
+        $this->templateDir = __DIR__.'/../../../../templates';
61 61
     }
62 62
 
63 63
     /** @return void */
@@ -82,11 +82,11 @@  discard block
 block discarded – undo
82 82
         $consolidated = new Consolidated($metrics);
83 83
 
84 84
         // history of builds
85
-        $today = (object)[
85
+        $today = (object) [
86 86
             'avg' => $consolidated->getAvg(),
87 87
             'sum' => $consolidated->getSum()
88 88
         ];
89
-        $files = glob($logDir . '/js/history-*.json');
89
+        $files = glob($logDir.'/js/history-*.json');
90 90
         if ($files === false) {
91 91
             throw new ShouldNotHappenException('Glob logdir operation return false');
92 92
         }
@@ -102,40 +102,40 @@  discard block
 block discarded – undo
102 102
         }
103 103
 
104 104
         // copy sources
105
-        if (!file_exists($logDir . '/js')) {
106
-            mkdir($logDir . '/js', 0755, true);
105
+        if (!file_exists($logDir.'/js')) {
106
+            mkdir($logDir.'/js', 0755, true);
107 107
         }
108
-        if (!file_exists($logDir . '/css')) {
109
-            mkdir($logDir . '/css', 0755, true);
108
+        if (!file_exists($logDir.'/css')) {
109
+            mkdir($logDir.'/css', 0755, true);
110 110
         }
111
-        if (!file_exists($logDir . '/images')) {
112
-            mkdir($logDir . '/images', 0755, true);
111
+        if (!file_exists($logDir.'/images')) {
112
+            mkdir($logDir.'/images', 0755, true);
113 113
         }
114
-        if (!file_exists($logDir . '/fonts')) {
115
-            mkdir($logDir . '/fonts', 0755, true);
114
+        if (!file_exists($logDir.'/fonts')) {
115
+            mkdir($logDir.'/fonts', 0755, true);
116 116
         }
117
-        recurse_copy($this->templateDir . '/html_report/js', $logDir . '/js');
118
-        recurse_copy($this->templateDir . '/html_report/css', $logDir . '/css');
119
-        recurse_copy($this->templateDir . '/html_report/images', $logDir . '/images');
120
-        recurse_copy($this->templateDir . '/html_report/fonts', $logDir . '/fonts');
117
+        recurse_copy($this->templateDir.'/html_report/js', $logDir.'/js');
118
+        recurse_copy($this->templateDir.'/html_report/css', $logDir.'/css');
119
+        recurse_copy($this->templateDir.'/html_report/images', $logDir.'/images');
120
+        recurse_copy($this->templateDir.'/html_report/fonts', $logDir.'/fonts');
121 121
 
122 122
         // render dynamic pages
123
-        $this->renderPage($this->templateDir . '/html_report/index.php', $logDir . '/index.html', $consolidated, $history);
124
-        $this->renderPage($this->templateDir . '/html_report/loc.php', $logDir . '/loc.html', $consolidated, $history);
125
-        $this->renderPage($this->templateDir . '/html_report/relations.php', $logDir . '/relations.html', $consolidated, $history);
126
-        $this->renderPage($this->templateDir . '/html_report/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
127
-        $this->renderPage($this->templateDir . '/html_report/all.php', $logDir . '/all.html', $consolidated, $history);
128
-        $this->renderPage($this->templateDir . '/html_report/oop.php', $logDir . '/oop.html', $consolidated, $history);
129
-        $this->renderPage($this->templateDir . '/html_report/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
130
-        $this->renderPage($this->templateDir . '/html_report/panel.php', $logDir . '/panel.html', $consolidated, $history);
131
-        $this->renderPage($this->templateDir . '/html_report/violations.php', $logDir . '/violations.html', $consolidated, $history);
132
-        $this->renderPage($this->templateDir . '/html_report/packages.php', $logDir . '/packages.html', $consolidated, $history);
133
-        $this->renderPage($this->templateDir . '/html_report/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
134
-        $this->renderPage($this->templateDir . '/html_report/composer.php', $logDir . '/composer.html', $consolidated, $history);
123
+        $this->renderPage($this->templateDir.'/html_report/index.php', $logDir.'/index.html', $consolidated, $history);
124
+        $this->renderPage($this->templateDir.'/html_report/loc.php', $logDir.'/loc.html', $consolidated, $history);
125
+        $this->renderPage($this->templateDir.'/html_report/relations.php', $logDir.'/relations.html', $consolidated, $history);
126
+        $this->renderPage($this->templateDir.'/html_report/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
127
+        $this->renderPage($this->templateDir.'/html_report/all.php', $logDir.'/all.html', $consolidated, $history);
128
+        $this->renderPage($this->templateDir.'/html_report/oop.php', $logDir.'/oop.html', $consolidated, $history);
129
+        $this->renderPage($this->templateDir.'/html_report/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
130
+        $this->renderPage($this->templateDir.'/html_report/panel.php', $logDir.'/panel.html', $consolidated, $history);
131
+        $this->renderPage($this->templateDir.'/html_report/violations.php', $logDir.'/violations.html', $consolidated, $history);
132
+        $this->renderPage($this->templateDir.'/html_report/packages.php', $logDir.'/packages.html', $consolidated, $history);
133
+        $this->renderPage($this->templateDir.'/html_report/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
134
+        $this->renderPage($this->templateDir.'/html_report/composer.php', $logDir.'/composer.html', $consolidated, $history);
135 135
         if ($this->config->has('git')) {
136
-            $this->renderPage($this->templateDir . '/html_report/git.php', $logDir . '/git.html', $consolidated, $consolidatedGroups, $history);
136
+            $this->renderPage($this->templateDir.'/html_report/git.php', $logDir.'/git.html', $consolidated, $consolidatedGroups, $history);
137 137
         }
138
-        $this->renderPage($this->templateDir . '/html_report/junit.php', $logDir . '/junit.html', $consolidated, $consolidatedGroups, $history);
138
+        $this->renderPage($this->templateDir.'/html_report/junit.php', $logDir.'/junit.html', $consolidated, $consolidatedGroups, $history);
139 139
 
140 140
         // js data
141 141
         file_put_contents(
@@ -149,8 +149,8 @@  discard block
 block discarded – undo
149 149
 
150 150
         // json data
151 151
         file_put_contents(
152
-            $logDir . '/classes.js',
153
-            'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
152
+            $logDir.'/classes.js',
153
+            'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
154 154
         );
155 155
 
156 156
         // HTML files to generate
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
 
172 172
         // consolidated by groups
173 173
         foreach ($consolidatedGroups as $name => $consolidated) {
174
-            $outDir = $logDir . DIRECTORY_SEPARATOR . $name;
174
+            $outDir = $logDir.DIRECTORY_SEPARATOR.$name;
175 175
             $this->currentGroup = $name;
176 176
             $this->assetPath = '../';
177 177
 
@@ -188,8 +188,8 @@  discard block
 block discarded – undo
188 188
                 );
189 189
 
190 190
                 file_put_contents(
191
-                    $outDir . '/classes.js',
192
-                    'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
191
+                    $outDir.'/classes.js',
192
+                    'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
193 193
                 );
194 194
             }
195 195
         }
@@ -267,7 +267,7 @@  discard block
 block discarded – undo
267 267
 
268 268
         $diff = $newValue - $oldValue;
269 269
         if ($diff > 0) {
270
-            $diff = '+' . $diff;
270
+            $diff = '+'.$diff;
271 271
         }
272 272
 
273 273
         $goodOrBad = 'neutral';
Please login to merge, or discard this patch.
phpmetrics/tests/Violation/examples/godobject.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -36,24 +36,24 @@
 block discarded – undo
36 36
 
37 37
     public function flow3_B()
38 38
     {
39
-        return $this->d  * 3;
39
+        return $this->d * 3;
40 40
     }
41 41
 
42 42
     public function flow3_C()
43 43
     {
44
-        return $this->d  * 4;
44
+        return $this->d * 4;
45 45
         $a = new E;
46 46
         $b = new F;
47 47
     }
48 48
 
49 49
     public function flow3_D()
50 50
     {
51
-        return $this->d  * 5;
51
+        return $this->d * 5;
52 52
     }
53 53
 
54 54
     public function flow3_E()
55 55
     {
56
-        return $this->d  * 6;
56
+        return $this->d * 6;
57 57
     }
58 58
 }
59 59
 
Please login to merge, or discard this patch.
phpmetrics/tests/Metric/System/UnitTesting/UnitTestingTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
     public function testICanParseJunitXmlFile()
15 15
     {
16 16
         $config = new Config();
17
-        $config->set('junit', __DIR__ . '/xml/junit1.xml');
17
+        $config->set('junit', __DIR__.'/xml/junit1.xml');
18 18
         $unit = new UnitTesting($config, []);
19 19
         $metrics = new Metrics();
20 20
         $unit->calculate($metrics);
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
     public function testExceptionIsThrownIfJunitFileDoesNotExist()
41 41
     {
42 42
         $config = new Config();
43
-        $config->set('junit', __DIR__ . '/xml/junit-not-found.xml');
43
+        $config->set('junit', __DIR__.'/xml/junit-not-found.xml');
44 44
         $unit = new UnitTesting($config, []);
45 45
         $metrics = new Metrics();
46 46
         $unit->calculate($metrics);
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
     public function testICanParseCodeceptionFile()
50 50
     {
51 51
         $config = new Config();
52
-        $config->set('junit', __DIR__ . '/xml/codeception1.xml');
52
+        $config->set('junit', __DIR__.'/xml/codeception1.xml');
53 53
         $unit = new UnitTesting($config, []);
54 54
         $metrics = new Metrics();
55 55
         $unit->calculate($metrics);
Please login to merge, or discard this patch.
phpmetrics/tests/Metric/PackageMetricTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@
 block discarded – undo
124 124
 
125 125
         $metric->setNormalizedDistance(1);
126 126
         $this->assertSame(1, $metric->getNormalizedDistance());
127
-        $this->assertEquals(1/sqrt(2), $metric->getDistance());
127
+        $this->assertEquals(1 / sqrt(2), $metric->getDistance());
128 128
     }
129 129
 
130 130
     public function testItMyaHasDependentInstabilities()
Please login to merge, or discard this patch.
phpmetrics/tests/Metric/Class_/ClassEnumVisitorTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@
 block discarded – undo
66 66
     public static function provideExamples()
67 67
     {
68 68
         return [
69
-            [__DIR__ . '/../examples/nbmethods1.php', 'A', 3, 1, 2, 7],
69
+            [__DIR__.'/../examples/nbmethods1.php', 'A', 3, 1, 2, 7],
70 70
         ];
71 71
     }
72 72
 
Please login to merge, or discard this patch.
tests/Metric/Class_/Component/MaintainabilityIndexVisitorTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -35,13 +35,13 @@  discard block
 block discarded – undo
35 35
         $prophet->get('volume')->willReturn($volume);
36 36
 
37 37
         // spy
38
-        $prophet->set('mIwoC', $mIwoC)->will(function () use ($prophet) {
38
+        $prophet->set('mIwoC', $mIwoC)->will(function() use ($prophet) {
39 39
             return $prophet->reveal();
40 40
         })->shouldBeCalled();
41
-        $prophet->set('mi', $mi)->will(function () use ($prophet) {
41
+        $prophet->set('mi', $mi)->will(function() use ($prophet) {
42 42
             return $prophet->reveal();
43 43
         })->shouldBeCalled();
44
-        $prophet->set('commentWeight', $weight)->will(function () use ($prophet) {
44
+        $prophet->set('commentWeight', $weight)->will(function() use ($prophet) {
45 45
             return $prophet->reveal();
46 46
         })->shouldBeCalled();
47 47
 
@@ -71,8 +71,8 @@  discard block
 block discarded – undo
71 71
     {
72 72
         return [
73 73
             //    CC    LLOC    CLOC        Volume      MIwoC      mi          commentWeight
74
-            [5     , 50    , 20       , 10         , 55.26,   92.1,      36.83 ],
75
-            [11    , 45   , 26       , 1777.49    , 39.7,     80.01,      40.3 ]
74
+            [5, 50, 20, 10, 55.26, 92.1, 36.83],
75
+            [11, 45, 26, 1777.49, 39.7, 80.01, 40.3]
76 76
         ];
77 77
     }
78 78
 }
Please login to merge, or discard this patch.
phpmetrics/tests/Metric/Class_/Text/HalsteadVisitorTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -63,10 +63,10 @@
 block discarded – undo
63 63
     public function provideExamples()
64 64
     {
65 65
         return [
66
-            [ __DIR__ . '/../../examples/halstead1.php', 'twice', 2, 3, 1.5],
67
-            [ __DIR__ . '/../../examples/halstead2.php', 'maxi', 4, 6, 4.5],
68
-            [ __DIR__ . '/../../examples/halstead3.php', 'f_switch', 3, 9, 1.29],
69
-            [ __DIR__ . '/../../examples/halstead4.php', 'f_while', 9, 13, 10.4],
66
+            [__DIR__.'/../../examples/halstead1.php', 'twice', 2, 3, 1.5],
67
+            [__DIR__.'/../../examples/halstead2.php', 'maxi', 4, 6, 4.5],
68
+            [__DIR__.'/../../examples/halstead3.php', 'f_switch', 3, 9, 1.29],
69
+            [__DIR__.'/../../examples/halstead4.php', 'f_while', 9, 13, 10.4],
70 70
         ];
71 71
     }
72 72
 }
Please login to merge, or discard this patch.
phpmetrics/tests/Metric/Class_/Text/LengthVisitorTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@
 block discarded – undo
51 51
     public function provideExamples()
52 52
     {
53 53
         return [
54
-            [ __DIR__ . '/../../examples/loc1.php', 'A', 21, 13, 8],
54
+            [__DIR__.'/../../examples/loc1.php', 'A', 21, 13, 8],
55 55
         ];
56 56
     }
57 57
 }
Please login to merge, or discard this patch.