bestit / PHP_CodeSniffer

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

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
      *
      * @var string
      */
-    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.';
 
     /**
@@ -65,7 +65,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;
@@ -84,7 +84,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/DocTags/ReturnTagSniff.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      *
      * @var string
      */
-    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.';
 
     /**

src/Standards/BestIt/Sniffs/SuppressingTrait.php

Spacing +1 added lines, -1 removed lines

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

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

Spacing +4 added lines, -4 removed lines

@@ -94,7 +94,7 @@ 
      *
      * @var string
      */
-    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.';
 
     /**
@@ -217,7 +217,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';
     }
 
@@ -254,7 +254,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);
         });
 
@@ -276,7 +276,7 @@ 
             $this->stackPos - 1
         );
 
-        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function (int $position) {
+        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function(int $position) {
             return $this->tokens[$position]['content'] === '@param';
         });