Browse Source

code grooming

SpacePossum 6 years ago
parent
commit
390c718a96

+ 0 - 1
src/AbstractNoUselessElseFixer.php

@@ -12,7 +12,6 @@
 
 namespace PhpCsFixer;
 
-use PhpCsFixer\Tokenizer\Token;
 use PhpCsFixer\Tokenizer\Tokens;
 
 /**

+ 1 - 1
src/Cache/SignatureInterface.php

@@ -47,7 +47,7 @@ interface SignatureInterface
     /**
      * @param SignatureInterface $signature
      *
-     * @return mixed
+     * @return bool
      */
     public function equals(self $signature);
 }

+ 1 - 0
src/Console/Command/DescribeCommand.php

@@ -251,6 +251,7 @@ final class DescribeCommand extends Command
             $output->writeln('');
         }
 
+        /** @var CodeSampleInterface[] $codeSamples */
         $codeSamples = array_filter($definition->getCodeSamples(), static function (CodeSampleInterface $codeSample) {
             if ($codeSample instanceof VersionSpecificCodeSampleInterface) {
                 return $codeSample->isSuitableFor(\PHP_VERSION_ID);

+ 1 - 1
src/Fixer/Basic/NonPrintableCharacterFixer.php

@@ -93,7 +93,7 @@ final class NonPrintableCharacterFixer extends AbstractFixer implements Configur
      */
     public function isCandidate(Tokens $tokens)
     {
-        return \count($tokens) > 1 && $tokens->isAnyTokenKindsFound(self::$tokens);
+        return $tokens->isAnyTokenKindsFound(self::$tokens);
     }
 
     /**

+ 1 - 1
src/Fixer/Phpdoc/PhpdocOrderFixer.php

@@ -45,7 +45,7 @@ final class PhpdocOrderFixer extends AbstractFixer
 /**
  * Hello there!
  *
- * @throws Exception|RuntimeException dfsdf
+ * @throws Exception|RuntimeException foo
  * @custom Test!
  * @return int  Return the number of changes.
  * @param string $foo

+ 0 - 1
src/Fixer/Phpdoc/PhpdocSummaryFixer.php

@@ -14,7 +14,6 @@ namespace PhpCsFixer\Fixer\Phpdoc;
 
 use PhpCsFixer\AbstractFixer;
 use PhpCsFixer\DocBlock\DocBlock;
-use PhpCsFixer\DocBlock\Line;
 use PhpCsFixer\DocBlock\ShortDescription;
 use PhpCsFixer\Fixer\WhitespacesAwareFixerInterface;
 use PhpCsFixer\FixerDefinition\CodeSample;

+ 2 - 0
src/FixerFactory.php

@@ -17,6 +17,7 @@ use PhpCsFixer\Fixer\ConfigurableFixerInterface;
 use PhpCsFixer\Fixer\FixerInterface;
 use PhpCsFixer\Fixer\WhitespacesAwareFixerInterface;
 use Symfony\Component\Finder\Finder as SymfonyFinder;
+use Symfony\Component\Finder\SplFileInfo;
 
 /**
  * Class provides a way to create a group of fixers.
@@ -94,6 +95,7 @@ final class FixerFactory
         if (null === $builtInFixers) {
             $builtInFixers = [];
 
+            /** @var SplFileInfo $file */
             foreach (SymfonyFinder::create()->files()->in(__DIR__.'/Fixer') as $file) {
                 $relativeNamespace = $file->getRelativePath();
                 $fixerClass = 'PhpCsFixer\\Fixer\\'.($relativeNamespace ? $relativeNamespace.'\\' : '').$file->getBasename('.php');

+ 2 - 0
src/Tokenizer/Transformers.php

@@ -14,6 +14,7 @@ namespace PhpCsFixer\Tokenizer;
 
 use PhpCsFixer\Utils;
 use Symfony\Component\Finder\Finder;
+use Symfony\Component\Finder\SplFileInfo;
 
 /**
  * Collection of Transformer classes.
@@ -101,6 +102,7 @@ final class Transformers
      */
     private function findBuiltInTransformers()
     {
+        /** @var SplFileInfo $file */
         foreach (Finder::create()->files()->in(__DIR__.'/Transformer') as $file) {
             $relativeNamespace = $file->getRelativePath();
             $class = __NAMESPACE__.'\\Transformer\\'.($relativeNamespace ? $relativeNamespace.'\\' : '').$file->getBasename('.php');

+ 3 - 3
tests/Fixtures/Integration/set/@PSR2.test-in.php

@@ -1,12 +1,12 @@
 <?
 namespace Vendor\Package;
-use FooInterface, FooInterfaceB;
+use FooInterfaceA, FooInterfaceB;
 use BarClass as Bar;
 use OtherVendor\OtherPackage\BazClass;
-class Foo extends Bar implements FooInterface{
+class Foo extends Bar implements FooInterfaceA{
     var $aaa = 1, $bbb = 2;
 
-    public function sampleFunction($a, $b = null)
+    public function sampleFunction($a, $arg1, $arg2, $arg3, $foo, $b = null)
     {
         if ($a === $b) {
             bar();

+ 3 - 3
tests/Fixtures/Integration/set/@PSR2.test-out.php

@@ -1,17 +1,17 @@
 <?php
 namespace Vendor\Package;
 
-use FooInterface;
+use FooInterfaceA;
 use FooInterfaceB;
 use BarClass as Bar;
 use OtherVendor\OtherPackage\BazClass;
 
-class Foo extends Bar implements FooInterface
+class Foo extends Bar implements FooInterfaceA
 {
     public $aaa = 1;
     public $bbb = 2;
 
-    public function sampleFunction($a, $b = null)
+    public function sampleFunction($a, $arg1, $arg2, $arg3, $foo, $b = null)
     {
         if ($a === $b) {
             bar();

Some files were not shown because too many files changed in this diff