phpmetrics / PhpMetrics

src/Hal/Application/Config/File/ConfigFileReaderJson.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
     private function resolvePath($path)
     {
         if (DIRECTORY_SEPARATOR !== $path[0]) {
-            $path = dirname($this->filename) . DIRECTORY_SEPARATOR . $path;
+            $path = dirname($this->filename).DIRECTORY_SEPARATOR.$path;
         }
 
         return $path;
@@ -78,7 +78,7 @@ 
 
         // Composer
         if (array_key_exists('composer', $jsonData)) {
-            $config->set('composer', (bool)$jsonData['composer']);
+            $config->set('composer', (bool) $jsonData['composer']);
         }
 
         // Search
@@ -111,7 +111,7 @@ 
         if (isset($jsonData['report']) && is_array($jsonData['report'])) {
             foreach ($jsonData['report'] as $reportType => $path) {
                 $path = $this->resolvePath($path);
-                $config->set('report-' . $reportType, $path);
+                $config->set('report-'.$reportType, $path);
             }
         }
     }

src/Hal/Metric/Class_/Coupling/ExternalsVisitor.php

Spacing +14 added lines, -14 removed lines

@@ -59,21 +59,21 @@ 
             // extends
             if (isset($node->extends)) {
                 if (is_array($node->extends)) {
-                    foreach ((array)$node->extends as $interface) {
-                        $this->pushToDependencies($dependencies, (string)$interface);
-                        array_push($parents, (string)$interface);
+                    foreach ((array) $node->extends as $interface) {
+                        $this->pushToDependencies($dependencies, (string) $interface);
+                        array_push($parents, (string) $interface);
                     }
                 } else {
-                    $this->pushToDependencies($dependencies, (string)$node->extends);
-                    array_push($parents, (string)$node->extends);
+                    $this->pushToDependencies($dependencies, (string) $node->extends);
+                    array_push($parents, (string) $node->extends);
                 }
             }
 
             // implements
             if (isset($node->implements)) {
                 foreach ($node->implements as $interface) {
-                    $this->pushToDependencies($dependencies, (string)$interface);
-                    array_push($interfaces, (string)$interface);
+                    $this->pushToDependencies($dependencies, (string) $interface);
+                    array_push($interfaces, (string) $interface);
                 }
             }
 
@@ -82,7 +82,7 @@ 
                     // return
                     if (isset($stmt->returnType)) {
                         if ($stmt->returnType instanceof Node\Name\FullyQualified) {
-                            $this->pushToDependencies($dependencies, (string)$stmt->returnType);
+                            $this->pushToDependencies($dependencies, (string) $stmt->returnType);
                         }
                     }
 
@@ -90,13 +90,13 @@ 
                     foreach ($stmt->params as $param) {
                         if ($param->type) {
                             if ($param->type instanceof Node\Name\FullyQualified) {
-                                $this->pushToDependencies($dependencies, (string)$param->type);
+                                $this->pushToDependencies($dependencies, (string) $param->type);
                             }
                         }
                     }
 
                     // instantiations, static calls
-                    \iterate_over_node($stmt, function ($node) use (&$dependencies) {
+                    \iterate_over_node($stmt, function($node) use (&$dependencies) {
                         switch (true) {
                             case $node instanceof Node\Expr\New_:
                                 // new MyClass
@@ -115,13 +115,13 @@ 
                         foreach ($matches[1] as $check) {
                             foreach ($this->uses as $use) {
                                 if (method_exists($use, 'getAlias')) {
-                                    if (((string)$use->getAlias()) === $check) {
-                                        $this->pushToDependencies($dependencies, (string)($use->name));
+                                    if (((string) $use->getAlias()) === $check) {
+                                        $this->pushToDependencies($dependencies, (string) ($use->name));
                                     }
                                     continue;
                                 }
                                 if ($use->alias === $check) {
-                                    $this->pushToDependencies($dependencies, (string)($use->name));
+                                    $this->pushToDependencies($dependencies, (string) ($use->name));
                                 }
                             }
                         }
@@ -141,6 +141,6 @@ 
         if ('self' === $lowercase || 'parent' === $lowercase) {
             return;
         }
-        array_push($dependencies, (string)$dependency);
+        array_push($dependencies, (string) $dependency);
     }
 }

src/Hal/Metric/Class_/Structural/LcomVisitor.php

Spacing +5 added lines, -5 removed lines

@@ -52,12 +52,12 @@ 
                         continue;
                     }
 
-                    if (!$graph->has($stmt->name . '()')) {
-                        $graph->insert(new TreeNode($stmt->name . '()'));
+                    if (!$graph->has($stmt->name.'()')) {
+                        $graph->insert(new TreeNode($stmt->name.'()'));
                     }
-                    $from = $graph->get($stmt->name . '()');
+                    $from = $graph->get($stmt->name.'()');
 
-                    \iterate_over_node($stmt, function ($node) use ($from, &$graph) {
+                    \iterate_over_node($stmt, function($node) use ($from, &$graph) {
                         if ($node instanceof Node\Expr\PropertyFetch && isset($node->var->name) && $node->var->name == 'this') {
                             $name = getNameOfNode($node);
                             // use of attribute $this->xxx;
@@ -73,7 +73,7 @@ 
                             if (!$node->var instanceof Node\Expr\New_ && isset($node->var->name) && getNameOfNode($node->var) === 'this') {
                                 // use of method call $this->xxx();
                                 // use of attribute $this->xxx;
-                                $name = getNameOfNode($node->name) . '()';
+                                $name = getNameOfNode($node->name).'()';
                                 if (!$graph->has($name)) {
                                     $graph->insert(new TreeNode($name));
                                 }

src/Hal/Metric/Class_/Complexity/CyclomaticComplexityVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
                     }
 
                     // iterate over children, recursively
-                    $cb = function ($node) use (&$cb) {
+                    $cb = function($node) use (&$cb) {
                         $ccn = 0;
 
                         foreach (get_object_vars($node) as $name => $member) {

src/Hal/Report/Cli/SearchReporter.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         );
 
         $config = $this->config->get('searches')->get($searchName)->getConfig();
-        if(!empty($foundSearch) && !empty($config->failIfFound) && true === $config->failIfFound) {
+        if (!empty($foundSearch) && !empty($config->failIfFound) && true === $config->failIfFound) {
             $title = sprintf(
                 '<error>[ERR] Found %d occurrences for search "%s"</error>',
                 sizeof($foundSearch),

src/Hal/Report/Violations/Xml/Reporter.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
 
     public function generate(Metrics $metrics)
     {
-        if(!class_exists('\DOMDocument')) {
+        if (!class_exists('\DOMDocument')) {
             $this->output->writeln('<error>The DOM extension is not available. Please install it if you want to use the Xml Violations report.</error>');
             return;
         }

src/Hal/Report/Html/Reporter.php

Spacing +35 added lines, -35 removed lines

@@ -54,7 +54,7 @@ 
     {
         $this->config = $config;
         $this->output = $output;
-        $this->templateDir = __DIR__ . '/../../../../templates';
+        $this->templateDir = __DIR__.'/../../../../templates';
     }
 
 
@@ -79,11 +79,11 @@ 
         $consolidated = new Consolidated($metrics);
 
         // history of builds
-        $today = (object)[
+        $today = (object) [
             'avg' => $consolidated->getAvg(),
             'sum' => $consolidated->getSum()
         ];
-        $files = glob($logDir . '/js/history-*.json');
+        $files = glob($logDir.'/js/history-*.json');
         $next = count($files) + 1;
         $history = [];
         natsort($files);
@@ -92,45 +92,45 @@ 
         }
 
         // copy sources
-        if (!file_exists($logDir . '/js')) {
-            mkdir($logDir . '/js', 0755, true);
+        if (!file_exists($logDir.'/js')) {
+            mkdir($logDir.'/js', 0755, true);
         }
-        if (!file_exists($logDir . '/css')) {
-            mkdir($logDir . '/css', 0755, true);
+        if (!file_exists($logDir.'/css')) {
+            mkdir($logDir.'/css', 0755, true);
         }
-        if (!file_exists($logDir . '/images')) {
-            mkdir($logDir . '/images', 0755, true);
+        if (!file_exists($logDir.'/images')) {
+            mkdir($logDir.'/images', 0755, true);
         }
-        if (!file_exists($logDir . '/fonts')) {
-            mkdir($logDir . '/fonts', 0755, true);
+        if (!file_exists($logDir.'/fonts')) {
+            mkdir($logDir.'/fonts', 0755, true);
         }
 
         if (!is_writable($logDir)) {
             throw new RuntimeExceptionAlias(sprintf('Unable to write in the directory "%s"', $logDir));
         }
 
-        recurse_copy($this->templateDir . '/html_report/js', $logDir . '/js');
-        recurse_copy($this->templateDir . '/html_report/css', $logDir . '/css');
-        recurse_copy($this->templateDir . '/html_report/images', $logDir . '/images');
-        recurse_copy($this->templateDir . '/html_report/fonts', $logDir . '/fonts');
+        recurse_copy($this->templateDir.'/html_report/js', $logDir.'/js');
+        recurse_copy($this->templateDir.'/html_report/css', $logDir.'/css');
+        recurse_copy($this->templateDir.'/html_report/images', $logDir.'/images');
+        recurse_copy($this->templateDir.'/html_report/fonts', $logDir.'/fonts');
 
         // render dynamic pages
-        $this->renderPage($this->templateDir . '/html_report/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
-        $this->renderPage($this->templateDir . '/html_report/composer.php', $logDir . '/composer.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
+        $this->renderPage($this->templateDir.'/html_report/composer.php', $logDir.'/composer.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage($this->templateDir . '/html_report/git.php', $logDir . '/git.html', $consolidated, $consolidatedGroups, $history);
+            $this->renderPage($this->templateDir.'/html_report/git.php', $logDir.'/git.html', $consolidated, $consolidatedGroups, $history);
         }
-        $this->renderPage($this->templateDir . '/html_report/junit.php', $logDir . '/junit.html', $consolidated, $consolidatedGroups, $history);
+        $this->renderPage($this->templateDir.'/html_report/junit.php', $logDir.'/junit.html', $consolidated, $consolidatedGroups, $history);
 
         // js data
         file_put_contents(
@@ -144,8 +144,8 @@ 
 
         // json data
         file_put_contents(
-            $logDir . '/classes.js',
-            'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+            $logDir.'/classes.js',
+            'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
         );
 
         // HTML files to generate
@@ -166,7 +166,7 @@ 
 
         // consolidated by groups
         foreach ($consolidatedGroups as $name => $consolidated) {
-            $outDir = $logDir . DIRECTORY_SEPARATOR . $name;
+            $outDir = $logDir.DIRECTORY_SEPARATOR.$name;
             $this->currentGroup = $name;
             $this->assetPath = '../';
 
@@ -183,8 +183,8 @@ 
                 );
 
                 file_put_contents(
-                    $outDir . '/classes.js',
-                    'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+                    $outDir.'/classes.js',
+                    'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
                 );
             }
         }
@@ -261,7 +261,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';

templates/html_report/all.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 $fullwidth = true;
-require __DIR__ . '/_header.php'; ?>
+require __DIR__.'/_header.php'; ?>
 
 <?php
 
@@ -37,4 +37,4 @@ 
             </div>
         </div>
     </div>
-<?php require __DIR__ . '/_footer.php'; ?>
+<?php require __DIR__.'/_footer.php'; ?>

templates/html_report/junit.php

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-require __DIR__ . '/_header.php'; ?>
+require __DIR__.'/_header.php'; ?>
 
 <?php
 if (!isset($project['unitTesting'])) {
@@ -8,7 +8,7 @@ 
 }
 
 $unit = $project['unitTesting'];
-$getMetricForClass = function ($classname, $metric) use ($classes) {
+$getMetricForClass = function($classname, $metric) use ($classes) {
     foreach ($classes as $class) {
         if ($classname !== $class['name']) {
             continue;
@@ -82,7 +82,7 @@ 
                                 <td><span class="path"><?php echo $class['name']; ?></span></td>
                                 <?php foreach (['ccn', 'bugs'] as $attribute) {?>
                                     <td>
-                                        <span class="badge" <?php echo gradientStyleFor($classes, $attribute, $class[$attribute]);?>>
+                                        <span class="badge" <?php echo gradientStyleFor($classes, $attribute, $class[$attribute]); ?>>
                                         <?php echo isset($class[$attribute]) ? $class[$attribute] : ''; ?>
                                         </span>
                                     </td>
@@ -146,7 +146,7 @@ 
         </div>
     </div>
 </div>
-    <?php require __DIR__ . '/_footer.php'; ?>
+    <?php require __DIR__.'/_footer.php'; ?>
 
     <script type="text/javascript">
         var clusterize = new Clusterize({
@@ -201,12 +201,12 @@ 
             foreach ($unit['tests'] as $test) {
                 array_push($unitTimeJson->children, [
                     'name' => $test->classname,
-                    'time' => (float)$test->time,
+                    'time' => (float) $test->time,
                 ]);
             }
             ?>
 
-            var json = <?php echo json_encode($unitTimeJson);?>;
+            var json = <?php echo json_encode($unitTimeJson); ?>;
             root = json;
             var nodes = treemap.nodes(json)
                 .filter(function (d) {