alanwillms / php-hound

src/Command.php

Doc Comments +1 added lines, -1 removed lines

@@ -293,7 +293,7 @@
     /**
      * Get argument value from user informed arguments.
      * @param string $name argument name.
-     * @return Mixed argument value.
+     * @return string argument value.
      */
     protected function getArgumentValue($name)
     {

Spacing +3 added lines, -3 removed lines

@@ -124,7 +124,7 @@ 
 
         if (!isset($formatClasses[$format])) {
             throw new UnexpectedValueException(
-                'Invalid format: "' . $format . '"'
+                'Invalid format: "'.$format.'"'
             );
         }
 
@@ -217,7 +217,7 @@ 
             if (0 === strpos($path, DIRECTORY_SEPARATOR)) {
                 continue;
             }
-            $path = $this->getWorkingDirectory() . DIRECTORY_SEPARATOR . $path;
+            $path = $this->getWorkingDirectory().DIRECTORY_SEPARATOR.$path;
         }
         $this->analysedPaths = $paths;
     }
@@ -255,7 +255,7 @@ 
      */
     public function getDescription()
     {
-        return 'PHP Hound ' . Analyser::VERSION;
+        return 'PHP Hound '.Analyser::VERSION;
     }
 
     /**

src/output/HtmlOutput.php

Spacing +7 added lines, -7 removed lines

@@ -64,7 +64,7 @@ 
                 'historyData' => array_values($chartData['historyData']),
             ]
         );
-        $fileName = $this->getOutputDirectory() . '/index.html';
+        $fileName = $this->getOutputDirectory().'/index.html';
         $file = new SplFileObject($fileName, 'w');
         $file->fwrite($indexHtml);
     }
@@ -87,8 +87,8 @@ 
                 'backButton' => true,
             ]
         );
-        $htmlFileName = $this->getOutputDirectory() . '/'
-            . str_replace(DIRECTORY_SEPARATOR, '_', $phpFilePath) . '.html';
+        $htmlFileName = $this->getOutputDirectory().'/'
+            . str_replace(DIRECTORY_SEPARATOR, '_', $phpFilePath).'.html';
 
         $file = new SplFileObject($htmlFileName, 'w');
         $file->fwrite($fileHtml);
@@ -109,8 +109,8 @@ 
             'layout',
             [
                 'content' => $content,
-                'phpHoundVersion' => 'PHP Hound ' . Analyser::VERSION,
-                'phpVersion' => 'PHP ' . phpversion(),
+                'phpHoundVersion' => 'PHP Hound '.Analyser::VERSION,
+                'phpVersion' => 'PHP '.phpversion(),
                 'generationTime' => $date->format('r'),
                 'backButton' => !empty($data['backButton']),
             ]
@@ -124,7 +124,7 @@ 
     protected function getPlatesEngine()
     {
         if (null === $this->platesEngine) {
-            $this->platesEngine = new Engine(__DIR__ . '/../templates');
+            $this->platesEngine = new Engine(__DIR__.'/../templates');
         }
 
         return $this->platesEngine;
@@ -136,7 +136,7 @@ 
      */
     protected function getOutputDirectory()
     {
-        $directory = $this->outputDirectory . '/phphound';
+        $directory = $this->outputDirectory.'/phphound';
 
         if (!file_exists($directory)) {
             mkdir($directory);

src/output/filter/DiffOutputFilter.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
     {
         $files = [];
         foreach ($this->getDiffsWithAddedCode() as $fileDiff) {
-            $files[] = $this->root . DIRECTORY_SEPARATOR . substr($fileDiff->getTo(), 2);
+            $files[] = $this->root.DIRECTORY_SEPARATOR.substr($fileDiff->getTo(), 2);
         }
         return $files;
     }
@@ -79,7 +79,7 @@ 
     {
         $resultFilter = [];
         foreach ($this->getDiffsWithAddedCode() as $fileDiff) {
-            $file = $this->root . DIRECTORY_SEPARATOR . substr($fileDiff->getTo(), 2);
+            $file = $this->root.DIRECTORY_SEPARATOR.substr($fileDiff->getTo(), 2);
             $lines = [];
             foreach ($fileDiff->getChunks() as $chunkDiff) {
                 $counter = -1;

src/output/html/FileHighlighter.php

Spacing +6 added lines, -6 removed lines

@@ -43,12 +43,12 @@ 
             $paddedLineNumber = str_pad($lineNumber, $paddingLength, '0', STR_PAD_LEFT);
             $hasIssues = isset($this->linesWithIssues[$lineNumber]);
             $lineCssClass = $hasIssues ? 'has-issues' : 'no-issues';
-            $lineId = 'line' . $lineNumber;
+            $lineId = 'line'.$lineNumber;
 
-            $html .= '<div class="' . $lineCssClass . '" id="' . $lineId . '">';
-            $html .= '<span class="line-number">' . $paddedLineNumber . '</span>';
+            $html .= '<div class="'.$lineCssClass.'" id="'.$lineId.'">';
+            $html .= '<span class="line-number">'.$paddedLineNumber.'</span>';
             $html .= $this->getIssuesTooltip($lineNumber);
-            $html .= $line . '</div>';
+            $html .= $line.'</div>';
         }
 
         $html .= "</span></code>";
@@ -87,11 +87,11 @@ 
         if (!isset($this->linesWithIssues[$lineNumber])) {
             return '';
         }
-        $html = '<div class="mdl-tooltip mdl-tooltip--large" for="line' . $lineNumber . '">';
+        $html = '<div class="mdl-tooltip mdl-tooltip--large" for="line'.$lineNumber.'">';
         $html .= '<ul>';
 
         foreach ($this->linesWithIssues[$lineNumber] as $issue) {
-            $html .= '<li>' . trim($issue['message']) . '</ul>';
+            $html .= '<li>'.trim($issue['message']).'</ul>';
         }
 
         $html .= '</ul></div>';

src/output/html/History.php

Spacing +1 added lines, -1 removed lines

@@ -125,6 +125,6 @@
      */
     protected function getHistoryFilePath()
     {
-        return $this->outputDirectory . '/history.json';
+        return $this->outputDirectory.'/history.json';
     }
 }

src/output/XmlOutput.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         $writer = new Writer;
         $writer->openMemory();
         $writer->write($this->getSabreXmlArrayFor($result));
-        return '<?xml version="1.0" encoding="UTF-8"?>' . $writer->outputMemory();
+        return '<?xml version="1.0" encoding="UTF-8"?>'.$writer->outputMemory();
     }
 
     /**

src/output/TextOutput.php

Spacing +3 added lines, -3 removed lines

@@ -19,14 +19,14 @@
             $issues = '(1 issue)';
 
             if ($issuesCount > 1) {
-                $issues = '(' . $issuesCount . ' issues)';
+                $issues = '('.$issuesCount.' issues)';
             }
 
-            $this->cli->yellowFlank($fileName . ' ' . $issues, '=', 2);
+            $this->cli->yellowFlank($fileName.' '.$issues, '=', 2);
 
             foreach ($lines as $line => $issues) {
                 foreach ($issues as $issue) {
-                    $this->cli->cyan()->inline($line . ': ');
+                    $this->cli->cyan()->inline($line.': ');
                     $this->cli->inline(trim($issue['message']));
                     $this->cli->br();
                 }

src/output/TextTriggerTrait.php

Spacing +2 added lines, -2 removed lines

@@ -23,13 +23,13 @@
                 if (!empty($message['ignoredPaths'])) {
                     $this->cli->out('(ignored paths:)');
                     foreach ($message['ignoredPaths'] as $ignoredPath) {
-                        $this->cli->red("\t" . $ignoredPath);
+                        $this->cli->red("\t".$ignoredPath);
                     }
                 }
                 break;
 
             case Analyser::EVENT_STARTING_TOOL:
-                $this->cli->inline('Running ' . $message['description'] . '... ');
+                $this->cli->inline('Running '.$message['description'].'... ');
                 break;
 
             case Analyser::EVENT_FINISHED_TOOL:

src/integration/PHPCodeSniffer.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
     public function getIgnoredArgument()
     {
         if (!empty($this->ignoredPaths)) {
-            return '--ignore=' . implode(',', $this->ignoredPaths) . ' ';
+            return '--ignore='.implode(',', $this->ignoredPaths).' ';
         }
         return '';
     }
@@ -35,9 +35,9 @@ 
      */
     public function getCommand($targetPaths)
     {
-        return $this->binariesPath . 'phpcs -p --standard=PSR2 --report=xml '
-            . $this->getIgnoredArgument() . '--report-file="'
-            . $this->temporaryFilePath . '" ' . implode(' ', $targetPaths);
+        return $this->binariesPath.'phpcs -p --standard=PSR2 --report=xml '
+            . $this->getIgnoredArgument().'--report-file="'
+            . $this->temporaryFilePath.'" '.implode(' ', $targetPaths);
     }
 
     /**