bestit / PHP_CodeSniffer

src/Standards/BestIt/CodeSniffer/Helper/DocDescriptionHelper.php

Spacing +1 added lines, -1 removed lines

@@ -300,7 +300,7 @@
 
         $this->file->getFixer()->addContent(
             $descEndPtr,
-            $this->file->getEolChar() . str_repeat('    ', $descEndToken['level']) . ' *'
+            $this->file->getEolChar().str_repeat('    ', $descEndToken['level']).' *'
         );
 
         $this->file->getFixer()->endChangeset();

src/Standards/BestIt/CodeSniffer/AbstractFileDecorator.php

Doc Comments +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
      * @param string $error The error message.
      * @param int $stackPtr The stack position where the error occurred.
      * @param string $code A violation code unique to the sniff message.
-     * @param array $data Replacements for the error message.
+     * @param string[] $data Replacements for the error message.
      * @param int $severity The severity level for this error. A value of 0
      *                         will be converted into the default severity level.
      *

Spacing +1 added lines, -1 removed lines

@@ -762,7 +762,7 @@
     {
         $baseProps = get_object_vars($baseFile);
 
-        array_walk($baseProps, function ($value, $key) {
+        array_walk($baseProps, function($value, $key) {
             $this->$key = $value;
         });
     }

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

Spacing +7 added lines, -7 removed lines

@@ -175,24 +175,24 @@ 
 
             // Insert new line between groups.
             if ($isGroupSwitch) {
-                $newContent .= $lineStartingPadding . '*' . $eolChar;
+                $newContent .= $lineStartingPadding.'*'.$eolChar;
             }
 
             // Create the new Tag.
             // WARNING We do not a line break in the tag summary.
-            $newContent .= $lineStartingPadding . '* ' .
+            $newContent .= $lineStartingPadding.'* '.
                 // Remove the "ending" whitespace if there are no more contents
                 trim(
-                    $thisTagContent . ' ' .
+                    $thisTagContent.' '.
                     implode(' ', array_column($tag['contents'] ?? [], 'content'))
-                ) .
+                ).
                 $eolChar;
 
             $prevTagContent = $thisTagContent;
             $withReturn = $isReturn;
         }
 
-        $newContent .= $lineStartingPadding . '*/' . $eolChar;
+        $newContent .= $lineStartingPadding.'*/'.$eolChar;
 
         return $newContent;
     }
@@ -244,7 +244,7 @@ 
 
         $fixer->addContentBefore(
             $token['pointer'],
-            $this->file->getEolChar() . $lineStartPadding . '* '
+            $this->file->getEolChar().$lineStartPadding.'* '
         );
 
         $fixer->endChangeset();
@@ -336,7 +336,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 = 0;
             $leftTagName = $leftToken['content'];
             $rightTagName = $rightToken['content'];

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

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         // Satisfy php md
         unset($callback);
 
-        return $this->isValidContentInTrait($tagContent, function (array $matches): bool {
+        return $this->isValidContentInTrait($tagContent, function(array $matches): bool {
             return (bool) filter_var($matches['mail'], FILTER_VALIDATE_EMAIL);
         });
     }

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

Spacing +1 added lines, -1 removed lines

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

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

Spacing +4 added lines, -4 removed lines

@@ -105,12 +105,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();
@@ -146,7 +146,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.
             }
 
@@ -288,7 +288,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/DocTags/TagContentFormatTrait.php

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
     protected function processTagContent(?string $tagContent = null): void
     {
         if (!$this->isValidContent($tagContent)) {
-            $this->getFile()->{'add' . ($this->asError() ? 'Error' : 'Warning')}(...$this->getReportData($tagContent));
+            $this->getFile()->{'add'.($this->asError() ? 'Error' : 'Warning')}(...$this->getReportData($tagContent));
         }
     }
 }

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

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     /**
      * The message for the error.
      */
-    private const MESSAGE_TAG_MISSING_DATES = 'Please provide the version since when its deprecated and when it will ' .
+    private const MESSAGE_TAG_MISSING_DATES = 'Please provide the version since when its deprecated and when it will '.
         'be removed (Pattern: %s).';
 
     /**

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

Spacing +4 added lines, -4 removed lines

@@ -74,7 +74,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.';
 
     /**
@@ -197,7 +197,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';
     }
 
@@ -234,7 +234,7 @@ 
             ))->setToken($this->token);
         }
 
-        $this->varTokens = array_filter($this->tokens, function (array $token) use ($varPositions): bool {
+        $this->varTokens = array_filter($this->tokens, function(array $token) use ($varPositions): bool {
             return in_array($token['pointer'], $varPositions, true);
         });
 
@@ -256,7 +256,7 @@ 
             $this->stackPos - 1
         );
 
-        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function (int $position) {
+        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function(int $position) {
             return $this->tokens[$position]['content'] === '@param';
         });