phpmetrics / PhpMetrics

src/Hal/Application/Analyze.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
     /**
      * Analyze constructor.
-     * @param OutputInterface $output
+     * @param Output $output
      */
     public function __construct(Config $config, Output $output, Issuer $issuer)
     {

src/Hal/Component/Issue/Issuer.php

Doc Comments +5 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
 
     /**
      * Issuer constructor.
-     * @param OutputInterface $output
+     * @param Output $output
      */
     public function __construct(Output $output)
     {
@@ -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/Cli/Reporter.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     /**
      * Reporter constructor.
      * @param Config $config
-     * @param OutputInterface $output
+     * @param Output $output
      */
     public function __construct(Config $config, Output $output)
     {

src/Hal/Component/Output/ProgressBar.php

Spacing +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
         if (DIRECTORY_SEPARATOR === '\\') {
             return
                 0 >= version_compare('10.0.10586',
-                    PHP_WINDOWS_VERSION_MAJOR . '.' . PHP_WINDOWS_VERSION_MINOR . '.' . PHP_WINDOWS_VERSION_BUILD)
+                    PHP_WINDOWS_VERSION_MAJOR.'.'.PHP_WINDOWS_VERSION_MINOR.'.'.PHP_WINDOWS_VERSION_BUILD)
                 || false !== getenv('ANSICON')
                 || 'ON' === getenv('ConEmuANSI')
                 || 'xterm' === getenv('TERM');

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/Hal/Application/Application.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
         $output = new CliOutput();
 
         // issues and debug
-        $issuer = (new Issuer($output));//->enable();
+        $issuer = (new Issuer($output)); //->enable();
 
         // config
         $config = (new Parser())->parse($argv);
@@ -57,7 +57,7 @@ 
         // analyze
         try {
             $metrics = (new Analyze($config, $output, $issuer))->run($files);
-        }catch(ConfigException $e) {
+        } catch (ConfigException $e) {
             $output->writeln(sprintf('<error>%s</error>', $e->getMessage()));
             exit(1);
         }

src/Hal/Report/Html/Reporter.php

Spacing +22 added lines, -22 removed lines

@@ -44,11 +44,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 = sizeof($files) + 1;
         $history = [];
         natsort($files);
@@ -57,34 +57,34 @@ 
         }
 
         // copy sources
-        if(!file_exists($logDir . '/js')) {
+        if (!file_exists($logDir.'/js')) {
             mkdir($logDir.'/js', 0755, true);
         }
-        if(!file_exists($logDir . '/css')) {
+        if (!file_exists($logDir.'/css')) {
             mkdir($logDir.'/css', 0755, true);
         }
-        if(!file_exists($logDir . '/images')) {
+        if (!file_exists($logDir.'/images')) {
             mkdir($logDir.'/images', 0755, true);
         }
-        recurse_copy(__DIR__ . '/template/js', $logDir . '/js');
-        recurse_copy(__DIR__ . '/template/css', $logDir . '/css');
-        recurse_copy(__DIR__ . '/template/images', $logDir . '/images');
+        recurse_copy(__DIR__.'/template/js', $logDir.'/js');
+        recurse_copy(__DIR__.'/template/css', $logDir.'/css');
+        recurse_copy(__DIR__.'/template/images', $logDir.'/images');
 
         // render dynamic pages
-        $this->renderPage(__DIR__ . '/template/index.php', $logDir . '/index.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/loc.php', $logDir . '/loc.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/relations.php', $logDir . '/relations.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/coupling.php', $logDir . '/coupling.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/all.php', $logDir . '/all.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/oop.php', $logDir . '/oop.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/complexity.php', $logDir . '/complexity.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/panel.php', $logDir . '/panel.html', $consolidated, $history);
-        $this->renderPage(__DIR__ . '/template/violations.php', $logDir . '/violations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/index.php', $logDir.'/index.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/loc.php', $logDir.'/loc.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/relations.php', $logDir.'/relations.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/coupling.php', $logDir.'/coupling.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/all.php', $logDir.'/all.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/oop.php', $logDir.'/oop.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/complexity.php', $logDir.'/complexity.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/panel.php', $logDir.'/panel.html', $consolidated, $history);
+        $this->renderPage(__DIR__.'/template/violations.php', $logDir.'/violations.html', $consolidated, $history);
         if ($this->config->has('git')) {
-            $this->renderPage(__DIR__ . '/template/git.php', $logDir . '/git.html', $consolidated, $history);
+            $this->renderPage(__DIR__.'/template/git.php', $logDir.'/git.html', $consolidated, $history);
         }
         if ($this->config->has('junit')) {
-            $this->renderPage(__DIR__ . '/template/junit.php', $logDir . '/junit.html', $consolidated, $history);
+            $this->renderPage(__DIR__.'/template/junit.php', $logDir.'/junit.html', $consolidated, $history);
         }
 
         // js data
@@ -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));
@@ -168,7 +168,7 @@ 
 
         $diff = $newValue - $oldValue;
         if ($diff > 0) {
-            $diff = '+' . $diff;
+            $diff = '+'.$diff;
         }
 
         $goodOrBad = 'neutral';

src/Hal/Report/Html/template/junit.php

Spacing +5 added lines, -5 removed lines

@@ -1,9 +1,9 @@ 
 <?php
-require __DIR__ . '/_header.php'; ?>
+require __DIR__.'/_header.php'; ?>
 
 <?php
 $unit = $project['unitTesting'];
-$getMetricForClass = function ($classname, $metric) use ($classes) {
+$getMetricForClass = function($classname, $metric) use ($classes) {
     foreach ($classes as $class) {
         if ($classname !== $class['name']) {
             continue;
@@ -133,7 +133,7 @@ 
         </div>
     </div>
 
-    <?php require __DIR__ . '/_footer.php'; ?>
+    <?php require __DIR__.'/_footer.php'; ?>
 
     <script type="text/javascript">
         var clusterize = new Clusterize({
@@ -188,12 +188,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) {