gskema / phpcs-type-sniff

src/Sniffs/CodeElement/FqcnConstSniff.php

Spacing +2 added lines, -2 removed lines

@@ -26,8 +26,8 @@
      */
     public function configure(array $config): void
     {
-        $this->reportType = (string)($config['reportType'] ?? 'warning');
-        $this->addViolationId = (bool)($config['addViolationId'] ?? false);
+        $this->reportType = (string) ($config['reportType'] ?? 'warning');
+        $this->addViolationId = (bool) ($config['addViolationId'] ?? false);
     }
 
     /**

src/Sniffs/CodeElement/FqcnPropSniff.php

Spacing +3 added lines, -3 removed lines

@@ -27,8 +27,8 @@ 
      */
     public function configure(array $config): void
     {
-        $this->reportType = (string)($config['reportType'] ?? 'warning');
-        $this->addViolationId = (bool)($config['addViolationId'] ?? false);
+        $this->reportType = (string) ($config['reportType'] ?? 'warning');
+        $this->addViolationId = (bool) ($config['addViolationId'] ?? false);
     }
 
     /**
@@ -72,7 +72,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/Sniffs/CodeElement/FqcnDescriptionSniff.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,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'] ?? []);
@@ -43,8 +43,8 @@ 
             $invalidTag = substr($invalidTag, 1);
         }
 
-        $this->reportType = (string)($config['reportType'] ?? 'warning');
-        $this->addViolationId = (bool)($config['addViolationId'] ?? false);
+        $this->reportType = (string) ($config['reportType'] ?? 'warning');
+        $this->addViolationId = (bool) ($config['addViolationId'] ?? false);
     }
 
     /**
@@ -68,7 +68,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);
                 }
             }
@@ -77,7 +77,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 +6 added lines, -6 removed lines

@@ -52,10 +52,10 @@ 
         $invalidTags = array_unique($invalidTags);
 
         $this->invalidTags = $invalidTags;
-        $this->reportMissingTags = (bool)($config['reportMissingTags'] ?? true);
-        $this->reportNullableBasicGetter = (bool)($config['reportNullableBasicGetter'] ?? true);
-        $this->reportType = (string)($config['reportType'] ?? 'warning');
-        $this->addViolationId = (bool)($config['addViolationId'] ?? false);
+        $this->reportMissingTags = (bool) ($config['reportMissingTags'] ?? true);
+        $this->reportNullableBasicGetter = (bool) ($config['reportNullableBasicGetter'] ?? true);
+        $this->reportType = (string) ($config['reportType'] ?? 'warning');
+        $this->addViolationId = (bool) ($config['addViolationId'] ?? false);
     }
 
     /**
@@ -111,7 +111,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);
@@ -215,7 +215,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/Sniffs/CompositeCodeElementSniff.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         $globalAddViolationId = $config['addViolationId'] ?? false;
 
         // 1. CompositeCodeElementSniff configuration
-        $this->useReflection = (bool)($config['useReflection'] ?? false);
+        $this->useReflection = (bool) ($config['useReflection'] ?? false);
 
         // 2. CodeElementSniff(s) configuration
         // Default sniffs. They can be removed by specifying <property name="FqcnMethodSniff.enabled" value="false"/>

src/Inspection/DocTypeInspector.php

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
             $subject->addDocTypeWarning(sprintf(
                 'Replace nullable type "%s" with compound type with null "%s" for :subject:.',
                 $docType->toString(),
-                str_replace('?', '', $docType->toString()) . '|null' // @TODO Not ideal
+                str_replace('?', '', $docType->toString()).'|null' // @TODO Not ideal
             ));
         }