bestit / PHP_CodeSniffer

src/Standards/BestIt/Sniffs/Functions/FluentSetterSniff.php

Spacing +2 added lines, -2 removed lines

@@ -110,7 +110,7 @@ 
     private function checkForFluentSetterErrors(File $phpcsFile, int $functionPos, int $classPos): void
     {
         $tokens = $phpcsFile->getTokens();
-        $errorData = $phpcsFile->getDeclarationName($classPos) . '::' . $phpcsFile->getDeclarationName($functionPos);
+        $errorData = $phpcsFile->getDeclarationName($classPos).'::'.$phpcsFile->getDeclarationName($functionPos);
 
         $functionToken = $tokens[$functionPos];
         $openBracePtr = $functionToken['scope_opener'];
@@ -226,7 +226,7 @@ 
 
         $phpcsFile->fixer->beginChangeset();
         $phpcsFile->fixer->addNewlineBefore($closingBracePtr - 1);
-        $phpcsFile->fixer->addContentBefore($closingBracePtr - 1, $expectedReturnSpaces . 'return $this;');
+        $phpcsFile->fixer->addContentBefore($closingBracePtr - 1, $expectedReturnSpaces.'return $this;');
         $phpcsFile->fixer->addNewlineBefore($closingBracePtr - 1);
         $phpcsFile->fixer->endChangeset();
     }

build/refreshReadmeTable.php

Spacing +4 added lines, -4 removed lines

@@ -2,7 +2,7 @@ 
 
 declare(strict_types=1);
 
-require_once __DIR__ . '/../vendor/autoload.php';
+require_once __DIR__.'/../vendor/autoload.php';
 
 $directory = new RecursiveDirectoryIterator($baseFolder = './src/Standards/BestIt/Sniffs');
 $iterator = new RecursiveIteratorIterator($directory);
@@ -74,8 +74,8 @@ 
     foreach ($regexIterator as $file) {
         [$file] = $file;
 
-        $simpleClassName = str_replace([$baseFolder . DIRECTORY_SEPARATOR, '.php', 'Sniff'], '', $file);
-        $fullQualifiedClassName = 'BestIt\\Sniffs\\' . str_replace('/', '\\', $simpleClassName) . 'Sniff';
+        $simpleClassName = str_replace([$baseFolder.DIRECTORY_SEPARATOR, '.php', 'Sniff'], '', $file);
+        $fullQualifiedClassName = 'BestIt\\Sniffs\\'.str_replace('/', '\\', $simpleClassName).'Sniff';
 
         $hasSuppresses = (bool) preg_match_all(
             '/->isSniffSuppressed\((?P<code>\s*.*\s*)\)/mU',
@@ -145,7 +145,7 @@ 
     ksort($codes);
 
     file_put_contents(
-        $tmpFile = __DIR__ . DIRECTORY_SEPARATOR . 'table.md',
+        $tmpFile = __DIR__.DIRECTORY_SEPARATOR.'table.md',
         <<<EOD
 | Sniff | Description | suppressable |
 | ----- | ----------- | ------------ |

src/Standards/BestIt/Sniffs/Formatting/TraitUseDeclarationSniff.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
         $commaPointers = TokenHelper::findNextAll($file, T_COMMA, $usePos + 1, $endPos);
         foreach ($commaPointers as $commaPos) {
             $pointerAfterComma = TokenHelper::findNextEffective($file, $commaPos + 1);
-            $fixer->replaceToken($commaPos, ';' . $file->eolChar . $indentation . 'use ');
+            $fixer->replaceToken($commaPos, ';'.$file->eolChar.$indentation.'use ');
             for ($i = $commaPos + 1; $i < $pointerAfterComma; $i++) {
                 $fixer->replaceToken($i, '');
             }

src/Standards/BestIt/Sniffs/Formatting/AlphabeticallySortedUsesSniff.php

Spacing +2 added lines, -2 removed lines

@@ -199,7 +199,7 @@ 
         return implode(
             $file->eolChar,
             array_map(
-                function (UseStatement $useStatement) use ($file): string {
+                function(UseStatement $useStatement) use ($file): string {
                     $unqualifiedName = NamespaceHelper::getUnqualifiedNameFromFullyQualifiedName(
                         $useStatement->getFullyQualifiedTypeName()
                     );
@@ -301,7 +301,7 @@ 
      */
     private function sortUseStatements(): void
     {
-        uasort($this->useStatements, function (UseStatement $prevStatement, UseStatement $nextStatement) {
+        uasort($this->useStatements, function(UseStatement $prevStatement, UseStatement $nextStatement) {
             return $this->compareUseStatements($prevStatement, $nextStatement);
         });
     }

src/Standards/BestIt/Sniffs/Commenting/AbstractDocSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
     /**
      * Returns the position of the summary or null.
      *
-     * @return int|null
+     * @return integer
      */
     private function getSummaryPosition(): ?int
     {

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 
         $this->file->fixer->addContent(
             $position,
-            $this->file->eolChar . str_repeat('    ', $token['level']) . ' *'
+            $this->file->eolChar.str_repeat('    ', $token['level']).' *'
         );
 
         $this->file->fixer->endChangeset();

src/Standards/BestIt/Sniffs/AbstractSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
     /**
      * Default method for fixing exceptions.
      *
-     * @param CodeWarning $exception
+     * @param CodeError $exception
      *
      * @return void
      */

src/Standards/BestIt/Sniffs/DocTags/VarTagSniff.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 namespace BestIt\Sniffs\DocTags;
 
-use BestIt\CodeSniffer\Helper\TokenHelper;
 use function array_column;
 use function array_map;
 use function implode;

src/Standards/BestIt/Sniffs/DocTags/TagSortingSniff.php

Spacing +9 added lines, -9 removed lines

@@ -230,27 +230,27 @@ 
 
             // Insert new line between groups.
             if ($isGroupSwitch) {
-                $newContent .= $lineStartingPadding . '*' . $eolChar;
+                $newContent .= $lineStartingPadding.'*'.$eolChar;
             }
 
             // Create the new Tag.
             // WARNING We do not need a line break in the tag summary.
-            $newContent .= $lineStartingPadding . '* ' . trim($thisTagContent);
+            $newContent .= $lineStartingPadding.'* '.trim($thisTagContent);
 
             if ($tag['contents']) {
                 $prevLine = $tag['line'];
                 foreach ($tag['contents'] as $subToken) {
                     // If we have a line switch, we need to create the correct indentation from before ...
                     if ($withLineSwitch = $subToken['line'] > $prevLine) {
-                        $newContent .= $eolChar .
-                            $lineStartingPadding . '*' .
+                        $newContent .= $eolChar.
+                            $lineStartingPadding.'*'.
                             str_repeat(' ', $subToken['column'] - strlen($lineStartingPadding) - 2);
 
                         $prevLine = $subToken['line'];
                     }
 
                     // ... if we have no line switch, then an additional whitespace is enough.
-                    $newContent .= ($withLineSwitch ? '' : ' ') . $subToken['content'];
+                    $newContent .= ($withLineSwitch ? '' : ' ').$subToken['content'];
                 }
             }
 
@@ -260,7 +260,7 @@ 
             $withReturn = $isReturn;
         }
 
-        $newContent .= $lineStartingPadding . '*/' . $eolChar;
+        $newContent .= $lineStartingPadding.'*/'.$eolChar;
 
         return $newContent;
     }
@@ -332,7 +332,7 @@ 
 
         $fixer->addContentBefore(
             $token['pointer'],
-            $this->file->eolChar . $lineStartPadding . '* '
+            $this->file->eolChar.$lineStartPadding.'* '
         );
 
         $fixer->endChangeset();
@@ -348,7 +348,7 @@ 
         $barrier = 0;
         $tokens = $this->docTagHelper->getTagTokens();
 
-        $tokens = array_filter($tokens, function (array $token) use (&$barrier): bool {
+        $tokens = array_filter($tokens, function(array $token) use (&$barrier): bool {
             $allowed = true;
 
             if ($barrier) {
@@ -443,7 +443,7 @@ 
     {
         $tagCounts = $this->docTagHelper->getTagCounts($tokens);
 
-        usort($tokens, function (array $leftToken, array $rightToken) use ($tagCounts): int {
+        usort($tokens, function(array $leftToken, array $rightToken) use ($tagCounts): int {
             return $this->compareTokensForSorting($leftToken, $rightToken, $tagCounts);
         });
 

src/Standards/BestIt/Sniffs/Commenting/RequiredDocBlockSniff.php

Spacing +2 added lines, -2 removed lines

@@ -99,7 +99,7 @@ 
             $tokenIdent = $this->getTokenName();
 
             throw (new CodeError(
-                static::CODE_MISSING_DOC_BLOCK_PREFIX . ucfirst($tokenIdent),
+                static::CODE_MISSING_DOC_BLOCK_PREFIX.ucfirst($tokenIdent),
                 self::MESSAGE_MISSING_DOC_BLOCK,
                 $this->stackPos
             ))->setPayload([lcfirst($tokenIdent)]);
@@ -123,7 +123,7 @@ 
             $tokenIdent = $this->getTokenName();
 
             $exception = (new CodeError(
-                static::CODE_NO_MULTI_LINE_DOC_BLOCK_PREFIX . ucfirst($tokenIdent),
+                static::CODE_NO_MULTI_LINE_DOC_BLOCK_PREFIX.ucfirst($tokenIdent),
                 self::MESSAGE_NO_MULTI_LINE_DOC_BLOCK_PREFIX,
                 $docCommentPos
             ))->setPayload([lcfirst($tokenIdent)]);