phpmetrics / PhpMetrics

src/Hal/Component/Issue/Issuer.php

Doc Comments +4 added lines, -3 removed lines

@@ -130,7 +130,7 @@ 
     }
 
     /**
-     * @param $status
+     * @param integer $status
      */
     protected function terminate($status)
     {
@@ -139,6 +139,7 @@ 
 
     /**
      * @param $log
+     * @param string $logfile
      * @return $this
      */
     protected function log($logfile, $log)
@@ -152,7 +153,7 @@ 
     }
 
     /**
-     * @param $debugKey
+     * @param string $debugKey
      * @param $value
      * @return $this
      */
@@ -163,7 +164,7 @@ 
     }
 
     /**
-     * @param $debugKey
+     * @param string $debugKey
      * @return $this
      */
     public function clear($debugKey)

src/Hal/Report/Html/Reporter.php

Doc Comments +2 added lines, -2 removed lines

@@ -109,8 +109,8 @@
     }
 
     /**
-     * @param $source
-     * @param $destination
+     * @param string $source
+     * @param string $destination
      * @return $this
      */
     public function renderPage($source, $destination, Consolidated $consolidated, $history)

Spacing +30 added lines, -30 removed lines

@@ -40,11 +40,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);
@@ -53,39 +53,39 @@ 
         }
 
         // 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);
         }
-        recurse_copy(PROJECT_DIR . '/templates/html_report/js', $logDir . '/js');
-        recurse_copy(PROJECT_DIR . '/templates/html_report/css', $logDir . '/css');
-        recurse_copy(PROJECT_DIR . '/templates/html_report/images', $logDir . '/images');
-        recurse_copy(PROJECT_DIR . '/templates/html_report/fonts', $logDir . '/fonts');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/js', $logDir.'/js');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/css', $logDir.'/css');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/images', $logDir.'/images');
+        recurse_copy(PROJECT_DIR.'/templates/html_report/fonts', $logDir.'/fonts');
 
         // render dynamic pages
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/violations.php', $logDir . '/violations.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/packages.php', $logDir . '/packages.html', $consolidated, $history);
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/package_relations.php', $logDir . '/package_relations.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/violations.php', $logDir.'/violations.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/packages.php', $logDir.'/packages.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/package_relations.php', $logDir.'/package_relations.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage(PROJECT_DIR . '/templates/html_report/git.php', $logDir . '/git.html', $consolidated, $history);
+            $this->renderPage(PROJECT_DIR.'/templates/html_report/git.php', $logDir.'/git.html', $consolidated, $history);
         }
-        $this->renderPage(PROJECT_DIR . '/templates/html_report/junit.php', $logDir . '/junit.html', $consolidated, $history);
+        $this->renderPage(PROJECT_DIR.'/templates/html_report/junit.php', $logDir.'/junit.html', $consolidated, $history);
 
         // js data
         file_put_contents(
@@ -99,8 +99,8 @@ 
 
         // json data
         file_put_contents(
-            $logDir . '/js/classes.js',
-            'var classes = ' . json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
+            $logDir.'/js/classes.js',
+            'var classes = '.json_encode($consolidated->getClasses(), JSON_PRETTY_PRINT)
         );
 
         $this->output->writeln(sprintf('HTML report generated in "%s" directory', $logDir));
@@ -167,7 +167,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';

src/Hal/Violation/Violations.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     {
         $string = '';
         foreach ($this->data as $violation) {
-            $string .= $violation->getName() . ',';
+            $string .= $violation->getName().',';
         }
         return $string;
     }

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

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
                     $output = 0;
                     $fanout = [];
 
-                    iterate_over_node($node, function ($node) use (&$output, &$fanout) {
+                    iterate_over_node($node, function($node) use (&$output, &$fanout) {
                         switch (true) {
                             case $node instanceof Stmt\Return_:
                                 $output++;

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

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
 
             $select = $while = $if = 0;
 
-            iterate_over_node($node, function ($node) use (&$while, &$select, &$if) {
+            iterate_over_node($node, function($node) use (&$while, &$select, &$if) {
                 switch (true) {
                     case $node instanceof Stmt\Do_:
                     case $node instanceof Stmt\Foreach_:
@@ -58,7 +58,7 @@ 
                 }
             });
 
-            $defect = 0.15 + 0.23 *  $while + 0.22 *  $select + 0.07 * $if;
+            $defect = 0.15 + 0.23 * $while + 0.22 * $select + 0.07 * $if;
             $class->set('kanDefect', round($defect, 2));
         }
     }

artifacts/phar/build.php

Spacing +5 added lines, -5 removed lines

@@ -1,8 +1,8 @@ 
 <?php
-chdir(__DIR__ . '/../../');
+chdir(__DIR__.'/../../');
 
 if (!file_exists('vendor/autoload.php')) {
-    echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!' . PHP_EOL;
+    echo '[ERROR] It\'s required to run "composer install" before building PhpMetrics!'.PHP_EOL;
     exit(1);
 }
 
@@ -21,7 +21,7 @@ 
     if (preg_match($exclude, $file)) {
         continue;
     }
-    $path = str_replace(__DIR__ . '/', '', $file);
+    $path = str_replace(__DIR__.'/', '', $file);
     $phar->addFromString($path, file_get_contents($file));
 }
 
@@ -52,8 +52,8 @@ 
 
 function rglob($pattern = '*', $flags = 0, $path = '')
 {
-    $paths = glob($path . '*', GLOB_MARK | GLOB_ONLYDIR | GLOB_NOSORT);
-    $files = glob($path . $pattern, $flags);
+    $paths = glob($path.'*', GLOB_MARK|GLOB_ONLYDIR|GLOB_NOSORT);
+    $files = glob($path.$pattern, $flags);
     foreach ($paths as $path) {
         $files = array_merge($files, rglob($pattern, $flags, $path));
     }

src/Hal/Component/Output/TestOutput.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function writeln($message)
     {
-        $this->write(PHP_EOL . $message);
+        $this->write(PHP_EOL.$message);
         return $this;
     }
 

src/Hal/Component/Output/CliOutput.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
      */
     public function writeln($message)
     {
-        $this->write(PHP_EOL . $message);
+        $this->write(PHP_EOL.$message);
         return $this;
     }
 
@@ -34,7 +34,7 @@ 
      */
     public function write($message)
     {
-        $this->quietMode||file_put_contents('php://stdout', $message);
+        $this->quietMode || file_put_contents('php://stdout', $message);
         return $this;
     }
 

src/functions.php

Spacing +7 added lines, -7 removed lines

@@ -90,7 +90,7 @@ 
     }
 
     if ($node instanceof \PhpParser\Node\Name\FullyQualified) {
-        return (string)$node;
+        return (string) $node;
     }
     if ($node instanceof \PhpParser\Node\Expr\New_) {
         return getNameOfNode($node->class);
@@ -101,7 +101,7 @@ 
     }
 
     if ($node instanceof \PhpParser\Node\Name) {
-        return (string)implode($node->parts);
+        return (string) implode($node->parts);
     }
 
     if (isset($node->name) && $node->name instanceof \PhpParser\Node\Expr\Variable) {
@@ -129,11 +129,11 @@ 
     }
 
     if (isset($node->name) && null === $node->name) {
-        return 'anonymous@' . spl_object_hash($node);
+        return 'anonymous@'.spl_object_hash($node);
     }
 
     if (isset($node->name)) {
-        return (string)$node->name;
+        return (string) $node->name;
     }
 
     return null;
@@ -151,10 +151,10 @@ 
     }
     while (false !== ($file = readdir($dir))) {
         if (($file != '.') && ($file != '..')) {
-            if (is_dir($src . '/' . $file)) {
-                recurse_copy($src . '/' . $file, $dst . '/' . $file);
+            if (is_dir($src.'/'.$file)) {
+                recurse_copy($src.'/'.$file, $dst.'/'.$file);
             } else {
-                copy($src . '/' . $file, $dst . '/' . $file);
+                copy($src.'/'.$file, $dst.'/'.$file);
             }
         }
     }