Skip to content
A set of custom fixers for PHP CS Fixer
PHP
Branch: master
Clone or download
Fetching latest commit…
Cannot retrieve the latest commit at this time.
Permalink
Type Name Latest commit message Commit time
Failed to load latest commit information.
dev-tools
src
tests
.codecov.yml
.editorconfig
.gitattributes
.gitignore
.php_cs
.travis.yml
CHANGELOG.md
LICENSE
README.md
composer.json
infection.json
phpmd.xml
phpstan.neon
phpunit.xml
psalm.xml

README.md

PHP CS Fixer: custom fixers

Latest stable version PHP version License Repository size Last commit

Build status Code coverage Tests Mutation testing badge Psalm type coverage

A set of custom fixers for PHP CS Fixer.

Installation

PHP CS Fixer: custom fixers can be installed by running:

composer require --dev kubawerlos/php-cs-fixer-custom-fixers

Usage

In your PHP CS Fixer configuration register fixers and use them:

 <?php
 return PhpCsFixer\Config::create()
+    ->registerCustomFixers(new PhpCsFixerCustomFixers\Fixers())
     ->setRules([
         '@PSR2' => true,
         'array_syntax' => ['syntax' => 'short'],
+        PhpCsFixerCustomFixers\Fixer\NoLeadingSlashInGlobalNamespaceFixer::name() => true,
+        PhpCsFixerCustomFixers\Fixer\PhpdocNoSuperfluousParamFixer::name() => true,
     ]);

Fixers

CommentSurroundedBySpacesFixer

Comment must be surrounded by spaces.

 <?php
-/*foo*/
+/* foo */

DataProviderNameFixer

Name of data provider that is used only once must match name of test. Risky: when relying on name of data provider function.

 <?php
 class FooTest extends TestCase {
     /**
-     * @dataProvider dataProvider
+     * @dataProvider provideHappyPathCases
      */
     function testHappyPath() {}
-    function dataProvider() {}
+    function provideHappyPathCases() {}
 }

DataProviderReturnTypeFixer

Return type of data provider must be iterable. Risky: when relying on signature of data provider.

 <?php
 class FooTest extends TestCase {
     /**
      * @dataProvider provideHappyPathCases
      */
     function testHappyPath() {}
-    function provideHappyPathCases(): array {}
+    function provideHappyPathCases(): iterable {}
 }

ImplodeCallFixer

Function implode must be called with 2 arguments in the documented order. DEPRECATED: use implode_call instead. Risky: when the function implode is overridden.

 <?php
-implode($foo, "") . implode($bar);
+implode("", $foo) . implode('', $bar);

InternalClassCasingFixer

Class defined internally by an extension, or the core should be called using the correct casing.

 <?php
-$foo = new STDClass();
+$foo = new stdClass();

MultilineCommentOpeningClosingAloneFixer

Multiline comment/PHPDoc must have opening and closing line without any extra content.

 <?php
-/** Hello
+/**
+ * Hello
  * World!
  */;

NoCommentedOutCodeFixer

There should be no commented out code.

 <?php
-//var_dump($_POST);
 print_r($_POST);

NoDoctrineMigrationsGeneratedCommentFixer

There must be no comment generated by Doctrine Migrations.

 <?php
 namespace Migrations;
 use Doctrine\DBAL\Schema\Schema;
-/**
- * Auto-generated Migration: Please modify to your needs!
- */
 final class Version20180609123456 extends AbstractMigration
 {
     public function up(Schema $schema)
     {
-        // this up() migration is auto-generated, please modify it to your needs
         $this->addSql("UPDATE t1 SET col1 = col1 + 1");
     }
     public function down(Schema $schema)
     {
-        // this down() migration is auto-generated, please modify it to your needs
         $this->addSql("UPDATE t1 SET col1 = col1 - 1");
     }
 }

NoDuplicatedImportsFixer

Duplicated use statements must be removed.

 <?php
 namespace FooBar;
 use Foo;
-use Foo;
 use Bar;

NoImportFromGlobalNamespaceFixer

There must be no import from global namespace.

 <?php
 namespace Foo;
-use DateTime;
 class Bar {
-    public function __construct(DateTime $dateTime) {}
+    public function __construct(\DateTime $dateTime) {}
 }

NoLeadingSlashInGlobalNamespaceFixer

When in global namespace there must be no leading slash for class.

 <?php
-$x = new \Foo();
+$x = new Foo();
 namespace Bar;
 $y = new \Baz();

NoNullableBooleanTypeFixer

There must be no nullable boolean type. Risky: when the null is used.

 <?php
-function foo(?bool $bar) : ?bool
+function foo(bool $bar) : bool
 {
      return $bar;
  }

NoPhpStormGeneratedCommentFixer

There must be no comment generated by PhpStorm.

 <?php
-/**
- * Created by PhpStorm.
- * User: root
- * Date: 01.01.70
- * Time: 12:00
- */
 namespace Foo;

NoReferenceInFunctionDefinitionFixer

There must be no reference in function definition. Risky: when rely on reference.

 <?php
-function foo(&$x) {}
+function foo($x) {}

NoTwoConsecutiveEmptyLinesFixer

There must be no two consecutive empty lines in code. DEPRECATED: use no_extra_blank_lines instead.

 <?php
 namespace Foo;

-
 class Bar {};

NoUnneededConcatenationFixer

There should not be inline concatenation of strings.

 <?php
-echo 'foo' . 'bar';
+echo 'foobar';

NoUselessClassCommentFixer

There must be no comment like: "Class FooBar". DEPRECATED: use NoUselessCommentFixer instead.

 <?php
 /**
- * Class FooBar
  * Class to do something
  */
 class FooBar {}

NoUselessCommentFixer

There must be no comment like "Class Foo".

 <?php
 /**
- * Class Foo
  * Class to do something
  */
 class Foo {
     /**
-     * Get bar
      */
     function getBar() {}
 }

NoUselessConstructorCommentFixer

There must be no comment like: "Foo constructor". DEPRECATED: use NoUselessCommentFixer instead.

 <?php
 class Foo {
     /**
-     * Foo constructor
      */
     public function __construct() {}
 }

NoUselessDoctrineRepositoryCommentFixer

There must be no comment generated by the Doctrine ORM.

 <?php
-/**
- * FooRepository
- *
- * This class was generated by the Doctrine ORM. Add your own custom
- * repository methods below.
- */
 class FooRepository extends EntityRepository {}

NoUselessSprintfFixer

Function sprintf without parameters should not be used. Risky: when the function sprintf is overridden.

 <?php
-$foo = sprintf('Foo');
+$foo = 'Foo';

NullableParamStyleFixer

Nullable parameters must be written in the consistent style. Configuration options:

  • style ('with_question_mark', 'without_question_mark'): whether nullable parameter type should be prefixed or not with question mark; defaults to 'with_question_mark'
 <?php
-function foo(int $x = null) {
+function foo(?int $x = null) {
 }

OperatorLinebreakFixer

Operators must always be at the beginning or at the end of the line. To be deprecated after this is merged and released. Configuration options:

  • only_booleans (bool): whether to limit operators to only boolean ones; defaults to false
  • position ('beginning', 'end'): whether to place operators at the beginning or at the end of the line; defaults to 'beginning'
 <?php
 function foo() {
-    return $bar ||
-        $baz;
+    return $bar
+        || $baz;
 }

PhpUnitNoUselessReturnFixer

PHPUnit's functions fail, markTestIncomplete and markTestSkipped should not be followed directly by return. Risky: when PHPUnit's native methods are overridden.

 <?php
 class FooTest extends TestCase {
     public function testFoo() {
         $this->markTestSkipped();
-        return;
     }
 }

PhpdocNoIncorrectVarAnnotationFixer

@var must be correct in the code.

 <?php
-/** @var Foo $foo */
 $bar = new Foo();

PhpdocNoSuperfluousParamFixer

There must be no superfluous parameters in PHPDoc.

 <?php
 /**
  * @param bool $b
- * @param int $i
  * @param string $s this is string
- * @param string $s duplicated
  */
 function foo($b, $s) {}

PhpdocOnlyAllowedAnnotationsFixer

Only listed annotations can be in PHPDoc. Configuration options:

  • elements (array): list of annotations to keep in PHPDoc; defaults to []
 <?php
 /**
  * @author John Doe
- * @package foo
- * @subpackage bar
  * @version 1.0
  */
 function foo_bar() {}

PhpdocParamOrderFixer

@param annotations must be in the same order as function's parameters.

 <?php
 /**
+ * @param int $a
  * @param int $b
- * @param int $a
  * @param int $c
  */
 function foo($a, $b, $c) {}

PhpdocParamTypeFixer

@param must have type.

 <?php
 /**
  * @param string $foo
- * @param        $bar
+ * @param mixed  $bar
  */
 function a($foo, $bar) {}

PhpdocSelfAccessorFixer

In PHPDoc inside class or interface element self should be preferred over the class name itself.

 <?php
 class Foo {
     /**
-     * @var Foo
+     * @var self
      */
      private $instance;
 }

PhpdocSingleLineVarFixer

@var annotation must be in single line when is the only content.

 <?php
 class Foo {
-    /**
-     * @var string
-     */
+    /** @var string */
     private $name;
 }

PhpdocVarAnnotationCorrectOrderFixer

@var and @type annotations must have type and name in the correct order. DEPRECATED: use phpdoc_var_annotation_correct_order instead.

 <?php
-/** @var $foo int */
+/** @var int $foo */
 $foo = 2 + 2;

SingleLineThrowFixer

throw must be single line. To be deprecated after this is merged and released.

 <?php
-throw new Exception(
-    'Error',
-    500
-);
+throw new Exception('Error', 500);

SingleSpaceAfterStatementFixer

Single space must follow - not followed by semicolon - statement. Configuration options:

  • allow_linebreak (bool): whether to allow statement followed by linebreak; defaults to false
 <?php
-$foo = new    Foo();
-echo$foo->__toString();
+$foo = new Foo();
+echo $foo->__toString();

SingleSpaceBeforeStatementFixer

Single space must precede - not preceded by linebreak - statement.

 <?php
-$foo =new Foo();
+$foo = new Foo();

Contributing

Request feature or report bug by creating issue.

Alternatively, fork the repo, develop your changes, regenerate README.md:

./dev-tools/readme > README.md

make sure all checks pass:

composer verify

and submit pull request.

You can’t perform that action at this time.