bestit / PHP_CodeSniffer

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

Doc Comments +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
     /**
      * Loads the tokens for the positions.
      *
-     * @param array $subTokenPoss
+     * @param integer[] $subTokenPoss
      *
      * @return array
      */

Spacing +4 added lines, -4 removed lines

@@ -42,7 +42,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).';
 
     /**
@@ -93,7 +93,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);
 
@@ -122,7 +122,7 @@ 
      */
     private function removeUnwantedTokens(array $subTokens): array
     {
-        return array_filter($subTokens, function (array $subToken): bool {
+        return array_filter($subTokens, function(array $subToken): bool {
             switch ($subToken['code']) {
                 case T_VARIABLE:
                     $return = (new PropertyHelper($this->file))->isProperty($subToken['pointer']);
@@ -161,7 +161,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/DocTags/ReturnTagSniff.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * The message for the mixed type warning.
      */
-    private const MESSAGE_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the ' .
+    private const MESSAGE_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the '.
         'required types in detail.';
 
     /**

src/Standards/BestIt/Sniffs/SuppressingTrait.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         );
 
         if ($sniffName) {
-            $sniffClassName .= '.' . $sniffName;
+            $sniffClassName .= '.'.$sniffName;
         }
 
         return $sniffClassName;

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

Spacing +8 added lines, -8 removed lines

@@ -86,7 +86,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)) {
@@ -101,7 +101,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.
-                            static::CODE_TAG_OCCURRENCE_MAX_PREFIX . ucfirst($tagContent),
+                            static::CODE_TAG_OCCURRENCE_MAX_PREFIX.ucfirst($tagContent),
                             [
                                 $tagContent,
                                 $maxCount,
@@ -126,7 +126,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));
 
@@ -136,7 +136,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.
-                        static::CODE_TAG_OCCURRENCE_MIN_PREFIX . ucfirst($tag),
+                        static::CODE_TAG_OCCURRENCE_MIN_PREFIX.ucfirst($tag),
                         [
                             $tag,
                             $minCount,
@@ -159,7 +159,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;
         });
     }
@@ -203,7 +203,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;
@@ -242,8 +242,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/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();
     }

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

Spacing +2 added lines, -2 removed lines

@@ -88,7 +88,7 @@ 
             $tokenIdent = $this->getTokenName();
 
             $exception = (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)]);
@@ -114,7 +114,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)]);

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 |
 | ----- | ----------- | ------------ |