bestit / PHP_CodeSniffer

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

Spacing +4 added lines, -4 removed lines

@@ -75,7 +75,7 @@ 
     /**
      * The message for the mixed type warning.
      */
-    private const MESSAGE_TAG_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the ' .
+    private const MESSAGE_TAG_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the '.
         'required types in detail.';
 
     /**
@@ -205,7 +205,7 @@ 
     {
         $varOfThisTag = $this->getArgumentTokenOfTag();
 
-        return '/(?P<type>[\w|\|\[\]]*) ?(?P<var>' . preg_quote($varOfThisTag['content'], '/') .
+        return '/(?P<type>[\w|\|\[\]]*) ?(?P<var>'.preg_quote($varOfThisTag['content'], '/').
             ') ?(?P<desc>.*)/m';
     }
 
@@ -244,7 +244,7 @@ 
 
         $this->varTokens = array_filter(
             $this->tokens,
-            function (int $tokenPos) use ($varPositions): bool {
+            function(int $tokenPos) use ($varPositions): bool {
                 return in_array($tokenPos, $varPositions, true);
             },
             ARRAY_FILTER_USE_KEY,
@@ -268,7 +268,7 @@ 
             $this->stackPos - 1,
         );
 
-        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function (int $position) {
+        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function(int $position) {
             return $this->tokens[$position]['content'] === '@param';
         });
 

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

Spacing +9 added lines, -9 removed lines

@@ -233,27 +233,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'];
                 }
             }
 
@@ -263,7 +263,7 @@ 
             $withReturn = $isReturn;
         }
 
-        $newContent .= $lineStartingPadding . '*/' . $eolChar;
+        $newContent .= $lineStartingPadding.'*/'.$eolChar;
 
         return $newContent;
     }
@@ -336,7 +336,7 @@ 
 
         $fixer->addContentBefore(
             $tokenPos,
-            $this->file->eolChar . $lineStartPadding . '* ',
+            $this->file->eolChar.$lineStartPadding.'* ',
         );
 
         $fixer->endChangeset();
@@ -352,7 +352,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) {
@@ -448,7 +448,7 @@ 
     {
         $tagCounts = $this->docTagHelper->getTagCounts($tokens);
 
-        uasort($tokens, function (array $leftToken, array $rightToken) use ($tagCounts): int {
+        uasort($tokens, function(array $leftToken, array $rightToken) use ($tagCounts): int {
             return $this->compareTokensForSorting($leftToken, $rightToken, $tagCounts);
         });
 

src/Standards/BestIt/Sniffs/TypeHints/ReturnTypeDeclarationSniff.php

Spacing +4 added lines, -4 removed lines

@@ -101,12 +101,12 @@ 
         $file->fixer->beginChangeset();
 
         if ($this->isCustomArrayType($returnTypeHint)) {
-            $returnTypeHint = ($returnTypeHint[0] === '?' ? '?' : '') . 'array';
+            $returnTypeHint = ($returnTypeHint[0] === '?' ? '?' : '').'array';
         }
 
         $file->fixer->addContent(
             $this->token['parenthesis_closer'],
-            ': ' . $returnTypeHint,
+            ': '.$returnTypeHint,
         );
 
         $file->fixer->endChangeset();
@@ -142,7 +142,7 @@ 
 
             // We add the question mark if there is a nullable type.
             if (in_array($type, self::NULL_TYPES, true) && ($typeCount > 1)) {
-                $returnTypeHint = '?' . $returnTypeHint;
+                $returnTypeHint = '?'.$returnTypeHint;
                 continue; // We still need this continue to prevent further execution of the questionmark.
             }
 
@@ -291,7 +291,7 @@ 
      */
     private function loadFixableTypes(?array $returnTypes): array
     {
-        return array_filter($returnTypes ?? [], function (string $returnType): bool {
+        return array_filter($returnTypes ?? [], function(string $returnType): bool {
             return $this->isFixableReturnType($returnType);
         });
     }

src/Standards/BestIt/Sniffs/TypeHints/ExplicitAssertionsSniff.php

Spacing +2 added lines, -2 removed lines

@@ -210,12 +210,12 @@
             if ($pointerToAddAssertion < $docCommentClosePointer && array_key_exists($pointerAfterDocComment + 1, $tokens)) {
                 $phpcsFile->fixer->addContentBefore(
                     $pointerAfterDocComment + 1,
-                    $indentation . $assertion . $phpcsFile->eolChar
+                    $indentation.$assertion.$phpcsFile->eolChar
                 );
             } else {
                 $phpcsFile->fixer->addContent(
                     $pointerToAddAssertion,
-                    $phpcsFile->eolChar . $indentation . $assertion
+                    $phpcsFile->eolChar.$indentation.$assertion
                 );
             }
 

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/Formatting/UCVFSortingSniff.php

Spacing +4 added lines, -4 removed lines

@@ -43,7 +43,7 @@ 
      *
      * @var string
      */
-    private const MESSAGE_WRONG_POSITION = 'Your php structure is at a wrong position. ' .
+    private const MESSAGE_WRONG_POSITION = 'Your php structure is at a wrong position. '.
         'The sorting order is: T_USE, T_CONST, T_VARIABLE, T_FUNCTION. We expect a %s (%s).';
 
     /**
@@ -94,7 +94,7 @@ 
      */
     private function loadTokensForPositions(array $subTokenPoss): array
     {
-        $subTokens = array_map(function (int $position): array {
+        $subTokens = array_map(function(int $position): array {
             return $this->tokens[$position];
         }, $subTokenPoss);
 
@@ -123,7 +123,7 @@ 
      */
     private function removeUnwantedTokens(array $subTokens): array
     {
-        return array_filter($subTokens, function (array $subToken, int $tokenPos): bool {
+        return array_filter($subTokens, function(array $subToken, int $tokenPos): bool {
             switch ($subToken['code']) {
                 case T_VARIABLE:
                     $return = (new PropertyHelper($this->file))->isProperty($tokenPos);
@@ -150,7 +150,7 @@ 
      */
     private function sortTokens(array $subTokens): array
     {
-        uasort($subTokens, function (array $leftToken, array $rightToken): int {
+        uasort($subTokens, function(array $leftToken, array $rightToken): int {
             // Don't change the structure by default.
             $return = $leftToken['line'] <=> $rightToken['line'];
 

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 ($iteratedPointer = $commaPos + 1; $iteratedPointer < $pointerAfterComma; $iteratedPointer++) {
                 $fixer->replaceToken($iteratedPointer, '');
             }

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

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      *
      * @var File|null
      */
-    protected File|null $file = null;
+    protected File | null $file = null;
 
     /**
      * A list of forbidden functions with their alternatives.

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

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
         $this->getFile()->fixer->addContent(
             $docTagClosingPos,
             sprintf(
-                "\n" . str_repeat(' ', ($this->tokens[$docTagClosingPos]['level']) * 4) . '%s',
+                "\n".str_repeat(' ', ($this->tokens[$docTagClosingPos]['level']) * 4).'%s',
                 trim($collectedAttrContent),
             ),
         );