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/Func/FunctionSignatureParser.php

Spacing +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
                 case T_CALLABLE:
                 case T_NULLABLE:
                     // these cannot be default
-                    $raw['type'] = ($raw['type'] ?? '') . $token['content'];
+                    $raw['type'] = ($raw['type'] ?? '').$token['content'];
                     break;
                 case T_EQUAL:
                     $raw['default'] = '';
@@ -87,7 +87,7 @@ 
                     if (isset($raw['default'])) {
                         $raw['default'] .= $token['content'];
                     } else {
-                        $raw['type'] = ($raw['type'] ?? '') . $token['content'];
+                        $raw['type'] = ($raw['type'] ?? '').$token['content'];
                     }
                     break;
                 case T_ELLIPSIS:

src/Core/CodeElement/CodeElementDetector.php

Spacing +4 added lines, -4 removed lines

@@ -70,7 +70,7 @@ 
                     break;
             }
 
-            $fqcn = ($namespace ? $namespace . '\\' : '') . $className;
+            $fqcn = ($namespace ? $namespace.'\\' : '').$className;
 
             // Instead of looking for doc blocks , we look for tokens
             // that should be preceded / followed by a doc block. This way we can
@@ -91,7 +91,7 @@ 
                 switch ($tokenCode) {
                     case T_CONST:
                         $constName = TokenHelper::getDeclarationName($file, $ptr);
-                        [$valueType,] = TokenHelper::getAssignmentType($file, $ptr);
+                        [$valueType, ] = TokenHelper::getAssignmentType($file, $ptr);
                         $docBlock = TokenHelper::getPrevDocBlock($file, $ptr, $skip);
                         $currentElement = new ConstElement($line, $docBlock, $namespace, $constName, $valueType);
                         $fileElement->addConstant($currentElement);
@@ -128,7 +128,7 @@ 
                 switch ($tokenCode) {
                     case T_CONST:
                         $docBlock = TokenHelper::getPrevDocBlock($file, $ptr, $skip);
-                        [$valueType,] = TokenHelper::getAssignmentType($file, $ptr);
+                        [$valueType, ] = TokenHelper::getAssignmentType($file, $ptr);
                         $currentElement = new ClassConstElement($line, $docBlock, $fqcn, $decName, $valueType);
                         $parentElement->addConstant($currentElement);
                         break;
@@ -178,7 +178,7 @@ 
                 switch ($tokenCode) {
                     case T_CONST:
                         $docBlock = TokenHelper::getPrevDocBlock($file, $ptr, $skip);
-                        [$valueType,] = TokenHelper::getAssignmentType($file, $ptr);
+                        [$valueType, ] = TokenHelper::getAssignmentType($file, $ptr);
                         $currentElement = new InterfaceConstElement($line, $docBlock, $fqcn, $decName, $valueType);
                         $parentElement->addConstant($currentElement);
                         break;

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