bestit / PHP_CodeSniffer

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';
         });
 

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

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
     /**
      * Error message for multiple returns.
      */
-    private const WARNING_MULTIPLE_RETURNS_FOUND = 'Multiple returns detected. Did you refactor your method? Please ' .
+    private const WARNING_MULTIPLE_RETURNS_FOUND = 'Multiple returns detected. Did you refactor your method? Please '.
         'do not use an early return if your method/function still is cluttered.';
 
     /**
@@ -61,7 +61,7 @@ 
             $this->token['scope_closer']
         );
 
-        return array_filter($returnPositions, function (int $returnPos): bool {
+        return array_filter($returnPositions, function(int $returnPos): bool {
             $possibleClosure = $this->file->findPrevious([T_CLOSURE, T_FUNCTION], $returnPos - 1, $this->stackPos);
 
             return $possibleClosure === $this->stackPos;
@@ -80,7 +80,7 @@ 
         if (count($returnPositions) > 1) {
             array_shift($returnPositions);
 
-            array_walk($returnPositions, function (int $returnPos): void {
+            array_walk($returnPositions, function(int $returnPos): void {
                 $this->file->addWarning(
                     self::WARNING_MULTIPLE_RETURNS_FOUND,
                     $returnPos,

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

Spacing +3 added lines, -3 removed lines

@@ -94,7 +94,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'];
@@ -136,7 +136,7 @@ 
         );
 
         if ($sniffName) {
-            $sniffClassName .= '.' . $sniffName;
+            $sniffClassName .= '.'.$sniffName;
         }
 
         return $sniffClassName;
@@ -214,7 +214,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();
     }