bestit / PHP_CodeSniffer

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

Doc Comments +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
      * @param string $rule
      * @param string $tag
      *
-     * @return bool|string
+     * @return string|false
      */
     private function hasFixCallback(string $rule, string $tag)
     {

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

Doc Comments +2 added lines, -2 removed lines

@@ -151,7 +151,7 @@ 
     /**
      * The callback to sort tokens.
      *
-     * 1. @return goes to the bottom
+     * 1. @return integer to the bottom
      * 2. Single tags are group alphabetically in the top group.
      * 3. Groups are sorted then by occurrence, that the largest group is the last one before the return.
      * 4. Same annotations are kept in the order of their code.
@@ -160,7 +160,7 @@ 
      * @param array $rightToken Provided by usort.
      * @param array $tagCounts Saves the occurence count for every tag.
      *
-     * @return int
+     * @return integer
      */
     private function compareTokensForSorting(array $leftToken, array $rightToken, array $tagCounts): int
     {

Spacing +9 added lines, -9 removed lines

@@ -230,27 +230,27 @@ 
 
             // Insert new line between groups.
             if ($isGroupSwitch) {
-                $newContent .= $lineStartingPadding . '*' . $eolChar;
+                $newContent .= $lineStartingPadding.'*'.$eolChar;
             }
 
             // Create the new Tag.
             // WARNING We do not need a line break in the tag summary.
-            $newContent .= $lineStartingPadding . '* ' . trim($thisTagContent);
+            $newContent .= $lineStartingPadding.'* '.trim($thisTagContent);
 
             if ($tag['contents']) {
                 $prevLine = $tag['line'];
                 foreach ($tag['contents'] as $subToken) {
                     // If we have a line switch, we need to create the correct indentation from before ...
                     if ($withLineSwitch = $subToken['line'] > $prevLine) {
-                        $newContent .= $eolChar .
-                            $lineStartingPadding . '*' .
+                        $newContent .= $eolChar.
+                            $lineStartingPadding.'*'.
                             str_repeat(' ', $subToken['column'] - strlen($lineStartingPadding) - 2);
 
                         $prevLine = $subToken['line'];
                     }
 
                     // ... if we have no line switch, then an additional whitespace is enough.
-                    $newContent .= ($withLineSwitch ? '' : ' ') . $subToken['content'];
+                    $newContent .= ($withLineSwitch ? '' : ' ').$subToken['content'];
                 }
             }
 
@@ -260,7 +260,7 @@ 
             $withReturn = $isReturn;
         }
 
-        $newContent .= $lineStartingPadding . '*/' . $eolChar;
+        $newContent .= $lineStartingPadding.'*/'.$eolChar;
 
         return $newContent;
     }
@@ -332,7 +332,7 @@ 
 
         $fixer->addContentBefore(
             $token['pointer'],
-            $this->file->getEolChar() . $lineStartPadding . '* '
+            $this->file->getEolChar().$lineStartPadding.'* '
         );
 
         $fixer->endChangeset();
@@ -348,7 +348,7 @@ 
         $barrier = 0;
         $tokens = $this->docTagHelper->getTagTokens();
 
-        $tokens = array_filter($tokens, function (array $token) use (&$barrier): bool {
+        $tokens = array_filter($tokens, function(array $token) use (&$barrier): bool {
             $allowed = true;
 
             if ($barrier) {
@@ -443,7 +443,7 @@ 
     {
         $tagCounts = $this->docTagHelper->getTagCounts($tokens);
 
-        usort($tokens, function (array $leftToken, array $rightToken) use ($tagCounts): int {
+        usort($tokens, function(array $leftToken, array $rightToken) use ($tagCounts): int {
             return $this->compareTokensForSorting($leftToken, $rightToken, $tagCounts);
         });
 

src/Standards/BestIt/Sniffs/Formatting/TraitUseDeclarationSniff.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
         $commaPointers = TokenHelper::findNextAll($file, T_COMMA, $usePos + 1, $endPos);
         foreach ($commaPointers as $commaPos) {
             $pointerAfterComma = TokenHelper::findNextEffective($file, $commaPos + 1);
-            $fixer->replaceToken($commaPos, ';' . $file->eolChar . $indentation . 'use ');
+            $fixer->replaceToken($commaPos, ';'.$file->eolChar.$indentation.'use ');
             for ($i = $commaPos + 1; $i < $pointerAfterComma; $i++) {
                 $fixer->replaceToken($i, '');
             }

src/Standards/BestIt/Sniffs/Formatting/AlphabeticallySortedUsesSniff.php

Spacing +2 added lines, -2 removed lines

@@ -199,7 +199,7 @@ 
         return implode(
             $file->eolChar,
             array_map(
-                function (UseStatement $useStatement) use ($file): string {
+                function(UseStatement $useStatement) use ($file): string {
                     $unqualifiedName = NamespaceHelper::getUnqualifiedNameFromFullyQualifiedName(
                         $useStatement->getFullyQualifiedTypeName()
                     );
@@ -301,7 +301,7 @@ 
      */
     private function sortUseStatements(): void
     {
-        uasort($this->useStatements, function (UseStatement $prevStatement, UseStatement $nextStatement) {
+        uasort($this->useStatements, function(UseStatement $prevStatement, UseStatement $nextStatement) {
             return $this->compareUseStatements($prevStatement, $nextStatement);
         });
     }