rector/docs/rector_rules_overview.md

All 510 Rectors Overview

• Projects
• General

---

Projects

• Architecture (4)
• Autodiscovery (4)
• CakePHP (6)
• Celebrity (3)
• CodeQuality (54)
• CodingStyle (32)
• DeadCode (40)
• Doctrine (16)
• DoctrineCodeQuality (2)
• DoctrineGedmoToKnplabs (7)
• DynamicTypeAnalysis (3)
• FileSystemRector (1)
• Guzzle (1)
• JMS (2)
• Laravel (6)
• Legacy (2)
• MockistaToMockery (2)
• MysqlToMysqli (4)
• Naming (1)
• Nette (12)
• NetteKdyby (4)
• NetteTesterToPHPUnit (3)
• NetteToSymfony (9)
• NetteUtilsCodeQuality (1)
• Order (4)
• PHPOffice (14)
• PHPStan (3)
• PHPUnit (37)
• PHPUnitSymfony (1)
• PSR4 (3)
• Performance (1)
• Phalcon (4)
• Php52 (2)
• Php53 (4)
• Php54 (2)
• Php55 (2)
• Php56 (2)
• Php70 (18)
• Php71 (9)
• Php72 (11)
• Php73 (10)
• Php74 (15)
• Php80 (10)
• PhpDeglobalize (1)
• PhpSpecToPHPUnit (7)
• Polyfill (2)
• Privatization (7)
• RemovingStatic (4)
• Renaming (10)
• Restoration (7)
• SOLID (12)
• Sensio (3)
• StrictCodeQuality (1)
• Symfony (29)
• SymfonyCodeQuality (1)
• SymfonyPHPUnit (1)
• Twig (1)
• TypeDeclaration (9)

Architecture

ConstructorInjectionToActionInjectionRector

• class: Rector\Architecture\Rector\Class_\ConstructorInjectionToActionInjectionRector
• test fixtures

Move constructor injection dependency in Controller to action injection

 final class SomeController
 {
-    /**
-     * @var ProductRepository
-     */
-    private $productRepository;
-
-    public function __construct(ProductRepository $productRepository)
+    public function default(ProductRepository $productRepository)
     {
-        $this->productRepository = $productRepository;
-    }
-
-    public function default()
-    {
-        $products = $this->productRepository->fetchAll();
+        $products = $productRepository->fetchAll();
     }
 }

MoveRepositoryFromParentToConstructorRector

• class: Rector\Architecture\Rector\Class_\MoveRepositoryFromParentToConstructorRector

Turns parent EntityRepository class to constructor dependency

 namespace App\Repository;

+use App\Entity\Post;
 use Doctrine\ORM\EntityRepository;

-final class PostRepository extends EntityRepository
+final class PostRepository
 {
+    /**
+     * @var \Doctrine\ORM\EntityRepository
+     */
+    private $repository;
+    public function __construct(\Doctrine\ORM\EntityManager $entityManager)
+    {
+        $this->repository = $entityManager->getRepository(\App\Entity\Post::class);
+    }
 }

ReplaceParentRepositoryCallsByRepositoryPropertyRector

• class: Rector\Architecture\Rector\MethodCall\ReplaceParentRepositoryCallsByRepositoryPropertyRector

Handles method calls in child of Doctrine EntityRepository and moves them to "$this->repository" property.

 <?php

 use Doctrine\ORM\EntityRepository;

 class SomeRepository extends EntityRepository
 {
     public function someMethod()
     {
-        return $this->findAll();
+        return $this->repository->findAll();
     }
 }

ServiceLocatorToDIRector

• class: Rector\Architecture\Rector\MethodCall\ServiceLocatorToDIRector

Turns "$this->getRepository()" in Symfony Controller to constructor injection and private property access.

 class ProductController extends Controller
 {
+    /**
+     * @var ProductRepository
+     */
+    private $productRepository;
+
+    public function __construct(ProductRepository $productRepository)
+    {
+        $this->productRepository = $productRepository;
+    }
+
     public function someAction()
     {
         $entityManager = $this->getDoctrine()->getManager();
-        $entityManager->getRepository('SomethingBundle:Product')->findSomething(...);
+        $this->productRepository->findSomething(...);
     }
 }

Autodiscovery

MoveEntitiesToEntityDirectoryRector

• class: Rector\Autodiscovery\Rector\FileSystem\MoveEntitiesToEntityDirectoryRector

Move entities to Entity namespace

-// file: app/Controller/Product.php
+// file: app/Entity/Product.php

-namespace App\Controller;
+namespace App\Entity;

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class Product
 {
 }

MoveInterfacesToContractNamespaceDirectoryRector

• class: Rector\Autodiscovery\Rector\FileSystem\MoveInterfacesToContractNamespaceDirectoryRector

Move interface to "Contract" namespace

-// file: app/Exception/Rule.php
+// file: app/Contract/Rule.php

-namespace App\Exception;
+namespace App\Contract;

 interface Rule
 {
 }

MoveServicesBySuffixToDirectoryRector

• class: Rector\Autodiscovery\Rector\FileSystem\MoveServicesBySuffixToDirectoryRector

Move classes by their suffix to their own group/directory

services:
    Rector\Autodiscovery\Rector\FileSystem\MoveServicesBySuffixToDirectoryRector:
        $groupNamesBySuffix:
            - Repository

↓

-// file: app/Entity/ProductRepository.php
+// file: app/Repository/ProductRepository.php

-namespace App/Entity;
+namespace App/Repository;

 class ProductRepository
 {
 }

MoveValueObjectsToValueObjectDirectoryRector

• class: Rector\Autodiscovery\Rector\FileSystem\MoveValueObjectsToValueObjectDirectoryRector

Move value object to ValueObject namespace/directory

-// app/Exception/Name.php
+// app/ValueObject/Name.php
 class Name
 {
     private $name;

     public function __construct(string $name)
     {
         $this->name = $name;
     }

     public function getName()
     {
         return $this->name;
     }
 }

CakePHP

AppUsesStaticCallToUseStatementRector

• class: Rector\CakePHP\Rector\StaticCall\AppUsesStaticCallToUseStatementRector
• test fixtures

Change App::uses() to use imports

-App::uses('NotificationListener', 'Event');
+use Event\NotificationListener;

 CakeEventManager::instance()->attach(new NotificationListener());

ArrayToFluentCallRector

• class: Rector\CakePHP\Rector\MethodCall\ArrayToFluentCallRector
• test fixtures

Moves array options to fluent setter method calls.

 class ArticlesTable extends \Cake\ORM\Table
 {
     public function initialize(array $config)
     {
-        $this->belongsTo('Authors', [
-            'foreignKey' => 'author_id',
-            'propertyName' => 'person'
-        ]);
+        $this->belongsTo('Authors')
+            ->setForeignKey('author_id')
+            ->setProperty('person');
     }
 }

ChangeSnakedFixtureNameToCamelRector

• class: Rector\CakePHP\Rector\Name\ChangeSnakedFixtureNameToCamelRector

Changes $fixtues style from snake_case to CamelCase.

 class SomeTest
 {
     protected $fixtures = [
-        'app.posts',
-        'app.users',
-        'some_plugin.posts/special_posts',
+        'app.Posts',
+        'app.Users',
+        'some_plugin.Posts/SpeectialPosts',
     ];

ImplicitShortClassNameUseStatementRector

• class: Rector\CakePHP\Rector\Name\ImplicitShortClassNameUseStatementRector
• test fixtures

Collect implicit class names and add imports

 use App\Foo\Plugin;
+use Cake\TestSuite\Fixture\TestFixture;

 class LocationsFixture extends TestFixture implements Plugin
 {
 }

ModalToGetSetRector

• class: Rector\CakePHP\Rector\MethodCall\ModalToGetSetRector
• test fixtures

Changes combined set/get value() to specific getValue() or setValue(x).

 $object = new InstanceConfigTrait;

-$config = $object->config();
-$config = $object->config('key');
+$config = $object->getConfig();
+$config = $object->getConfig('key');

-$object->config('key', 'value');
-$object->config(['key' => 'value']);
+$object->setConfig('key', 'value');
+$object->setConfig(['key' => 'value']);

RenameMethodCallBasedOnParameterRector

• class: Rector\CakePHP\Rector\MethodCall\RenameMethodCallBasedOnParameterRector
• test fixtures

Changes method calls based on matching the first parameter value.

services:
    Rector\CakePHP\Rector\MethodCall\RenameMethodCallBasedOnParameterRector:
        $methodNamesByTypes:
            getParam:
                match_parameter: paging
                replace_with: getAttribute
            withParam:
                match_parameter: paging
                replace_with: withAttribute

↓

 $object = new ServerRequest();

-$config = $object->getParam('paging');
-$object = $object->withParam('paging', ['a value']);
+$config = $object->getAttribute('paging');
+$object = $object->withAttribute('paging', ['a value']);

Celebrity

CommonNotEqualRector

• class: Rector\Celebrity\Rector\NotEqual\CommonNotEqualRector
• test fixtures

Use common != instead of less known <> with same meaning

 final class SomeClass
 {
     public function run($one, $two)
     {
-        return $one <> $two;
+        return $one != $two;
     }
 }

LogicalToBooleanRector

• class: Rector\Celebrity\Rector\BooleanOp\LogicalToBooleanRector
• test fixtures

Change OR, AND to ||, && with more common understanding

-if ($f = false or true) {
+if (($f = false) || true) {
     return $f;
 }

SetTypeToCastRector

• class: Rector\Celebrity\Rector\FuncCall\SetTypeToCastRector
• test fixtures

Changes settype() to (type) where possible

 class SomeClass
 {
-    public function run($foo)
+    public function run(array $items)
     {
-        settype($foo, 'string');
+        $foo = (string) $foo;

-        return settype($foo, 'integer');
+        return (int) $foo;
     }
 }

CodeQuality

AbsolutizeRequireAndIncludePathRector

• class: Rector\CodeQuality\Rector\Include_\AbsolutizeRequireAndIncludePathRector
• test fixtures

include/require to absolute path. This Rector might introduce backwards incompatible code, when the include/require beeing changed depends on the current working directory.

 class SomeClass
 {
     public function run()
     {
-        require 'autoload.php';
+        require __DIR__ . '/autoload.php';

         require $variable;
     }
 }

AddPregQuoteDelimiterRector

• class: Rector\CodeQuality\Rector\FuncCall\AddPregQuoteDelimiterRector
• test fixtures

Add preg_quote delimiter when missing

-'#' . preg_quote('name') . '#';
+'#' . preg_quote('name', '#') . '#';

AndAssignsToSeparateLinesRector

• class: Rector\CodeQuality\Rector\LogicalAnd\AndAssignsToSeparateLinesRector
• test fixtures

Split 2 assigns ands to separate line

 class SomeClass
 {
     public function run()
     {
         $tokens = [];
-        $token = 4 and $tokens[] = $token;
+        $token = 4;
+        $tokens[] = $token;
     }
 }

ArrayKeyExistsTernaryThenValueToCoalescingRector

• class: Rector\CodeQuality\Rector\Ternary\ArrayKeyExistsTernaryThenValueToCoalescingRector
• test fixtures

Change array_key_exists() ternary to coalesing

 class SomeClass
 {
     public function run($values, $keyToMatch)
     {
-        $result = array_key_exists($keyToMatch, $values) ? $values[$keyToMatch] : null;
+        $result = $values[$keyToMatch] ?? null;
     }
 }

ArrayKeysAndInArrayToArrayKeyExistsRector

• class: Rector\CodeQuality\Rector\FuncCall\ArrayKeysAndInArrayToArrayKeyExistsRector
• test fixtures

Replace array_keys() and in_array() to array_key_exists()

 class SomeClass
 {
     public function run($packageName, $values)
     {
-        $keys = array_keys($values);
-        return in_array($packageName, $keys, true);
+        return array_key_exists($packageName, $values);
     }
 }

ArrayMergeOfNonArraysToSimpleArrayRector

• class: Rector\CodeQuality\Rector\FuncCall\ArrayMergeOfNonArraysToSimpleArrayRector
• test fixtures

Change array_merge of non arrays to array directly

 class SomeClass
 {
     public function go()
     {
         $value = 5;
         $value2 = 10;

-        return array_merge([$value], [$value2]);
+        return [$value, $value2];
     }
 }

ArrayThisCallToThisMethodCallRector

• class: Rector\CodeQuality\Rector\Array_\ArrayThisCallToThisMethodCallRector
• test fixtures

Change [$this, someMethod] without any args to $this->someMethod()

 class SomeClass
 {
     public function run()
     {
-        $values = [$this, 'giveMeMore'];
+        $values = $this->giveMeMore();
     }

     public function giveMeMore()
     {
         return 'more';
     }
 }

BooleanNotIdenticalToNotIdenticalRector

• class: Rector\CodeQuality\Rector\Identical\BooleanNotIdenticalToNotIdenticalRector
• test fixtures

Negated identical boolean compare to not identical compare (does not apply to non-bool values)

 class SomeClass
 {
     public function run()
     {
         $a = true;
         $b = false;

-        var_dump(! $a === $b); // true
-        var_dump(! ($a === $b)); // true
+        var_dump($a !== $b); // true
+        var_dump($a !== $b); // true
         var_dump($a !== $b); // true
     }
 }

CallableThisArrayToAnonymousFunctionRector

• class: Rector\CodeQuality\Rector\Array_\CallableThisArrayToAnonymousFunctionRector
• test fixtures

Convert [$this, "method"] to proper anonymous function

 class SomeClass
 {
     public function run()
     {
         $values = [1, 5, 3];
-        usort($values, [$this, 'compareSize']);
+        usort($values, function ($first, $second) {
+            return $this->compareSize($first, $second);
+        });

         return $values;
     }

     private function compareSize($first, $second)
     {
         return $first <=> $second;
     }
 }

ChangeArrayPushToArrayAssignRector

• class: Rector\CodeQuality\Rector\FuncCall\ChangeArrayPushToArrayAssignRector
• test fixtures

Change array_push() to direct variable assign

 class SomeClass
 {
     public function run()
     {
         $items = [];
-        array_push($items, $item);
+        $items[] = $item;
     }
 }

CombineIfRector

• class: Rector\CodeQuality\Rector\If_\CombineIfRector
• test fixtures

Merges nested if statements

 class SomeClass {
     public function run()
     {
-        if ($cond1) {
-            if ($cond2) {
-                return 'foo';
-            }
+        if ($cond1 && $cond2) {
+            return 'foo';
         }
     }
 }

CombinedAssignRector

• class: Rector\CodeQuality\Rector\Assign\CombinedAssignRector
• test fixtures

Simplify $value = $value + 5; assignments to shorter ones

-$value = $value + 5;
+$value += 5;

CompactToVariablesRector

• class: Rector\CodeQuality\Rector\FuncCall\CompactToVariablesRector
• test fixtures

Change compact() call to own array

 class SomeClass
 {
     public function run()
     {
         $checkout = 'one';
         $form = 'two';

-        return compact('checkout', 'form');
+        return ['checkout' => $checkout, 'form' => $form];
     }
 }

CompleteDynamicPropertiesRector

• class: Rector\CodeQuality\Rector\Class_\CompleteDynamicPropertiesRector
• test fixtures

Add missing dynamic properties

 class SomeClass
 {
+    /**
+     * @var int
+     */
+    public $value;
     public function set()
     {
         $this->value = 5;
     }
 }

ConsecutiveNullCompareReturnsToNullCoalesceQueueRector

• class: Rector\CodeQuality\Rector\If_\ConsecutiveNullCompareReturnsToNullCoalesceQueueRector
• test fixtures

Change multiple null compares to ?? queue

 class SomeClass
 {
     public function run()
     {
-        if (null !== $this->orderItem) {
-            return $this->orderItem;
-        }
-
-        if (null !== $this->orderItemUnit) {
-            return $this->orderItemUnit;
-        }
-
-        return null;
+        return $this->orderItem ?? $this->orderItemUnit;
     }
 }

ExplicitBoolCompareRector

• class: Rector\CodeQuality\Rector\If_\ExplicitBoolCompareRector
• test fixtures

Make if conditions more explicit

 final class SomeController
 {
     public function run($items)
     {
-        if (!count($items)) {
+        if (count($items) === 0) {
             return 'no items';
         }
     }
 }

ForRepeatedCountToOwnVariableRector

• class: Rector\CodeQuality\Rector\For_\ForRepeatedCountToOwnVariableRector
• test fixtures

Change count() in for function to own variable

 class SomeClass
 {
     public function run($items)
     {
-        for ($i = 5; $i <= count($items); $i++) {
+        $itemsCount = count($items);
+        for ($i = 5; $i <= $itemsCount; $i++) {
             echo $items[$i];
         }
     }
 }

ForToForeachRector

• class: Rector\CodeQuality\Rector\For_\ForToForeachRector
• test fixtures

Change for() to foreach() where useful

 class SomeClass
 {
     public function run($tokens)
     {
-        for ($i = 0, $c = count($tokens); $i < $c; ++$i) {
-            if ($tokens[$i][0] === T_STRING && $tokens[$i][1] === 'fn') {
+        foreach ($tokens as $i => $token) {
+            if ($token[0] === T_STRING && $token[1] === 'fn') {
                 $previousNonSpaceToken = $this->getPreviousNonSpaceToken($tokens, $i);
                 if ($previousNonSpaceToken !== null && $previousNonSpaceToken[0] === T_OBJECT_OPERATOR) {
                     continue;
                 }
                 $tokens[$i][0] = self::T_FN;
             }
         }
     }
 }

ForeachItemsAssignToEmptyArrayToAssignRector

• class: Rector\CodeQuality\Rector\Foreach_\ForeachItemsAssignToEmptyArrayToAssignRector
• test fixtures

Change foreach() items assign to empty array to direct assign

 class SomeClass
 {
     public function run($items)
     {
         $items2 = [];
-        foreach ($items as $item) {
-             $items2[] = $item;
-        }
+        $items2 = $items;
     }
 }

ForeachToInArrayRector

• class: Rector\CodeQuality\Rector\Foreach_\ForeachToInArrayRector
• test fixtures

Simplify foreach loops into in_array when possible

-foreach ($items as $item) {
-    if ($item === "something") {
-        return true;
-    }
-}
-
-return false;
+in_array("something", $items, true);

GetClassToInstanceOfRector

• class: Rector\CodeQuality\Rector\Identical\GetClassToInstanceOfRector
• test fixtures

Changes comparison with get_class to instanceof

-if (EventsListener::class === get_class($event->job)) { }
+if ($event->job instanceof EventsListener) { }

InArrayAndArrayKeysToArrayKeyExistsRector

• class: Rector\CodeQuality\Rector\FuncCall\InArrayAndArrayKeysToArrayKeyExistsRector
• test fixtures

Simplify in_array and array_keys functions combination into array_key_exists when array_keys has one argument only

-in_array("key", array_keys($array), true);
+array_key_exists("key", $array);

InlineIfToExplicitIfRector

• class: Rector\CodeQuality\Rector\BinaryOp\InlineIfToExplicitIfRector
• test fixtures

Change inline if to explicit if

 class SomeClass
 {
     public function run()
     {
         $userId = null;

-        is_null($userId) && $userId = 5;
+        if (is_null($userId)) {
+            $userId = 5;
+        }
     }
 }

IntvalToTypeCastRector

• class: Rector\CodeQuality\Rector\FuncCall\IntvalToTypeCastRector
• test fixtures

Change intval() to faster and readable (int) $value

 class SomeClass
 {
     public function run($value)
     {
-        return intval($value);
+        return (int) $value;
     }
 }

IsAWithStringWithThirdArgumentRector

• class: Rector\CodeQuality\Rector\FuncCall\IsAWithStringWithThirdArgumentRector
• test fixtures

Complete missing 3rd argument in case is_a() function in case of strings

 class SomeClass
 {
     public function __construct(string $value)
     {
-        return is_a($value, 'stdClass');
+        return is_a($value, 'stdClass', true);
     }
 }

JoinStringConcatRector

• class: Rector\CodeQuality\Rector\Concat\JoinStringConcatRector
• test fixtures

Joins concat of 2 strings

 class SomeClass
 {
     public function run()
     {
-        $name = 'Hi' . ' Tom';
+        $name = 'Hi Tom';
     }
 }

RemoveAlwaysTrueConditionSetInConstructorRector

• class: Rector\CodeQuality\Rector\If_\RemoveAlwaysTrueConditionSetInConstructorRector
• test fixtures

If conditions is always true, perform the content right away

 final class SomeClass
 {
     private $value;

     public function __construct($value)
     {
         $this->value = $value;
     }

     public function go()
     {
-        if ($this->value) {
-            return 'yes';
-        }
+        return 'yes';
     }
 }

RemoveSoleValueSprintfRector

• class: Rector\CodeQuality\Rector\FuncCall\RemoveSoleValueSprintfRector
• test fixtures

Remove sprintf() wrapper if not needed

 class SomeClass
 {
     public function run()
     {
-        $value = sprintf('%s', 'hi');
+        $value = 'hi';

         $welcome = 'hello';
-        $value = sprintf('%s', $welcome);
+        $value = $welcome;
     }
 }

ShortenElseIfRector

• class: Rector\CodeQuality\Rector\If_\ShortenElseIfRector
• test fixtures

Shortens else/if to elseif

 class SomeClass
 {
     public function run()
     {
         if ($cond1) {
             return $action1;
-        } else {
-            if ($cond2) {
-                return $action2;
-            }
+        } elseif ($cond2) {
+            return $action2;
         }
     }
 }

SimplifyArraySearchRector

• class: Rector\CodeQuality\Rector\Identical\SimplifyArraySearchRector
• test fixtures

Simplify array_search to in_array

-array_search("searching", $array) !== false;
+in_array("searching", $array);

-array_search("searching", $array, true) !== false;
+in_array("searching", $array, true);

SimplifyBoolIdenticalTrueRector

• class: Rector\CodeQuality\Rector\Identical\SimplifyBoolIdenticalTrueRector
• test fixtures

Symplify bool value compare to true or false

 class SomeClass
 {
     public function run(bool $value, string $items)
     {
-         $match = in_array($value, $items, TRUE) === TRUE;
-         $match = in_array($value, $items, TRUE) !== FALSE;
+         $match = in_array($value, $items, TRUE);
+         $match = in_array($value, $items, TRUE);
     }
 }

SimplifyConditionsRector

• class: Rector\CodeQuality\Rector\Identical\SimplifyConditionsRector
• test fixtures

Simplify conditions

-if (! ($foo !== 'bar')) {...
+if ($foo === 'bar') {...

SimplifyDeMorganBinaryRector

• class: Rector\CodeQuality\Rector\BinaryOp\SimplifyDeMorganBinaryRector
• test fixtures

Simplify negated conditions with de Morgan theorem

 <?php

 $a = 5;
 $b = 10;
-$result = !($a > 20 || $b <= 50);
+$result = $a <= 20 && $b > 50;

SimplifyDuplicatedTernaryRector

• class: Rector\CodeQuality\Rector\Ternary\SimplifyDuplicatedTernaryRector
• test fixtures

Remove ternary that duplicated return value of true : false

 class SomeClass
 {
     public function run(bool $value, string $name)
     {
-         $isTrue = $value ? true : false;
+         $isTrue = $value;
          $isName = $name ? true : false;
     }
 }

SimplifyEmptyArrayCheckRector

• class: Rector\CodeQuality\Rector\BooleanAnd\SimplifyEmptyArrayCheckRector
• test fixtures

Simplify is_array and empty functions combination into a simple identical check for an empty array

-is_array($values) && empty($values)
+$values === []

SimplifyForeachToArrayFilterRector

• class: Rector\CodeQuality\Rector\Foreach_\SimplifyForeachToArrayFilterRector
• test fixtures

Simplify foreach with function filtering to array filter

-$directories = [];
 $possibleDirectories = [];
-foreach ($possibleDirectories as $possibleDirectory) {
-    if (file_exists($possibleDirectory)) {
-        $directories[] = $possibleDirectory;
-    }
-}
+$directories = array_filter($possibleDirectories, 'file_exists');

SimplifyForeachToCoalescingRector

• class: Rector\CodeQuality\Rector\Foreach_\SimplifyForeachToCoalescingRector
• test fixtures

Changes foreach that returns set value to ??

-foreach ($this->oldToNewFunctions as $oldFunction => $newFunction) {
-    if ($currentFunction === $oldFunction) {
-        return $newFunction;
-    }
-}
-
-return null;
+return $this->oldToNewFunctions[$currentFunction] ?? null;

SimplifyFuncGetArgsCountRector

• class: Rector\CodeQuality\Rector\FuncCall\SimplifyFuncGetArgsCountRector
• test fixtures

Simplify count of func_get_args() to func_num_args()

-count(func_get_args());
+func_num_args();

SimplifyIfElseToTernaryRector

• class: Rector\CodeQuality\Rector\If_\SimplifyIfElseToTernaryRector
• test fixtures

Changes if/else for same value as assign to ternary

 class SomeClass
 {
     public function run()
     {
-        if (empty($value)) {
-            $this->arrayBuilt[][$key] = true;
-        } else {
-            $this->arrayBuilt[][$key] = $value;
-        }
+        $this->arrayBuilt[][$key] = empty($value) ? true : $value;
     }
 }

SimplifyIfIssetToNullCoalescingRector

• class: Rector\CodeQuality\Rector\If_\SimplifyIfIssetToNullCoalescingRector
• test fixtures

Simplify binary if to null coalesce

 final class SomeController
 {
     public function run($possibleStatieYamlFile)
     {
-        if (isset($possibleStatieYamlFile['import'])) {
-            $possibleStatieYamlFile['import'] = array_merge($possibleStatieYamlFile['import'], $filesToImport);
-        } else {
-            $possibleStatieYamlFile['import'] = $filesToImport;
-        }
+        $possibleStatieYamlFile['import'] = array_merge($possibleStatieYamlFile['import'] ?? [], $filesToImport);
     }
 }

SimplifyIfNotNullReturnRector

• class: Rector\CodeQuality\Rector\If_\SimplifyIfNotNullReturnRector
• test fixtures

Changes redundant null check to instant return

 $newNode = 'something ;
-if ($newNode !== null) {
-    return $newNode;
-}
-
-return null;
+return $newNode;

SimplifyIfReturnBoolRector

• class: Rector\CodeQuality\Rector\If_\SimplifyIfReturnBoolRector
• test fixtures

Shortens if return false/true to direct return

-if (strpos($docToken->getContent(), "\n") === false) {
-    return true;
-}
-
-return false;
+return strpos($docToken->getContent(), "\n") === false;

SimplifyInArrayValuesRector

• class: Rector\CodeQuality\Rector\FuncCall\SimplifyInArrayValuesRector
• test fixtures

Removes unneeded array_values() in in_array() call

-in_array("key", array_values($array), true);
+in_array("key", $array, true);

SimplifyRegexPatternRector

• class: Rector\CodeQuality\Rector\FuncCall\SimplifyRegexPatternRector
• test fixtures

Simplify regex pattern to known ranges

 class SomeClass
 {
     public function run($value)
     {
-        preg_match('#[a-zA-Z0-9+]#', $value);
+        preg_match('#[\w\d+]#', $value);
     }
 }

SimplifyStrposLowerRector

• class: Rector\CodeQuality\Rector\FuncCall\SimplifyStrposLowerRector
• test fixtures

Simplify strpos(strtolower(), "...") calls

-strpos(strtolower($var), "...")"
+stripos($var, "...")"

SimplifyTautologyTernaryRector

• class: Rector\CodeQuality\Rector\Ternary\SimplifyTautologyTernaryRector
• test fixtures

Simplify tautology ternary to value

-$value = ($fullyQualifiedTypeHint !== $typeHint) ? $fullyQualifiedTypeHint : $typeHint;
+$value = $fullyQualifiedTypeHint;

SimplifyUselessVariableRector

• class: Rector\CodeQuality\Rector\Return_\SimplifyUselessVariableRector
• test fixtures

Removes useless variable assigns

 function () {
-    $a = true;
-    return $a;
+    return true;
 };

SingleInArrayToCompareRector

• class: Rector\CodeQuality\Rector\FuncCall\SingleInArrayToCompareRector
• test fixtures

Changes in_array() with single element to ===

 class SomeClass
 {
     public function run()
     {
-        if (in_array(strtolower($type), ['$this'], true)) {
+        if (strtolower($type) === '$this') {
             return strtolower($type);
         }
     }
 }

SplitListAssignToSeparateLineRector

• class: Rector\CodeQuality\Rector\Assign\SplitListAssignToSeparateLineRector
• test fixtures

Splits [$a, $b] = [5, 10] scalar assign to standalone lines

 final class SomeClass
 {
     public function run(): void
     {
-        [$a, $b] = [1, 2];
+        $a = 1;
+        $b = 2;
     }
 }

StrlenZeroToIdenticalEmptyStringRector

• class: Rector\CodeQuality\Rector\FuncCall\StrlenZeroToIdenticalEmptyStringRector
• test fixtures

Changes strlen comparison to 0 to direct empty string compare

 class SomeClass
 {
     public function run($value)
     {
-        $empty = strlen($value) === 0;
+        $empty = $value === '';
     }
 }

ThrowWithPreviousExceptionRector

• class: Rector\CodeQuality\Rector\Catch_\ThrowWithPreviousExceptionRector
• test fixtures

When throwing into a catch block, checks that the previous exception is passed to the new throw clause

 class SomeClass
 {
     public function run()
     {
         try {
             $someCode = 1;
         } catch (Throwable $throwable) {
-            throw new AnotherException('ups');
+            throw new AnotherException('ups', $throwable->getCode(), $throwable);
         }
     }
 }

UnnecessaryTernaryExpressionRector

• class: Rector\CodeQuality\Rector\Ternary\UnnecessaryTernaryExpressionRector
• test fixtures

Remove unnecessary ternary expressions.

-$foo === $bar ? true : false;
+$foo === $bar;

UnusedForeachValueToArrayKeysRector

• class: Rector\CodeQuality\Rector\Foreach_\UnusedForeachValueToArrayKeysRector
• test fixtures

Change foreach with unused $value but only $key, to array_keys()

 class SomeClass
 {
     public function run()
     {
         $items = [];
-        foreach ($values as $key => $value) {
+        foreach (array_keys($values) as $key) {
             $items[$key] = null;
         }
     }
 }

UseIdenticalOverEqualWithSameTypeRector

• class: Rector\CodeQuality\Rector\Equal\UseIdenticalOverEqualWithSameTypeRector
• test fixtures

Use ===/!== over ==/!=, it values have the same type

 class SomeClass
 {
     public function run(int $firstValue, int $secondValue)
     {
-         $isSame = $firstValue == $secondValue;
-         $isDiffernt = $firstValue != $secondValue;
+         $isSame = $firstValue === $secondValue;
+         $isDiffernt = $firstValue !== $secondValue;
     }
 }

CodingStyle

AddArrayDefaultToArrayPropertyRector

• class: Rector\CodingStyle\Rector\Class_\AddArrayDefaultToArrayPropertyRector
• test fixtures

Adds array default value to property to prevent foreach over null error

 class SomeClass
 {
     /**
      * @var int[]
      */
-    private $values;
+    private $values = [];

     public function isEmpty()
     {
-        return $this->values === null;
+        return $this->values === [];
     }
 }

AnnotateThrowablesRector

• class: Rector\CodingStyle\Rector\Throw_\AnnotateThrowablesRector
• test fixtures

Adds @throws DocBlock comments to methods that thrwo \Throwables.

 class RootExceptionInMethodWithDocblock
 {
     /**
      * This is a comment.
      *
      * @param int $code
+     * @throws \RuntimeException
      */
     public function throwException(int $code)
     {
         throw new \RuntimeException('', $code);
     }
 }

BinarySwitchToIfElseRector

• class: Rector\CodingStyle\Rector\Switch_\BinarySwitchToIfElseRector
• test fixtures

Changes switch with 2 options to if-else

-switch ($foo) {
-    case 'my string':
-        $result = 'ok';
-    break;
-
-    default:
-        $result = 'not ok';
+if ($foo == 'my string') {
+    $result = 'ok;
+} else {
+    $result = 'not ok';
 }

CallUserFuncCallToVariadicRector

• class: Rector\CodingStyle\Rector\FuncCall\CallUserFuncCallToVariadicRector
• test fixtures

Replace call_user_func_call with variadic

 class SomeClass
 {
     public function run()
     {
-        call_user_func_array('some_function', $items);
+        some_function(...$items);
     }
 }

CamelCaseFunctionNamingToUnderscoreRector

• class: Rector\CodingStyle\Rector\Function_\CamelCaseFunctionNamingToUnderscoreRector
• test fixtures

Change CamelCase naming of functions to under_score naming

-function someCamelCaseFunction()
+function some_camel_case_function()
 {
 }

-someCamelCaseFunction();
+some_camel_case_function();

CatchExceptionNameMatchingTypeRector

• class: Rector\CodingStyle\Rector\Catch_\CatchExceptionNameMatchingTypeRector
• test fixtures

Type and name of catch exception should match

 class SomeClass
 {
     public function run()
     {
         try {
             // ...
-        } catch (SomeException $typoException) {
-            $typoException->getMessage();
+        } catch (SomeException $someException) {
+            $someException->getMessage();
         }
     }
 }

ConsistentImplodeRector

• class: Rector\CodingStyle\Rector\FuncCall\ConsistentImplodeRector
• test fixtures

Changes various implode forms to consistent one

 class SomeClass
 {
     public function run(array $items)
     {
-        $itemsAsStrings = implode($items);
-        $itemsAsStrings = implode($items, '|');
+        $itemsAsStrings = implode('', $items);
+        $itemsAsStrings = implode('|', $items);

         $itemsAsStrings = implode('|', $items);
     }
 }

ConsistentPregDelimiterRector

• class: Rector\CodingStyle\Rector\FuncCall\ConsistentPregDelimiterRector
• test fixtures

Replace PREG delimiter with configured one

 class SomeClass
 {
     public function run()
     {
-        preg_match('~value~', $value);
-        preg_match_all('~value~im', $value);
+        preg_match('#value#', $value);
+        preg_match_all('#value#im', $value);
     }
 }

EncapsedStringsToSprintfRector

• class: Rector\CodingStyle\Rector\Encapsed\EncapsedStringsToSprintfRector
• test fixtures

Convert enscaped {$string} to more readable sprintf

 final class SomeClass
 {
     public function run(string $format)
     {
-        return "Unsupported format {$format}";
+        return sprintf('Unsupported format %s', $format);
     }
 }

FollowRequireByDirRector

• class: Rector\CodingStyle\Rector\Include_\FollowRequireByDirRector
• test fixtures

include/require should be followed by absolute path

 class SomeClass
 {
     public function run()
     {
-        require 'autoload.php';
+        require __DIR__ . '/autoload.php';
     }
 }

FunctionCallToConstantRector

• class: Rector\CodingStyle\Rector\FuncCall\FunctionCallToConstantRector
• test fixtures

Changes use of function calls to use constants

 class SomeClass
 {
     public function run()
     {
-        $value = php_sapi_name();
+        $value = PHP_SAPI;
     }
 }

 class SomeClass
 {
     public function run()
     {
-        $value = pi();
+        $value = M_PI;
     }
 }

IdenticalFalseToBooleanNotRector

• class: Rector\CodingStyle\Rector\Identical\IdenticalFalseToBooleanNotRector
• test fixtures

Changes === false to negate !

-if ($something === false) {}
+if (! $something) {}

MakeInheritedMethodVisibilitySameAsParentRector

• class: Rector\CodingStyle\Rector\ClassMethod\MakeInheritedMethodVisibilitySameAsParentRector
• test fixtures

Make method visibility same as parent one

 class ChildClass extends ParentClass
 {
-    public function run()
+    protected function run()
     {
     }
 }

 class ParentClass
 {
     protected function run()
     {
     }
 }

ManualJsonStringToJsonEncodeArrayRector

• class: Rector\CodingStyle\Rector\String_\ManualJsonStringToJsonEncodeArrayRector
• test fixtures

Add extra space before new assign set

 final class SomeClass
 {
     public function run()
     {
-        $someJsonAsString = '{"role_name":"admin","numberz":{"id":"10"}}';
+        $data = [
+            'role_name' => 'admin',
+            'numberz' => ['id' => 10]
+        ];
+
+        $someJsonAsString = Nette\Utils\Json::encode($data);
     }
 }

NewlineBeforeNewAssignSetRector

• class: Rector\CodingStyle\Rector\ClassMethod\NewlineBeforeNewAssignSetRector
• test fixtures

Add extra space before new assign set

 final class SomeClass
 {
     public function run()
     {
         $value = new Value;
         $value->setValue(5);
+
         $value2 = new Value;
         $value2->setValue(1);
     }
 }

NullableCompareToNullRector

• class: Rector\CodingStyle\Rector\If_\NullableCompareToNullRector
• test fixtures

Changes negate of empty comparison of nullable value to explicit === or !== compare

 /** @var stdClass|null $value */
-if ($value) {
+if ($value !== null) {
 }

-if (!$value) {
+if ($value === null) {
 }

PreferThisOrSelfMethodCallRector

• class: Rector\CodingStyle\Rector\MethodCall\PreferThisOrSelfMethodCallRector
• test fixtures

Changes $this->... to self:: or vise versa for specific types

services:
    Rector\CodingStyle\Rector\MethodCall\PreferThisOrSelfMethodCallRector:
        PHPUnit\TestCase: self

↓

 class SomeClass extends PHPUnit\TestCase
 {
     public function run()
     {
-        $this->assertThis();
+        self::assertThis();
     }
 }

RemoveDoubleUnderscoreInMethodNameRector

• class: Rector\CodingStyle\Rector\ClassMethod\RemoveDoubleUnderscoreInMethodNameRector
• test fixtures

Non-magic PHP object methods cannot start with "__"

 class SomeClass
 {
-    public function __getName($anotherObject)
+    public function getName($anotherObject)
     {
-        $anotherObject->__getSurname();
+        $anotherObject->getSurname();
     }
 }

RemoveUnusedAliasRector

• class: Rector\CodingStyle\Rector\Use_\RemoveUnusedAliasRector
• test fixtures

Removes unused use aliases. Keep annotation aliases like "Doctrine\ORM\Mapping as ORM" to keep convention format

-use Symfony\Kernel as BaseKernel;
+use Symfony\Kernel;

-class SomeClass extends BaseKernel
+class SomeClass extends Kernel
 {
 }

ReturnArrayClassMethodToYieldRector

• class: Rector\CodingStyle\Rector\ClassMethod\ReturnArrayClassMethodToYieldRector
• test fixtures

Turns array return to yield return in specific type and method

services:
    Rector\CodingStyle\Rector\ClassMethod\ReturnArrayClassMethodToYieldRector:
        EventSubscriberInterface:
            - getSubscribedEvents

↓

 class SomeEventSubscriber implements EventSubscriberInterface
 {
     public static function getSubscribedEvents()
     {
-        return ['event' => 'callback'];
+        yield 'event' => 'callback';
     }
 }

SimpleArrayCallableToStringRector

• class: Rector\CodingStyle\Rector\FuncCall\SimpleArrayCallableToStringRector
• test fixtures

Changes redundant anonymous bool functions to simple calls

-$paths = array_filter($paths, function ($path): bool {
-    return is_dir($path);
-});
+array_filter($paths, "is_dir");

SplitDoubleAssignRector

• class: Rector\CodingStyle\Rector\Assign\SplitDoubleAssignRector
• test fixtures

Split multiple inline assigns to each own lines default value, to prevent undefined array issues

 class SomeClass
 {
     public function run()
     {
-        $one = $two = 1;
+        $one = 1;
+        $two = 1;
     }
 }

SplitGroupedConstantsAndPropertiesRector

• class: Rector\CodingStyle\Rector\ClassConst\SplitGroupedConstantsAndPropertiesRector
• test fixtures

Separate constant and properties to own lines

 class SomeClass
 {
-    const HI = true, AHOJ = 'true';
+    const HI = true;
+    const AHOJ = 'true';

     /**
      * @var string
      */
-    public $isIt, $isIsThough;
+    public $isIt;
+
+    /**
+     * @var string
+     */
+    public $isIsThough;
 }

SplitGroupedUseImportsRector

• class: Rector\CodingStyle\Rector\Use_\SplitGroupedUseImportsRector
• test fixtures

Split grouped use imports and trait statements to standalone lines

-use A, B;
+use A;
+use B;

 class SomeClass
 {
-    use SomeTrait, AnotherTrait;
+    use SomeTrait;
+    use AnotherTrait;
 }

SplitStringClassConstantToClassConstFetchRector

• class: Rector\CodingStyle\Rector\String_\SplitStringClassConstantToClassConstFetchRector
• test fixtures

Separate class constant in a string to class constant fetch and string

 class SomeClass
 {
     const HI = true;
 }

 class AnotherClass
 {
     public function get()
     {
-        return 'SomeClass::HI';
+        return SomeClass::class . '::HI';
     }
 }

StrictArraySearchRector

• class: Rector\CodingStyle\Rector\FuncCall\StrictArraySearchRector
• test fixtures

Makes array_search search for identical elements

-array_search($value, $items);
+array_search($value, $items, true);

SymplifyQuoteEscapeRector

• class: Rector\CodingStyle\Rector\String_\SymplifyQuoteEscapeRector
• test fixtures

Prefer quote that are not inside the string

 class SomeClass
 {
     public function run()
     {
-         $name = "\" Tom";
-         $name = '\' Sara';
+         $name = '" Tom';
+         $name = "' Sara";
     }
 }

UnderscoreToPascalCaseVariableAndPropertyNameRector

• class: Rector\CodingStyle\Rector\Variable\UnderscoreToPascalCaseVariableAndPropertyNameRector
• test fixtures

Change under_score names to pascalCase

 final class SomeClass
 {
-    public function run($a_b)
+    public function run($aB)
     {
-        $some_value = 5;
+        $someValue = 5;

-        $this->run($a_b);
+        $this->run($aB);
     }
 }

UseIncrementAssignRector

• class: Rector\CodingStyle\Rector\Assign\UseIncrementAssignRector
• test fixtures

Use ++ increment instead of $var += 1.

 class SomeClass
 {
     public function run()
     {
-        $style += 1;
+        ++$style
     }
 }

VarConstantCommentRector

• class: Rector\CodingStyle\Rector\ClassConst\VarConstantCommentRector
• test fixtures

Constant should have a @var comment with type

 class SomeClass
 {
+    /**
+     * @var string
+     */
     const HI = 'hi';
 }

VersionCompareFuncCallToConstantRector

• class: Rector\CodingStyle\Rector\FuncCall\VersionCompareFuncCallToConstantRector
• test fixtures

Changes use of call to version compare function to use of PHP version constant

 class SomeClass
 {
     public function run()
     {
-        version_compare(PHP_VERSION, '5.3.0', '<');
+        PHP_VERSION_ID < 50300;
     }
 }

YieldClassMethodToArrayClassMethodRector

• class: Rector\CodingStyle\Rector\ClassMethod\YieldClassMethodToArrayClassMethodRector
• test fixtures

Turns yield return to array return in specific type and method

services:
    Rector\CodingStyle\Rector\ClassMethod\YieldClassMethodToArrayClassMethodRector:
        EventSubscriberInterface:
            - getSubscribedEvents

↓

 class SomeEventSubscriber implements EventSubscriberInterface
 {
     public static function getSubscribedEvents()
     {
-        yield 'event' => 'callback';
+        return ['event' => 'callback'];
     }
 }

DeadCode

RemoveAlwaysTrueIfConditionRector

• class: Rector\DeadCode\Rector\If_\RemoveAlwaysTrueIfConditionRector
• test fixtures

Remove if condition that is always true

 final class SomeClass
 {
     public function go()
     {
-        if (1 === 1) {
-            return 'yes';
-        }
+        return 'yes';

         return 'no';
     }
 }

RemoveAndTrueRector

• class: Rector\DeadCode\Rector\BooleanAnd\RemoveAndTrueRector
• test fixtures

Remove and true that has no added value

 class SomeClass
 {
     public function run()
     {
-        return true && 5 === 1;
+        return 5 === 1;
     }
 }

RemoveAssignOfVoidReturnFunctionRector

• class: Rector\DeadCode\Rector\Assign\RemoveAssignOfVoidReturnFunctionRector
• test fixtures

Remove assign of void function/method to variable

 class SomeClass
 {
     public function run()
     {
-        $value = $this->getOne();
+        $this->getOne();
     }

     private function getOne(): void
     {
     }
 }

RemoveCodeAfterReturnRector

• class: Rector\DeadCode\Rector\FunctionLike\RemoveCodeAfterReturnRector
• test fixtures

Remove dead code after return statement

 class SomeClass
 {
     public function run(int $a)
     {
          return $a;
-         $a++;
     }
 }

RemoveConcatAutocastRector

• class: Rector\DeadCode\Rector\Concat\RemoveConcatAutocastRector
• test fixtures

Remove (string) casting when it comes to concat, that does this by default

 class SomeConcatingClass
 {
     public function run($value)
     {
-        return 'hi ' . (string) $value;
+        return 'hi ' . $value;
     }
 }

RemoveDeadConstructorRector

• class: Rector\DeadCode\Rector\ClassMethod\RemoveDeadConstructorRector
• test fixtures

Remove empty constructor

 class SomeClass
 {
-    public function __construct()
-    {
-    }
 }

RemoveDeadIfForeachForRector

• class: Rector\DeadCode\Rector\For_\RemoveDeadIfForeachForRector
• test fixtures

Remove if, foreach and for that does not do anything

 class SomeClass
 {
     public function run($someObject)
     {
         $value = 5;
-        if ($value) {
-        }
-
         if ($someObject->run()) {
-        }
-
-        foreach ($values as $value) {
         }

         return $value;
     }
 }

RemoveDeadRecursiveClassMethodRector

• class: Rector\DeadCode\Rector\ClassMethod\RemoveDeadRecursiveClassMethodRector
• test fixtures

Remove unused public method that only calls itself recursively

 class SomeClass
 {
-    public function run()
-    {
-        return $this->run();
-    }
 }

RemoveDeadReturnRector

• class: Rector\DeadCode\Rector\FunctionLike\RemoveDeadReturnRector
• test fixtures

Remove last return in the functions, since does not do anything

 class SomeClass
 {
     public function run()
     {
         $shallWeDoThis = true;

         if ($shallWeDoThis) {
             return;
         }
-
-        return;
     }
 }

RemoveDeadStmtRector

• class: Rector\DeadCode\Rector\Stmt\RemoveDeadStmtRector
• test fixtures

Removes dead code statements

-$value = 5;
-$value;
+$value = 5;

RemoveDeadTryCatchRector

• class: Rector\DeadCode\Rector\TryCatch\RemoveDeadTryCatchRector
• test fixtures

Remove dead try/catch

 class SomeClass
 {
     public function run()
     {
-        try {
-            // some code
-        }
-        catch (Throwable $throwable) {
-            throw $throwable;
-        }
+        // some code
     }
 }

RemoveDeadZeroAndOneOperationRector

• class: Rector\DeadCode\Rector\Plus\RemoveDeadZeroAndOneOperationRector
• test fixtures

Remove operation with 1 and 0, that have no effect on the value

 class SomeClass
 {
     public function run()
     {
-        $value = 5 * 1;
-        $value = 5 + 0;
+        $value = 5;
+        $value = 5;
     }
 }

RemoveDefaultArgumentValueRector

• class: Rector\DeadCode\Rector\MethodCall\RemoveDefaultArgumentValueRector
• test fixtures

Remove argument value, if it is the same as default value

 class SomeClass
 {
     public function run()
     {
-        $this->runWithDefault([]);
-        $card = self::runWithStaticDefault([]);
+        $this->runWithDefault();
+        $card = self::runWithStaticDefault();
     }

     public function runWithDefault($items = [])
     {
         return $items;
     }

     public function runStaticWithDefault($cards = [])
     {
         return $cards;
     }
 }

RemoveDelegatingParentCallRector

• class: Rector\DeadCode\Rector\ClassMethod\RemoveDelegatingParentCallRector
• test fixtures

Removed dead parent call, that does not change anything

 class SomeClass
 {
-    public function prettyPrint(array $stmts): string
-    {
-        return parent::prettyPrint($stmts);
-    }
 }

RemoveDoubleAssignRector

• class: Rector\DeadCode\Rector\Assign\RemoveDoubleAssignRector
• test fixtures

Simplify useless double assigns

-$value = 1;
 $value = 1;

RemoveDuplicatedArrayKeyRector

• class: Rector\DeadCode\Rector\Array_\RemoveDuplicatedArrayKeyRector
• test fixtures

Remove duplicated key in defined arrays.

 $item = [
-    1 => 'A',
     1 => 'B'
 ];

RemoveDuplicatedCaseInSwitchRector

• class: Rector\DeadCode\Rector\Switch_\RemoveDuplicatedCaseInSwitchRector
• test fixtures

2 following switch keys with identical will be reduced to one result

 class SomeClass
 {
     public function run()
     {
         switch ($name) {
              case 'clearHeader':
                  return $this->modifyHeader($node, 'remove');
              case 'clearAllHeaders':
-                 return $this->modifyHeader($node, 'replace');
              case 'clearRawHeaders':
                  return $this->modifyHeader($node, 'replace');
              case '...':
                  return 5;
         }
     }
 }

RemoveDuplicatedIfReturnRector

• class: Rector\DeadCode\Rector\FunctionLike\RemoveDuplicatedIfReturnRector
• test fixtures

Remove duplicated if stmt with return in function/method body

 class SomeClass
 {
     public function run($value)
     {
         if ($value) {
             return true;
         }

         $value2 = 100;
-
-        if ($value) {
-            return true;
-        }
     }
 }

RemoveDuplicatedInstanceOfRector

• class: Rector\DeadCode\Rector\Instanceof_\RemoveDuplicatedInstanceOfRector
• test fixtures

Remove duplicated instanceof in one call

 class SomeClass
 {
     public function run($value)
     {
-        $isIt = $value instanceof A || $value instanceof A;
-        $isIt = $value instanceof A && $value instanceof A;
+        $isIt = $value instanceof A;
+        $isIt = $value instanceof A;
     }
 }

RemoveEmptyClassMethodRector

• class: Rector\DeadCode\Rector\ClassMethod\RemoveEmptyClassMethodRector
• test fixtures

Remove empty method calls not required by parents

 class OrphanClass
 {
-    public function __construct()
-    {
-    }
 }

RemoveNullPropertyInitializationRector

• class: Rector\DeadCode\Rector\Property\RemoveNullPropertyInitializationRector
• test fixtures

Remove initialization with null value from property declarations

 class SunshineCommand extends ParentClassWithNewConstructor
 {
-    private $myVar = null;
+    private $myVar;
 }

RemoveOverriddenValuesRector

• class: Rector\DeadCode\Rector\ClassMethod\RemoveOverriddenValuesRector
• test fixtures

Remove initial assigns of overridden values

 final class SomeController
 {
     public function run()
     {
-         $directories = [];
          $possibleDirectories = [];
          $directories = array_filter($possibleDirectories, 'file_exists');
     }
 }

RemoveParentCallWithoutParentRector

• class: Rector\DeadCode\Rector\StaticCall\RemoveParentCallWithoutParentRector
• test fixtures

Remove unused parent call with no parent class

 class OrphanClass
 {
     public function __construct()
     {
-         parent::__construct();
     }
 }

RemoveSetterOnlyPropertyAndMethodCallRector

• class: Rector\DeadCode\Rector\Class_\RemoveSetterOnlyPropertyAndMethodCallRector
• test fixtures

Removes method that set values that are never used

 class SomeClass
 {
-    private $name;
-
-    public function setName($name)
-    {
-        $this->name = $name;
-    }
 }

 class ActiveOnlySetter
 {
     public function run()
     {
         $someClass = new SomeClass();
-        $someClass->setName('Tom');
     }
 }

RemoveUnreachableStatementRector

• class: Rector\DeadCode\Rector\Stmt\RemoveUnreachableStatementRector
• test fixtures

Remove unreachable statements

 class SomeClass
 {
     public function run()
     {
         return 5;
-
-        $removeMe = 10;
     }
 }

RemoveUnusedAssignVariableRector

• class: Rector\DeadCode\Rector\Assign\RemoveUnusedAssignVariableRector
• test fixtures

Remove assigned unused variable

 class SomeClass
 {
     public function run()
     {
-        $value = $this->process();
+        $this->process();
     }

     public function process()
     {
         // something going on
         return 5;
     }
 }

RemoveUnusedClassConstantRector

• class: Rector\DeadCode\Rector\ClassConst\RemoveUnusedClassConstantRector
• test fixtures

Remove unused class constants

 class SomeClass
 {
-    private const SOME_CONST = 'dead';
-
     public function run()
     {
     }
 }

RemoveUnusedClassesRector

• class: Rector\DeadCode\Rector\Class_\RemoveUnusedClassesRector
• test fixtures

Remove unused classes without interface

 interface SomeInterface
 {
 }

 class SomeClass implements SomeInterface
 {
     public function run($items)
     {
         return null;
     }
-}
-
-class NowhereUsedClass
-{
 }

RemoveUnusedDoctrineEntityMethodAndPropertyRector

• class: Rector\DeadCode\Rector\Class_\RemoveUnusedDoctrineEntityMethodAndPropertyRector
• test fixtures

Removes unused methods and properties from Doctrine entity classes

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class UserEntity
 {
-    /**
-     * @ORM\Column
-     */
-    private $name;
-
-    public function getName()
-    {
-        return $this->name;
-    }
-
-    public function setName($name)
-    {
-        $this->name = $name;
-    }
 }

RemoveUnusedForeachKeyRector

• class: Rector\DeadCode\Rector\Foreach_\RemoveUnusedForeachKeyRector
• test fixtures

Remove unused key in foreach

 $items = [];
-foreach ($items as $key => $value) {
+foreach ($items as $value) {
     $result = $value;
 }

RemoveUnusedFunctionRector

• class: Rector\DeadCode\Rector\Function_\RemoveUnusedFunctionRector
• test fixtures

Remove unused function

-function removeMe()
-{
-}
-
 function useMe()
 {
 }

 useMe();

RemoveUnusedNonEmptyArrayBeforeForeachRector

• class: Rector\DeadCode\Rector\If_\RemoveUnusedNonEmptyArrayBeforeForeachRector
• test fixtures

Remove unused if check to non-empty array before foreach of the array

 class SomeClass
 {
     public function run()
     {
         $values = [];
-        if ($values !== []) {
-            foreach ($values as $value) {
-                echo $value;
-            }
+        foreach ($values as $value) {
+            echo $value;
         }
     }
 }

RemoveUnusedParameterRector

• class: Rector\DeadCode\Rector\ClassMethod\RemoveUnusedParameterRector
• test fixtures

Remove unused parameter, if not required by interface or parent class

 class SomeClass
 {
-    public function __construct($value, $value2)
+    public function __construct($value)
     {
          $this->value = $value;
     }
 }

RemoveUnusedPrivateConstantRector

• class: Rector\DeadCode\Rector\ClassConst\RemoveUnusedPrivateConstantRector
• test fixtures

Remove unused private constant

 final class SomeController
 {
-    private const SOME_CONSTANT = 5;
     public function run()
     {
         return 5;
     }
 }

RemoveUnusedPrivateMethodRector

• class: Rector\DeadCode\Rector\ClassMethod\RemoveUnusedPrivateMethodRector
• test fixtures

Remove unused private method

 final class SomeController
 {
     public function run()
     {
         return 5;
     }
-
-    private function skip()
-    {
-        return 10;
-    }
 }

RemoveUnusedPrivatePropertyRector

• class: Rector\DeadCode\Rector\Property\RemoveUnusedPrivatePropertyRector
• test fixtures

Remove unused private properties

 class SomeClass
 {
-    private $property;
 }

RemoveUnusedVariableAssignRector

• class: Rector\DeadCode\Rector\Assign\RemoveUnusedVariableAssignRector
• test fixtures

Remove unused assigns to variables

 class SomeClass
 {
     public function run()
     {
-        $value = 5;
     }
 }

SimplifyIfElseWithSameContentRector

• class: Rector\DeadCode\Rector\If_\SimplifyIfElseWithSameContentRector
• test fixtures

Remove if/else if they have same content

 class SomeClass
 {
     public function run()
     {
-        if (true) {
-            return 1;
-        } else {
-            return 1;
-        }
+        return 1;
     }
 }

SimplifyMirrorAssignRector

• class: Rector\DeadCode\Rector\Expression\SimplifyMirrorAssignRector
• test fixtures

Removes unneeded $a = $a assigns

-$a = $a;

TernaryToBooleanOrFalseToBooleanAndRector

• class: Rector\DeadCode\Rector\Ternary\TernaryToBooleanOrFalseToBooleanAndRector
• test fixtures

Change ternary of bool : false to && bool

 class SomeClass
 {
     public function go()
     {
-        return $value ? $this->getBool() : false;
+        return $value && $this->getBool();
     }

     private function getBool(): bool
     {
         return (bool) 5;
     }
 }

Doctrine

AddEntityIdByConditionRector

• class: Rector\Doctrine\Rector\Class_\AddEntityIdByConditionRector
• test fixtures

Add entity id with annotations when meets condition

services:
    Rector\Doctrine\Rector\Class_\AddEntityIdByConditionRector: {  }

↓

 class SomeClass
 {
     use SomeTrait;
+
+    /**
+      * @ORM\Id
+      * @ORM\Column(type="integer")
+      * @ORM\GeneratedValue(strategy="AUTO")
+      */
+     private $id;
+
+    public function getId(): int
+    {
+        return $this->id;
+    }
 }

AddUuidAnnotationsToIdPropertyRector

• class: Rector\Doctrine\Rector\Property\AddUuidAnnotationsToIdPropertyRector
• test fixtures

Add uuid annotations to $id property

 use Doctrine\ORM\Attributes as ORM;

 /**
  * @ORM\Entity
  */
 class SomeClass
 {
     /**
-     * @var int
+     * @var \Ramsey\Uuid\UuidInterface
      * @ORM\Id
-     * @ORM\Column(type="integer")
-     * @ORM\GeneratedValue(strategy="AUTO")
-     * @Serializer\Type("int")
+     * @ORM\Column(type="uuid_binary")
+     * @Serializer\Type("string")
      */
     public $id;
 }

AddUuidMirrorForRelationPropertyRector

• class: Rector\Doctrine\Rector\Class_\AddUuidMirrorForRelationPropertyRector
• test fixtures

Adds $uuid property to entities, that already have $id with integer type.Require for step-by-step migration from int to uuid.

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class SomeEntity
 {
     /**
      * @ORM\ManyToOne(targetEntity="AnotherEntity", cascade={"persist", "merge"})
      * @ORM\JoinColumn(nullable=false)
      */
     private $amenity;
+
+    /**
+     * @ORM\ManyToOne(targetEntity="AnotherEntity", cascade={"persist", "merge"})
+     * @ORM\JoinColumn(nullable=true, referencedColumnName="uuid")
+     */
+    private $amenityUuid;
 }

 /**
  * @ORM\Entity
  */
 class AnotherEntity
 {
     /**
      * @var int
      * @ORM\Id
      * @ORM\Column(type="integer")
      * @ORM\GeneratedValue(strategy="AUTO")
      */
     private $id;

     private $uuid;
 }

AddUuidToEntityWhereMissingRector

• class: Rector\Doctrine\Rector\Class_\AddUuidToEntityWhereMissingRector
• test fixtures

Adds $uuid property to entities, that already have $id with integer type.Require for step-by-step migration from int to uuid. In following step it should be renamed to $id and replace it

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class SomeEntityWithIntegerId
 {
     /**
+     * @var \Ramsey\Uuid\UuidInterface
+     * @ORM\Column(type="uuid_binary", unique=true, nullable=true)
+     */
+    private $uuid;
+    /**
      * @var int
      * @ORM\Id
      * @ORM\Column(type="integer")
      * @ORM\GeneratedValue(strategy="AUTO")
      */
     private $id;
 }

AlwaysInitializeUuidInEntityRector

• class: Rector\Doctrine\Rector\Class_\AlwaysInitializeUuidInEntityRector
• test fixtures

Add uuid initializion to all entities that misses it

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class AddUuidInit
 {
     /**
      * @ORM\Id
      * @var UuidInterface
      */
     private $superUuid;
+    public function __construct()
+    {
+        $this->superUuid = \Ramsey\Uuid\Uuid::uuid4();
+    }
 }

ChangeGetIdTypeToUuidRector

• class: Rector\Doctrine\Rector\ClassMethod\ChangeGetIdTypeToUuidRector
• test fixtures

Change return type of getId() to uuid interface

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class GetId
 {
-    public function getId(): int
+    public function getId(): \Ramsey\Uuid\UuidInterface
     {
         return $this->id;
     }
 }

ChangeGetUuidMethodCallToGetIdRector

• class: Rector\Doctrine\Rector\MethodCall\ChangeGetUuidMethodCallToGetIdRector
• test fixtures

Change getUuid() method call to getId()

 use Doctrine\ORM\Mapping as ORM;
 use Ramsey\Uuid\Uuid;
 use Ramsey\Uuid\UuidInterface;

 class SomeClass
 {
     public function run()
     {
         $buildingFirst = new Building();

-        return $buildingFirst->getUuid()->toString();
+        return $buildingFirst->getId()->toString();
     }
 }

 /**
  * @ORM\Entity
  */
 class UuidEntity
 {
     private $uuid;
     public function getUuid(): UuidInterface
     {
         return $this->uuid;
     }
 }

ChangeIdenticalUuidToEqualsMethodCallRector

• class: Rector\Doctrine\Rector\Identical\ChangeIdenticalUuidToEqualsMethodCallRector
• test fixtures

Change $uuid === 1 to $uuid->equals(\Ramsey\Uuid\Uuid::fromString(1))

 class SomeClass
 {
     public function match($checkedId): int
     {
         $building = new Building();

-        return $building->getId() === $checkedId;
+        return $building->getId()->equals(\Ramsey\Uuid\Uuid::fromString($checkedId));
     }
 }

ChangeReturnTypeOfClassMethodWithGetIdRector

• class: Rector\Doctrine\Rector\ClassMethod\ChangeReturnTypeOfClassMethodWithGetIdRector
• test fixtures

Change getUuid() method call to getId()

 class SomeClass
 {
-    public function getBuildingId(): int
+    public function getBuildingId(): \Ramsey\Uuid\UuidInterface
     {
         $building = new Building();

         return $building->getId();
     }
 }

ChangeSetIdToUuidValueRector

• class: Rector\Doctrine\Rector\MethodCall\ChangeSetIdToUuidValueRector
• test fixtures

Change set id to uuid values

 use Doctrine\ORM\Mapping as ORM;
 use Ramsey\Uuid\Uuid;

 class SomeClass
 {
     public function run()
     {
         $buildingFirst = new Building();
-        $buildingFirst->setId(1);
-        $buildingFirst->setUuid(Uuid::fromString('a3bfab84-e207-4ddd-b96d-488151de9e96'));
+        $buildingFirst->setId(Uuid::fromString('a3bfab84-e207-4ddd-b96d-488151de9e96'));
     }
 }

 /**
  * @ORM\Entity
  */
 class Building
 {
 }

ChangeSetIdTypeToUuidRector

• class: Rector\Doctrine\Rector\ClassMethod\ChangeSetIdTypeToUuidRector
• test fixtures

Change param type of setId() to uuid interface

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class SetId
 {
     private $id;

-    public function setId(int $uuid): int
+    public function setId(\Ramsey\Uuid\UuidInterface $uuid): int
     {
         return $this->id = $uuid;
     }
 }

EntityAliasToClassConstantReferenceRector

• class: Rector\Doctrine\Rector\MethodCall\EntityAliasToClassConstantReferenceRector
• test fixtures

Replaces doctrine alias with class.

 $entityManager = new Doctrine\ORM\EntityManager();
-$entityManager->getRepository("AppBundle:Post");
+$entityManager->getRepository(\App\Entity\Post::class);

ManagerRegistryGetManagerToEntityManagerRector

• class: Rector\Doctrine\Rector\Class_\ManagerRegistryGetManagerToEntityManagerRector
• test fixtures

Changes ManagerRegistry intermediate calls directly to EntityManager calls

-use Doctrine\Common\Persistence\ManagerRegistry;
+use Doctrine\ORM\EntityManagerInterface;

 class CustomRepository
 {
     /**
-     * @var ManagerRegistry
+     * @var EntityManagerInterface
      */
-    private $managerRegistry;
+    private $entityManager;

-    public function __construct(ManagerRegistry $managerRegistry)
+    public function __construct(EntityManagerInterface $entityManager)
     {
-        $this->managerRegistry = $managerRegistry;
+        $this->entityManager = $entityManager;
     }

     public function run()
     {
-        $entityManager = $this->managerRegistry->getManager();
-        $someRepository = $entityManager->getRepository('Some');
+        $someRepository = $this->entityManager->getRepository('Some');
     }
 }

RemoveRepositoryFromEntityAnnotationRector

• class: Rector\Doctrine\Rector\Class_\RemoveRepositoryFromEntityAnnotationRector
• test fixtures

Removes repository class from @Entity annotation

 use Doctrine\ORM\Mapping as ORM;

 /**
- * @ORM\Entity(repositoryClass="ProductRepository")
+ * @ORM\Entity
  */
 class Product
 {
 }

RemoveTemporaryUuidColumnPropertyRector

• class: Rector\Doctrine\Rector\Property\RemoveTemporaryUuidColumnPropertyRector
• test fixtures

Remove temporary $uuid property

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class Column
 {
     /**
      * @ORM\Column
      */
     public $id;
-
-    /**
-     * @ORM\Column
-     */
-    public $uuid;
 }

RemoveTemporaryUuidRelationPropertyRector

• class: Rector\Doctrine\Rector\Property\RemoveTemporaryUuidRelationPropertyRector
• test fixtures

Remove temporary *Uuid relation properties

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class Column
 {
     /**
      * @ORM\ManyToMany(targetEntity="Phonenumber")
      */
     private $apple;
-
-    /**
-     * @ORM\ManyToMany(targetEntity="Phonenumber")
-     */
-    private $appleUuid;
 }

DoctrineCodeQuality

ChangeQuerySetParametersMethodParameterFromArrayToArrayCollectionRector

• class: Rector\DoctrineCodeQuality\Rector\Class_\ChangeQuerySetParametersMethodParameterFromArrayToArrayCollectionRector
• test fixtures

Change array to ArrayCollection in setParameters method of query builder

-
+use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\ORM\EntityRepository;
+use Doctrine\ORM\Query\Parameter;

 class SomeRepository extends EntityRepository
 {
     public function getSomething()
     {
         return $this
             ->createQueryBuilder('sm')
             ->select('sm')
             ->where('sm.foo = :bar')
-            ->setParameters([
-                'bar' => 'baz'
-            ])
+            ->setParameters(new ArrayCollection([
+                new  Parameter('bar', 'baz'),
+            ]))
             ->getQuery()
             ->getResult()
         ;
     }
 }

InitializeDefaultEntityCollectionRector

• class: Rector\DoctrineCodeQuality\Rector\Class_\InitializeDefaultEntityCollectionRector
• test fixtures

Initialize collection property in Entity constructor

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
 class SomeClass
 {
     /**
      * @ORM\OneToMany(targetEntity="MarketingEvent")
      */
     private $marketingEvents = [];
+
+    public function __construct()
+    {
+        $this->marketingEvents = new ArrayCollection();
+    }
 }

DoctrineGedmoToKnplabs

BlameableBehaviorRector

• class: Rector\DoctrineGedmoToKnplabs\Rector\Class_\BlameableBehaviorRector
• test fixtures

Change Blameable from gedmo/doctrine-extensions to knplabs/doctrine-behaviors

-use Gedmo\Mapping\Annotation as Gedmo;
 use Doctrine\ORM\Mapping as ORM;
+use Knp\DoctrineBehaviors\Contract\Entity\BlameableInterface;
+use Knp\DoctrineBehaviors\Model\Blameable\BlameableTrait;

 /**
  * @ORM\Entity
  */
-class SomeClass
+class SomeClass implements BlameableInterface
 {
-    /**
-     * @Gedmo\Blameable(on="create")
-     */
-    private $createdBy;
-
-    /**
-     * @Gedmo\Blameable(on="update")
-     */
-    private $updatedBy;
-
-    /**
-     * @Gedmo\Blameable(on="change", field={"title", "body"})
-     */
-    private $contentChangedBy;
-
-    public function getCreatedBy()
-    {
-        return $this->createdBy;
-    }
-
-    public function getUpdatedBy()
-    {
-        return $this->updatedBy;
-    }
-
-    public function getContentChangedBy()
-    {
-        return $this->contentChangedBy;
-    }
+    use BlameableTrait;
 }

LoggableBehaviorRector

• class: Rector\DoctrineGedmoToKnplabs\Rector\Class_\LoggableBehaviorRector
• test fixtures

Change Loggable from gedmo/doctrine-extensions to knplabs/doctrine-behaviors

-use Gedmo\Mapping\Annotation as Gedmo;
 use Doctrine\ORM\Mapping as ORM;
+use Knp\DoctrineBehaviors\Model\Loggable\LoggableTrait;
+use Knp\DoctrineBehaviors\Contract\Entity\LoggableInterface;

 /**
  * @ORM\Entity
- * @Gedmo\Loggable
  */
-class SomeClass
+class SomeClass implements LoggableInterface
 {
+    use LoggableTrait;
+
     /**
-     * @Gedmo\Versioned
      * @ORM\Column(name="title", type="string", length=8)
      */
     private $title;
 }

SluggableBehaviorRector

• class: Rector\DoctrineGedmoToKnplabs\Rector\Class_\SluggableBehaviorRector
• test fixtures

Change Sluggable from gedmo/doctrine-extensions to knplabs/doctrine-behaviors

 use Gedmo\Mapping\Annotation as Gedmo;
+use Knp\DoctrineBehaviors\Model\Sluggable\SluggableTrait;
+use Knp\DoctrineBehaviors\Contract\Entity\SluggableInterface;

-class SomeClass
+class SomeClass implements SluggableInterface
 {
+    use SluggableTrait;
+
     /**
-     * @Gedmo\Slug(fields={"name"})
+     * @return string[]
      */
-    private $slug;
-
-    public function getSlug(): ?string
+    public function getSluggableFields(): array
     {
-        return $this->slug;
-    }
-
-    public function setSlug(?string $slug): void
-    {
-        $this->slug = $slug;
+        return ['name'];
     }
 }

SoftDeletableBehaviorRector

• class: Rector\DoctrineGedmoToKnplabs\Rector\Class_\SoftDeletableBehaviorRector
• test fixtures

Change SoftDeletable from gedmo/doctrine-extensions to knplabs/doctrine-behaviors

-use Gedmo\Mapping\Annotation as Gedmo;
+use Knp\DoctrineBehaviors\Contract\Entity\SoftDeletableInterface;
+use Knp\DoctrineBehaviors\Model\SoftDeletable\SoftDeletableTrait;

-/**
- * @Gedmo\SoftDeleteable(fieldName="deletedAt", timeAware=false, hardDelete=true)
- */
-class SomeClass
+class SomeClass implements SoftDeletableInterface
 {
-    /**
-     * @ORM\Column(name="deletedAt", type="datetime", nullable=true)
-     */
-    private $deletedAt;
-
-    public function getDeletedAt()
-    {
-        return $this->deletedAt;
-    }
-
-    public function setDeletedAt($deletedAt)
-    {
-        $this->deletedAt = $deletedAt;
-    }
+    use SoftDeletableTrait;
 }

TimestampableBehaviorRector

• class: Rector\DoctrineGedmoToKnplabs\Rector\Class_\TimestampableBehaviorRector
• test fixtures

Change Timestampable from gedmo/doctrine-extensions to knplabs/doctrine-behaviors

-use Gedmo\Timestampable\Traits\TimestampableEntity;
+use Knp\DoctrineBehaviors\Model\Timestampable\TimestampableTrait;
+use Knp\DoctrineBehaviors\Contract\Entity\TimestampableInterface;

-class SomeClass
+class SomeClass implements TimestampableInterface
 {
-    use TimestampableEntity;
+    use TimestampableTrait;
 }

TranslationBehaviorRector

• class: Rector\DoctrineGedmoToKnplabs\Rector\Class_\TranslationBehaviorRector
• test fixtures

Change Translation from gedmo/doctrine-extensions to knplabs/doctrine-behaviors

-use Gedmo\Mapping\Annotation as Gedmo;
-use Doctrine\ORM\Mapping as ORM;
-use Gedmo\Translatable\Translatable;
+use Knp\DoctrineBehaviors\Model\Translatable\TranslatableTrait;
+use Knp\DoctrineBehaviors\Contract\Entity\TranslatableInterface;

-/**
- * @ORM\Table
- */
-class Article implements Translatable
+class SomeClass implements TranslatableInterface
 {
+    use TranslatableTrait;
+}
+
+
+use Knp\DoctrineBehaviors\Contract\Entity\TranslationInterface;
+use Knp\DoctrineBehaviors\Model\Translatable\TranslationTrait;
+
+class SomeClassTranslation implements TranslationInterface
+{
+    use TranslationTrait;
+
     /**
-     * @Gedmo\Translatable
      * @ORM\Column(length=128)
      */
     private $title;

     /**
-     * @Gedmo\Translatable
      * @ORM\Column(type="text")
      */
     private $content;
-
-    /**
-     * @Gedmo\Locale
-     * Used locale to override Translation listener`s locale
-     * this is not a mapped field of entity metadata, just a simple property
-     * and it is not necessary because globally locale can be set in listener
-     */
-    private $locale;
-
-    public function setTitle($title)
-    {
-        $this->title = $title;
-    }
-
-    public function getTitle()
-    {
-        return $this->title;
-    }
-
-    public function setContent($content)
-    {
-        $this->content = $content;
-    }
-
-    public function getContent()
-    {
-        return $this->content;
-    }
-
-    public function setTranslatableLocale($locale)
-    {
-        $this->locale = $locale;
-    }
 }

TreeBehaviorRector

• class: Rector\DoctrineGedmoToKnplabs\Rector\Class_\TreeBehaviorRector
• test fixtures

Change Tree from gedmo/doctrine-extensions to knplabs/doctrine-behaviors

-use Doctrine\Common\Collections\Collection;
-use Gedmo\Mapping\Annotation as Gedmo;
+use Knp\DoctrineBehaviors\Contract\Entity\TreeNodeInterface;
+use Knp\DoctrineBehaviors\Model\Tree\TreeNodeTrait;

-/**
- * @Gedmo\Tree(type="nested")
- */
-class SomeClass
+class SomeClass implements TreeNodeInterface
 {
-    /**
-     * @Gedmo\TreeLeft
-     * @ORM\Column(name="lft", type="integer")
-     * @var int
-     */
-    private $lft;
-
-    /**
-     * @Gedmo\TreeRight
-     * @ORM\Column(name="rgt", type="integer")
-     * @var int
-     */
-    private $rgt;
-
-    /**
-     * @Gedmo\TreeLevel
-     * @ORM\Column(name="lvl", type="integer")
-     * @var int
-     */
-    private $lvl;
-
-    /**
-     * @Gedmo\TreeRoot
-     * @ORM\ManyToOne(targetEntity="Category")
-     * @ORM\JoinColumn(name="tree_root", referencedColumnName="id", onDelete="CASCADE")
-     * @var Category
-     */
-    private $root;
-
-    /**
-     * @Gedmo\TreeParent
-     * @ORM\ManyToOne(targetEntity="Category", inversedBy="children")
-     * @ORM\JoinColumn(name="parent_id", referencedColumnName="id", onDelete="CASCADE")
-     * @var Category
-     */
-    private $parent;
-
-    /**
-     * @ORM\OneToMany(targetEntity="Category", mappedBy="parent")
-     * @var Category[]|Collection
-     */
-    private $children;
-
-    public function getRoot(): self
-    {
-        return $this->root;
-    }
-
-    public function setParent(self $category): void
-    {
-        $this->parent = $category;
-    }
-
-    public function getParent(): self
-    {
-        return $this->parent;
-    }
+    use TreeNodeTrait;
 }

DynamicTypeAnalysis

AddArgumentTypeWithProbeDataRector

• class: Rector\DynamicTypeAnalysis\Rector\ClassMethod\AddArgumentTypeWithProbeDataRector
• test fixtures

Add argument type based on probed data

 class SomeClass
 {
-    public function run($arg)
+    public function run(string $arg)
     {
     }
 }

DecorateMethodWithArgumentTypeProbeRector

• class: Rector\DynamicTypeAnalysis\Rector\ClassMethod\DecorateMethodWithArgumentTypeProbeRector
• test fixtures

Add probe that records argument types to each method

 class SomeClass
 {
     public function run($arg)
     {
+        \Rector\DynamicTypeAnalysis\Probe\TypeStaticProbe::recordArgumentType($arg, __METHOD__, 0);
     }
 }

RemoveArgumentTypeProbeRector

• class: Rector\DynamicTypeAnalysis\Rector\StaticCall\RemoveArgumentTypeProbeRector
• test fixtures

Clean up probe that records argument types

-use Rector\DynamicTypeAnalysis\Probe\TypeStaticProbe;
-
 class SomeClass
 {
     public function run($arg)
     {
-        TypeStaticProbe::recordArgumentType($arg, __METHOD__, 0);
     }
 }

FileSystemRector

RemoveProjectFileRector

• class: Rector\FileSystemRector\Rector\Removing\RemoveProjectFileRector

Remove file relative to project directory

-// someFile/ToBeRemoved.txt

Guzzle

MessageAsArrayRector

• class: Rector\Guzzle\Rector\MethodCall\MessageAsArrayRector
• test fixtures

Changes getMessage(..., true) to getMessageAsArray()

 /** @var GuzzleHttp\Message\MessageInterface */
-$value = $message->getMessage('key', true);
+$value = $message->getMessageAsArray('key');

JMS

RemoveJmsInjectParamsAnnotationRector

• class: Rector\JMS\Rector\ClassMethod\RemoveJmsInjectParamsAnnotationRector
• test fixtures

Removes JMS\DiExtraBundle\Annotation\InjectParams annotation

 use JMS\DiExtraBundle\Annotation as DI;

 class SomeClass
 {
-    /**
-     * @DI\InjectParams({
-     *     "subscribeService" = @DI\Inject("app.email.service.subscribe"),
-     *     "ipService" = @DI\Inject("app.util.service.ip")
-     * })
-     */
     public function __construct()
     {
     }
-}
+}

RemoveJmsInjectServiceAnnotationRector

• class: Rector\JMS\Rector\Class_\RemoveJmsInjectServiceAnnotationRector
• test fixtures

Removes JMS\DiExtraBundle\Annotation\Services annotation

 use JMS\DiExtraBundle\Annotation as DI;

-/**
- * @DI\Service("email.web.services.subscribe_token", public=true)
- */
 class SomeClass
 {
 }

Laravel

FacadeStaticCallToConstructorInjectionRector

• class: Rector\Laravel\Rector\StaticCall\FacadeStaticCallToConstructorInjectionRector
• test fixtures

Move Illuminate\Support\Facades* static calls to constructor injection

 use Illuminate\Support\Facades\Response;

 class ExampleController extends Controller
 {
+    /**
+     * @var \Illuminate\Contracts\Routing\ResponseFactory
+     */
+    private $responseFactory;
+
+    public function __construct(\Illuminate\Contracts\Routing\ResponseFactory $responseFactory)
+    {
+        $this->responseFactory = $responseFactory;
+    }
+
     public function store()
     {
-        return Response::view('example', ['new_example' => 123]);
+        return $this->responseFactory->view('example', ['new_example' => 123]);
     }
 }

HelperFunctionToConstructorInjectionRector

• class: Rector\Laravel\Rector\FuncCall\HelperFunctionToConstructorInjectionRector
• test fixtures

Move help facade-like function calls to constructor injection

 class SomeController
 {
+    /**
+     * @var \Illuminate\Contracts\View\Factory
+     */
+    private $viewFactory;
+
+    public function __construct(\Illuminate\Contracts\View\Factory $viewFactory)
+    {
+        $this->viewFactory = $viewFactory;
+    }
+
     public function action()
     {
-        $template = view('template.blade');
-        $viewFactory = view();
+        $template = $this->viewFactory->make('template.blade');
+        $viewFactory = $this->viewFactory;
     }
 }

InlineValidationRulesToArrayDefinitionRector

• class: Rector\Laravel\Rector\Class_\InlineValidationRulesToArrayDefinitionRector
• test fixtures

Transforms inline validation rules to array definition

 use Illuminate\Foundation\Http\FormRequest;

 class SomeClass extends FormRequest
 {
     public function rules(): array
     {
         return [
-            'someAttribute' => 'required|string|exists:' . SomeModel::class . 'id',
+            'someAttribute' => ['required', 'string', \Illuminate\Validation\Rule::exists(SomeModel::class, 'id')],
         ];
     }
 }

MinutesToSecondsInCacheRector

• class: Rector\Laravel\Rector\StaticCall\MinutesToSecondsInCacheRector
• test fixtures

Change minutes argument to seconds in Illuminate\Contracts\Cache\Store and Illuminate\Support\Facades\Cache

 class SomeClass
 {
     public function run()
     {
-        Illuminate\Support\Facades\Cache::put('key', 'value', 60);
+        Illuminate\Support\Facades\Cache::put('key', 'value', 60 * 60);
     }
 }

Redirect301ToPermanentRedirectRector

• class: Rector\Laravel\Rector\StaticCall\Redirect301ToPermanentRedirectRector
• test fixtures

Change "redirect" call with 301 to "permanentRedirect"

 class SomeClass
 {
     public function run()
     {
-        Illuminate\Routing\Route::redirect('/foo', '/bar', 301);
+        Illuminate\Routing\Route::permanentRedirect('/foo', '/bar');
     }
 }

RequestStaticValidateToInjectRector

• class: Rector\Laravel\Rector\StaticCall\RequestStaticValidateToInjectRector
• test fixtures

Change static validate() method to $request->validate()

 use Illuminate\Http\Request;

 class SomeClass
 {
-    public function store()
+    public function store(\Illuminate\Http\Request $request)
     {
-        $validatedData = Request::validate(['some_attribute' => 'required']);
+        $validatedData = $request->validate(['some_attribute' => 'required']);
     }
 }

Legacy

ChangeSingletonToServiceRector

• class: Rector\Legacy\Rector\ClassMethod\ChangeSingletonToServiceRector
• test fixtures

Change singleton class to normal class that can be registered as a service

 class SomeClass
 {
-    private static $instance;
-
-    private function __construct()
+    public function __construct()
     {
-    }
-
-    public static function getInstance()
-    {
-        if (null === static::$instance) {
-            static::$instance = new static();
-        }
-
-        return static::$instance;
     }
 }

FunctionToStaticMethodRector

• class: Rector\Legacy\Rector\FileSystem\FunctionToStaticMethodRector

Change functions to static calls, so composer can autoload them

-function some_function()
+class SomeUtilsClass
 {
+    public static function someFunction()
+    {
+    }
 }

-some_function('lol');
+SomeUtilsClass::someFunction('lol');

MockistaToMockery

MockeryTearDownRector

• class: Rector\MockistaToMockery\Rector\Class_\MockeryTearDownRector
• test fixtures

Add Mockery::close() in tearDown() method if not yet

 use PHPUnit\Framework\TestCase;

 class SomeTest extends TestCase
 {
+    protected function tearDown(): void
+    {
+        Mockery::close();
+    }
     public function test()
     {
         $mockUser = mock(User::class);
     }
 }

MockistaMockToMockeryMockRector

• class: Rector\MockistaToMockery\Rector\ClassMethod\MockistaMockToMockeryMockRector
• test fixtures

Change functions to static calls, so composer can autoload them

 class SomeTest
 {
     public function run()
     {
-        $mockUser = mock(User::class);
-        $mockUser->getId()->once->andReturn(1);
-        $mockUser->freeze();
+        $mockUser = Mockery::mock(User::class);
+        $mockUser->expects()->getId()->once()->andReturn(1);
     }
 }

MysqlToMysqli

MysqlAssignToMysqliRector

• class: Rector\MysqlToMysqli\Rector\Assign\MysqlAssignToMysqliRector
• test fixtures

Converts more complex mysql functions to mysqli

-$data = mysql_db_name($result, $row);
+mysqli_data_seek($result, $row);
+$fetch = mysql_fetch_row($result);
+$data = $fetch[0];

MysqlFuncCallToMysqliRector

• class: Rector\MysqlToMysqli\Rector\FuncCall\MysqlFuncCallToMysqliRector
• test fixtures

Converts more complex mysql functions to mysqli

-mysql_drop_db($database);
+mysqli_query('DROP DATABASE ' . $database);

MysqlPConnectToMysqliConnectRector

• class: Rector\MysqlToMysqli\Rector\FuncCall\MysqlPConnectToMysqliConnectRector
• test fixtures

Replace mysql_pconnect() with mysqli_connect() with host p: prefix

 final class SomeClass
 {
     public function run($host, $username, $password)
     {
-        return mysql_pconnect($host, $username, $password);
+        return mysqli_connect('p:' . $host, $username, $password);
     }
 }

MysqlQueryMysqlErrorWithLinkRector

• class: Rector\MysqlToMysqli\Rector\FuncCall\MysqlQueryMysqlErrorWithLinkRector
• test fixtures

Add mysql_query and mysql_error with connection

 class SomeClass
 {
     public function run()
     {
         $conn = mysqli_connect('host', 'user', 'pass');

-        mysql_error();
+        mysqli_error($conn);
         $sql = 'SELECT';

-        return mysql_query($sql);
+        return mysqli_query($conn, $sql);
     }
 }

Naming

RenamePropertyToMatchTypeRector

• class: Rector\Naming\Rector\Class_\RenamePropertyToMatchTypeRector
• test fixtures

Rename property and method param to match its type

 class SomeClass
 {
     /**
      * @var EntityManager
      */
-    private $eventManager;
+    private $entityManager;

-    public function __construct(EntityManager $eventManager)
+    public function __construct(EntityManager $entityManager)
     {
-        $this->eventManager = $eventManager;
+        $this->entityManager = $entityManager;
     }
 }

Nette

AddDatePickerToDateControlRector

• class: Rector\Nette\Rector\MethodCall\AddDatePickerToDateControlRector
• test fixtures

Nextras/Form upgrade of addDatePicker method call to DateControl assign

 use Nette\Application\UI\Form;

 class SomeClass
 {
     public function run()
     {
         $form = new Form();
-        $form->addDatePicker('key', 'Label');
+        $form['key'] = new \Nextras\FormComponents\Controls\DateControl('Label');
     }
 }

ContextGetByTypeToConstructorInjectionRector

• class: Rector\Nette\Rector\MethodCall\ContextGetByTypeToConstructorInjectionRector
• test fixtures

Move dependency passed to all children to parent as @inject/@required dependency

 class SomeClass
 {
     /**
      * @var \Nette\DI\Container
      */
     private $context;

+    /**
+     * @var SomeTypeToInject
+     */
+    private $someTypeToInject;
+
+    public function __construct(SomeTypeToInject $someTypeToInject)
+    {
+        $this->someTypeToInject = $someTypeToInject;
+    }
+
     public function run()
     {
-        $someTypeToInject = $this->context->getByType(SomeTypeToInject::class);
+        $someTypeToInject = $this->someTypeToInject;
     }
 }

EndsWithFunctionToNetteUtilsStringsRector

• class: Rector\Nette\Rector\Identical\EndsWithFunctionToNetteUtilsStringsRector
• test fixtures

Use Nette\Utils\Strings::endWith() over bare string-functions

 class SomeClass
 {
     public function end($needle)
     {
         $content = 'Hi, my name is Tom';

-        $yes = substr($content, -strlen($needle)) === $needle;
+        $yes = \Nette\Utils\Strings::endsWith($content, $needle);
     }
 }

FilePutContentsToFileSystemWriteRector

• class: Rector\Nette\Rector\FuncCall\FilePutContentsToFileSystemWriteRector
• test fixtures

Change file_put_contents() to FileSystem::write()

 class SomeClass
 {
     public function run()
     {
-        file_put_contents('file.txt', 'content');
+        \Nette\Utils\FileSystem::write('file.txt', 'content');

         file_put_contents('file.txt', 'content_to_append', FILE_APPEND);
     }
 }

JsonDecodeEncodeToNetteUtilsJsonDecodeEncodeRector

• class: Rector\Nette\Rector\FuncCall\JsonDecodeEncodeToNetteUtilsJsonDecodeEncodeRector
• test fixtures

Changes json_encode()/json_decode() to safer and more verbose Nette\Utils\Json::encode()/decode() calls

 class SomeClass
 {
     public function decodeJson(string $jsonString)
     {
-        $stdClass = json_decode($jsonString);
+        $stdClass = \Nette\Utils\Json::decode($jsonString);

-        $array = json_decode($jsonString, true);
-        $array = json_decode($jsonString, false);
+        $array = \Nette\Utils\Json::decode($jsonString, \Nette\Utils\Json::FORCE_ARRAY);
+        $array = \Nette\Utils\Json::decode($jsonString);
     }

     public function encodeJson(array $data)
     {
-        $jsonString = json_encode($data);
+        $jsonString = \Nette\Utils\Json::encode($data);

-        $prettyJsonString = json_encode($data, JSON_PRETTY_PRINT);
+        $prettyJsonString = \Nette\Utils\Json::encode($data, \Nette\Utils\Json::PRETTY);
     }
 }

PregFunctionToNetteUtilsStringsRector

• class: Rector\Nette\Rector\FuncCall\PregFunctionToNetteUtilsStringsRector
• test fixtures

Use Nette\Utils\Strings over bare preg_split() and preg_replace() functions

+use Nette\Utils\Strings;
+
 class SomeClass
 {
     public function run()
     {
         $content = 'Hi my name is Tom';
-        $splitted = preg_split('#Hi#', $content);
+        $splitted = \Nette\Utils\Strings::split($content, '#Hi#');
     }
 }

PregMatchFunctionToNetteUtilsStringsRector

• class: Rector\Nette\Rector\FuncCall\PregMatchFunctionToNetteUtilsStringsRector
• test fixtures

Use Nette\Utils\Strings over bare preg_match() and preg_match_all() functions

+use Nette\Utils\Strings;
+
 class SomeClass
 {
     public function run()
     {
         $content = 'Hi my name is Tom';
-        preg_match('#Hi#', $content, $matches);
+        $matches = Strings::match($content, '#Hi#');
     }
 }

SetClassWithArgumentToSetFactoryRector

• class: Rector\Nette\Rector\MethodCall\SetClassWithArgumentToSetFactoryRector
• test fixtures

Change setClass with class and arguments to separated methods

 use Nette\DI\ContainerBuilder;

 class SomeClass
 {
     public function run(ContainerBuilder $containerBuilder)
     {
         $containerBuilder->addDefinition('...')
-            ->setClass('SomeClass', [1, 2]);
+            ->setFactory('SomeClass', [1, 2]);
     }
 }

StartsWithFunctionToNetteUtilsStringsRector

• class: Rector\Nette\Rector\Identical\StartsWithFunctionToNetteUtilsStringsRector
• test fixtures

Use Nette\Utils\Strings::startsWith() over bare string-functions

 class SomeClass
 {
     public function start($needle)
     {
         $content = 'Hi, my name is Tom';

-        $yes = substr($content, 0, strlen($needle)) === $needle;
+        $yes = \Nette\Utils\Strings::startsWith($content, $needle);
     }
 }

StrposToStringsContainsRector

• class: Rector\Nette\Rector\NotIdentical\StrposToStringsContainsRector
• test fixtures

Use Nette\Utils\Strings over bare string-functions

 class SomeClass
 {
     public function run()
     {
         $name = 'Hi, my name is Tom';
-        return strpos($name, 'Hi') !== false;
+        return \Nette\Utils\Strings::contains($name, 'Hi');
     }
 }

SubstrStrlenFunctionToNetteUtilsStringsRector

• class: Rector\Nette\Rector\FuncCall\SubstrStrlenFunctionToNetteUtilsStringsRector
• test fixtures

Use Nette\Utils\Strings over bare string-functions

 class SomeClass
 {
     public function run()
     {
-        return substr($value, 0, 3);
+        return \Nette\Utils\Strings::substring($value, 0, 3);
     }
 }

TemplateMagicAssignToExplicitVariableArrayRector

• class: Rector\Nette\Rector\ClassMethod\TemplateMagicAssignToExplicitVariableArrayRector
• test fixtures

Change $this->templates->{magic} to $this->template->render(..., $values)

 use Nette\Application\UI\Control;

 class SomeControl extends Control
 {
     public function render()
     {
-        $this->template->param = 'some value';
-        $this->template->render(__DIR__ . '/poll.latte');
+        $this->template->render(__DIR__ . '/poll.latte', ['param' => 'some value']);
     }
 }

NetteKdyby

ChangeNetteEventNamesInGetSubscribedEventsRector

• class: Rector\NetteKdyby\Rector\ClassMethod\ChangeNetteEventNamesInGetSubscribedEventsRector
• test fixtures

Change EventSubscriber from Kdyby to Contributte

+use Contributte\Events\Extra\Event\Application\ShutdownEvent;
 use Kdyby\Events\Subscriber;
 use Nette\Application\Application;
-use Nette\Application\UI\Presenter;

 class GetApplesSubscriber implements Subscriber
 {
-    public function getSubscribedEvents()
+    public static function getSubscribedEvents()
     {
         return [
-            Application::class . '::onShutdown',
+            ShutdownEvent::class => 'onShutdown',
         ];
     }

-    public function onShutdown(Presenter $presenter)
+    public function onShutdown(ShutdownEvent $shutdownEvent)
     {
+        $presenter = $shutdownEvent->getPresenter();
         $presenterName = $presenter->getName();
         // ...
     }
 }

ReplaceEventManagerWithEventSubscriberRector

• class: Rector\NetteKdyby\Rector\MethodCall\ReplaceEventManagerWithEventSubscriberRector
• test fixtures

Change Kdyby EventManager to EventDispatcher

 use Kdyby\Events\EventManager;

 final class SomeClass
 {
     /**
      * @var EventManager
      */
     private $eventManager;

     public function __construct(EventManager $eventManager)
     {
         $this->eventManager = eventManager;
     }

     public function run()
     {
         $key = '2000';
-        $this->eventManager->dispatchEvent(static::class . '::onCopy', new EventArgsList([$this, $key]));
+        $this->eventManager->dispatch(new SomeClassCopyEvent($this, $key));
     }
 }

ReplaceMagicEventPropertySubscriberWithEventClassSubscriberRector

• class: Rector\NetteKdyby\Rector\ClassMethod\ReplaceMagicEventPropertySubscriberWithEventClassSubscriberRector
• test fixtures

Change getSubscribedEvents() from on magic property, to Event class

 use Kdyby\Events\Subscriber;

 final class ActionLogEventSubscriber implements Subscriber
 {
     public function getSubscribedEvents(): array
     {
         return [
-            AlbumService::class . '::onApprove' => 'onAlbumApprove',
+            AlbumServiceApproveEvent::class => 'onAlbumApprove',
         ];
     }

-    public function onAlbumApprove(Album $album, int $adminId): void
+    public function onAlbumApprove(AlbumServiceApproveEventAlbum $albumServiceApproveEventAlbum): void
     {
+        $album = $albumServiceApproveEventAlbum->getAlbum();
         $album->play();
     }
 }

ReplaceMagicPropertyEventWithEventClassRector

• class: Rector\NetteKdyby\Rector\MethodCall\ReplaceMagicPropertyEventWithEventClassRector
• test fixtures

Change $onProperty magic call with event disptacher and class dispatch

 final class FileManager
 {
-    public $onUpload;
+    use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;

+    public function __construct(EventDispatcherInterface $eventDispatcher)
+    {
+        $this->eventDispatcher = $eventDispatcher;
+    }
+
     public function run(User $user)
     {
-        $this->onUpload($user);
+        $onFileManagerUploadEvent = new FileManagerUploadEvent($user);
+        $this->eventDispatcher->dispatch($onFileManagerUploadEvent);
     }
 }

NetteTesterToPHPUnit

NetteAssertToPHPUnitAssertRector

• class: Rector\NetteTesterToPHPUnit\Rector\StaticCall\NetteAssertToPHPUnitAssertRector

Migrate Nette/Assert calls to PHPUnit

 use Tester\Assert;

 function someStaticFunctions()
 {
-    Assert::true(10 == 5);
+    \PHPUnit\Framework\Assert::assertTrue(10 == 5);
 }

NetteTesterClassToPHPUnitClassRector

• class: Rector\NetteTesterToPHPUnit\Rector\Class_\NetteTesterClassToPHPUnitClassRector

Migrate Nette Tester test case to PHPUnit

 namespace KdybyTests\Doctrine;

 use Tester\TestCase;
 use Tester\Assert;

-require_once __DIR__ . '/../bootstrap.php';
-
-class ExtensionTest extends TestCase
+class ExtensionTest extends \PHPUnit\Framework\TestCase
 {
     public function testFunctionality()
     {
-        Assert::true($default instanceof Kdyby\Doctrine\EntityManager);
-        Assert::true(5);
-        Assert::same($container->getService('kdyby.doctrine.default.entityManager'), $default);
+        $this->assertInstanceOf(\Kdyby\Doctrine\EntityManager::cllass, $default);
+        $this->assertTrue(5);
+        $this->same($container->getService('kdyby.doctrine.default.entityManager'), $default);
     }
-}
-
-(new \ExtensionTest())->run();
+}

RenameTesterTestToPHPUnitToTestFileRector

• class: Rector\NetteTesterToPHPUnit\Rector\RenameTesterTestToPHPUnitToTestFileRector

Rename "*.phpt" file to "*Test.php" file

-// tests/SomeTestCase.phpt
+// tests/SomeTestCase.php

NetteToSymfony

DeleteFactoryInterfaceRector

• class: Rector\NetteToSymfony\Rector\FileSystem\DeleteFactoryInterfaceRector

Interface factories are not needed in Symfony. Clear constructor injection is used instead

-interface SomeControlFactoryInterface
-{
-    public function create();
-}

FormControlToControllerAndFormTypeRector

• class: Rector\NetteToSymfony\Rector\Assign\FormControlToControllerAndFormTypeRector
• test fixtures

Change Form that extends Control to Controller and decoupled FormType

-use Nette\Application\UI\Form;
-use Nette\Application\UI\Control;
-
-class SomeForm extends Control
+class SomeFormController extends \Symfony\Bundle\FrameworkBundle\Controller\AbstractController
 {
-    public function createComponentForm()
+    /**
+     * @Route(...)
+     */
+    public function actionSomeForm(\Symfony\Component\HttpFoundation\Request $request): \Symfony\Component\HttpFoundation\Response
     {
-        $form = new Form();
-        $form->addText('name', 'Your name');
+        $form = $this->createForm(SomeFormType::class);
+        $form->handleRequest($request);

-        $form->onSuccess[] = [$this, 'processForm'];
-    }
-
-    public function processForm(Form $form)
-    {
-        // process me
+        if ($form->isSuccess() && $form->isValid()) {
+            // process me
+        }
     }
 }

New file

use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\Form\FormBuilderInterface;

class SomeFormType extends AbstractType
{
    public function buildForm(FormBuilderInterface $formBuilder, array $options)
    {
        $formBuilder->add('name', TextType::class, [
            'label' => 'Your name'
        ]);
    }
}

FromHttpRequestGetHeaderToHeadersGetRector

• class: Rector\NetteToSymfony\Rector\MethodCall\FromHttpRequestGetHeaderToHeadersGetRector
• test fixtures

Changes getHeader() to $request->headers->get()

 use Nette\Request;

 final class SomeController
 {
     public static function someAction(Request $request)
     {
-        $header = $this->httpRequest->getHeader('x');
+        $header = $request->headers->get('x');
     }
 }

FromRequestGetParameterToAttributesGetRector

• class: Rector\NetteToSymfony\Rector\MethodCall\FromRequestGetParameterToAttributesGetRector
• test fixtures

Changes "getParameter()" to "attributes->get()" from Nette to Symfony

 use Nette\Request;

 final class SomeController
 {
     public static function someAction(Request $request)
     {
-        $value = $request->getParameter('abz');
+        $value = $request->attribute->get('abz');
     }
 }

NetteControlToSymfonyControllerRector

• class: Rector\NetteToSymfony\Rector\Class_\NetteControlToSymfonyControllerRector
• test fixtures

Migrate Nette Component to Symfony Controller

-use Nette\Application\UI\Control;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Response;

-class SomeControl extends Control
+class SomeController extends AbstractController
 {
-    public function render()
-    {
-        $this->template->param = 'some value';
-        $this->template->render(__DIR__ . '/poll.latte');
-    }
+     public function some(): Response
+     {
+         return $this->render(__DIR__ . '/poll.latte', ['param' => 'some value']);
+     }
 }

NetteFormToSymfonyFormRector

• class: Rector\NetteToSymfony\Rector\Class_\NetteFormToSymfonyFormRector
• test fixtures

Migrate Nette\Forms in Presenter to Symfony

 use Nette\Application\UI;

 class SomePresenter extends UI\Presenter
 {
     public function someAction()
     {
-        $form = new UI\Form;
-        $form->addText('name', 'Name:');
-        $form->addPassword('password', 'Password:');
-        $form->addSubmit('login', 'Sign up');
+        $form = $this->createFormBuilder();
+        $form->add('name', \Symfony\Component\Form\Extension\Core\Type\TextType::class, [
+            'label' => 'Name:'
+        ]);
+        $form->add('password', \Symfony\Component\Form\Extension\Core\Type\PasswordType::class, [
+            'label' => 'Password:'
+        ]);
+        $form->add('login', \Symfony\Component\Form\Extension\Core\Type\SubmitType::class, [
+            'label' => 'Sign up'
+        ]);
     }
 }

RenameEventNamesInEventSubscriberRector

• class: Rector\NetteToSymfony\Rector\ClassMethod\RenameEventNamesInEventSubscriberRector
• test fixtures

Changes event names from Nette ones to Symfony ones

 use Symfony\Component\EventDispatcher\EventSubscriberInterface;

 final class SomeClass implements EventSubscriberInterface
 {
     public static function getSubscribedEvents()
     {
-        return ['nette.application' => 'someMethod'];
+        return [\SymfonyEvents::KERNEL => 'someMethod'];
     }
 }

RouterListToControllerAnnotationsRector

• class: Rector\NetteToSymfony\Rector\ClassMethod\RouterListToControllerAnnotationsRector
• test fixtures

Change new Route() from RouteFactory to @Route annotation above controller method

 final class RouterFactory
 {
     public function create(): RouteList
     {
         $routeList = new RouteList();
+
+        // case of single action controller, usually get() or __invoke() method
         $routeList[] = new Route('some-path', SomePresenter::class);

         return $routeList;
     }
 }

+use Symfony\Component\Routing\Annotation\Route;
+
 final class SomePresenter
 {
+    /**
+     * @Route(path="some-path")
+     */
     public function run()
     {
     }
 }

WrapTransParameterNameRector

• class: Rector\NetteToSymfony\Rector\MethodCall\WrapTransParameterNameRector
• test fixtures

Adds %% to placeholder name of trans() method if missing

 use Symfony\Component\Translation\Translator;

 final class SomeController
 {
     public function run()
     {
         $translator = new Translator('');
         $translated = $translator->trans(
             'Hello %name%',
-            ['name' => $name]
+            ['%name%' => $name]
         );
     }
 }

NetteUtilsCodeQuality

ReplaceTimeNumberWithDateTimeConstantRector

• class: Rector\NetteUtilsCodeQuality\Rector\LNumber\ReplaceTimeNumberWithDateTimeConstantRector
• test fixtures

Replace time numbers with Nette\Utils\DateTime constants

 final class SomeClass
 {
     public function run()
     {
-        return 86400;
+        return \Nette\Utils\DateTime::DAY;
     }
 }

Order

OrderClassConstantsByIntegerValueRector

• class: Rector\Order\Rector\Class_\OrderClassConstantsByIntegerValueRector
• test fixtures

Order class constant order by their integer value

 class SomeClass
 {
     const MODE_ON = 0;

+    const MODE_MAYBE = 1;
+
     const MODE_OFF = 2;
-
-    const MODE_MAYBE = 1;
 }

OrderPrivateMethodsByUseRector

• class: Rector\Order\Rector\Class_\OrderPrivateMethodsByUseRector
• test fixtures

Order private methods in order of their use

 class SomeClass
 {
     public function run()
     {
         $this->call1();
         $this->call2();
     }

-    private function call2()
+    private function call1()
     {
     }

-    private function call1()
+    private function call2()
     {
     }
 }

OrderPropertyByComplexityRector

• class: Rector\Order\Rector\Class_\OrderPropertyByComplexityRector
• test fixtures

Order properties by complexity, from the simplest like scalars to the most complex, like union or collections

-class SomeClass
+class SomeClass implements FoodRecipeInterface
 {
     /**
      * @var string
      */
     private $name;

     /**
-     * @var Type
+     * @var int
      */
-    private $service;
+    private $price;

     /**
-     * @var int
+     * @var Type
      */
-    private $price;
+    private $service;
 }

OrderPublicInterfaceMethodRector

• class: Rector\Order\Rector\Class_\OrderPublicInterfaceMethodRector
• test fixtures

Order public methods required by interface in custom orderer

services:
    Rector\Order\Rector\Class_\OrderPublicInterfaceMethodRector:
        $methodOrderByInterfaces:
            FoodRecipeInterface:
                - getDescription
                - process

↓

 class SomeClass implements FoodRecipeInterface
 {
-    public function process()
+    public function getDescription()
     {
     }
-
-    public function getDescription()
+    public function process()
     {
     }
 }

PHPOffice

AddRemovedDefaultValuesRector

• class: Rector\PHPOffice\Rector\StaticCall\AddRemovedDefaultValuesRector
• test fixtures

Complete removed default values explicitly

 final class SomeClass
 {
     public function run(): void
     {
         $logger = new \PHPExcel_CalcEngine_Logger;
-        $logger->setWriteDebugLog();
+        $logger->setWriteDebugLog(false);
     }
 }

CellStaticToCoordinateRector

• class: Rector\PHPOffice\Rector\StaticCall\CellStaticToCoordinateRector
• test fixtures

Methods to manipulate coordinates that used to exists in PHPExcel_Cell to PhpOffice\PhpSpreadsheet\Cell\Coordinate

 class SomeClass
 {
     public function run()
     {
-        \PHPExcel_Cell::stringFromColumnIndex();
+        \PhpOffice\PhpSpreadsheet\Cell\Coordinate::stringFromColumnIndex();
     }
 }

ChangeChartRendererRector

• class: Rector\PHPOffice\Rector\StaticCall\ChangeChartRendererRector
• test fixtures

Change chart renderer

 final class SomeClass
 {
     public function run(): void
     {
-        \PHPExcel_Settings::setChartRenderer($rendererName, $rendererLibraryPath);
+        \PHPExcel_Settings::setChartRenderer(\PhpOffice\PhpSpreadsheet\Chart\Renderer\JpGraph::class);
     }
 }

ChangeConditionalGetConditionRector

• class: Rector\PHPOffice\Rector\MethodCall\ChangeConditionalGetConditionRector
• test fixtures

Change argument PHPExcel_Style_Conditional->getCondition() to getConditions()

 final class SomeClass
 {
     public function run(): void
     {
         $conditional = new \PHPExcel_Style_Conditional;
-        $someCondition = $conditional->getCondition();
+        $someCondition = $conditional->getConditions()[0] ?? '';
     }
 }

ChangeConditionalReturnedCellRector

• class: Rector\PHPOffice\Rector\MethodCall\ChangeConditionalReturnedCellRector
• test fixtures

Change conditional call to getCell()

 final class SomeClass
 {
     public function run(): void
     {
         $worksheet = new \PHPExcel_Worksheet();
-        $cell = $worksheet->setCellValue('A1', 'value', true);
+        $cell = $worksheet->getCell('A1')->setValue('value');
     }
 }

ChangeConditionalSetConditionRector

• class: Rector\PHPOffice\Rector\MethodCall\ChangeConditionalSetConditionRector
• test fixtures

Change argument PHPExcel_Style_Conditional->setCondition() to setConditions()

 final class SomeClass
 {
     public function run(): void
     {
         $conditional = new \PHPExcel_Style_Conditional;
-        $someCondition = $conditional->setCondition(1);
+        $someCondition = $conditional->setConditions((array) 1);
     }
 }

ChangeDataTypeForValueRector

• class: Rector\PHPOffice\Rector\StaticCall\ChangeDataTypeForValueRector
• test fixtures

Change argument DataType::dataTypeForValue() to DefaultValueBinder

 final class SomeClass
 {
     public function run(): void
     {
-        $type = \PHPExcel_Cell_DataType::dataTypeForValue('value');
+        $type = \PhpOffice\PhpSpreadsheet\Cell\DefaultValueBinder::dataTypeForValue('value');
     }
 }

ChangeDuplicateStyleArrayToApplyFromArrayRector

• class: Rector\PHPOffice\Rector\MethodCall\ChangeDuplicateStyleArrayToApplyFromArrayRector
• test fixtures

Change method call duplicateStyleArray() to getStyle() + applyFromArray()

 final class SomeClass
 {
     public function run(): void
     {
         $worksheet = new \PHPExcel_Worksheet();
-        $worksheet->duplicateStyleArray($styles, $range, $advanced);
+        $worksheet->getStyle($range)->applyFromArray($styles, $advanced);
     }
 }

ChangeIOFactoryArgumentRector

• class: Rector\PHPOffice\Rector\StaticCall\ChangeIOFactoryArgumentRector
• test fixtures

Change argument of PHPExcel_IOFactory::createReader(), PHPExcel_IOFactory::createWriter() and PHPExcel_IOFactory::identify()

 final class SomeClass
 {
     public function run(): void
     {
-        $writer = \PHPExcel_IOFactory::createWriter('CSV');
+        $writer = \PHPExcel_IOFactory::createWriter('Csv');
     }
 }

ChangePdfWriterRector

• class: Rector\PHPOffice\Rector\StaticCall\ChangePdfWriterRector
• test fixtures

Change init of PDF writer

 final class SomeClass
 {
     public function run(): void
     {
-        \PHPExcel_Settings::setPdfRendererName(PHPExcel_Settings::PDF_RENDERER_MPDF);
-        \PHPExcel_Settings::setPdfRenderer($somePath);
-        $writer = \PHPExcel_IOFactory::createWriter($spreadsheet, 'PDF');
+        $writer = new \PhpOffice\PhpSpreadsheet\Writer\Pdf\Mpdf($spreadsheet);
     }
 }

ChangeSearchLocationToRegisterReaderRector

• class: Rector\PHPOffice\Rector\StaticCall\ChangeSearchLocationToRegisterReaderRector
• test fixtures

Change argument addSearchLocation() to registerReader()

 final class SomeClass
 {
     public function run(): void
     {
-        \PHPExcel_IOFactory::addSearchLocation($type, $location, $classname);
+        \PhpOffice\PhpSpreadsheet\IOFactory::registerReader($type, $classname);
     }
 }

GetDefaultStyleToGetParentRector

• class: Rector\PHPOffice\Rector\MethodCall\GetDefaultStyleToGetParentRector
• test fixtures

Methods to (new Worksheet())->getDefaultStyle() to getParent()->getDefaultStyle()

 class SomeClass
 {
     public function run()
     {
         $worksheet = new \PHPExcel_Worksheet();
-        $worksheet->getDefaultStyle();
+        $worksheet->getParent()->getDefaultStyle();
     }
 }

IncreaseColumnIndexRector

• class: Rector\PHPOffice\Rector\MethodCall\IncreaseColumnIndexRector
• test fixtures

Column index changed from 0 to 1 - run only ONCE! changes current value without memory

 final class SomeClass
 {
     public function run(): void
     {
         $worksheet = new \PHPExcel_Worksheet();
-        $worksheet->setCellValueByColumnAndRow(0, 3, '1150');
+        $worksheet->setCellValueByColumnAndRow(1, 3, '1150');
     }
 }

RemoveSetTempDirOnExcelWriterRector

• class: Rector\PHPOffice\Rector\MethodCall\RemoveSetTempDirOnExcelWriterRector
• test fixtures

Remove setTempDir() on PHPExcel_Writer_Excel5

 final class SomeClass
 {
     public function run(): void
     {
         $writer = new \PHPExcel_Writer_Excel5;
-        $writer->setTempDir();
     }
 }

PHPStan

PHPStormVarAnnotationRector

• class: Rector\PHPStan\Rector\Assign\PHPStormVarAnnotationRector
• test fixtures

Change various @var annotation formats to one PHPStorm understands

-$config = 5;
-/** @var \Shopsys\FrameworkBundle\Model\Product\Filter\ProductFilterConfig $config */
+/** @var \Shopsys\FrameworkBundle\Model\Product\Filter\ProductFilterConfig $config */
+$config = 5;

RecastingRemovalRector

• class: Rector\PHPStan\Rector\Cast\RecastingRemovalRector
• test fixtures

Removes recasting of the same type

 $string = '';
-$string = (string) $string;
+$string = $string;

 $array = [];
-$array = (array) $array;
+$array = $array;

RemoveNonExistingVarAnnotationRector

• class: Rector\PHPStan\Rector\Node\RemoveNonExistingVarAnnotationRector
• test fixtures

Removes non-existing @var annotations above the code

 class SomeClass
 {
     public function get()
     {
-        /** @var Training[] $trainings */
         return $this->getData();
     }
 }

PHPUnit

AddDoesNotPerformAssertionToNonAssertingTestRector

• class: Rector\PHPUnit\Rector\ClassMethod\AddDoesNotPerformAssertionToNonAssertingTestRector
• test fixtures

Tests without assertion will have @doesNotPerformAssertion

 class SomeClass extends PHPUnit\Framework\TestCase
 {
+    /**
+     * @doesNotPerformAssertions
+     */
     public function test()
     {
         $nothing = 5;
     }
 }

AddProphecyTraitRector

• class: Rector\PHPUnit\Rector\Class_\AddProphecyTraitRector
• test fixtures

Add Prophecy trait for method using $this->prophesize()

 use PHPUnit\Framework\TestCase;
+use Prophecy\PhpUnit\ProphecyTrait;

 final class ExampleTest extends TestCase
 {
+    use ProphecyTrait;
+
     public function testOne(): void
     {
         $prophecy = $this->prophesize(\AnInterface::class);
     }
 }

AddSeeTestAnnotationRector

• class: Rector\PHPUnit\Rector\Class_\AddSeeTestAnnotationRector
• test fixtures

Add @see annotation test of the class for faster jump to test. Make it FQN, so it stays in the annotation, not in the PHP source code.

+/**
+ * @see \SomeServiceTest
+ */
 class SomeService
 {
 }

 use PHPUnit\Framework\TestCase;

 class SomeServiceTest extends TestCase
 {
 }

ArrayArgumentInTestToDataProviderRector

• class: Rector\PHPUnit\Rector\Class_\ArrayArgumentInTestToDataProviderRector
• test fixtures

Move array argument from tests into data provider [configurable]

services:
    Rector\PHPUnit\Rector\Class_\ArrayArgumentInTestToDataProviderRector:
        $configuration:
            -
                class: PHPUnit\Framework\TestCase
                old_method: doTestMultiple
                new_method: doTestSingle
                variable_name: number

↓

 use PHPUnit\Framework\TestCase;

 class SomeServiceTest extends TestCase
 {
-    public function test()
+    /**
+     * @dataProvider provideData()
+     */
+    public function test(int $number)
     {
-        $this->doTestMultiple([1, 2, 3]);
+        $this->doTestSingle($number);
+    }
+
+    public function provideData(): \Iterator
+    {
+        yield [1];
+        yield [2];
+        yield [3];
     }
 }

AssertCompareToSpecificMethodRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertCompareToSpecificMethodRector
• test fixtures

Turns vague php-only method in PHPUnit TestCase to more specific

-$this->assertSame(10, count($anything), "message");
+$this->assertCount(10, $anything, "message");

-$this->assertNotEquals(get_class($value), stdClass::class);
+$this->assertNotInstanceOf(stdClass::class, $value);

AssertComparisonToSpecificMethodRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertComparisonToSpecificMethodRector
• test fixtures

Turns comparison operations to their method name alternatives in PHPUnit TestCase

-$this->assertTrue($foo === $bar, "message");
+$this->assertSame($bar, $foo, "message");

-$this->assertFalse($foo >= $bar, "message");
+$this->assertLessThanOrEqual($bar, $foo, "message");

AssertEqualsParameterToSpecificMethodsTypeRector

• class: Rector\PHPUnit\Rector\MethodCall\AssertEqualsParameterToSpecificMethodsTypeRector
• test fixtures

Change assertEquals()/assertNotEquals() method parameters to new specific alternatives

 final class SomeTest extends \PHPUnit\Framework\TestCase
 {
     public function test()
     {
         $value = 'value';
-        $this->assertEquals('string', $value, 'message', 5.0);
+        $this->assertEqualsWithDelta('string', $value, 5.0, 'message');

-        $this->assertEquals('string', $value, 'message', 0.0, 20);
+        $this->assertEquals('string', $value, 'message', 0.0);

-        $this->assertEquals('string', $value, 'message', 0.0, 10, true);
+        $this->assertEqualsCanonicalizing('string', $value, 'message');

-        $this->assertEquals('string', $value, 'message', 0.0, 10, false, true);
+        $this->assertEqualsIgnoringCase('string', $value, 'message');
     }
 }

AssertFalseStrposToContainsRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertFalseStrposToContainsRector
• test fixtures

Turns strpos/stripos comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertFalse(strpos($anything, "foo"), "message");
+$this->assertNotContains("foo", $anything, "message");

-$this->assertNotFalse(stripos($anything, "foo"), "message");
+$this->assertContains("foo", $anything, "message");

AssertInstanceOfComparisonRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertInstanceOfComparisonRector
• test fixtures

Turns instanceof comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertTrue($foo instanceof Foo, "message");
+$this->assertInstanceOf("Foo", $foo, "message");

-$this->assertFalse($foo instanceof Foo, "message");
+$this->assertNotInstanceOf("Foo", $foo, "message");

AssertIssetToSpecificMethodRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertIssetToSpecificMethodRector
• test fixtures

Turns isset comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertTrue(isset($anything->foo));
+$this->assertObjectHasAttribute("foo", $anything);

-$this->assertFalse(isset($anything["foo"]), "message");
+$this->assertArrayNotHasKey("foo", $anything, "message");

AssertNotOperatorRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertNotOperatorRector
• test fixtures

Turns not-operator comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertTrue(!$foo, "message");
+$this->assertFalse($foo, "message");

-$this->assertFalse(!$foo, "message");
+$this->assertTrue($foo, "message");

AssertPropertyExistsRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertPropertyExistsRector
• test fixtures

Turns property_exists comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertTrue(property_exists(new Class, "property"), "message");
+$this->assertClassHasAttribute("property", "Class", "message");

-$this->assertFalse(property_exists(new Class, "property"), "message");
+$this->assertClassNotHasAttribute("property", "Class", "message");

AssertRegExpRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertRegExpRector
• test fixtures

Turns preg_match comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertSame(1, preg_match("/^Message for ".*"\.$/", $string), $message);
+$this->assertRegExp("/^Message for ".*"\.$/", $string, $message);

-$this->assertEquals(false, preg_match("/^Message for ".*"\.$/", $string), $message);
+$this->assertNotRegExp("/^Message for ".*"\.$/", $string, $message);

AssertSameBoolNullToSpecificMethodRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertSameBoolNullToSpecificMethodRector
• test fixtures

Turns same bool and null comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertSame(null, $anything);
+$this->assertNull($anything);

-$this->assertNotSame(false, $anything);
+$this->assertNotFalse($anything);

AssertTrueFalseInternalTypeToSpecificMethodRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertTrueFalseInternalTypeToSpecificMethodRector
• test fixtures

Turns true/false with internal type comparisons to their method name alternatives in PHPUnit TestCase

-$this->assertTrue(is_{internal_type}($anything), "message");
+$this->assertInternalType({internal_type}, $anything, "message");

-$this->assertFalse(is_{internal_type}($anything), "message");
+$this->assertNotInternalType({internal_type}, $anything, "message");

AssertTrueFalseToSpecificMethodRector

• class: Rector\PHPUnit\Rector\SpecificMethod\AssertTrueFalseToSpecificMethodRector
• test fixtures

Turns true/false comparisons to their method name alternatives in PHPUnit TestCase when possible

-$this->assertTrue(is_readable($readmeFile), "message");
+$this->assertIsReadable($readmeFile, "message");

CreateMockToCreateStubRector

• class: Rector\PHPUnit\Rector\MethodCall\CreateMockToCreateStubRector
• test fixtures

Replaces createMock() with createStub() when relevant

 use PHPUnit\Framework\TestCase

 class MyTest extends TestCase
 {
     public function testItBehavesAsExpected(): void
     {
-        $stub = $this->createMock(\Exception::class);
+        $stub = $this->createStub(\Exception::class);
         $stub->method('getMessage')
             ->willReturn('a message');

         $mock = $this->createMock(\Exception::class);
         $mock->expects($this->once())
             ->method('getMessage')
             ->willReturn('a message');

         self::assertSame('a message', $stub->getMessage());
         self::assertSame('a message', $mock->getMessage());
     }
 }

DelegateExceptionArgumentsRector

• class: Rector\PHPUnit\Rector\DelegateExceptionArgumentsRector
• test fixtures

Takes setExpectedException() 2nd and next arguments to own methods in PHPUnit.

-$this->setExpectedException(Exception::class, "Message", "CODE");
+$this->setExpectedException(Exception::class);
+$this->expectExceptionMessage("Message");
+$this->expectExceptionCode("CODE");

EnsureDataProviderInDocBlockRector

• class: Rector\PHPUnit\Rector\ClassMethod\EnsureDataProviderInDocBlockRector
• test fixtures

Data provider annotation must be in doc block

 class SomeClass extends PHPUnit\Framework\TestCase
 {
-    /*
+    /**
      * @dataProvider testProvideData()
      */
     public function test()
     {
         $nothing = 5;
     }
 }

ExceptionAnnotationRector

• class: Rector\PHPUnit\Rector\ExceptionAnnotationRector
• test fixtures

Changes `@expectedException annotations to expectException*() methods

-/**
- * @expectedException Exception
- * @expectedExceptionMessage Message
- */
 public function test()
 {
+    $this->expectException('Exception');
+    $this->expectExceptionMessage('Message');
     // tested code
 }

ExplicitPhpErrorApiRector

• class: Rector\PHPUnit\Rector\MethodCall\ExplicitPhpErrorApiRector
• test fixtures

Use explicit API for expecting PHP errors, warnings, and notices

 final class SomeTest extends \PHPUnit\Framework\TestCase
 {
     public function test()
     {
-        $this->expectException(\PHPUnit\Framework\TestCase\Deprecated::class);
-        $this->expectException(\PHPUnit\Framework\TestCase\Error::class);
-        $this->expectException(\PHPUnit\Framework\TestCase\Notice::class);
-        $this->expectException(\PHPUnit\Framework\TestCase\Warning::class);
+        $this->expectDeprecation();
+        $this->expectError();
+        $this->expectNotice();
+        $this->expectWarning();
     }
 }

GetMockBuilderGetMockToCreateMockRector

• class: Rector\PHPUnit\Rector\MethodCall\GetMockBuilderGetMockToCreateMockRector
• test fixtures

Remove getMockBuilder() to createMock()

 class SomeTest extends \PHPUnit\Framework\TestCase
 {
     public function test()
     {
-        $applicationMock = $this->getMockBuilder('SomeClass')
-           ->disableOriginalConstructor()
-           ->getMock();
+        $applicationMock = $this->createMock('SomeClass');
     }
 }

GetMockRector

• class: Rector\PHPUnit\Rector\GetMockRector
• test fixtures

Turns getMock*() methods to createMock()

-$this->getMock("Class");
+$this->createMock("Class");

-$this->getMockWithoutInvokingTheOriginalConstructor("Class");
+$this->createMock("Class");

RemoveDataProviderTestPrefixRector

• class: Rector\PHPUnit\Rector\Class_\RemoveDataProviderTestPrefixRector
• test fixtures

Data provider methods cannot start with "test" prefix

 class SomeClass extends PHPUnit\Framework\TestCase
 {
     /**
-     * @dataProvider testProvideData()
+     * @dataProvider provideData()
      */
     public function test()
     {
         $nothing = 5;
     }

-    public function testProvideData()
+    public function provideData()
     {
         return ['123'];
     }
 }

RemoveEmptyTestMethodRector

• class: Rector\PHPUnit\Rector\ClassMethod\RemoveEmptyTestMethodRector
• test fixtures

Remove empty test methods

 class SomeTest extends \PHPUnit\Framework\TestCase
 {
-    /**
-     * testGetTranslatedModelField method
-     *
-     * @return void
-     */
-    public function testGetTranslatedModelField()
-    {
-    }
 }

RemoveExpectAnyFromMockRector

• class: Rector\PHPUnit\Rector\MethodCall\RemoveExpectAnyFromMockRector
• test fixtures

Remove expect($this->any()) from mocks as it has no added value

 use PHPUnit\Framework\TestCase;

 class SomeClass extends TestCase
 {
     public function test()
     {
         $translator = $this->getMock('SomeClass');
-        $translator->expects($this->any())
-            ->method('trans')
+        $translator->method('trans')
             ->willReturn('translated max {{ max }}!');
     }
 }

ReplaceAssertArraySubsetRector

• class: Rector\PHPUnit\Rector\MethodCall\ReplaceAssertArraySubsetRector
• test fixtures

Replace deprecated "assertArraySubset()" method with alternative methods

 class SomeTest extends \PHPUnit\Framework\TestCase
 {
     public function test()
     {
         $checkedArray = [];

-        $this->assertArraySubset([
-           'cache_directory' => 'new_value',
-        ], $checkedArray, true);
+        $this->assertArrayHasKey('cache_directory', $checkedArray);
+        $this->assertSame('new_value', $checkedArray['cache_directory']);
     }
 }

ReplaceAssertArraySubsetWithDmsPolyfillRector

• class: Rector\PHPUnit\Rector\MethodCall\ReplaceAssertArraySubsetWithDmsPolyfillRector
• test fixtures

Change assertArraySubset() to static call of DMS\PHPUnitExtensions\ArraySubset\Assert

 use PHPUnit\Framework\TestCase;

 class SomeClass extends TestCase
 {
     public function test()
     {
-        self::assertArraySubset(['bar' => 0], ['bar' => '0'], true);
+        \DMS\PHPUnitExtensions\ArraySubset\Assert::assertArraySubset(['bar' => 0], ['bar' => '0'], true);

-        $this->assertArraySubset(['bar' => 0], ['bar' => '0'], true);
+        \DMS\PHPUnitExtensions\ArraySubset\Assert::assertArraySubset(['bar' => 0], ['bar' => '0'], true);
     }
 }

SelfContainerGetMethodCallFromTestToInjectPropertyRector

• class: Rector\PHPUnit\Rector\Class_\SelfContainerGetMethodCallFromTestToInjectPropertyRector
• test fixtures

Change $container->get() calls in PHPUnit to @inject properties autowired by jakzal/phpunit-injector

 use PHPUnit\Framework\TestCase;
 class SomeClassTest extends TestCase {
+    /**
+     * @var SomeService
+     * @inject
+     */
+    private $someService;
     public function test()
     {
-        $someService = $this->getContainer()->get(SomeService::class);
+        $someService = $this->someService;
     }
 }
 class SomeService { }

SimplifyForeachInstanceOfRector

• class: Rector\PHPUnit\Rector\Foreach_\SimplifyForeachInstanceOfRector
• test fixtures

Simplify unnecessary foreach check of instances

-foreach ($foos as $foo) {
-    $this->assertInstanceOf(\SplFileInfo::class, $foo);
-}
+$this->assertContainsOnlyInstancesOf(\SplFileInfo::class, $foos);

SpecificAssertContainsRector

• class: Rector\PHPUnit\Rector\MethodCall\SpecificAssertContainsRector
• test fixtures

Change assertContains()/assertNotContains() method to new string and iterable alternatives

 <?php

 final class SomeTest extends \PHPUnit\Framework\TestCase
 {
     public function test()
     {
-        $this->assertContains('foo', 'foo bar');
-        $this->assertNotContains('foo', 'foo bar');
+        $this->assertStringContainsString('foo', 'foo bar');
+        $this->assertStringNotContainsString('foo', 'foo bar');
     }
 }

SpecificAssertContainsWithoutIdentityRector

• class: Rector\PHPUnit\Rector\MethodCall\SpecificAssertContainsWithoutIdentityRector
• test fixtures

Change assertContains()/assertNotContains() with non-strict comparison to new specific alternatives

 <?php

-final class SomeTest extends \PHPUnit\Framework\TestCase
+final class SomeTest extends TestCase
 {
     public function test()
     {
         $objects = [ new \stdClass(), new \stdClass(), new \stdClass() ];
-        $this->assertContains(new \stdClass(), $objects, 'message', false, false);
-        $this->assertNotContains(new \stdClass(), $objects, 'message', false, false);
+        $this->assertContainsEquals(new \stdClass(), $objects, 'message');
+        $this->assertNotContainsEquals(new \stdClass(), $objects, 'message');
     }
 }

SpecificAssertInternalTypeRector

• class: Rector\PHPUnit\Rector\MethodCall\SpecificAssertInternalTypeRector
• test fixtures

Change assertInternalType()/assertNotInternalType() method to new specific alternatives

 final class SomeTest extends \PHPUnit\Framework\TestCase
 {
     public function test()
     {
         $value = 'value';
-        $this->assertInternalType('string', $value);
-        $this->assertNotInternalType('array', $value);
+        $this->assertIsString($value);
+        $this->assertIsNotArray($value);
     }
 }

TestListenerToHooksRector

• class: Rector\PHPUnit\Rector\Class_\TestListenerToHooksRector
• test fixtures

Refactor "*TestListener.php" to particular "*Hook.php" files

 namespace App\Tests;

-use PHPUnit\Framework\TestListener;
-
-final class BeforeListHook implements TestListener
+final class BeforeListHook implements \PHPUnit\Runner\BeforeTestHook, \PHPUnit\Runner\AfterTestHook
 {
-    public function addError(Test $test, \Throwable $t, float $time): void
+    public function executeBeforeTest(Test $test): void
     {
-    }
-
-    public function addWarning(Test $test, Warning $e, float $time): void
-    {
-    }
-
-    public function addFailure(Test $test, AssertionFailedError $e, float $time): void
-    {
-    }
-
-    public function addIncompleteTest(Test $test, \Throwable $t, float $time): void
-    {
-    }
-
-    public function addRiskyTest(Test $test, \Throwable $t, float $time): void
-    {
-    }
-
-    public function addSkippedTest(Test $test, \Throwable $t, float $time): void
-    {
-    }
-
-    public function startTestSuite(TestSuite $suite): void
-    {
-    }
-
-    public function endTestSuite(TestSuite $suite): void
-    {
-    }
-
-    public function startTest(Test $test): void
-    {
         echo 'start test!';
     }

-    public function endTest(Test $test, float $time): void
+    public function executeAfterTest(Test $test, float $time): void
     {
         echo $time;
     }
 }

TryCatchToExpectExceptionRector

• class: Rector\PHPUnit\Rector\TryCatchToExpectExceptionRector
• test fixtures

Turns try/catch to expectException() call

-try {
-	$someService->run();
-} catch (Throwable $exception) {
-    $this->assertInstanceOf(RuntimeException::class, $e);
-    $this->assertContains('There was an error executing the following script', $e->getMessage());
-}
+$this->expectException(RuntimeException::class);
+$this->expectExceptionMessage('There was an error executing the following script');
+$someService->run();

UseSpecificWillMethodRector

• class: Rector\PHPUnit\Rector\MethodCall\UseSpecificWillMethodRector
• test fixtures

Changes ->will($this->xxx()) to one specific method

 class SomeClass extends PHPUnit\Framework\TestCase
 {
     public function test()
     {
         $translator = $this->getMockBuilder('Symfony\Component\Translation\TranslatorInterface')->getMock();
         $translator->expects($this->any())
             ->method('trans')
-            ->with($this->equalTo('old max {{ max }}!'))
-            ->will($this->returnValue('translated max {{ max }}!'));
+            ->with('old max {{ max }}!')
+            ->willReturnValue('translated max {{ max }}!');
     }
 }

WithConsecutiveArgToArrayRector

• class: Rector\PHPUnit\Rector\MethodCall\WithConsecutiveArgToArrayRector
• test fixtures

Split withConsecutive() arg to array

 class SomeClass
 {
     public function run($one, $two)
     {
     }
 }

 class SomeTestCase extends \PHPUnit\Framework\TestCase
 {
     public function test()
     {
         $someClassMock = $this->createMock(SomeClass::class);
         $someClassMock
             ->expects($this->exactly(2))
             ->method('run')
-            ->withConsecutive(1, 2, 3, 5);
+            ->withConsecutive([1, 2], [3, 5]);
     }
 }

PHPUnitSymfony

AddMessageToEqualsResponseCodeRector

• class: Rector\PHPUnitSymfony\Rector\StaticCall\AddMessageToEqualsResponseCodeRector
• test fixtures

Add response content to response code assert, so it is easier to debug

 use PHPUnit\Framework\TestCase;
 use Symfony\Component\HttpFoundation\Response;

 final class SomeClassTest extends TestCase
 {
     public function test(Response $response)
     {
         $this->assertEquals(
             Response::HTTP_NO_CONTENT,
             $response->getStatusCode()
+            $response->getContent()
         );
     }
 }

PSR4

MultipleClassFileToPsr4ClassesRector

• class: Rector\PSR4\Rector\MultipleClassFileToPsr4ClassesRector

Turns namespaced classes in one file to standalone PSR-4 classes.

+// new file: "app/Exceptions/FirstException.php"
 namespace App\Exceptions;

 use Exception;

 final class FirstException extends Exception
 {

 }
+
+// new file: "app/Exceptions/SecondException.php"
+namespace App\Exceptions;
+
+use Exception;

 final class SecondException extends Exception
 {

 }

NormalizeNamespaceByPSR4ComposerAutoloadFileSystemRector

• class: Rector\PSR4\Rector\FileSystem\NormalizeNamespaceByPSR4ComposerAutoloadFileSystemRector
• test fixtures

Adds namespace to namespace-less files to match PSR-4 in composer.json autoload section. Run with combination with Rector\PSR4\Rector\MultipleClassFileToPsr4ClassesRector

 // src/SomeClass.php

+namespace App\CustomNamespace;
+
 class SomeClass
 {
 }

composer.json

{
    "autoload": {
        "psr-4": {
            "App\\CustomNamespace\\": "src"
        }
    }
}

NormalizeNamespaceByPSR4ComposerAutoloadRector

• class: Rector\PSR4\Rector\Namespace_\NormalizeNamespaceByPSR4ComposerAutoloadRector
• test fixtures

Changes namespace and class names to match PSR-4 in composer.json autoload section

 // src/SomeClass.php

-namespace App\DifferentNamespace;
+namespace App\CustomNamespace;

 class SomeClass
 {
 }

composer.json

{
    "autoload": {
        "psr-4": {
            "App\\CustomNamespace\\": "src"
        }
    }
}

Performance

PreslashSimpleFunctionRector

• class: Rector\Performance\Rector\FuncCall\PreslashSimpleFunctionRector
• test fixtures

Add pre-slash to short named functions to improve performance

 class SomeClass
 {
     public function shorten($value)
     {
-        return trim($value);
+        return \trim($value);
     }
 }

Phalcon

AddRequestToHandleMethodCallRector

• class: Rector\Phalcon\Rector\MethodCall\AddRequestToHandleMethodCallRector
• test fixtures

Add $_SERVER REQUEST_URI to method call

 class SomeClass {
     public function run($di)
     {
         $application = new \Phalcon\Mvc\Application();
-        $response = $application->handle();
+        $response = $application->handle($_SERVER["REQUEST_URI"]);
     }
 }

DecoupleSaveMethodCallWithArgumentToAssignRector

• class: Rector\Phalcon\Rector\MethodCall\DecoupleSaveMethodCallWithArgumentToAssignRector
• test fixtures

Decouple Phalcon\Mvc\Model::save() with argument to assign()

 class SomeClass
 {
     public function run(\Phalcon\Mvc\Model $model, $data)
     {
-        $model->save($data);
+        $model->save();
+        $model->assign($data);
     }
 }

FlashWithCssClassesToExtraCallRector

• class: Rector\Phalcon\Rector\Assign\FlashWithCssClassesToExtraCallRector
• test fixtures

Add $cssClasses in Flash to separated method call

 class SomeClass {
     public function run()
     {
         $cssClasses = [];
-        $flash = new Phalcon\Flash($cssClasses);
+        $flash = new Phalcon\Flash();
+        $flash->setCssClasses($cssClasses);
     }
 }

NewApplicationToToFactoryWithDefaultContainerRector

• class: Rector\Phalcon\Rector\Assign\NewApplicationToToFactoryWithDefaultContainerRector
• test fixtures

Change new application to default factory with application

 class SomeClass
 {
     public function run($di)
     {
-        $application = new \Phalcon\Mvc\Application($di);
+        $container = new \Phalcon\Di\FactoryDefault();
+        $application = new \Phalcon\Mvc\Application($container);

-        $response = $application->handle();
+        $response = $application->handle($_SERVER["REQUEST_URI"]);
     }
 }

Php52

ContinueToBreakInSwitchRector

• class: Rector\Php52\Rector\Switch_\ContinueToBreakInSwitchRector
• test fixtures

Use break instead of continue in switch statements

 function some_run($value)
 {
     switch ($value) {
         case 1:
             echo 'Hi';
-            continue;
+            break;
         case 2:
             echo 'Hello';
             break;
     }
 }

VarToPublicPropertyRector

• class: Rector\Php52\Rector\Property\VarToPublicPropertyRector
• test fixtures

Remove unused private method

 final class SomeController
 {
-    var $name = 'Tom';
+    public $name = 'Tom';
 }

Php53

ClearReturnNewByReferenceRector

• class: Rector\Php53\Rector\Assign\ClearReturnNewByReferenceRector
• test fixtures

Remove reference from "$assign = &new Value;"

-$assign = &new Value;
+$assign = new Value;

DirNameFileConstantToDirConstantRector

• class: Rector\Php53\Rector\FuncCall\DirNameFileConstantToDirConstantRector
• test fixtures

Convert dirname(FILE) to DIR

 class SomeClass
 {
     public function run()
     {
-        return dirname(__FILE__);
+        return __DIR__;
     }
 }

ReplaceHttpServerVarsByServerRector

• class: Rector\Php53\Rector\Variable\ReplaceHttpServerVarsByServerRector
• test fixtures

Rename old $HTTP_* variable names to new replacements

-$serverVars = $HTTP_SERVER_VARS;
+$serverVars = $_SERVER;

TernaryToElvisRector

• class: Rector\Php53\Rector\Ternary\TernaryToElvisRector
• test fixtures

Use ?: instead of ?, where useful

 function elvis()
 {
-    $value = $a ? $a : false;
+    $value = $a ?: false;
 }

Php54

RemoveReferenceFromCallRector

• class: Rector\Php54\Rector\FuncCall\RemoveReferenceFromCallRector
• test fixtures

Remove & from function and method calls

 final class SomeClass
 {
     public function run($one)
     {
-        return strlen(&$one);
+        return strlen($one);
     }
 }

RemoveZeroBreakContinueRector

• class: Rector\Php54\Rector\Break_\RemoveZeroBreakContinueRector
• test fixtures

Remove 0 from break and continue

 class SomeClass
 {
     public function run($random)
     {
-        continue 0;
-        break 0;
+        continue;
+        break;

         $five = 5;
-        continue $five;
+        continue 5;

-        break $random;
+        break;
     }
 }

Php55

PregReplaceEModifierRector

• class: Rector\Php55\Rector\FuncCall\PregReplaceEModifierRector
• test fixtures

The /e modifier is no longer supported, use preg_replace_callback instead

 class SomeClass
 {
     public function run()
     {
-        $comment = preg_replace('~\b(\w)(\w+)~e', '"$1".strtolower("$2")', $comment);
+        $comment = preg_replace_callback('~\b(\w)(\w+)~', function ($matches) {
+              return($matches[1].strtolower($matches[2]));
+        }, , $comment);
     }
 }

StringClassNameToClassConstantRector

• class: Rector\Php55\Rector\String_\StringClassNameToClassConstantRector
• test fixtures

Replace string class names by ::class constant

 class AnotherClass
 {
 }

 class SomeClass
 {
     public function run()
     {
-        return 'AnotherClass';
+        return \AnotherClass::class;
     }
 }

Php56

AddDefaultValueForUndefinedVariableRector

• class: Rector\Php56\Rector\FunctionLike\AddDefaultValueForUndefinedVariableRector
• test fixtures

Adds default value for undefined variable

 class SomeClass
 {
     public function run()
     {
+        $a = null;
         if (rand(0, 1)) {
             $a = 5;
         }
         echo $a;
     }
 }

PowToExpRector

• class: Rector\Php56\Rector\FuncCall\PowToExpRector
• test fixtures

Changes pow(val, val2) to ** (exp) parameter

-pow(1, 2);
+1**2;

Php70

BreakNotInLoopOrSwitchToReturnRector

• class: Rector\Php70\Rector\Break_\BreakNotInLoopOrSwitchToReturnRector
• test fixtures

Convert break outside for/foreach/switch context to return

 class SomeClass
 {
     public function run()
     {
         if ($isphp5)
             return 1;
         else
             return 2;
-        break;
+        return;
     }
 }

CallUserMethodRector

• class: Rector\Php70\Rector\FuncCall\CallUserMethodRector
• test fixtures

Changes call_user_method()/call_user_method_array() to call_user_func()/call_user_func_array()

-call_user_method($method, $obj, "arg1", "arg2");
+call_user_func(array(&$obj, "method"), "arg1", "arg2");

EmptyListRector

• class: Rector\Php70\Rector\List_\EmptyListRector
• test fixtures

list() cannot be empty

-'list() = $values;'
+'list($unusedGenerated) = $values;'

EregToPregMatchRector

• class: Rector\Php70\Rector\FuncCall\EregToPregMatchRector
• test fixtures

Changes ereg*() to preg*() calls

-ereg("hi")
+preg_match("#hi#");

ExceptionHandlerTypehintRector

• class: Rector\Php70\Rector\FunctionLike\ExceptionHandlerTypehintRector
• test fixtures

Changes property @var annotations from annotation to type.

-function handler(Exception $exception) { ... }
+function handler(Throwable $exception) { ... }
 set_exception_handler('handler');

IfToSpaceshipRector

• class: Rector\Php70\Rector\If_\IfToSpaceshipRector
• test fixtures

Changes if/else to spaceship <=> where useful

 class SomeClass
 {
     public function run()
     {
         usort($languages, function ($a, $b) {
-            if ($a[0] === $b[0]) {
-                return 0;
-            }
-
-            return ($a[0] < $b[0]) ? 1 : -1;
+            return $b[0] <=> $a[0];
         });
     }
 }

ListSplitStringRector

• class: Rector\Php70\Rector\List_\ListSplitStringRector
• test fixtures

list() cannot split string directly anymore, use str_split()

-list($foo) = "string";
+list($foo) = str_split("string");

ListSwapArrayOrderRector

• class: Rector\Php70\Rector\List_\ListSwapArrayOrderRector
• test fixtures

list() assigns variables in reverse order - relevant in array assign

-list($a[], $a[]) = [1, 2];
+list($a[], $a[]) = array_reverse([1, 2]);

MultiDirnameRector

• class: Rector\Php70\Rector\FuncCall\MultiDirnameRector
• test fixtures

Changes multiple dirname() calls to one with nesting level

-dirname(dirname($path));
+dirname($path, 2);

NonVariableToVariableOnFunctionCallRector

• class: Rector\Php70\Rector\FuncCall\NonVariableToVariableOnFunctionCallRector
• test fixtures

Transform non variable like arguments to variable where a function or method expects an argument passed by reference

-reset(a());
+$a = a(); reset($a);

Php4ConstructorRector

• class: Rector\Php70\Rector\FunctionLike\Php4ConstructorRector
• test fixtures

Changes PHP 4 style constructor to __construct.

 class SomeClass
 {
-    public function SomeClass()
+    public function __construct()
     {
     }
 }

RandomFunctionRector

• class: Rector\Php70\Rector\FuncCall\RandomFunctionRector
• test fixtures

Changes rand, srand and getrandmax by new mt_* alternatives.

-rand();
+mt_rand();

ReduceMultipleDefaultSwitchRector

• class: Rector\Php70\Rector\Switch_\ReduceMultipleDefaultSwitchRector
• test fixtures

Remove first default switch, that is ignored

 switch ($expr) {
     default:
-         echo "Hello World";
-
-    default:
          echo "Goodbye Moon!";
          break;
 }

RenameMktimeWithoutArgsToTimeRector

• class: Rector\Php70\Rector\FuncCall\RenameMktimeWithoutArgsToTimeRector
• test fixtures

Renames mktime() without arguments to time()

 class SomeClass
 {
     public function run()
     {
         $time = mktime(1, 2, 3);
-        $nextTime = mktime();
+        $nextTime = time();
     }
 }

StaticCallOnNonStaticToInstanceCallRector

• class: Rector\Php70\Rector\StaticCall\StaticCallOnNonStaticToInstanceCallRector
• test fixtures

Changes static call to instance call, where not useful

 class Something
 {
     public function doWork()
     {
     }
 }

 class Another
 {
     public function run()
     {
-        return Something::doWork();
+        return (new Something)->doWork();
     }
 }

TernaryToNullCoalescingRector

• class: Rector\Php70\Rector\Ternary\TernaryToNullCoalescingRector
• test fixtures

Changes unneeded null check to ?? operator

-$value === null ? 10 : $value;
+$value ?? 10;

-isset($value) ? $value : 10;
+$value ?? 10;

TernaryToSpaceshipRector

• class: Rector\Php70\Rector\Ternary\TernaryToSpaceshipRector
• test fixtures

Use <=> spaceship instead of ternary with same effect

 function order_func($a, $b) {
-    return ($a < $b) ? -1 : (($a > $b) ? 1 : 0);
+    return $a <=> $b;
 }

ThisCallOnStaticMethodToStaticCallRector

• class: Rector\Php70\Rector\MethodCall\ThisCallOnStaticMethodToStaticCallRector
• test fixtures

Changes $this->call() to static method to static call

 class SomeClass
 {
     public static function run()
     {
-        $this->eat();
+        self::eat();
     }

     public static function eat()
     {
     }
 }

Php71

AssignArrayToStringRector

• class: Rector\Php71\Rector\Assign\AssignArrayToStringRector
• test fixtures

String cannot be turned into array by assignment anymore

-$string = '';
+$string = [];
 $string[] = 1;

BinaryOpBetweenNumberAndStringRector

• class: Rector\Php71\Rector\BinaryOp\BinaryOpBetweenNumberAndStringRector
• test fixtures

Change binary operation between some number + string to PHP 7.1 compatible version

 class SomeClass
 {
     public function run()
     {
-        $value = 5 + '';
-        $value = 5.0 + 'hi';
+        $value = 5 + 0;
+        $value = 5.0 + 0
     }
 }

CountOnNullRector

• class: Rector\Php71\Rector\FuncCall\CountOnNullRector
• test fixtures

Changes count() on null to safe ternary check

 $values = null;
-$count = count($values);
+$count = is_array($values) || $values instanceof Countable ? count($values) : 0;

IsIterableRector

• class: Rector\Php71\Rector\BinaryOp\IsIterableRector
• test fixtures

Changes is_array + Traversable check to is_iterable

-is_array($foo) || $foo instanceof Traversable;
+is_iterable($foo);

ListToArrayDestructRector

• class: Rector\Php71\Rector\List_\ListToArrayDestructRector
• test fixtures

Remove & from new &X

 class SomeClass
 {
     public function run()
     {
-        list($id1, $name1) = $data;
+        [$id1, $name1] = $data;

-        foreach ($data as list($id, $name)) {
+        foreach ($data as [$id, $name]) {
         }
     }
 }

MultiExceptionCatchRector

• class: Rector\Php71\Rector\TryCatch\MultiExceptionCatchRector
• test fixtures

Changes multi catch of same exception to single one | separated.

 try {
    // Some code...
-} catch (ExceptionType1 $exception) {
-   $sameCode;
-} catch (ExceptionType2 $exception) {
+} catch (ExceptionType1 | ExceptionType2 $exception) {
    $sameCode;
 }

PublicConstantVisibilityRector

• class: Rector\Php71\Rector\ClassConst\PublicConstantVisibilityRector
• test fixtures

Add explicit public constant visibility.

 class SomeClass
 {
-    const HEY = 'you';
+    public const HEY = 'you';
 }

RemoveExtraParametersRector

• class: Rector\Php71\Rector\FuncCall\RemoveExtraParametersRector
• test fixtures

Remove extra parameters

-strlen("asdf", 1);
+strlen("asdf");

ReservedObjectRector

• class: Rector\Php71\Rector\Name\ReservedObjectRector
• test fixtures

Changes reserved "Object" name to "Object" where can be configured

-class Object
+class SmartObject
 {
 }

Php72

BarewordStringRector

• class: Rector\Php72\Rector\ConstFetch\BarewordStringRector
• test fixtures

Changes unquoted non-existing constants to strings

-var_dump(VAR);
+var_dump("VAR");

CreateFunctionToAnonymousFunctionRector

• class: Rector\Php72\Rector\FuncCall\CreateFunctionToAnonymousFunctionRector
• test fixtures

Use anonymous functions instead of deprecated create_function()

 class ClassWithCreateFunction
 {
     public function run()
     {
-        $callable = create_function('$matches', "return '$delimiter' . strtolower(\$matches[1]);");
+        $callable = function($matches) use ($delimiter) {
+            return $delimiter . strtolower($matches[1]);
+        };
     }
 }

GetClassOnNullRector

• class: Rector\Php72\Rector\FuncCall\GetClassOnNullRector
• test fixtures

Null is no more allowed in get_class()

 final class SomeClass
 {
     public function getItem()
     {
         $value = null;
-        return get_class($value);
+        return $value !== null ? get_class($value) : self::class;
     }
 }

IsObjectOnIncompleteClassRector

• class: Rector\Php72\Rector\FuncCall\IsObjectOnIncompleteClassRector
• test fixtures

Incomplete class returns inverted bool on is_object()

 $incompleteObject = new __PHP_Incomplete_Class;
-$isObject = is_object($incompleteObject);
+$isObject = ! is_object($incompleteObject);

ListEachRector

• class: Rector\Php72\Rector\Each\ListEachRector
• test fixtures

each() function is deprecated, use key() and current() instead

-list($key, $callback) = each($callbacks);
+$key = key($callbacks);
+$callback = current($callbacks);
+next($callbacks);

ParseStrWithResultArgumentRector

• class: Rector\Php72\Rector\FuncCall\ParseStrWithResultArgumentRector
• test fixtures

Use $result argument in parse_str() function

-parse_str($this->query);
-$data = get_defined_vars();
+parse_str($this->query, $result);
+$data = $result;

ReplaceEachAssignmentWithKeyCurrentRector

• class: Rector\Php72\Rector\Each\ReplaceEachAssignmentWithKeyCurrentRector
• test fixtures

Replace each() assign outside loop

 $array = ['b' => 1, 'a' => 2];
-$eachedArray = each($array);
+$eachedArray[1] = current($array);
+$eachedArray['value'] = current($array);
+$eachedArray[0] = key($array);
+$eachedArray['key'] = key($array);
+next($array);

StringifyDefineRector

• class: Rector\Php72\Rector\FuncCall\StringifyDefineRector
• test fixtures

Make first argument of define() string

 class SomeClass
 {
     public function run(int $a)
     {
-         define(CONSTANT_2, 'value');
+         define('CONSTANT_2', 'value');
          define('CONSTANT', 'value');
     }
 }

StringsAssertNakedRector

• class: Rector\Php72\Rector\FuncCall\StringsAssertNakedRector
• test fixtures

String asserts must be passed directly to assert()

 function nakedAssert()
 {
-    assert('true === true');
-    assert("true === true");
+    assert(true === true);
+    assert(true === true);
 }

UnsetCastRector

• class: Rector\Php72\Rector\Unset_\UnsetCastRector
• test fixtures

Removes (unset) cast

-$different = (unset) $value;
+$different = null;

-$value = (unset) $value;
+unset($value);

WhileEachToForeachRector

• class: Rector\Php72\Rector\Each\WhileEachToForeachRector
• test fixtures

each() function is deprecated, use foreach() instead.

-while (list($key, $callback) = each($callbacks)) {
+foreach ($callbacks as $key => $callback) {
     // ...
 }

-while (list($key) = each($callbacks)) {
+foreach (array_keys($callbacks) as $key) {
     // ...
 }

Php73

ArrayKeyFirstLastRector

• class: Rector\Php73\Rector\FuncCall\ArrayKeyFirstLastRector
• test fixtures

Make use of array_key_first() and array_key_last()

-reset($items);
-$firstKey = key($items);
+$firstKey = array_key_first($items);

-end($items);
-$lastKey = key($items);
+$lastKey = array_key_last($items);

IsCountableRector

• class: Rector\Php73\Rector\BinaryOp\IsCountableRector
• test fixtures

Changes is_array + Countable check to is_countable

-is_array($foo) || $foo instanceof Countable;
+is_countable($foo);

JsonThrowOnErrorRector

• class: Rector\Php73\Rector\FuncCall\JsonThrowOnErrorRector
• test fixtures

Adds JSON_THROW_ON_ERROR to json_encode() and json_decode() to throw JsonException on error

-json_encode($content);
-json_decode($json);
+json_encode($content, JSON_THROW_ON_ERROR);
+json_decode($json, null, null, JSON_THROW_ON_ERROR);

RegexDashEscapeRector

• class: Rector\Php73\Rector\FuncCall\RegexDashEscapeRector
• test fixtures

Escape - in some cases

-preg_match("#[\w-()]#", 'some text');
+preg_match("#[\w\-()]#", 'some text');

RemoveMissingCompactVariableRector

• class: Rector\Php73\Rector\FuncCall\RemoveMissingCompactVariableRector
• test fixtures

Remove non-existing vars from compact()

 class SomeClass
 {
     public function run()
     {
         $value = 'yes';

-        compact('value', 'non_existing');
+        compact('value');
     }
 }

SensitiveConstantNameRector

• class: Rector\Php73\Rector\ConstFetch\SensitiveConstantNameRector
• test fixtures

Changes case insensitive constants to sensitive ones.

 define('FOO', 42, true);
 var_dump(FOO);
-var_dump(foo);
+var_dump(FOO);

SensitiveDefineRector

• class: Rector\Php73\Rector\FuncCall\SensitiveDefineRector
• test fixtures

Changes case insensitive constants to sensitive ones.

-define('FOO', 42, true);
+define('FOO', 42);

SensitiveHereNowDocRector

• class: Rector\Php73\Rector\String_\SensitiveHereNowDocRector
• test fixtures

Changes heredoc/nowdoc that contains closing word to safe wrapper name

-$value = <<<A
+$value = <<<A_WRAP
     A
-A
+A_WRAP

SetCookieRector

• class: Rector\Php73\Rector\FuncCall\SetCookieRector
• test fixtures

Convert setcookie argument to PHP7.3 option array

-setcookie('name', $value, 360);
+setcookie('name', $value, ['expires' => 360]);

-setcookie('name', $name, 0, '', '', true, true);
+setcookie('name', $name, ['expires' => 0, 'path' => '', 'domain' => '', 'secure' => true, 'httponly' => true]);

StringifyStrNeedlesRector

• class: Rector\Php73\Rector\FuncCall\StringifyStrNeedlesRector
• test fixtures

Makes needles explicit strings

 $needle = 5;
-$fivePosition = strpos('725', $needle);
+$fivePosition = strpos('725', (string) $needle);

Php74

AddLiteralSeparatorToNumberRector

• class: Rector\Php74\Rector\LNumber\AddLiteralSeparatorToNumberRector
• test fixtures

Add "_" as thousands separator in numbers

 class SomeClass
 {
     public function run()
     {
-        $int = 1000;
-        $float = 1000500.001;
+        $int = 1_000;
+        $float = 1_000_500.001;
     }
 }

ArrayKeyExistsOnPropertyRector

• class: Rector\Php74\Rector\FuncCall\ArrayKeyExistsOnPropertyRector
• test fixtures

Change array_key_exists() on property to property_exists()

 class SomeClass {
      public $value;
 }
 $someClass = new SomeClass;

-array_key_exists('value', $someClass);
+property_exists($someClass, 'value');

ArraySpreadInsteadOfArrayMergeRector

• class: Rector\Php74\Rector\FuncCall\ArraySpreadInsteadOfArrayMergeRector
• test fixtures

Change array_merge() to spread operator, except values with possible string key values

 class SomeClass
 {
     public function run($iter1, $iter2)
     {
-        $values = array_merge(iterator_to_array($iter1), iterator_to_array($iter2));
+        $values = [...$iter1, ...$iter2];

         // Or to generalize to all iterables
-        $anotherValues = array_merge(
-            is_array($iter1) ? $iter1 : iterator_to_array($iter1),
-            is_array($iter2) ? $iter2 : iterator_to_array($iter2)
-        );
+        $anotherValues = [...$iter1, ...$iter2];
     }
 }

ChangeReflectionTypeToStringToGetNameRector

• class: Rector\Php74\Rector\MethodCall\ChangeReflectionTypeToStringToGetNameRector
• test fixtures

Change string calls on ReflectionType

 class SomeClass
 {
     public function go(ReflectionFunction $reflectionFunction)
     {
         $parameterReflection = $reflectionFunction->getParameters()[0];

-        $paramType = (string) $parameterReflection->getType();
+        $paramType = (string) ($parameterReflection->getType() ? $parameterReflection->getType()->getName() : null);

-        $stringValue = 'hey' . $reflectionFunction->getReturnType();
+        $stringValue = 'hey' . ($reflectionFunction->getReturnType() ? $reflectionFunction->getReturnType()->getName() : null);

         // keep
         return $reflectionFunction->getReturnType();
     }
 }

ClassConstantToSelfClassRector

• class: Rector\Php74\Rector\MagicConstClass\ClassConstantToSelfClassRector
• test fixtures

Change CLASS to self::class

 class SomeClass
 {
    public function callOnMe()
    {
-       var_dump(__CLASS__);
+       var_dump(self::class);
    }
 }

ClosureToArrowFunctionRector

• class: Rector\Php74\Rector\Closure\ClosureToArrowFunctionRector
• test fixtures

Change closure to arrow function

 class SomeClass
 {
     public function run($meetups)
     {
-        return array_filter($meetups, function (Meetup $meetup) {
-            return is_object($meetup);
-        });
+        return array_filter($meetups, fn(Meetup $meetup) => is_object($meetup));
     }
 }

ExportToReflectionFunctionRector

• class: Rector\Php74\Rector\StaticCall\ExportToReflectionFunctionRector
• test fixtures

Change export() to ReflectionFunction alternatives

-$reflectionFunction = ReflectionFunction::export('foo');
-$reflectionFunctionAsString = ReflectionFunction::export('foo', true);
+$reflectionFunction = new ReflectionFunction('foo');
+$reflectionFunctionAsString = (string) new ReflectionFunction('foo');

FilterVarToAddSlashesRector

• class: Rector\Php74\Rector\FuncCall\FilterVarToAddSlashesRector
• test fixtures

Change filter_var() with slash escaping to addslashes()

 $var= "Satya's here!";
-filter_var($var, FILTER_SANITIZE_MAGIC_QUOTES);
+addslashes($var);

GetCalledClassToStaticClassRector

• class: Rector\Php74\Rector\FuncCall\GetCalledClassToStaticClassRector
• test fixtures

Change CLASS to self::class

 class SomeClass
 {
    public function callOnMe()
    {
-       var_dump(get_called_class());
+       var_dump(static::class);
    }
 }

MbStrrposEncodingArgumentPositionRector

• class: Rector\Php74\Rector\FuncCall\MbStrrposEncodingArgumentPositionRector
• test fixtures

Change mb_strrpos() encoding argument position

-mb_strrpos($text, "abc", "UTF-8");
+mb_strrpos($text, "abc", 0, "UTF-8");

NullCoalescingOperatorRector

• class: Rector\Php74\Rector\Assign\NullCoalescingOperatorRector
• test fixtures

Use null coalescing operator ??=

 $array = [];
-$array['user_id'] = $array['user_id'] ?? 'value';
+$array['user_id'] ??= 'value';

RealToFloatTypeCastRector

• class: Rector\Php74\Rector\Double\RealToFloatTypeCastRector
• test fixtures

Change deprecated (real) to (float)

 class SomeClass
 {
     public function run()
     {
-        $number = (real) 5;
+        $number = (float) 5;
         $number = (float) 5;
         $number = (double) 5;
     }
 }

ReservedFnFunctionRector

• class: Rector\Php74\Rector\Function_\ReservedFnFunctionRector
• test fixtures

Change fn() function name, since it will be reserved keyword

 class SomeClass
 {
     public function run()
     {
-        function fn($value)
+        function f($value)
         {
             return $value;
         }

-        fn(5);
+        f(5);
     }
 }

RestoreDefaultNullToNullableTypePropertyRector

• class: Rector\Php74\Rector\Property\RestoreDefaultNullToNullableTypePropertyRector
• test fixtures

Add null default to properties with PHP 7.4 property nullable type

 class SomeClass
 {
-    public ?string $name;
+    public ?string $name = null;
 }

TypedPropertyRector

• class: Rector\Php74\Rector\Property\TypedPropertyRector
• test fixtures

Changes property @var annotations from annotation to type.

 final class SomeClass
 {
-    /**
-     * @var int
-     */
-    private count;
+    private int count;
 }

Php80

AnnotationToAttributeRector

• class: Rector\Php80\Rector\Class_\AnnotationToAttributeRector
• test fixtures

Change annotation to attribute

 use Doctrine\ORM\Attributes as ORM;

-/**
-  * @ORM\Entity
-  */
+<<ORM\Entity>>
 class SomeClass
 {
 }

ClassOnObjectRector

• class: Rector\Php80\Rector\FuncCall\ClassOnObjectRector
• test fixtures

Change get_class($object) to faster $object::class

 class SomeClass
 {
     public function run($object)
     {
-        return get_class($object);
+        return $object::class;
     }
 }

GetDebugTypeRector

• class: Rector\Php80\Rector\Ternary\GetDebugTypeRector
• test fixtures

Change ternary type resolve to get_debug_type()

 class SomeClass
 {
     public function run($value)
     {
-        return is_object($value) ? get_class($value) : gettype($value);
+        return get_debug_type($value);
     }
 }

RemoveUnusedVariableInCatchRector

• class: Rector\Php80\Rector\Catch_\RemoveUnusedVariableInCatchRector
• test fixtures

Remove unused variable in catch()

 final class SomeClass
 {
     public function run()
     {
         try {
-        } catch (Throwable $notUsedThrowable) {
+        } catch (Throwable) {
         }
     }
 }

StrContainsRector

• class: Rector\Php80\Rector\NotIdentical\StrContainsRector
• test fixtures

Replace strpos() !== false and strstr() with str_contains()

 class SomeClass
 {
     public function run()
     {
-        return strpos('abc', 'a') !== false;
+        return str_contains('abc', 'a');
     }
 }

StrEndsWithRector

• class: Rector\Php80\Rector\Identical\StrEndsWithRector
• test fixtures

Change helper functions to str_ends_with()

 class SomeClass
 {
     public function run()
     {
-        $isMatch = substr($haystack, -strlen($needle)) === $needle;
+        $isMatch = str_ends_with($haystack, $needle);
     }
 }

StrStartsWithRector

• class: Rector\Php80\Rector\Identical\StrStartsWithRector
• test fixtures

Change helper functions to str_starts_with()

 class SomeClass
 {
     public function run()
     {
-        $isMatch = substr($haystack, 0, strlen($needle)) === $needle;
+        $isMatch = str_starts_with($haystack, $needle);

-        $isNotMatch = substr($haystack, 0, strlen($needle)) !== $needle;
+        $isNotMatch = ! str_starts_with($haystack, $needle);
     }
 }

StringableForToStringRector

• class: Rector\Php80\Rector\Class_\StringableForToStringRector
• test fixtures

Add Stringable interface to classes with __toString() method

-class SomeClass
+class SomeClass implements Stringable
 {
-    public function __toString()
+    public function __toString(): string
     {
         return 'I can stringz';
     }
 }

TokenGetAllToObjectRector

• class: Rector\Php80\Rector\FuncCall\TokenGetAllToObjectRector
• test fixtures

Complete missing constructor dependency instance by type

 final class SomeClass
 {
     public function run()
     {
-        $tokens = token_get_all($code);
-        foreach ($tokens as $token) {
-            if (is_array($token)) {
-               $name = token_name($token[0]);
-               $text = $token[1];
-            } else {
-               $name = null;
-               $text = $token;
-            }
+        $tokens = \PhpToken::getAll($code);
+        foreach ($tokens as $phpToken) {
+           $name = $phpToken->getTokenName();
+           $text = $phpToken->text;
         }
     }
 }

UnionTypesRector

• class: Rector\Php80\Rector\FunctionLike\UnionTypesRector
• test fixtures

Change docs types to union types, where possible (properties are covered by TypedPropertiesRector)

 class SomeClass {
     /**
      * @param array|int $number
      * @return bool|float
      */
-    public function go($number)
+    public function go(array|int $number): bool|float
     {
     }
 }

PhpDeglobalize

ChangeGlobalVariablesToPropertiesRector

• class: Rector\PhpDeglobalize\Rector\Class_\ChangeGlobalVariablesToPropertiesRector
• test fixtures

Change global $variables to private properties

 class SomeClass
 {
+    private $variable;
     public function go()
     {
-        global $variable;
-        $variable = 5;
+        $this->variable = 5;
     }

     public function run()
     {
-        global $variable;
-        var_dump($variable);
+        var_dump($this->variable);
     }
 }

PhpSpecToPHPUnit

AddMockPropertiesRector

• class: Rector\PhpSpecToPHPUnit\Rector\Class_\AddMockPropertiesRector

Migrate PhpSpec behavior to PHPUnit test

 namespace spec\SomeNamespaceForThisTest;

-use PhpSpec\ObjectBehavior;
-
 class OrderSpec extends ObjectBehavior
 {
-    public function let(OrderFactory $factory, ShippingMethod $shippingMethod)
+    /**
+     * @var \SomeNamespaceForThisTest\Order
+     */
+    private $order;
+    protected function setUp()
     {
-        $factory->createShippingMethodFor(Argument::any())->shouldBeCalled()->willReturn($shippingMethod);
+        /** @var OrderFactory|\PHPUnit\Framework\MockObject\MockObject $factory */
+        $factory = $this->createMock(OrderFactory::class);
+
+        /** @var ShippingMethod|\PHPUnit\Framework\MockObject\MockObject $shippingMethod */
+        $shippingMethod = $this->createMock(ShippingMethod::class);
+
+        $factory->expects($this->once())->method('createShippingMethodFor')->willReturn($shippingMethod);
     }
 }

MockVariableToPropertyFetchRector

• class: Rector\PhpSpecToPHPUnit\Rector\ClassMethod\MockVariableToPropertyFetchRector

Migrate PhpSpec behavior to PHPUnit test

 namespace spec\SomeNamespaceForThisTest;

-use PhpSpec\ObjectBehavior;
-
 class OrderSpec extends ObjectBehavior
 {
-    public function let(OrderFactory $factory, ShippingMethod $shippingMethod)
+    /**
+     * @var \SomeNamespaceForThisTest\Order
+     */
+    private $order;
+    protected function setUp()
     {
-        $factory->createShippingMethodFor(Argument::any())->shouldBeCalled()->willReturn($shippingMethod);
+        /** @var OrderFactory|\PHPUnit\Framework\MockObject\MockObject $factory */
+        $factory = $this->createMock(OrderFactory::class);
+
+        /** @var ShippingMethod|\PHPUnit\Framework\MockObject\MockObject $shippingMethod */
+        $shippingMethod = $this->createMock(ShippingMethod::class);
+
+        $factory->expects($this->once())->method('createShippingMethodFor')->willReturn($shippingMethod);
     }
 }

PhpSpecClassToPHPUnitClassRector

• class: Rector\PhpSpecToPHPUnit\Rector\Class_\PhpSpecClassToPHPUnitClassRector

Migrate PhpSpec behavior to PHPUnit test

 namespace spec\SomeNamespaceForThisTest;

-use PhpSpec\ObjectBehavior;
-
 class OrderSpec extends ObjectBehavior
 {
-    public function let(OrderFactory $factory, ShippingMethod $shippingMethod)
+    /**
+     * @var \SomeNamespaceForThisTest\Order
+     */
+    private $order;
+    protected function setUp()
     {
-        $factory->createShippingMethodFor(Argument::any())->shouldBeCalled()->willReturn($shippingMethod);
+        /** @var OrderFactory|\PHPUnit\Framework\MockObject\MockObject $factory */
+        $factory = $this->createMock(OrderFactory::class);
+
+        /** @var ShippingMethod|\PHPUnit\Framework\MockObject\MockObject $shippingMethod */
+        $shippingMethod = $this->createMock(ShippingMethod::class);
+
+        $factory->expects($this->once())->method('createShippingMethodFor')->willReturn($shippingMethod);
     }
 }

PhpSpecMethodToPHPUnitMethodRector

• class: Rector\PhpSpecToPHPUnit\Rector\ClassMethod\PhpSpecMethodToPHPUnitMethodRector

Migrate PhpSpec behavior to PHPUnit test

 namespace spec\SomeNamespaceForThisTest;

-use PhpSpec\ObjectBehavior;
-
 class OrderSpec extends ObjectBehavior
 {
-    public function let(OrderFactory $factory, ShippingMethod $shippingMethod)
+    /**
+     * @var \SomeNamespaceForThisTest\Order
+     */
+    private $order;
+    protected function setUp()
     {
-        $factory->createShippingMethodFor(Argument::any())->shouldBeCalled()->willReturn($shippingMethod);
+        /** @var OrderFactory|\PHPUnit\Framework\MockObject\MockObject $factory */
+        $factory = $this->createMock(OrderFactory::class);
+
+        /** @var ShippingMethod|\PHPUnit\Framework\MockObject\MockObject $shippingMethod */
+        $shippingMethod = $this->createMock(ShippingMethod::class);
+
+        $factory->expects($this->once())->method('createShippingMethodFor')->willReturn($shippingMethod);
     }
 }

PhpSpecMocksToPHPUnitMocksRector

• class: Rector\PhpSpecToPHPUnit\Rector\MethodCall\PhpSpecMocksToPHPUnitMocksRector

Migrate PhpSpec behavior to PHPUnit test

 namespace spec\SomeNamespaceForThisTest;

-use PhpSpec\ObjectBehavior;
-
 class OrderSpec extends ObjectBehavior
 {
-    public function let(OrderFactory $factory, ShippingMethod $shippingMethod)
+    /**
+     * @var \SomeNamespaceForThisTest\Order
+     */
+    private $order;
+    protected function setUp()
     {
-        $factory->createShippingMethodFor(Argument::any())->shouldBeCalled()->willReturn($shippingMethod);
+        /** @var OrderFactory|\PHPUnit\Framework\MockObject\MockObject $factory */
+        $factory = $this->createMock(OrderFactory::class);
+
+        /** @var ShippingMethod|\PHPUnit\Framework\MockObject\MockObject $shippingMethod */
+        $shippingMethod = $this->createMock(ShippingMethod::class);
+
+        $factory->expects($this->once())->method('createShippingMethodFor')->willReturn($shippingMethod);
     }
 }

PhpSpecPromisesToPHPUnitAssertRector

• class: Rector\PhpSpecToPHPUnit\Rector\MethodCall\PhpSpecPromisesToPHPUnitAssertRector

Migrate PhpSpec behavior to PHPUnit test

 namespace spec\SomeNamespaceForThisTest;

-use PhpSpec\ObjectBehavior;
-
 class OrderSpec extends ObjectBehavior
 {
-    public function let(OrderFactory $factory, ShippingMethod $shippingMethod)
+    /**
+     * @var \SomeNamespaceForThisTest\Order
+     */
+    private $order;
+    protected function setUp()
     {
-        $factory->createShippingMethodFor(Argument::any())->shouldBeCalled()->willReturn($shippingMethod);
+        /** @var OrderFactory|\PHPUnit\Framework\MockObject\MockObject $factory */
+        $factory = $this->createMock(OrderFactory::class);
+
+        /** @var ShippingMethod|\PHPUnit\Framework\MockObject\MockObject $shippingMethod */
+        $shippingMethod = $this->createMock(ShippingMethod::class);
+
+        $factory->expects($this->once())->method('createShippingMethodFor')->willReturn($shippingMethod);
     }
 }

RenameSpecFileToTestFileRector

• class: Rector\PhpSpecToPHPUnit\Rector\FileSystem\RenameSpecFileToTestFileRector

Rename "*Spec.php" file to "*Test.php" file

-// tests/SomeSpec.php
+// tests/SomeTest.php

Polyfill

UnwrapFutureCompatibleIfFunctionExistsRector

• class: Rector\Polyfill\Rector\If_\UnwrapFutureCompatibleIfFunctionExistsRector
• test fixtures

Remove functions exists if with else for always existing

 class SomeClass
 {
     public function run()
     {
         // session locking trough other addons
-        if (function_exists('session_abort')) {
-            session_abort();
-        } else {
-            session_write_close();
-        }
+        session_abort();
     }
 }

UnwrapFutureCompatibleIfPhpVersionRector

• class: Rector\Polyfill\Rector\If_\UnwrapFutureCompatibleIfPhpVersionRector
• test fixtures

Remove php version checks if they are passed

 // current PHP: 7.2
-if (version_compare(PHP_VERSION, '7.2', '<')) {
-    return 'is PHP 7.1-';
-} else {
-    return 'is PHP 7.2+';
-}
+return 'is PHP 7.2+';

Privatization

ChangeLocalPropertyToVariableRector

• class: Rector\Privatization\Rector\Class_\ChangeLocalPropertyToVariableRector
• test fixtures

Change local property used in single method to local variable

 class SomeClass
 {
-    private $count;
     public function run()
     {
-        $this->count = 5;
-        return $this->count;
+        $count = 5;
+        return $count;
     }
 }

PrivatizeFinalClassMethodRector

• class: Rector\Privatization\Rector\ClassMethod\PrivatizeFinalClassMethodRector
• test fixtures

Change protected class method to private if possible

 final class SomeClass
 {
-    protected function someMethod()
+    private function someMethod()
     {
     }
 }

PrivatizeFinalClassPropertyRector

• class: Rector\Privatization\Rector\Property\PrivatizeFinalClassPropertyRector
• test fixtures

Change property to private if possible

 final class SomeClass
 {
-    protected $value;
+    private $value;
 }

PrivatizeLocalClassConstantRector

• class: Rector\Privatization\Rector\ClassConst\PrivatizeLocalClassConstantRector
• test fixtures

Finalize every class constant that is used only locally

 class ClassWithConstantUsedOnlyHere
 {
-    const LOCAL_ONLY = true;
+    private const LOCAL_ONLY = true;

     public function isLocalOnly()
     {
         return self::LOCAL_ONLY;
     }
 }

PrivatizeLocalGetterToPropertyRector

• class: Rector\Privatization\Rector\MethodCall\PrivatizeLocalGetterToPropertyRector
• test fixtures

Privatize getter of local property to property

 class SomeClass
 {
     private $some;

     public function run()
     {
-        return $this->getSome() + 5;
+        return $this->some + 5;
     }

     private function getSome()
     {
         return $this->some;
     }
 }

PrivatizeLocalOnlyMethodRector

• class: Rector\Privatization\Rector\ClassMethod\PrivatizeLocalOnlyMethodRector
• test fixtures

Privatize local-only use methods

 class SomeClass
 {
     /**
      * @api
      */
     public function run()
     {
         return $this->useMe();
     }

-    public function useMe()
+    private function useMe()
     {
     }
 }

PrivatizeLocalPropertyToPrivatePropertyRector

• class: Rector\Privatization\Rector\Property\PrivatizeLocalPropertyToPrivatePropertyRector
• test fixtures

Privatize local-only property to private property

 class SomeClass
 {
-    public $value;
+    private $value;

     public function run()
     {
         return $this->value;
     }
 }

RemovingStatic

NewUniqueObjectToEntityFactoryRector

• class: Rector\RemovingStatic\Rector\Class_\NewUniqueObjectToEntityFactoryRector

Convert new X to new factories

-<?php
-
 class SomeClass
 {
+    public function __construct(AnotherClassFactory $anotherClassFactory)
+    {
+        $this->anotherClassFactory = $anotherClassFactory;
+    }
+
     public function run()
     {
-        return new AnotherClass;
+        return $this->anotherClassFactory->create();
     }
 }

 class AnotherClass
 {
     public function someFun()
     {
         return StaticClass::staticMethod();
     }
 }

PHPUnitStaticToKernelTestCaseGetRector

• class: Rector\RemovingStatic\Rector\Class_\PHPUnitStaticToKernelTestCaseGetRector
• test fixtures

Convert static calls in PHPUnit test cases, to get() from the container of KernelTestCase

services:
    Rector\RemovingStatic\Rector\Class_\PHPUnitStaticToKernelTestCaseGetRector:
        staticClassTypes:
            - EntityFactory

↓

-<?php
+use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;

-use PHPUnit\Framework\TestCase;
+final class SomeTestCase extends KernelTestCase
+{
+    /**
+     * @var EntityFactory
+     */
+    private $entityFactory;

-final class SomeTestCase extends TestCase
-{
+    protected function setUp(): void
+    {
+        parent::setUp();
+        $this->entityFactory = self::$container->get(EntityFactory::class);
+    }
+
     public function test()
     {
-        $product = EntityFactory::create('product');
+        $product = $this->entityFactory->create('product');
     }
 }

PassFactoryToUniqueObjectRector

• class: Rector\RemovingStatic\Rector\Class_\PassFactoryToUniqueObjectRector

Convert new X/Static::call() to factories in entities, pass them via constructor to each other

services:
    Rector\RemovingStatic\Rector\Class_\PassFactoryToUniqueObjectRector:
        $typesToServices:
            - StaticClass

↓

-<?php
-
 class SomeClass
 {
+    public function __construct(AnotherClassFactory $anotherClassFactory)
+    {
+        $this->anotherClassFactory = $anotherClassFactory;
+    }
+
     public function run()
     {
-        return new AnotherClass;
+        return $this->anotherClassFactory->create();
     }
 }

 class AnotherClass
 {
+    public function __construct(StaticClass $staticClass)
+    {
+        $this->staticClass = $staticClass;
+    }
+
     public function someFun()
     {
-        return StaticClass::staticMethod();
+        return $this->staticClass->staticMethod();
+    }
+}
+
+final class AnotherClassFactory
+{
+    /**
+     * @var StaticClass
+     */
+    private $staticClass;
+
+    public function __construct(StaticClass $staticClass)
+    {
+        $this->staticClass = $staticClass;
+    }
+
+    public function create(): AnotherClass
+    {
+        return new AnotherClass($this->staticClass);
     }
 }

StaticTypeToSetterInjectionRector

• class: Rector\RemovingStatic\Rector\Class_\StaticTypeToSetterInjectionRector
• test fixtures

Changes types to setter injection

services:
    Rector\RemovingStatic\Rector\Class_\StaticTypeToSetterInjectionRector:
        $staticTypes:
            - SomeStaticClass

↓

 <?php

 final class CheckoutEntityFactory
 {
+    /**
+     * @var SomeStaticClass
+     */
+    private $someStaticClass;
+
+    public function setSomeStaticClass(SomeStaticClass $someStaticClass)
+    {
+        $this->someStaticClass = $someStaticClass;
+    }
+
     public function run()
     {
-        return SomeStaticClass::go();
+        return $this->someStaticClass->go();
     }
 }

Renaming

RenameAnnotationRector

• class: Rector\Renaming\Rector\Annotation\RenameAnnotationRector
• test fixtures

Turns defined annotations above properties and methods to their new values.

services:
    Rector\Renaming\Rector\Annotation\RenameAnnotationRector:
        $classToAnnotationMap:
            PHPUnit\Framework\TestCase:
                test: scenario

↓

 class SomeTest extends PHPUnit\Framework\TestCase
 {
     /**
-     * @test
+     * @scenario
      */
     public function someMethod()
     {
     }
 }

RenameClassConstantRector

• class: Rector\Renaming\Rector\Constant\RenameClassConstantRector
• test fixtures

Replaces defined class constants in their calls.

services:
    Rector\Renaming\Rector\Constant\RenameClassConstantRector:
        SomeClass:
            OLD_CONSTANT: NEW_CONSTANT
            OTHER_OLD_CONSTANT: 'DifferentClass::NEW_CONSTANT'

↓

-$value = SomeClass::OLD_CONSTANT;
-$value = SomeClass::OTHER_OLD_CONSTANT;
+$value = SomeClass::NEW_CONSTANT;
+$value = DifferentClass::NEW_CONSTANT;

RenameClassRector

• class: Rector\Renaming\Rector\Class_\RenameClassRector
• test fixtures

Replaces defined classes by new ones.

services:
    Rector\Renaming\Rector\Class_\RenameClassRector:
        $oldToNewClasses:
            App\SomeOldClass: App\SomeNewClass

↓

 namespace App;

-use SomeOldClass;
+use SomeNewClass;

-function someFunction(SomeOldClass $someOldClass): SomeOldClass
+function someFunction(SomeNewClass $someOldClass): SomeNewClass
 {
-    if ($someOldClass instanceof SomeOldClass) {
-        return new SomeOldClass;
+    if ($someOldClass instanceof SomeNewClass) {
+        return new SomeNewClass;
     }
 }

RenameConstantRector

• class: Rector\Renaming\Rector\ConstFetch\RenameConstantRector
• test fixtures

Replace constant by new ones

 final class SomeClass
 {
     public function run()
     {
-        return MYSQL_ASSOC;
+        return MYSQLI_ASSOC;
     }
 }

RenameFuncCallToStaticCallRector

• class: Rector\Renaming\Rector\FuncCall\RenameFuncCallToStaticCallRector
• test fixtures

Rename func call to static call

services:
    Rector\Renaming\Rector\FuncCall\RenameFuncCallToStaticCallRector:
        $functionsToStaticCalls:
            strPee:
                - Strings
                - strPaa

↓

 class SomeClass
 {
     public function run()
     {
-        strPee('...');
+        \Strings::strPaa('...');
     }
 }

RenameFunctionRector

• class: Rector\Renaming\Rector\Function_\RenameFunctionRector
• test fixtures

Turns defined function call new one.

services:
    Rector\Renaming\Rector\Function_\RenameFunctionRector:
        $oldFunctionToNewFunction:
            view: Laravel\Templating\render

↓

-view("...", []);
+Laravel\Templating\render("...", []);

RenameMethodCallRector

• class: Rector\Renaming\Rector\MethodCall\RenameMethodCallRector
• test fixtures

Turns method call names to new ones.

services:
    Rector\Renaming\Rector\MethodCall\RenameMethodCallRector:
        SomeExampleClass:
            oldMethod: newMethod

↓

 $someObject = new SomeExampleClass;
-$someObject->oldMethod();
+$someObject->newMethod();

RenameMethodRector

• class: Rector\Renaming\Rector\MethodCall\RenameMethodRector
• test fixtures

Turns method names to new ones.

services:
    Rector\Renaming\Rector\MethodCall\RenameMethodRector:
        SomeExampleClass:
            $oldToNewMethodsByClass:
                oldMethod: newMethod

↓

 $someObject = new SomeExampleClass;
-$someObject->oldMethod();
+$someObject->newMethod();

RenameNamespaceRector

• class: Rector\Renaming\Rector\Namespace_\RenameNamespaceRector
• test fixtures

Replaces old namespace by new one.

services:
    Rector\Renaming\Rector\Namespace_\RenameNamespaceRector:
        $oldToNewNamespaces:
            SomeOldNamespace: SomeNewNamespace

↓

-$someObject = new SomeOldNamespace\SomeClass;
+$someObject = new SomeNewNamespace\SomeClass;

RenameStaticMethodRector

• class: Rector\Renaming\Rector\MethodCall\RenameStaticMethodRector
• test fixtures

Turns method names to new ones.

services:
    Rector\Renaming\Rector\MethodCall\RenameStaticMethodRector:
        SomeClass:
            oldMethod:
                - AnotherExampleClass
                - newStaticMethod

↓

-SomeClass::oldStaticMethod();
+AnotherExampleClass::newStaticMethod();

services:
    Rector\Renaming\Rector\MethodCall\RenameStaticMethodRector:
        $oldToNewMethodByClasses:
            SomeClass:
                oldMethod: newStaticMethod

↓

-SomeClass::oldStaticMethod();
+SomeClass::newStaticMethod();

Restoration

CompleteImportForPartialAnnotationRector

• class: Rector\Restoration\Rector\Namespace_\CompleteImportForPartialAnnotationRector
• test fixtures

In case you have accidentally removed use imports but code still contains partial use statements, this will save you

services:
    Rector\Restoration\Rector\Namespace_\CompleteImportForPartialAnnotationRector:
        $useImportsToRestore:
            -
                - Doctrine\ORM\Mapping
                - ORM

↓

+use Doctrine\ORM\Mapping as ORM;
+
 class SomeClass
 {
     /**
      * @ORM\Id
      */
     public $id;
 }

CompleteMissingDependencyInNewRector

• class: Rector\Restoration\Rector\New_\CompleteMissingDependencyInNewRector
• test fixtures

Complete missing constructor dependency instance by type

services:
    Rector\Restoration\Rector\New_\CompleteMissingDependencyInNewRector:
        $classToInstantiateByType:
            RandomDependency: RandomDependency

↓

 final class SomeClass
 {
     public function run()
     {
-        $valueObject = new RandomValueObject();
+        $valueObject = new RandomValueObject(new RandomDependency());
     }
 }

 class RandomValueObject
 {
     public function __construct(RandomDependency $randomDependency)
     {
     }
 }

MakeTypedPropertyNullableIfCheckedRector

• class: Rector\Restoration\Rector\Property\MakeTypedPropertyNullableIfCheckedRector
• test fixtures

Make typed property nullable if checked

 final class SomeClass
 {
-    private AnotherClass $anotherClass;
+    private ?AnotherClass $anotherClass = null;

     public function run()
     {
         if ($this->anotherClass === null) {
             $this->anotherClass = new AnotherClass;
         }
     }
 }

MissingClassConstantReferenceToStringRector

• class: Rector\Restoration\Rector\ClassConstFetch\MissingClassConstantReferenceToStringRector
• test fixtures

Convert missing class reference to string

 class SomeClass
 {
     public function run()
     {
-        return NonExistingClass::class;
+        return 'NonExistingClass';
     }
 }

RemoveFinalFromEntityRector

• class: Rector\Restoration\Rector\Class_\RemoveFinalFromEntityRector
• test fixtures

Remove final from Doctrine entities

 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity
  */
-final class SomeClass
+class SomeClass
 {
 }

RemoveUselessJustForSakeInterfaceRector

• class: Rector\Restoration\Rector\Class_\RemoveUselessJustForSakeInterfaceRector
• test fixtures

Remove interface, that are added just for its sake, but nowhere useful

-class SomeClass implements OnlyHereUsedInterface
+class SomeClass
 {
 }

-interface OnlyHereUsedInterface
-{
-}
-
 class SomePresenter
 {
-    public function __construct(OnlyHereUsedInterface $onlyHereUsed)
+    public function __construct(SomeClass $onlyHereUsed)
     {
     }
 }

UpdateFileNameByClassNameFileSystemRector

• class: Rector\Restoration\Rector\FileSystem\UpdateFileNameByClassNameFileSystemRector
• test fixtures

Rename file to respect class name

-// app/SomeClass.php
+// app/AnotherClass.php
 class AnotherClass
 {
 }

SOLID

AddFalseDefaultToBoolPropertyRector

• class: Rector\SOLID\Rector\Property\AddFalseDefaultToBoolPropertyRector
• test fixtures

Add false default to bool properties, to prevent null compare errors

 class SomeClass
 {
     /**
      * @var bool
      */
-    private $isDisabled;
+    private $isDisabled = false;
 }

ChangeIfElseValueAssignToEarlyReturnRector

• class: Rector\SOLID\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector
• test fixtures

Change if/else value to early return

 class SomeClass
 {
     public function run()
     {
         if ($this->hasDocBlock($tokens, $index)) {
-            $docToken = $tokens[$this->getDocBlockIndex($tokens, $index)];
-        } else {
-            $docToken = null;
+            return $tokens[$this->getDocBlockIndex($tokens, $index)];
         }
-
-        return $docToken;
+        return null;
     }
 }

ChangeNestedForeachIfsToEarlyContinueRector

• class: Rector\SOLID\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector
• test fixtures

Change nested ifs to foreach with continue

 class SomeClass
 {
     public function run()
     {
         $items = [];

         foreach ($values as $value) {
-            if ($value === 5) {
-                if ($value2 === 10) {
-                    $items[] = 'maybe';
-                }
+            if ($value !== 5) {
+                continue;
             }
+            if ($value2 !== 10) {
+                continue;
+            }
+
+            $items[] = 'maybe';
         }
     }
 }

ChangeNestedIfsToEarlyReturnRector

• class: Rector\SOLID\Rector\If_\ChangeNestedIfsToEarlyReturnRector
• test fixtures

Change nested ifs to early return

 class SomeClass
 {
     public function run()
     {
-        if ($value === 5) {
-            if ($value2 === 10) {
-                return 'yes';
-            }
+        if ($value !== 5) {
+            return 'no';
+        }
+
+        if ($value2 === 10) {
+            return 'yes';
         }

         return 'no';
     }
 }

ChangeReadOnlyPropertyWithDefaultValueToConstantRector

• class: Rector\SOLID\Rector\Property\ChangeReadOnlyPropertyWithDefaultValueToConstantRector
• test fixtures

Change property with read only status with default value to constant

 class SomeClass
 {
     /**
      * @var string[]
      */
-    private $magicMethods = [
+    private const MAGIC_METHODS = [
         '__toString',
         '__wakeup',
     ];

     public function run()
     {
-        foreach ($this->magicMethods as $magicMethod) {
+        foreach (self::MAGIC_METHODS as $magicMethod) {
             echo $magicMethod;
         }
     }
 }

ChangeReadOnlyVariableWithDefaultValueToConstantRector

• class: Rector\SOLID\Rector\ClassMethod\ChangeReadOnlyVariableWithDefaultValueToConstantRector
• test fixtures

Change variable with read only status with default value to constant

 class SomeClass
 {
+    /**
+     * @var string[]
+     */
+    private const REPLACEMENTS = [
+        'PHPUnit\Framework\TestCase\Notice' => 'expectNotice',
+        'PHPUnit\Framework\TestCase\Deprecated' => 'expectDeprecation',
+    ];
+
     public function run()
     {
-        $replacements = [
-            'PHPUnit\Framework\TestCase\Notice' => 'expectNotice',
-            'PHPUnit\Framework\TestCase\Deprecated' => 'expectDeprecation',
-        ];
-
-        foreach ($replacements as $class => $method) {
+        foreach (self::REPLACEMENTS as $class => $method) {
         }
     }
 }

FinalizeClassesWithoutChildrenRector

• class: Rector\SOLID\Rector\Class_\FinalizeClassesWithoutChildrenRector
• test fixtures

Finalize every class that has no children

-class FirstClass
+final class FirstClass
 {
 }

 class SecondClass
 {
 }

-class ThirdClass extends SecondClass
+final class ThirdClass extends SecondClass
 {
 }

MakeUnusedClassesWithChildrenAbstractRector

• class: Rector\SOLID\Rector\Class_\MakeUnusedClassesWithChildrenAbstractRector
• test fixtures

Classes that have no children nor are used, should have abstract

 class SomeClass extends PossibleAbstractClass
 {
 }

-class PossibleAbstractClass
+abstract class PossibleAbstractClass
 {
 }

MultiParentingToAbstractDependencyRector

• class: Rector\SOLID\Rector\Class_\MultiParentingToAbstractDependencyRector
• test fixtures

Move dependency passed to all children to parent as @inject/@required dependency

services:
    Rector\SOLID\Rector\Class_\MultiParentingToAbstractDependencyRector:
        $framework: nette

↓

 abstract class AbstractParentClass
 {
-    private $someDependency;
-
-    public function __construct(SomeDependency $someDependency)
-    {
-        $this->someDependency = $someDependency;
-    }
+    /**
+     * @inject
+     * @var SomeDependency
+     */
+    public $someDependency;
 }

 class FirstChild extends AbstractParentClass
 {
-    public function __construct(SomeDependency $someDependency)
-    {
-        parent::__construct($someDependency);
-    }
 }

 class SecondChild extends AbstractParentClass
 {
-    public function __construct(SomeDependency $someDependency)
-    {
-        parent::__construct($someDependency);
-    }
 }

RemoveAlwaysElseRector

• class: Rector\SOLID\Rector\If_\RemoveAlwaysElseRector
• test fixtures

Split if statement, when if condition always break execution flow

 class SomeClass
 {
     public function run($value)
     {
         if ($value) {
             throw new \InvalidStateException;
-        } else {
-            return 10;
         }
+
+        return 10;
     }
 }

RepeatedLiteralToClassConstantRector

• class: Rector\SOLID\Rector\Class_\RepeatedLiteralToClassConstantRector
• test fixtures

Replace repeated strings with constant

 class SomeClass
 {
+    /**
+     * @var string
+     */
+    private const REQUIRES = 'requires';
     public function run($key, $items)
     {
-        if ($key === 'requires') {
-            return $items['requires'];
+        if ($key === self::REQUIRES) {
+            return $items[self::REQUIRES];
         }
     }
 }

UseInterfaceOverImplementationInConstructorRector

• class: Rector\SOLID\Rector\ClassMethod\UseInterfaceOverImplementationInConstructorRector
• test fixtures

Use interface instead of specific class

 class SomeClass
 {
-    public function __construct(SomeImplementation $someImplementation)
+    public function __construct(SomeInterface $someImplementation)
     {
     }
 }

 class SomeImplementation implements SomeInterface
 {
 }

 interface SomeInterface
 {
 }

Sensio

RemoveServiceFromSensioRouteRector

• class: Rector\Sensio\Rector\ClassMethod\RemoveServiceFromSensioRouteRector
• test fixtures

Remove service from Sensio @Route

 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;

 final class SomeClass
 {
     /**
-     * @Route(service="some_service")
+     * @Route()
      */
     public function run()
     {
     }
 }

ReplaceSensioRouteAnnotationWithSymfonyRector

• class: Rector\Sensio\Rector\ClassMethod\ReplaceSensioRouteAnnotationWithSymfonyRector
• test fixtures

Replace Sensio @Route annotation with Symfony one

-use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
+use Symfony\Component\Routing\Annotation\Route;

 final class SomeClass
 {
     /**
      * @Route()
      */
     public function run()
     {
     }
 }

TemplateAnnotationRector

• class: Rector\Sensio\Rector\FrameworkExtraBundle\TemplateAnnotationRector

Turns @Template annotation to explicit method call in Controller of FrameworkExtraBundle in Symfony

-/**
- * @Template()
- */
 public function indexAction()
 {
+    return $this->render("index.html.twig");
 }

StrictCodeQuality

VarInlineAnnotationToAssertRector

• class: Rector\StrictCodeQuality\Rector\Stmt\VarInlineAnnotationToAssertRector
• test fixtures

Turn @var inline checks above code to assert() of hte type

 class SomeClass
 {
     public function run()
     {
         /** @var SpecificClass $value */
+        assert($value instanceof SpecificClass);
         $value->call();
     }
 }

Symfony

ActionSuffixRemoverRector

• class: Rector\Symfony\Rector\Controller\ActionSuffixRemoverRector
• test fixtures

Removes Action suffixes from methods in Symfony Controllers

 class SomeController
 {
-    public function indexAction()
+    public function index()
     {
     }
 }

AddFlashRector

• class: Rector\Symfony\Rector\Controller\AddFlashRector
• test fixtures

Turns long flash adding to short helper method in Controller in Symfony

 class SomeController extends Controller
 {
     public function some(Request $request)
     {
-        $request->getSession()->getFlashBag()->add("success", "something");
+        $this->addFlash("success", "something");
     }
 }

CascadeValidationFormBuilderRector

• class: Rector\Symfony\Rector\MethodCall\CascadeValidationFormBuilderRector
• test fixtures

Change "cascade_validation" option to specific node attribute

 class SomeController
 {
     public function someMethod()
     {
-        $form = $this->createFormBuilder($article, ['cascade_validation' => true])
-            ->add('author', new AuthorType())
+        $form = $this->createFormBuilder($article)
+            ->add('author', new AuthorType(), [
+                'constraints' => new \Symfony\Component\Validator\Constraints\Valid(),
+            ])
             ->getForm();
     }

     protected function createFormBuilder()
     {
         return new FormBuilder();
     }
 }

ConsoleExceptionToErrorEventConstantRector

• class: Rector\Symfony\Rector\Console\ConsoleExceptionToErrorEventConstantRector
• test fixtures

Turns old event name with EXCEPTION to ERROR constant in Console in Symfony

-"console.exception"
+Symfony\Component\Console\ConsoleEvents::ERROR

-Symfony\Component\Console\ConsoleEvents::EXCEPTION
+Symfony\Component\Console\ConsoleEvents::ERROR

ConsoleExecuteReturnIntRector

• class: Rector\Symfony\Rector\Console\ConsoleExecuteReturnIntRector
• test fixtures

Returns int from Command::execute command

 class SomeCommand extends Command
 {
-    public function execute(InputInterface $input, OutputInterface $output)
+    public function index(InputInterface $input, OutputInterface $output): int
     {
-        return null;
+        return 0;
     }
 }

ConstraintUrlOptionRector

• class: Rector\Symfony\Rector\Validator\ConstraintUrlOptionRector
• test fixtures

Turns true value to Url::CHECK_DNS_TYPE_ANY in Validator in Symfony.

-$constraint = new Url(["checkDNS" => true]);
+$constraint = new Url(["checkDNS" => Url::CHECK_DNS_TYPE_ANY]);

ContainerBuilderCompileEnvArgumentRector

• class: Rector\Symfony\Rector\DependencyInjection\ContainerBuilderCompileEnvArgumentRector
• test fixtures

Turns old default value to parameter in ContinerBuilder->build() method in DI in Symfony

 use Symfony\Component\DependencyInjection\ContainerBuilder;

 $containerBuilder = new ContainerBuilder();
-$containerBuilder->compile();
+$containerBuilder->compile(true);

ContainerGetToConstructorInjectionRector

• class: Rector\Symfony\Rector\FrameworkBundle\ContainerGetToConstructorInjectionRector
• test fixtures

Turns fetching of dependencies via $container->get() in ContainerAware to constructor injection in Command and Controller in Symfony

-final class SomeCommand extends ContainerAwareCommand
+final class SomeCommand extends Command
 {
+    public function __construct(SomeService $someService)
+    {
+        $this->someService = $someService;
+    }
+
     public function someMethod()
     {
         // ...
-        $this->getContainer()->get('some_service');
-        $this->container->get('some_service');
+        $this->someService;
+        $this->someService;
     }
 }

FormIsValidRector

• class: Rector\Symfony\Rector\Form\FormIsValidRector
• test fixtures

Adds $form->isSubmitted() validation to all $form->isValid() calls in Form in Symfony

-if ($form->isValid()) {
+if ($form->isSubmitted() && $form->isValid()) {
 }

FormTypeGetParentRector

• class: Rector\Symfony\Rector\Form\FormTypeGetParentRector
• test fixtures

Turns string Form Type references to their CONSTANT alternatives in getParent() and getExtendedType() methods in Form in Symfony

-function getParent() { return "collection"; }
+function getParent() { return CollectionType::class; }

-function getExtendedType() { return "collection"; }
+function getExtendedType() { return CollectionType::class; }

FormTypeInstanceToClassConstRector

• class: Rector\Symfony\Rector\MethodCall\FormTypeInstanceToClassConstRector
• test fixtures

Changes createForm(new FormType), add(new FormType) to ones with "FormType::class"

 class SomeController
 {
     public function action()
     {
-        $form = $this->createForm(new TeamType, $entity, [
+        $form = $this->createForm(TeamType::class, $entity, [
             'action' => $this->generateUrl('teams_update', ['id' => $entity->getId()]),
             'method' => 'PUT',
         ]);
     }
 }

GetParameterToConstructorInjectionRector

• class: Rector\Symfony\Rector\FrameworkBundle\GetParameterToConstructorInjectionRector
• test fixtures

Turns fetching of parameters via getParameter() in ContainerAware to constructor injection in Command and Controller in Symfony

-class MyCommand extends ContainerAwareCommand
+class MyCommand extends Command
 {
+    private $someParameter;
+
+    public function __construct($someParameter)
+    {
+        $this->someParameter = $someParameter;
+    }
+
     public function someMethod()
     {
-        $this->getParameter('someParameter');
+        $this->someParameter;
     }
 }

GetRequestRector

• class: Rector\Symfony\Rector\HttpKernel\GetRequestRector
• test fixtures

Turns fetching of dependencies via $this->get() to constructor injection in Command and Controller in Symfony

+use Symfony\Component\HttpFoundation\Request;
+
 class SomeController
 {
-    public function someAction()
+    public function someAction(Request $request)
     {
-        $this->getRequest()->...();
+        $request->...();
     }
 }

GetToConstructorInjectionRector

• class: Rector\Symfony\Rector\FrameworkBundle\GetToConstructorInjectionRector
• test fixtures

Turns fetching of dependencies via $this->get() to constructor injection in Command and Controller in Symfony

-class MyCommand extends ContainerAwareCommand
+class MyCommand extends Command
 {
+    public function __construct(SomeService $someService)
+    {
+        $this->someService = $someService;
+    }
+
     public function someMethod()
     {
-        // ...
-        $this->get('some_service');
+        $this->someService;
     }
 }

MakeCommandLazyRector

• class: Rector\Symfony\Rector\Class_\MakeCommandLazyRector
• test fixtures

Make Symfony commands lazy

 use Symfony\Component\Console\Command\Command

 class SunshineCommand extends Command
 {
+    protected static $defaultName = 'sunshine';
     public function configure()
     {
-        $this->setName('sunshine');
     }
 }

MakeDispatchFirstArgumentEventRector

• class: Rector\Symfony\Rector\MethodCall\MakeDispatchFirstArgumentEventRector
• test fixtures

Make event object a first argument of dispatch() method, event name as second

 use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;

 class SomeClass
 {
     public function run(EventDispatcherInterface $eventDispatcher)
     {
-        $eventDispatcher->dispatch('event_name', new Event());
+        $eventDispatcher->dispatch(new Event(), 'event_name');
     }
 }

MergeMethodAnnotationToRouteAnnotationRector

• class: Rector\Symfony\Rector\ClassMethod\MergeMethodAnnotationToRouteAnnotationRector
• test fixtures

Merge removed @Method annotation to @Route one

-use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
 use Symfony\Component\Routing\Annotation\Route;

 class DefaultController extends Controller
 {
     /**
-     * @Route("/show/{id}")
-     * @Method({"GET", "HEAD"})
+     * @Route("/show/{id}", methods={"GET","HEAD"})
      */
     public function show($id)
     {
     }
 }

OptionNameRector

• class: Rector\Symfony\Rector\Form\OptionNameRector
• test fixtures

Turns old option names to new ones in FormTypes in Form in Symfony

 $builder = new FormBuilder;
-$builder->add("...", ["precision" => "...", "virtual" => "..."];
+$builder->add("...", ["scale" => "...", "inherit_data" => "..."];

ParseFileRector

• class: Rector\Symfony\Rector\Yaml\ParseFileRector
• test fixtures

session > use_strict_mode is true by default and can be removed

-session > use_strict_mode: true
+session:

ProcessBuilderGetProcessRector

• class: Rector\Symfony\Rector\Process\ProcessBuilderGetProcessRector
• test fixtures

Removes $processBuilder->getProcess() calls to $processBuilder in Process in Symfony, because ProcessBuilder was removed. This is part of multi-step Rector and has very narrow focus.

 $processBuilder = new Symfony\Component\Process\ProcessBuilder;
-$process = $processBuilder->getProcess();
-$commamdLine = $processBuilder->getProcess()->getCommandLine();
+$process = $processBuilder;
+$commamdLine = $processBuilder->getCommandLine();

ProcessBuilderInstanceRector

• class: Rector\Symfony\Rector\Process\ProcessBuilderInstanceRector
• test fixtures

Turns ProcessBuilder::instance() to new ProcessBuilder in Process in Symfony. Part of multi-step Rector.

-$processBuilder = Symfony\Component\Process\ProcessBuilder::instance($args);
+$processBuilder = new Symfony\Component\Process\ProcessBuilder($args);

ReadOnlyOptionToAttributeRector

• class: Rector\Symfony\Rector\MethodCall\ReadOnlyOptionToAttributeRector
• test fixtures

Change "read_only" option in form to attribute

 use Symfony\Component\Form\FormBuilderInterface;

 function buildForm(FormBuilderInterface $builder, array $options)
 {
-    $builder->add('cuid', TextType::class, ['read_only' => true]);
+    $builder->add('cuid', TextType::class, ['attr' => ['read_only' => true]]);
 }

RedirectToRouteRector

• class: Rector\Symfony\Rector\Controller\RedirectToRouteRector
• test fixtures

Turns redirect to route to short helper method in Controller in Symfony

-$this->redirect($this->generateUrl("homepage"));
+$this->redirectToRoute("homepage");

ResponseStatusCodeRector

• class: Rector\Symfony\Rector\BinaryOp\ResponseStatusCodeRector
• test fixtures

Turns status code numbers to constants

 class SomeController
 {
     public function index()
     {
         $response = new \Symfony\Component\HttpFoundation\Response();
-        $response->setStatusCode(200);
+        $response->setStatusCode(\Symfony\Component\HttpFoundation\Response::HTTP_OK);

-        if ($response->getStatusCode() === 200) {}
+        if ($response->getStatusCode() === \Symfony\Component\HttpFoundation\Response::HTTP_OK) {}
     }
 }

RootNodeTreeBuilderRector

• class: Rector\Symfony\Rector\New_\RootNodeTreeBuilderRector
• test fixtures

Changes Process string argument to an array

 use Symfony\Component\Config\Definition\Builder\TreeBuilder;

-$treeBuilder = new TreeBuilder();
-$rootNode = $treeBuilder->root('acme_root');
+$treeBuilder = new TreeBuilder('acme_root');
+$rootNode = $treeBuilder->getRootNode();
 $rootNode->someCall();

SimplifyWebTestCaseAssertionsRector

• class: Rector\Symfony\Rector\MethodCall\SimplifyWebTestCaseAssertionsRector
• test fixtures

Simplify use of assertions in WebTestCase

 use PHPUnit\Framework\TestCase;

 class SomeClass extends TestCase
 {
     public function test()
     {
-        $this->assertSame(200, $client->getResponse()->getStatusCode());
+         $this->assertResponseIsSuccessful();
     }

     public function testUrl()
     {
-        $this->assertSame(301, $client->getResponse()->getStatusCode());
-        $this->assertSame('https://example.com', $client->getResponse()->headers->get('Location'));
+        $this->assertResponseRedirects('https://example.com', 301);
     }

     public function testContains()
     {
-        $this->assertContains('Hello World', $crawler->filter('h1')->text());
+        $this->assertSelectorTextContains('h1', 'Hello World');
     }
 }

StringFormTypeToClassRector

• class: Rector\Symfony\Rector\Form\StringFormTypeToClassRector
• test fixtures

Turns string Form Type references to their CONSTANT alternatives in FormTypes in Form in Symfony

 $formBuilder = new Symfony\Component\Form\FormBuilder;
-$formBuilder->add('name', 'form.type.text');
+$formBuilder->add('name', \Symfony\Component\Form\Extension\Core\Type\TextType::class);

StringToArrayArgumentProcessRector

• class: Rector\Symfony\Rector\New_\StringToArrayArgumentProcessRector
• test fixtures

Changes Process string argument to an array

 use Symfony\Component\Process\Process;
-$process = new Process('ls -l');
+$process = new Process(['ls', '-l']);

VarDumperTestTraitMethodArgsRector

• class: Rector\Symfony\Rector\VarDumper\VarDumperTestTraitMethodArgsRector
• test fixtures

Adds a new $filter argument in VarDumperTestTrait->assertDumpEquals() and VarDumperTestTrait->assertDumpMatchesFormat() in Validator in Symfony.

-$varDumperTestTrait->assertDumpEquals($dump, $data, $message = "");
+$varDumperTestTrait->assertDumpEquals($dump, $data, $filter = 0, $message = "");

-$varDumperTestTrait->assertDumpMatchesFormat($dump, $data, $message = "");
+$varDumperTestTrait->assertDumpMatchesFormat($dump, $data, $filter = 0, $message = "");

SymfonyCodeQuality

EventListenerToEventSubscriberRector

• class: Rector\SymfonyCodeQuality\Rector\Class_\EventListenerToEventSubscriberRector
• test fixtures

Change Symfony Event listener class to Event Subscriber based on configuration in service.yaml file

 <?php

-class SomeListener
+use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+
+class SomeEventSubscriber implements EventSubscriberInterface
 {
+     /**
+      * @return string[]
+      */
+     public static function getSubscribedEvents(): array
+     {
+         return ['some_event' => 'methodToBeCalled'];
+     }
+
      public function methodToBeCalled()
      {
      }
-}
-
-// in config.yaml
-services:
-    SomeListener:
-        tags:
-            - { name: kernel.event_listener, event: 'some_event', method: 'methodToBeCalled' }
+}

SymfonyPHPUnit

SelfContainerGetMethodCallFromTestToSetUpMethodRector

• class: Rector\SymfonyPHPUnit\Rector\Class_\SelfContainerGetMethodCallFromTestToSetUpMethodRector
• test fixtures

Move self::$container service fetching from test methods up to setUp method

 use ItemRepository;
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;

 class SomeTest extends KernelTestCase
 {
+    /**
+     * @var \ItemRepository
+     */
+    private $itemRepository;
+
+    protected function setUp()
+    {
+        parent::setUp();
+        $this->itemRepository = self::$container->get(ItemRepository::class);
+    }
+
     public function testOne()
     {
-        $itemRepository = self::$container->get(ItemRepository::class);
-        $itemRepository->doStuff();
+        $this->itemRepository->doStuff();
     }

     public function testTwo()
     {
-        $itemRepository = self::$container->get(ItemRepository::class);
-        $itemRepository->doAnotherStuff();
+        $this->itemRepository->doAnotherStuff();
     }
 }

Twig

SimpleFunctionAndFilterRector

• class: Rector\Twig\Rector\SimpleFunctionAndFilterRector
• test fixtures

Changes Twig_Function_Method to Twig_SimpleFunction calls in Twig_Extension.

 class SomeExtension extends Twig_Extension
 {
     public function getFunctions()
     {
         return [
-            'is_mobile' => new Twig_Function_Method($this, 'isMobile'),
+             new Twig_SimpleFunction('is_mobile', [$this, 'isMobile']),
         ];
     }

     public function getFilters()
     {
         return [
-            'is_mobile' => new Twig_Filter_Method($this, 'isMobile'),
+             new Twig_SimpleFilter('is_mobile', [$this, 'isMobile']),
         ];
     }
 }

TypeDeclaration

AddArrayParamDocTypeRector

• class: Rector\TypeDeclaration\Rector\ClassMethod\AddArrayParamDocTypeRector
• test fixtures

Adds @param annotation to array parameters inferred from the rest of the code

 class SomeClass
 {
     /**
      * @var int[]
      */
     private $values;

+    /**
+     * @param int[] $values
+     */
     public function __construct(array $values)
     {
         $this->values = $values;
     }
 }

AddArrayReturnDocTypeRector

• class: Rector\TypeDeclaration\Rector\ClassMethod\AddArrayReturnDocTypeRector
• test fixtures

Adds @return annotation to array parameters inferred from the rest of the code

 class SomeClass
 {
     /**
      * @var int[]
      */
     private $values;

+    /**
+     * @return int[]
+     */
     public function getValues(): array
     {
         return $this->values;
     }
 }

AddClosureReturnTypeRector

• class: Rector\TypeDeclaration\Rector\Closure\AddClosureReturnTypeRector
• test fixtures

Add known return type to functions

 class SomeClass
 {
     public function run($meetups)
     {
-        return array_filter($meetups, function (Meetup $meetup) {
+        return array_filter($meetups, function (Meetup $meetup): bool {
             return is_object($meetup);
         });
     }
 }

AddMethodCallBasedParamTypeRector

• class: Rector\TypeDeclaration\Rector\ClassMethod\AddMethodCallBasedParamTypeRector
• test fixtures

Change param type of passed getId() to UuidInterface type declaration

 class SomeClass
 {
-    public function getById($id)
+    public function getById(\Ramsey\Uuid\UuidInterface $id)
     {
     }
 }

 class CallerClass
 {
     public function run()
     {
         $building = new Building();
         $someClass = new SomeClass();
         $someClass->getById($building->getId());
     }
 }

AddParamTypeDeclarationRector

• class: Rector\TypeDeclaration\Rector\ClassMethod\AddParamTypeDeclarationRector
• test fixtures

Add param types where needed

services:
    Rector\TypeDeclaration\Rector\ClassMethod\AddParamTypeDeclarationRector:
        $typehintForParameterByMethodByClass:
            SomeClass:
                process:
                    - string

↓

 class SomeClass
 {
-    public function process($name)
+    public function process(string $name)
     {
     }
 }

CompleteVarDocTypePropertyRector

• class: Rector\TypeDeclaration\Rector\Property\CompleteVarDocTypePropertyRector
• test fixtures

Complete property @var annotations or correct the old ones

 final class SomeClass
 {
+    /**
+     * @var EventDispatcher
+     */
     private $eventDispatcher;

     public function __construct(EventDispatcher $eventDispatcher)
     {
         $this->eventDispatcher = $eventDispatcher;
     }
 }

ParamTypeDeclarationRector

• class: Rector\TypeDeclaration\Rector\FunctionLike\ParamTypeDeclarationRector
• test fixtures

Change @param types to type declarations if not a BC-break

 <?php

 class ParentClass
 {
     /**
      * @param int $number
      */
     public function keep($number)
     {
     }
 }

 final class ChildClass extends ParentClass
 {
     /**
      * @param int $number
      */
     public function keep($number)
     {
     }

     /**
      * @param int $number
      */
-    public function change($number)
+    public function change(int $number)
     {
     }
 }

PropertyTypeDeclarationRector

• class: Rector\TypeDeclaration\Rector\Property\PropertyTypeDeclarationRector
• test fixtures

Add @var to properties that are missing it

 class SomeClass
 {
+    /**
+     * @var int
+     */
     private $value;

     public function run()
     {
         $this->value = 123;
     }
 }

ReturnTypeDeclarationRector

• class: Rector\TypeDeclaration\Rector\FunctionLike\ReturnTypeDeclarationRector
• test fixtures

Change @return types and type from static analysis to type declarations if not a BC-break

 <?php

 class SomeClass
 {
     /**
      * @return int
      */
-    public function getCount()
+    public function getCount(): int
     {
     }
 }

---

General

• Core (44)

Core

ActionInjectionToConstructorInjectionRector

• class: Rector\Core\Rector\Architecture\DependencyInjection\ActionInjectionToConstructorInjectionRector
• test fixtures

Turns action injection in Controllers to constructor injection

 final class SomeController
 {
-    public function default(ProductRepository $productRepository)
+    /**
+     * @var ProductRepository
+     */
+    private $productRepository;
+    public function __construct(ProductRepository $productRepository)
     {
-        $products = $productRepository->fetchAll();
+        $this->productRepository = $productRepository;
+    }
+
+    public function default()
+    {
+        $products = $this->productRepository->fetchAll();
     }
 }

AddInterfaceByTraitRector

• class: Rector\Core\Rector\Class_\AddInterfaceByTraitRector
• test fixtures

Add interface by used trait

services:
    Rector\Core\Rector\Class_\AddInterfaceByTraitRector:
        $interfaceByTrait:
            SomeTrait: SomeInterface

↓

-class SomeClass
+class SomeClass implements SomeInterface
 {
     use SomeTrait;
 }

AddMethodParentCallRector

• class: Rector\Core\Rector\ClassMethod\AddMethodParentCallRector
• test fixtures

Add method parent call, in case new parent method is added

 class SunshineCommand extends ParentClassWithNewConstructor
 {
     public function __construct()
     {
         $value = 5;
+
+        parent::__construct();
     }
 }

AddReturnTypeDeclarationRector

• class: Rector\Core\Rector\ClassMethod\AddReturnTypeDeclarationRector
• test fixtures

Changes defined return typehint of method and class.

services:
    Rector\Core\Rector\ClassMethod\AddReturnTypeDeclarationRector:
        $typehintForMethodByClass:
            SomeClass:
                getData: array

↓

 class SomeClass
 {
-    public getData()
+    public getData(): array
     {
     }
 }

AnnotatedPropertyInjectToConstructorInjectionRector

• class: Rector\Core\Rector\Architecture\DependencyInjection\AnnotatedPropertyInjectToConstructorInjectionRector
• test fixtures

Turns non-private properties with @annotation to private properties and constructor injection

 /**
  * @var SomeService
- * @inject
  */
-public $someService;
+private $someService;
+
+public function __construct(SomeService $someService)
+{
+    $this->someService = $someService;
+}

ArgumentAdderRector

• class: Rector\Core\Rector\Argument\ArgumentAdderRector
• test fixtures

This Rector adds new default arguments in calls of defined methods and class types.

services:
    Rector\Core\Rector\Argument\ArgumentAdderRector:
        $positionWithDefaultValueByMethodNamesByClassTypes:
            SomeExampleClass:
                someMethod:
                    -
                        name: someArgument
                        default_value: 'true'
                        type: SomeType

↓

 $someObject = new SomeExampleClass;
-$someObject->someMethod();
+$someObject->someMethod(true);

services:
    Rector\Core\Rector\Argument\ArgumentAdderRector:
        $positionWithDefaultValueByMethodNamesByClassTypes:
            SomeExampleClass:
                someMethod:
                    -
                        name: someArgument
                        default_value: 'true'
                        type: SomeType

↓

 class MyCustomClass extends SomeExampleClass
 {
-    public function someMethod()
+    public function someMethod($value = true)
     {
     }
 }

ArgumentDefaultValueReplacerRector

• class: Rector\Core\Rector\Argument\ArgumentDefaultValueReplacerRector
• test fixtures

Replaces defined map of arguments in defined methods and their calls.

services:
    Rector\Core\Rector\Argument\ArgumentDefaultValueReplacerRector:
        SomeExampleClass:
            someMethod:
                -
                    -
                        before: 'SomeClass::OLD_CONSTANT'
                        after: 'false'

↓

 $someObject = new SomeClass;
-$someObject->someMethod(SomeClass::OLD_CONSTANT);
+$someObject->someMethod(false);'

ArgumentRemoverRector

• class: Rector\Core\Rector\Argument\ArgumentRemoverRector
• test fixtures

Removes defined arguments in defined methods and their calls.

services:
    Rector\Core\Rector\Argument\ArgumentRemoverRector:
        ExampleClass:
            someMethod:
                -
                    value: 'true'

↓

 $someObject = new SomeClass;
-$someObject->someMethod(true);
+$someObject->someMethod();'

ChangeConstantVisibilityRector

• class: Rector\Core\Rector\Visibility\ChangeConstantVisibilityRector
• test fixtures

Change visibility of constant from parent class.

services:
    Rector\Core\Rector\Visibility\ChangeConstantVisibilityRector:
        ParentObject:
            SOME_CONSTANT: protected

↓

 class FrameworkClass
 {
     protected const SOME_CONSTANT = 1;
 }

 class MyClass extends FrameworkClass
 {
-    public const SOME_CONSTANT = 1;
+    protected const SOME_CONSTANT = 1;
 }

ChangeContractMethodSingleToManyRector

• class: Rector\Core\Rector\ClassMethod\ChangeContractMethodSingleToManyRector
• test fixtures

Change method that returns single value to multiple values

services:
    Rector\Core\Rector\ClassMethod\ChangeContractMethodSingleToManyRector:
        $oldToNewMethodByType:
            SomeClass:
                getNode: getNodes

↓

 class SomeClass
 {
-    public function getNode(): string
+    /**
+     * @return string[]
+     */
+    public function getNodes(): array
     {
-        return 'Echo_';
+        return ['Echo_'];
     }
 }

ChangeMethodVisibilityRector

• class: Rector\Core\Rector\Visibility\ChangeMethodVisibilityRector
• test fixtures

Change visibility of method from parent class.

services:
    Rector\Core\Rector\Visibility\ChangeMethodVisibilityRector:
        $methodToVisibilityByClass:
            FrameworkClass:
                someMethod: protected

↓

 class FrameworkClass
 {
     protected someMethod()
     {
     }
 }

 class MyClass extends FrameworkClass
 {
-    public someMethod()
+    protected someMethod()
     {
     }
 }

ChangePropertyVisibilityRector

• class: Rector\Core\Rector\Visibility\ChangePropertyVisibilityRector
• test fixtures

Change visibility of property from parent class.

services:
    Rector\Core\Rector\Visibility\ChangePropertyVisibilityRector:
        FrameworkClass:
            someProperty: protected

↓

 class FrameworkClass
 {
     protected $someProperty;
 }

 class MyClass extends FrameworkClass
 {
-    public $someProperty;
+    protected $someProperty;
 }

FluentReplaceRector

• class: Rector\Core\Rector\MethodBody\FluentReplaceRector
• test fixtures

Turns fluent interface calls to classic ones.

services:
    Rector\Core\Rector\MethodBody\FluentReplaceRector:
        $classesToDefluent:
            - SomeExampleClass

↓

 $someClass = new SomeClass();
-$someClass->someFunction()
-            ->otherFunction();
+$someClass->someFunction();
+$someClass->otherFunction();

FunctionToMethodCallRector

• class: Rector\Core\Rector\Function_\FunctionToMethodCallRector
• test fixtures

Turns defined function calls to local method calls.

services:
    Rector\Core\Rector\Function_\FunctionToMethodCallRector:
        view:
            - this
            - render

↓

-view("...", []);
+$this->render("...", []);

FunctionToNewRector

• class: Rector\Core\Rector\FuncCall\FunctionToNewRector
• test fixtures

Change configured function calls to new Instance

 class SomeClass
 {
     public function run()
     {
-        $array = collection([]);
+        $array = new \Collection([]);
     }
 }

FunctionToStaticCallRector

• class: Rector\Core\Rector\Function_\FunctionToStaticCallRector
• test fixtures

Turns defined function call to static method call.

services:
    Rector\Core\Rector\Function_\FunctionToStaticCallRector:
        view:
            - SomeStaticClass
            - render

↓

-view("...", []);
+SomeClass::render("...", []);

GetAndSetToMethodCallRector

• class: Rector\Core\Rector\MagicDisclosure\GetAndSetToMethodCallRector
• test fixtures

Turns defined __get/__set to specific method calls.

services:
    Rector\Core\Rector\MagicDisclosure\GetAndSetToMethodCallRector:
        SomeContainer:
            set: addService

↓

 $container = new SomeContainer;
-$container->someService = $someService;
+$container->setService("someService", $someService);

services:
    Rector\Core\Rector\MagicDisclosure\GetAndSetToMethodCallRector:
        $typeToMethodCalls:
            SomeContainer:
                get: getService

↓

 $container = new SomeContainer;
-$someService = $container->someService;
+$someService = $container->getService("someService");

InjectAnnotationClassRector

• class: Rector\Core\Rector\Property\InjectAnnotationClassRector
• test fixtures

Changes properties with specified annotations class to constructor injection

services:
    Rector\Core\Rector\Property\InjectAnnotationClassRector:
        $annotationClasses:
            - DI\Annotation\Inject
            - JMS\DiExtraBundle\Annotation\Inject

↓

 use JMS\DiExtraBundle\Annotation as DI;

 class SomeController
 {
     /**
-     * @DI\Inject("entity.manager")
+     * @var EntityManager
      */
     private $entityManager;
+
+    public function __construct(EntityManager $entityManager)
+    {
+        $this->entityManager = entityManager;
+    }
 }

MergeInterfacesRector

• class: Rector\Core\Rector\Interface_\MergeInterfacesRector
• test fixtures

Merges old interface to a new one, that already has its methods

services:
    Rector\Core\Rector\Interface_\MergeInterfacesRector:
        SomeOldInterface: SomeInterface

↓

-class SomeClass implements SomeInterface, SomeOldInterface
+class SomeClass implements SomeInterface
 {
 }

MethodCallToAnotherMethodCallWithArgumentsRector

• class: Rector\Core\Rector\MethodCall\MethodCallToAnotherMethodCallWithArgumentsRector
• test fixtures

Turns old method call with specific types to new one with arguments

services:
    Rector\Core\Rector\MethodCall\MethodCallToAnotherMethodCallWithArgumentsRector:
        Nette\DI\ServiceDefinition:
            setInject:
                -
                    - addTag
                    -
                        - inject

↓

 $serviceDefinition = new Nette\DI\ServiceDefinition;
-$serviceDefinition->setInject();
+$serviceDefinition->addTag('inject');

MethodCallToReturnRector

• class: Rector\Core\Rector\MethodCall\MethodCallToReturnRector
• test fixtures

Wrap method call to return

services:
    Rector\Core\Rector\MethodCall\MethodCallToReturnRector:
        $methodNamesByType:
            SomeClass:
                - deny

↓

 class SomeClass
 {
     public function run()
     {
-        $this->deny();
+        return $this->deny();
     }

     public function deny()
     {
         return 1;
     }
 }

NewObjectToFactoryCreateRector

• class: Rector\Core\Rector\Architecture\Factory\NewObjectToFactoryCreateRector
• test fixtures

Replaces creating object instances with "new" keyword with factory method.

services:
    Rector\Core\Rector\Architecture\Factory\NewObjectToFactoryCreateRector:
        MyClass:
            class: MyClassFactory
            method: create

↓

 class SomeClass
 {
+	/**
+	 * @var \MyClassFactory
+	 */
+	private $myClassFactory;
+
 	public function example() {
-		new MyClass($argument);
+		$this->myClassFactory->create($argument);
 	}
 }

NewToStaticCallRector

• class: Rector\Core\Rector\New_\NewToStaticCallRector
• test fixtures

Change new Object to static call

services:
    Rector\Core\Rector\New_\NewToStaticCallRector:
        $typeToStaticCalls:
            Cookie:
                - Cookie
                - create

↓

 class SomeClass
 {
     public function run()
     {
-        new Cookie($name);
+        Cookie::create($name);
     }
 }

NormalToFluentRector

• class: Rector\Core\Rector\MethodBody\NormalToFluentRector
• test fixtures

Turns fluent interface calls to classic ones.

services:
    Rector\Core\Rector\MethodBody\NormalToFluentRector:
        SomeClass:
            - someFunction
            - otherFunction

↓

 $someObject = new SomeClass();
-$someObject->someFunction();
-$someObject->otherFunction();
+$someObject->someFunction()
+    ->otherFunction();

ParentClassToTraitsRector

• class: Rector\Core\Rector\Class_\ParentClassToTraitsRector
• test fixtures

Replaces parent class to specific traits

services:
    Rector\Core\Rector\Class_\ParentClassToTraitsRector:
        Nette\Object:
            - Nette\SmartObject

↓

-class SomeClass extends Nette\Object
+class SomeClass
 {
+    use Nette\SmartObject;
 }

PropertyAssignToMethodCallRector

• class: Rector\Core\Rector\Assign\PropertyAssignToMethodCallRector
• test fixtures

Turns property assign of specific type and property name to method call

services:
    Rector\Core\Rector\Assign\PropertyAssignToMethodCallRector:
        $oldPropertiesToNewMethodCallsByType:
            SomeClass:
                oldPropertyName: oldProperty
                newMethodName: newMethodCall

↓

 $someObject = new SomeClass;
-$someObject->oldProperty = false;
+$someObject->newMethodCall(false);

PropertyToMethodRector

• class: Rector\Core\Rector\Property\PropertyToMethodRector
• test fixtures

Replaces properties assign calls be defined methods.

services:
    Rector\Core\Rector\Property\PropertyToMethodRector:
        $perClassPropertyToMethods:
            SomeObject:
                property:
                    get: getProperty
                    set: setProperty

↓

-$result = $object->property;
-$object->property = $value;
+$result = $object->getProperty();
+$object->setProperty($value);

services:
    Rector\Core\Rector\Property\PropertyToMethodRector:
        $perClassPropertyToMethods:
            SomeObject:
                property:
                    get:
                        method: getConfig
                        arguments:
                            - someArg

↓

-$result = $object->property;
+$result = $object->getProperty('someArg');

PseudoNamespaceToNamespaceRector

• class: Rector\Core\Rector\Namespace_\PseudoNamespaceToNamespaceRector
• test fixtures

Replaces defined Pseudo_Namespaces by Namespace\Ones.

services:
    Rector\Core\Rector\Namespace_\PseudoNamespaceToNamespaceRector:
        $namespacePrefixesWithExcludedClasses:
            Some_:
                - Some_Class_To_Keep

↓

-/** @var Some_Chicken $someService */
-$someService = new Some_Chicken;
+/** @var Some\Chicken $someService */
+$someService = new Some\Chicken;
 $someClassToKeep = new Some_Class_To_Keep;

RemoveFuncCallArgRector

• class: Rector\Core\Rector\FuncCall\RemoveFuncCallArgRector
• test fixtures

Remove argument by position by function name

services:
    Rector\Core\Rector\FuncCall\RemoveFuncCallArgRector:
        $argumentPositionByFunctionName:
            remove_last_arg:
                - 1

↓

-remove_last_arg(1, 2);
+remove_last_arg(1);

RemoveIniGetSetFuncCallRector

• class: Rector\Core\Rector\FuncCall\RemoveIniGetSetFuncCallRector
• test fixtures

Remove ini_get by configuration

services:
    Rector\Core\Rector\FuncCall\RemoveIniGetSetFuncCallRector:
        $keysToRemove:
            - y2k_compliance

↓

-ini_get('y2k_compliance');
-ini_set('y2k_compliance', 1);

RemoveInterfacesRector

• class: Rector\Core\Rector\Interface_\RemoveInterfacesRector
• test fixtures

Removes interfaces usage from class.

services:
    Rector\Core\Rector\Interface_\RemoveInterfacesRector:
        - SomeInterface

↓

-class SomeClass implements SomeInterface
+class SomeClass
 {
 }

RemoveTraitRector

• class: Rector\Core\Rector\ClassLike\RemoveTraitRector
• test fixtures

Remove specific traits from code

 class SomeClass
 {
-    use SomeTrait;
 }

RenameClassConstantsUseToStringsRector

• class: Rector\Core\Rector\Constant\RenameClassConstantsUseToStringsRector
• test fixtures

Replaces constant by value

services:
    Rector\Core\Rector\Constant\RenameClassConstantsUseToStringsRector:
        Nette\Configurator:
            DEVELOPMENT: development
            PRODUCTION: production

↓

-$value === Nette\Configurator::DEVELOPMENT
+$value === "development"

RenamePropertyRector

• class: Rector\Core\Rector\Property\RenamePropertyRector
• test fixtures

Replaces defined old properties by new ones.

services:
    Rector\Core\Rector\Property\RenamePropertyRector:
        $oldToNewPropertyByTypes:
            SomeClass:
                someOldProperty: someNewProperty

↓

-$someObject->someOldProperty;
+$someObject->someNewProperty;

ReplaceVariableByPropertyFetchRector

• class: Rector\Core\Rector\Architecture\DependencyInjection\ReplaceVariableByPropertyFetchRector

Turns variable in controller action to property fetch, as follow up to action injection variable to property change.

 final class SomeController
 {
     /**
      * @var ProductRepository
      */
     private $productRepository;

     public function __construct(ProductRepository $productRepository)
     {
         $this->productRepository = $productRepository;
     }

     public function default()
     {
-        $products = $productRepository->fetchAll();
+        $products = $this->productRepository->fetchAll();
     }
 }

ReturnThisRemoveRector

• class: Rector\Core\Rector\MethodBody\ReturnThisRemoveRector
• test fixtures

Removes "return $this;" from fluent interfaces for specified classes.

services:
    Rector\Core\Rector\MethodBody\ReturnThisRemoveRector:
        -
            - SomeExampleClass

↓

 class SomeExampleClass
 {
     public function someFunction()
     {
-        return $this;
     }

     public function otherFunction()
     {
-        return $this;
     }
 }

ServiceGetterToConstructorInjectionRector

• class: Rector\Core\Rector\MethodCall\ServiceGetterToConstructorInjectionRector
• test fixtures

Get service call to constructor injection

services:
    Rector\Core\Rector\MethodCall\ServiceGetterToConstructorInjectionRector:
        $methodNamesByTypesToServiceTypes:
            FirstService:
                getAnotherService: AnotherService

↓

 final class SomeClass
 {
     /**
      * @var FirstService
      */
     private $firstService;

-    public function __construct(FirstService $firstService)
-    {
-        $this->firstService = $firstService;
-    }
-
-    public function run()
-    {
-        $anotherService = $this->firstService->getAnotherService();
-        $anotherService->run();
-    }
-}
-
-class FirstService
-{
     /**
      * @var AnotherService
      */
     private $anotherService;

-    public function __construct(AnotherService $anotherService)
+    public function __construct(FirstService $firstService, AnotherService $anotherService)
     {
+        $this->firstService = $firstService;
         $this->anotherService = $anotherService;
     }

-    public function getAnotherService(): AnotherService
+    public function run()
     {
-         return $this->anotherService;
+        $anotherService = $this->anotherService;
+        $anotherService->run();
     }
 }

StaticCallToFunctionRector

• class: Rector\Core\Rector\StaticCall\StaticCallToFunctionRector
• test fixtures

Turns static call to function call.

services:
    Rector\Core\Rector\StaticCall\StaticCallToFunctionRector:
        $staticCallToFunction:
            OldClass:
                oldMethod: new_function

↓

-OldClass::oldMethod("args");
+new_function("args");

StringToClassConstantRector

• class: Rector\Core\Rector\String_\StringToClassConstantRector
• test fixtures

Changes strings to specific constants

services:
    Rector\Core\Rector\String_\StringToClassConstantRector:
        compiler.post_dump:
            - Yet\AnotherClass
            - CONSTANT

↓

 final class SomeSubscriber
 {
     public static function getSubscribedEvents()
     {
-        return ['compiler.post_dump' => 'compile'];
+        return [\Yet\AnotherClass::CONSTANT => 'compile'];
     }
 }

SwapClassMethodArgumentsRector

• class: Rector\Core\Rector\StaticCall\SwapClassMethodArgumentsRector
• test fixtures

Reorder class method arguments, including their calls

services:
    Rector\Core\Rector\StaticCall\SwapClassMethodArgumentsRector:
        $newArgumentPositionsByMethodAndClass:
            SomeClass:
                run:
                    - 1
                    - 0

↓

 class SomeClass
 {
-    public static function run($first, $second)
+    public static function run($second, $first)
     {
-        self::run($first, $second);
+        self::run($second, $first);
     }
 }

SwapFuncCallArgumentsRector

• class: Rector\Core\Rector\Argument\SwapFuncCallArgumentsRector
• test fixtures

Swap arguments in function calls

 final class SomeClass
 {
     public function run($one, $two)
     {
-        return some_function($one, $two);
+        return some_function($two, $one);
     }
 }

ToStringToMethodCallRector

• class: Rector\Core\Rector\MagicDisclosure\ToStringToMethodCallRector
• test fixtures

Turns defined code uses of "__toString()" method to specific method calls.

services:
    Rector\Core\Rector\MagicDisclosure\ToStringToMethodCallRector:
        SomeObject: getPath

↓

 $someValue = new SomeObject;
-$result = (string) $someValue;
-$result = $someValue->__toString();
+$result = $someValue->getPath();
+$result = $someValue->getPath();

UnsetAndIssetToMethodCallRector

• class: Rector\Core\Rector\MagicDisclosure\UnsetAndIssetToMethodCallRector
• test fixtures

Turns defined __isset/__unset calls to specific method calls.

services:
    Rector\Core\Rector\MagicDisclosure\UnsetAndIssetToMethodCallRector:
        SomeContainer:
            isset: hasService

↓

 $container = new SomeContainer;
-isset($container["someKey"]);
+$container->hasService("someKey");

services:
    Rector\Core\Rector\MagicDisclosure\UnsetAndIssetToMethodCallRector:
        SomeContainer:
            unset: removeService

↓

 $container = new SomeContainer;
-unset($container["someKey"]);
+$container->removeService("someKey");

WrapReturnRector

• class: Rector\Core\Rector\ClassMethod\WrapReturnRector
• test fixtures

Wrap return value of specific method

services:
    Rector\Core\Rector\ClassMethod\WrapReturnRector:
        SomeClass:
            getItem: array

↓

 final class SomeClass
 {
     public function getItem()
     {
-        return 1;
+        return [1];
     }
 }