DaveLiddament / sarb

src/Domain/ResultsParser/AnalysisResults.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function __construct(array $analysisResults)
     {
-        usort($analysisResults, function (AnalysisResult $a, AnalysisResult $b): int {
+        usort($analysisResults, function(AnalysisResult $a, AnalysisResult $b): int {
             return $a->getLocation()->compareTo($b->getLocation());
         });
 

tests/Unit/Core/Analyser/LocationSortTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
             $foo16,
         ];
 
-        usort($list, function (Location $a, Location $b): int {
+        usort($list, function(Location $a, Location $b): int {
             return $a->compareTo($b);
         });
 

tests/Integration/UpgradeV0BaselineFilesTest.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
         return [
             ['phan', new PhanJsonIdentifier()],
             ['phpcs-json', new PhpCodeSnifferJsonIdentifier()],
-            ['phpcs-txt',  new PhpCodeSnifferJsonIdentifier()],
+            ['phpcs-txt', new PhpCodeSnifferJsonIdentifier()],
             ['phpmd', new PhpmdJsonIdentifier()],
             ['phpstan-json', new PhpstanJsonIdentifier()],
             ['phpstan-text', new PhpstanJsonIdentifier()],

example/eslint2sarb.php

Spacing +2 added lines, -2 removed lines

@@ -8,11 +8,11 @@
 
 $issues = [];
 
-foreach($asJson as $fileWithIssues) {
+foreach ($asJson as $fileWithIssues) {
 
     $fileName = $fileWithIssues['filePath'];
 
-    foreach($fileWithIssues['messages'] as $issue) {
+    foreach ($fileWithIssues['messages'] as $issue) {
         $issues[] = [
             'file' => $fileName,
             'line' => $issue['line'],

Braces +6 added lines, -2 removed lines

@@ -1,10 +1,14 @@
 <?php
 
 $inputData = stream_get_contents(STDIN);
-if ($inputData === false) die("Could not read input");
+if ($inputData === false) {
+    die("Could not read input");
+}
 
 $asJson = json_decode($inputData, true);
-if (!is_array($asJson)) die ("Could not parse JSON");
+if (!is_array($asJson)) {
+    die ("Could not parse JSON");
+}
 
 $issues = [];
 

tests/Unit/Core/Utils/PathTest.php

Spacing +1 added lines, -1 removed lines

@@ -364,7 +364,7 @@
     /** @return array<int,array{string,string, string}> */
     public function provideMakeRelativeTests(): array
     {
-        $paths = array_map(function (array $arguments) {
+        $paths = array_map(function(array $arguments) {
             return [$arguments[2], $arguments[1], $arguments[0]];
         }, $this->providePathTests());
 

src/Domain/Utils/Path.php

Spacing +1 added lines, -2 removed lines

@@ -119,8 +119,7 @@
         }
 
         // Add the root directory again
-        self::$buffer[$path] = $canonicalPath = $root.implode('/', $canonicalParts);
-        ++self::$bufferSize;
+        self::$buffer[$path] = $canonicalPath = $root.implode('/', $canonicalParts);++self::$bufferSize;
 
         // Clean up regularly to prevent memory leaks
         if (self::$bufferSize > self::CLEANUP_THRESHOLD) {

src/Domain/BaseLiner/BaseLineImporter.php

Spacing +5 added lines, -5 removed lines

@@ -80,11 +80,11 @@
 
             return new BaseLine($historyFactory, $analysisResults, $resultsParser, $historyMarker);
         } catch (
-            ArrayParseException|
-            InvalidResultsParserException|
-            InvalidHistoryFactoryException|
-            InvalidHistoryMarkerException|
-            InvalidContentTypeException|
+            ArrayParseException |
+            InvalidResultsParserException |
+            InvalidHistoryFactoryException |
+            InvalidHistoryMarkerException |
+            InvalidContentTypeException |
             ParseAtLocationException $e) {
                 throw BaseLineImportException::fromException($fileName, $e);
             }

src/Domain/ResultsParser/AnalysisResultsImporter.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     ): AnalysisResults {
         try {
             return $resultsParser->convertFromString($analysisResultsAsString, $projectRoot);
-        } catch (InvalidContentTypeException|ParseAtLocationException $e) {
+        } catch (InvalidContentTypeException | ParseAtLocationException $e) {
             throw AnalysisResultsImportException::fromException($resultsParser->getIdentifier(), $e);
         }
     }

src/Plugins/ResultsParsers/SarbJsonResultsParser/SarbRelativeFileJsonResultsParser.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
                 ArrayUtils::assertArray($analysisResultAsArray);
                 $analysisResult = $this->convertAnalysisResultFromArray($analysisResultAsArray, $projectRoot);
                 $analysisResultsBuilder->addAnalysisResult($analysisResult);
-            } catch (ArrayParseException|InvalidPathException $e) {
+            } catch (ArrayParseException | InvalidPathException $e) {
                 throw ParseAtLocationException::issueAtPosition($e, $resultsCount);
             }
         }