bestit / PHP_CodeSniffer

src/Standards/BestIt/Sniffs/DocTags/AuthorTagSniff.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         // Satisfy php md
         unset($callback);
 
-        return $this->isValidContentInTrait($tagContent, function (array $matches): bool {
+        return $this->isValidContentInTrait($tagContent, function(array $matches): bool {
             return (bool) filter_var($matches['mail'], FILTER_VALIDATE_EMAIL);
         });
     }

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/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/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/DocTags/AbstractRequiredTagsSniff.php

Spacing +8 added lines, -8 removed lines

@@ -86,7 +86,7 @@ 
         array_walk(
             $allTags,
             // TODO: Removed Duplicates
-            function (array $tag, int $tagPos) use (&$checkedTags, $tagRules): void {
+            function(array $tag, int $tagPos) use (&$checkedTags, $tagRules): void {
                 $tagContent = substr($tag['content'], 1);
 
                 if (!in_array($tagContent, $checkedTags)) {
@@ -101,7 +101,7 @@ 
                             $tagPos,
                             // We use an error code containing the tag name because we can't configure this rules from
                             // the outside and need  specific code to exclude the rule for this special tag.
-                            static::CODE_TAG_OCCURRENCE_MAX_PREFIX . ucfirst($tagContent),
+                            static::CODE_TAG_OCCURRENCE_MAX_PREFIX.ucfirst($tagContent),
                             [
                                 $tagContent,
                                 $maxCount,
@@ -126,7 +126,7 @@ 
 
         array_walk(
             $rulesWithReq,
-            function (array $tagRule, string $tag) use ($checkedRule): void {
+            function(array $tagRule, string $tag) use ($checkedRule): void {
                 $minCount = $tagRule[$checkedRule];
                 $tagCount = count($this->findTokensForTag($tag));
 
@@ -136,7 +136,7 @@ 
                         $this->getDocCommentPos(),
                         // We use an error code containing the tag name because we can't configure this rules from the
                         // outside and need  specific code to exclude the rule for this special tag.
-                        static::CODE_TAG_OCCURRENCE_MIN_PREFIX . ucfirst($tag),
+                        static::CODE_TAG_OCCURRENCE_MIN_PREFIX.ucfirst($tag),
                         [
                             $tag,
                             $minCount,
@@ -159,7 +159,7 @@ 
     {
         $allTags = $this->getAllTags();
 
-        return array_filter($allTags, function (array $tag) use ($tagName): bool {
+        return array_filter($allTags, function(array $tag) use ($tagName): bool {
             return substr($tag['content'], 1) === $tagName;
         });
     }
@@ -203,7 +203,7 @@ 
     {
         $processedTagRules = $this->getProcessedTagRules();
 
-        $processedTagRules = array_filter($processedTagRules, function (array $tagRule) use ($requiredRule): bool {
+        $processedTagRules = array_filter($processedTagRules, function(array $tagRule) use ($requiredRule): bool {
             return array_key_exists($requiredRule, $tagRule);
         });
         return $processedTagRules;
@@ -242,8 +242,8 @@ 
     {
         $processedTagRules = $this->getTagRules();
 
-        array_walk($processedTagRules, function (&$tagRule) {
-            $tagRule = array_map(function ($valueOrCallback) {
+        array_walk($processedTagRules, function(&$tagRule) {
+            $tagRule = array_map(function($valueOrCallback) {
                 return is_callable($valueOrCallback, true)
                     ? Closure::fromCallable($valueOrCallback)->call($this)
                     : $valueOrCallback;

build/refreshReadmeTable.php

Spacing +4 added lines, -4 removed lines

@@ -2,7 +2,7 @@ 
 
 declare(strict_types=1);
 
-require_once __DIR__ . '/../vendor/autoload.php';
+require_once __DIR__.'/../vendor/autoload.php';
 
 $directory = new RecursiveDirectoryIterator($baseFolder = './src/Standards/BestIt/Sniffs');
 $iterator = new RecursiveIteratorIterator($directory);
@@ -74,8 +74,8 @@ 
     foreach ($regexIterator as $file) {
         [$file] = $file;
 
-        $simpleClassName = str_replace([$baseFolder . DIRECTORY_SEPARATOR, '.php', 'Sniff'], '', $file);
-        $fullQualifiedClassName = 'BestIt\\Sniffs\\' . str_replace('/', '\\', $simpleClassName) . 'Sniff';
+        $simpleClassName = str_replace([$baseFolder.DIRECTORY_SEPARATOR, '.php', 'Sniff'], '', $file);
+        $fullQualifiedClassName = 'BestIt\\Sniffs\\'.str_replace('/', '\\', $simpleClassName).'Sniff';
 
         $hasSuppresses = (bool) preg_match_all(
             '/->isSniffSuppressed\((?P<code>\s*.*\s*)\)/mU',
@@ -145,7 +145,7 @@ 
     ksort($codes);
 
     file_put_contents(
-        $tmpFile = __DIR__ . DIRECTORY_SEPARATOR . 'table.md',
+        $tmpFile = __DIR__.DIRECTORY_SEPARATOR.'table.md',
         <<<EOD
 | Sniff | Description | suppressable |
 | ----- | ----------- | ------------ |

src/Standards/BestIt/Sniffs/Strings/ConcatCalculationSniff.php

Spacing +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
      */
     private function isCheckAlreadyDone(int $mathPos): bool
     {
-        $checkMarker = $this->file->getFilename() . $mathPos;
+        $checkMarker = $this->file->getFilename().$mathPos;
         $isDone = @self::$alreadyDoneChecks[$checkMarker];
 
         self::$alreadyDoneChecks[$checkMarker] = true;

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

Spacing +4 added lines, -4 removed lines

@@ -152,7 +152,7 @@ 
     private function isTooSimpleGetter(string $propertyName, int $methodPosition): bool
     {
         return $this->matchesMethodContentToRegex(
-            '/return \$this->' . $propertyName . '\s*;$/m',
+            '/return \$this->'.$propertyName.'\s*;$/m',
             $methodPosition,
         );
     }
@@ -176,7 +176,7 @@ 
             $parameterName = current($parameters);
 
             $isTooSimpleSetter = $this->matchesMethodContentToRegex(
-                '/\$this->' . $propertyName . '\s*=\s*\\' . $parameterName . '\s*;' .
+                '/\$this->'.$propertyName.'\s*=\s*\\'.$parameterName.'\s*;'.
                 // finish with the assignment or wait for the fluent return.
                 '($|\s*return\s*\$this\s*;$)/m',
                 $methodPosition,
@@ -224,8 +224,8 @@ 
         }
 
         $propertyName = substr($this->tokens[$propertyPointer]['content'], 1);
-        $setterPosition = $this->findMethod(self::METHOD_PREFIX_SETTER . ucfirst($propertyName));
-        $getterPosition = $this->findMethod(self::METHOD_PREFIX_GETTER . ucfirst($propertyName));
+        $setterPosition = $this->findMethod(self::METHOD_PREFIX_SETTER.ucfirst($propertyName));
+        $getterPosition = $this->findMethod(self::METHOD_PREFIX_GETTER.ucfirst($propertyName));
 
         if (
             $getterPosition && $setterPosition && $this->isTooSimpleSetter($propertyName, $setterPosition) &&

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,