gskema / phpcs-type-sniff

src/Sniffs/AbstractConfigurableSniff.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
                 }
             }
 
-            array_walk_recursive($opts, function (&$val) {
+            array_walk_recursive($opts, function(&$val) {
                 if ('true' === $val) {
                     $val = true;
                 } elseif ('false' === $val) {

src/Sniffs/CodeElement/FqcnPropSniff.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
         $varTag = $subject->getDocBlock()->getTagsByName('var')[0] ?? null;
 
         if ($varTag && null !== $varTag->getParamName()) {
-            $subject->addDocTypeWarning('Remove property name $' . $varTag->getParamName() . ' from @var tag');
+            $subject->addDocTypeWarning('Remove property name $'.$varTag->getParamName().' from @var tag');
         }
     }
 

src/Core/Type/TypeHelper.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
                 $typedArrayType->getType() instanceof ArrayType
                 || $typedArrayType->getType() instanceof UndefinedType
             ) {
-                return $typedArrayType;  // e.g. array[][] or [][]
+                return $typedArrayType; // e.g. array[][] or [][]
             }
         }
 

src/Core/Type/Declaration/NullableType.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function toString(): string
     {
-        return '?' . $this->type->toString();
+        return '?'.$this->type->toString();
     }
 
     public function toDocString(): string
@@ -37,6 +37,6 @@ 
         $rawType = $this->type->toString();
 
         // This must match sorting in CompoundType::toString() for raw comparisons.
-        return $rawType > 'null' ? 'null|' . $rawType : $rawType . '|null';
+        return $rawType > 'null' ? 'null|'.$rawType : $rawType.'|null';
     }
 }

src/Core/Type/DocBlock/TypedArrayType.php

Spacing +1 added lines, -1 removed lines

@@ -38,6 +38,6 @@
             $innerType = sprintf('(%s)', $innerType);
         }
 
-        return $innerType . str_repeat('[]', $this->depth);
+        return $innerType.str_repeat('[]', $this->depth);
     }
 }

src/Core/DocBlock/DocBlockParser.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             }
 
             if (null !== $tagLineNum) {
-                $rawTags[$tagLineNum] = ($rawTags[$tagLineNum] ?? '') . ' ' . trim($rawLine);
+                $rawTags[$tagLineNum] = ($rawTags[$tagLineNum] ?? '').' '.trim($rawLine);
             } elseif (!empty($rawLine) || !empty($descLines)) {
                 // Always append non-empty lines, but also append empty lines if we have
                 // non-empty lines already. More non-empty lines might follow.

src/Sniffs/CodeElement/FqcnDescriptionSniff.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
     {
         $this->invalidPatterns = array_merge($this->invalidPatterns, $config['invalidPatterns'] ?? []);
         foreach ($this->invalidPatterns as &$invalidPattern) {
-            $invalidPattern = '#' . $invalidPattern . '#i';
+            $invalidPattern = '#'.$invalidPattern.'#i';
         }
 
         $this->invalidTags = array_merge($this->invalidTags, $config['invalidTags'] ?? []);
@@ -70,7 +70,7 @@ 
         foreach ($element->getDocBlock()->getDescriptionLines() as $lineNum => $descriptionLine) {
             foreach ($this->invalidPatterns as $invalidPattern) {
                 if (preg_match($invalidPattern, $descriptionLine)) {
-                    $originId = $this->addViolationId ? $element->getFqcn() . $descriptionLine : null;
+                    $originId = $this->addViolationId ? $element->getFqcn().$descriptionLine : null;
                     SniffHelper::addViolation($file, 'Useless description', $lineNum, static::CODE, $this->reportType, $originId);
                 }
             }
@@ -79,7 +79,7 @@ 
         foreach ($element->getDocBlock()->getTags() as $tag) {
             foreach ($this->invalidTags as $invalidTagName) {
                 if ($tag->getName() === $invalidTagName) {
-                    $originId = $this->addViolationId ? $element->getFqcn() . $invalidTagName : null;
+                    $originId = $this->addViolationId ? $element->getFqcn().$invalidTagName : null;
                     SniffHelper::addViolation($file, 'Useless tag', $tag->getLine(), static::CODE, $this->reportType, $originId);
                 }
             }

src/Sniffs/CodeElement/FqcnMethodSniff.php

Spacing +2 added lines, -2 removed lines

@@ -115,7 +115,7 @@ 
         // @param
         foreach ($fnSig->getParams() as $fnParam) {
             $paramTag = $docBlock->getParamTag($fnParam->getName());
-            $id = $method->getId() . $fnParam->getName();
+            $id = $method->getId().$fnParam->getName();
             $subject = ParamTypeSubject::fromParam($fnParam, $paramTag, $docBlock, $id);
             $this->processSigType($subject);
             $subject->writeViolationsTo($file, static::CODE, $this->reportType, $this->addViolationId);
@@ -220,7 +220,7 @@ 
         foreach ($method->getDocBlock()->getTags() as $tag) {
             foreach ($invalidTags as $invalidTagName) {
                 if ($tag->getName() === $invalidTagName) {
-                    $originId = $this->addViolationId ? $method->getId() . $invalidTagName : null;
+                    $originId = $this->addViolationId ? $method->getId().$invalidTagName : null;
                     SniffHelper::addViolation($file, 'Useless tag', $tag->getLine(), static::CODE, $this->reportType, $originId);
                 }
             }

src/Core/SniffHelper.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         ?string $originId
     ): void {
         if (null !== $originId) {
-            $violationId = substr(md5($message . $originId), 0, 16);
+            $violationId = substr(md5($message.$originId), 0, 16);
             $message = sprintf('%s [%s]', $message, $violationId);
         }