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/Sniffs/Functions/FluentSetterSniff.php

Spacing +3 added lines, -3 removed lines

@@ -107,7 +107,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'];
@@ -149,7 +149,7 @@ 
         );
 
         if ($sniffName) {
-            $sniffFQCN .= '.' . $sniffName;
+            $sniffFQCN .= '.'.$sniffName;
         }
 
         return $sniffFQCN;
@@ -224,7 +224,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();
     }

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/DeprecatedTagSniff.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      *
      * @var string
      */
-    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/PackageTagSniff.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     {
         $this->file->getFixer()->replaceToken(
             TokenHelper::findNext($this->file->getBaseFile(), [T_DOC_COMMENT_STRING], $this->stackPos),
-            ' ' . $currentNamespace
+            ' '.$currentNamespace
         );
     }
 

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/AbstractRequiredTagsSniff.php

Spacing +8 added lines, -8 removed lines

@@ -90,7 +90,7 @@ 
         array_walk(
             $allTags,
             // TODO: Removed Duplicates
-            function (array $tag, int $tagPos) use (&$checkedTags, $tagRules): void {
+            function(array $tag, int $tagPos) use (&$checkedTags, $tagRules): void {
                 $tagContent = substr($tag['content'], 1);
 
                 if (!in_array($tagContent, $checkedTags)) {
@@ -105,7 +105,7 @@ 
                             $tagPos,
                             // We use an error code containing the tag name because we can't configure this rules from
                             // the outside and need  specific code to exclude the rule for this special tag.
-                            self::CODE_TAG_OCCURRENCE_MAX_PREFIX . ucfirst($tagContent),
+                            self::CODE_TAG_OCCURRENCE_MAX_PREFIX.ucfirst($tagContent),
                             [
                                 $tagContent,
                                 $maxCount,
@@ -130,7 +130,7 @@ 
 
         array_walk(
             $rulesWithReq,
-            function (array $tagRule, string $tag) use ($checkedRule): void {
+            function(array $tagRule, string $tag) use ($checkedRule): void {
                 $minCount = $tagRule[$checkedRule];
                 $tagCount = count($this->findTokensForTag($tag));
 
@@ -140,7 +140,7 @@ 
                         $this->getDocCommentPos(),
                         // We use an error code containing the tag name because we can't configure this rules from the
                         // outside and need  specific code to exclude the rule for this special tag.
-                        self::CODE_TAG_OCCURRENCE_MIN_PREFIX . ucfirst($tag),
+                        self::CODE_TAG_OCCURRENCE_MIN_PREFIX.ucfirst($tag),
                         [
                             $tag,
                             $minCount,
@@ -163,7 +163,7 @@ 
     {
         $allTags = $this->getAllTags();
 
-        return array_filter($allTags, function (array $tag) use ($tagName): bool {
+        return array_filter($allTags, function(array $tag) use ($tagName): bool {
             return substr($tag['content'], 1) === $tagName;
         });
     }
@@ -207,7 +207,7 @@ 
     {
         $processedTagRules = $this->getProcessedTagRules();
 
-        $processedTagRules = array_filter($processedTagRules, function (array $tagRule) use ($requiredRule): bool {
+        $processedTagRules = array_filter($processedTagRules, function(array $tagRule) use ($requiredRule): bool {
             return array_key_exists($requiredRule, $tagRule);
         });
         return $processedTagRules;
@@ -246,8 +246,8 @@ 
     {
         $processedTagRules = $this->getTagRules();
 
-        array_walk($processedTagRules, function (&$tagRule) {
-            $tagRule = array_map(function ($valueOrCallback) {
+        array_walk($processedTagRules, function(&$tagRule) {
+            $tagRule = array_map(function($valueOrCallback) {
                 return is_callable($valueOrCallback, true)
                     ? Closure::fromCallable($valueOrCallback)->call($this)
                     : $valueOrCallback;

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/RequiredDocBlockSniff.php

Spacing +2 added lines, -2 removed lines

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