2019-05-26 21:26:33 +02:00
# All 300 Rectors Overview
2018-04-29 11:03:47 +02:00
2018-08-01 22:09:34 +02:00
- [Projects ](#projects )
- [General ](#general )
## Projects
2019-02-04 01:32:53 +01:00
- [CakePHP ](#cakephp )
2019-03-09 13:24:30 +00:00
- [Celebrity ](#celebrity )
2019-02-04 01:32:53 +01:00
- [CodeQuality ](#codequality )
- [CodingStyle ](#codingstyle )
- [DeadCode ](#deadcode )
2018-08-01 22:09:34 +02:00
- [Doctrine ](#doctrine )
2019-02-04 01:32:53 +01:00
- [DomainDrivenDesign ](#domaindrivendesign )
- [Guzzle ](#guzzle )
2019-03-09 13:24:30 +00:00
- [Laravel ](#laravel )
2019-05-26 18:38:09 +02:00
- [Legacy ](#legacy )
2019-02-21 15:36:16 +01:00
- [MysqlToMysqli ](#mysqltomysqli )
2019-04-02 15:35:35 +02:00
- [Nette ](#nette )
2019-03-16 21:31:46 +01:00
- [NetteTesterToPHPUnit ](#nettetestertophpunit )
2019-02-02 17:22:15 +01:00
- [NetteToSymfony ](#nettetosymfony )
2019-02-04 01:32:53 +01:00
- [PHPStan ](#phpstan )
2018-07-31 23:47:59 +02:00
- [PHPUnit ](#phpunit )
2019-02-04 01:32:53 +01:00
- [Php ](#php )
2018-08-01 22:09:34 +02:00
- [PhpParser ](#phpparser )
2019-03-16 21:31:46 +01:00
- [PhpSpecToPHPUnit ](#phpspectophpunit )
2019-05-02 01:56:58 +02:00
- [RemovingStatic ](#removingstatic )
- [SOLID ](#solid )
2019-02-04 01:32:53 +01:00
- [Sensio ](#sensio )
2019-03-16 21:31:46 +01:00
- [Shopware ](#shopware )
2018-09-29 00:33:35 +08:00
- [Silverstripe ](#silverstripe )
2019-02-04 01:32:53 +01:00
- [Sylius ](#sylius )
- [Symfony ](#symfony )
2018-09-29 00:33:35 +08:00
- [Twig ](#twig )
2019-05-19 10:27:38 +02:00
- [TypeDeclaration ](#typedeclaration )
2018-09-29 00:33:35 +08:00
2019-02-04 01:32:53 +01:00
## CakePHP
2018-09-29 00:33:35 +08:00
### `ModalToGetSetRector`
- class: `Rector\CakePHP\Rector\MethodCall\ModalToGetSetRector`
Changes combined set/get `value()` to specific `getValue()` or `setValue(x)` .
```diff
$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']);
```
2018-08-01 22:09:34 +02:00
2018-11-11 13:22:35 +01:00
< br >
2019-03-31 14:25:39 +02:00
### `ChangeSnakedFixtureNameToCamelRector`
- class: `Rector\CakePHP\Rector\Name\ChangeSnakedFixtureNameToCamelRector`
Changes $fixtues style from snake_case to CamelCase.
```diff
class SomeTest
{
protected $fixtures = [
- 'app.posts',
- 'app.users',
- 'some_plugin.posts/special_posts',
+ 'app.Posts',
+ 'app.Users',
+ 'some_plugin.Posts/SpeectialPosts',
];
```
< br >
2019-03-09 13:24:30 +00:00
## Celebrity
### `SetTypeToCastRector`
- class: `Rector\Celebrity\Rector\FuncCall\SetTypeToCastRector`
Changes settype() to (type) where possible
```diff
class SomeClass
{
- public function run($foo)
+ public function run(array $items)
{
- settype($foo, 'string');
+ $foo = (string) $foo;
- return settype($foo, 'integer');
+ return (int) $foo;
}
}
```
< br >
### `CommonNotEqualRector`
- class: `Rector\Celebrity\Rector\NotEqual\CommonNotEqualRector`
Use common != instead of less known < > with same meaning
```diff
final class SomeClass
{
public function run($one, $two)
{
- return $one < > $two;
+ return $one != $two;
}
}
```
< br >
### `LogicalToBooleanRector`
- class: `Rector\Celebrity\Rector\BooleanOp\LogicalToBooleanRector`
Change OR, AND to ||, & & with more common understanding
```diff
-if ($f = false or true) {
+if (($f = false) || true) {
return $f;
}
```
< br >
2019-02-04 01:32:53 +01:00
## CodeQuality
2018-10-22 00:26:45 +02:00
### `CombinedAssignRector`
- class: `Rector\CodeQuality\Rector\Assign\CombinedAssignRector`
Simplify $value = $value + 5; assignments to shorter ones
```diff
-$value = $value + 5;
+$value += 5;
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-21 15:36:16 +01:00
### `UseIdenticalOverEqualWithSameTypeRector`
- class: `Rector\CodeQuality\Rector\Equal\UseIdenticalOverEqualWithSameTypeRector`
Use ===/!== over ==/!=, it values have the same type
```diff
class SomeClass
{
public function run(int $firstValue, int $secondValue)
{
- $isSame = $firstValue == $secondValue;
- $isDiffernt = $firstValue != $secondValue;
+ $isSame = $firstValue === $secondValue;
+ $isDiffernt = $firstValue !== $secondValue;
}
}
```
< br >
2019-03-09 13:24:30 +00:00
### `SimplifyDuplicatedTernaryRector`
- class: `Rector\CodeQuality\Rector\Ternary\SimplifyDuplicatedTernaryRector`
Remove ternary that duplicated return value of true : false
```diff
class SomeClass
{
public function run(bool $value, string $name)
{
- $isTrue = $value ? true : false;
+ $isTrue = $value;
$isName = $name ? true : false;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `TernaryToElvisRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodeQuality\Rector\Ternary\TernaryToElvisRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
Use ?: instead of ?, where useful
2018-12-22 12:22:28 +01:00
```diff
2019-02-04 01:32:53 +01:00
function elvis()
{
- $value = $a ? $a : false;
+ $value = $a ?: false;
}
2018-12-22 12:22:28 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `SimplifyTautologyTernaryRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodeQuality\Rector\Ternary\SimplifyTautologyTernaryRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
Simplify tautology ternary to value
2018-11-11 13:22:35 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$value = ($fullyQualifiedTypeHint !== $typeHint) ? $fullyQualifiedTypeHint : $typeHint;
+$value = $fullyQualifiedTypeHint;
2018-11-11 13:22:35 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `UnnecessaryTernaryExpressionRector`
2019-01-03 13:47:44 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodeQuality\Rector\Ternary\UnnecessaryTernaryExpressionRector`
2019-01-03 13:47:44 +01:00
2019-02-04 01:32:53 +01:00
Remove unnecessary ternary expressions.
2019-01-03 13:47:44 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$foo === $bar ? true : false;
+$foo === $bar;
2019-01-03 13:47:44 +01:00
```
< br >
2018-10-23 15:58:57 -03:00
### `ForeachToInArrayRector`
- class: `Rector\CodeQuality\Rector\Foreach_\ForeachToInArrayRector`
Simplify `foreach` loops into `in_array` when possible
```diff
-foreach ($items as $item) {
2018-11-11 13:22:35 +01:00
- if ($item === "something") {
2018-10-23 15:58:57 -03:00
- return true;
- }
-}
-
-return false;
2018-11-11 13:22:35 +01:00
+in_array("something", $items, true);
2018-10-23 15:58:57 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-12-31 12:50:32 +01:00
### `SimplifyForeachToArrayFilterRector`
- class: `Rector\CodeQuality\Rector\Foreach_\SimplifyForeachToArrayFilterRector`
Simplify foreach with function filtering to array filter
```diff
-$directories = [];
$possibleDirectories = [];
-foreach ($possibleDirectories as $possibleDirectory) {
- if (file_exists($possibleDirectory)) {
- $directories[] = $possibleDirectory;
- }
-}
+$directories = array_filter($possibleDirectories, 'file_exists');
```
< br >
2018-10-22 00:26:45 +02:00
### `SimplifyForeachToCoalescingRector`
- class: `Rector\CodeQuality\Rector\Foreach_\SimplifyForeachToCoalescingRector`
Changes foreach that returns set value to ??
```diff
-foreach ($this->oldToNewFunctions as $oldFunction => $newFunction) {
- if ($currentFunction === $oldFunction) {
- return $newFunction;
- }
-}
-
-return null;
+return $this->oldToNewFunctions[$currentFunction] ?? null;
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-02 01:56:58 +02:00
### `CallableThisArrayToAnonymousFunctionRector`
- class: `Rector\CodeQuality\Rector\Array_\CallableThisArrayToAnonymousFunctionRector`
Convert [$this, "method"] to proper anonymous function
```diff
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;
}
}
```
< br >
2019-05-19 10:27:38 +02:00
### `ForToForeachRector`
- class: `Rector\CodeQuality\Rector\For_\ForToForeachRector`
Change for() to foreach() where useful
```diff
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;
}
}
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `SimplifyUselessVariableRector`
- class: `Rector\CodeQuality\Rector\Return_\SimplifyUselessVariableRector`
Removes useless variable assigns
```diff
function () {
- $a = true;
- return $a;
+ return true;
};
```
< br >
2018-10-22 00:26:45 +02:00
### `InArrayAndArrayKeysToArrayKeyExistsRector`
- class: `Rector\CodeQuality\Rector\FuncCall\InArrayAndArrayKeysToArrayKeyExistsRector`
Simplify `in_array` and `array_keys` functions combination into `array_key_exists` when `array_keys` has one argument only
```diff
-in_array("key", array_keys($array), true);
+array_key_exists("key", $array);
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-23 17:13:13 -03:00
2018-11-11 13:22:35 +01:00
### `SimplifyInArrayValuesRector`
2018-10-23 17:13:13 -03:00
2018-11-11 13:22:35 +01:00
- class: `Rector\CodeQuality\Rector\FuncCall\SimplifyInArrayValuesRector`
Removes unneeded array_values() in in_array() call
2018-10-23 17:13:13 -03:00
```diff
2018-11-11 13:22:35 +01:00
-in_array("key", array_values($array), true);
+in_array("key", $array, true);
2018-10-23 17:13:13 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-19 10:27:38 +02:00
### `CompactToVariablesRector`
- class: `Rector\CodeQuality\Rector\FuncCall\CompactToVariablesRector`
Change compact() call to own array
```diff
class SomeClass
{
public function run()
{
$checkout = 'one';
$form = 'two';
- return compact('checkout', 'form');
+ return ['checkout' => $checkout, 'form' => $form];
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `SimplifyFuncGetArgsCountRector`
- class: `Rector\CodeQuality\Rector\FuncCall\SimplifyFuncGetArgsCountRector`
Simplify count of func_get_args() to fun_num_args()
```diff
-count(func_get_args());
+func_num_args();
```
< br >
2018-12-31 12:50:32 +01:00
### `SingleInArrayToCompareRector`
- class: `Rector\CodeQuality\Rector\FuncCall\SingleInArrayToCompareRector`
Changes in_array() with single element to ===
```diff
class SomeClass
{
public function run()
{
- if (in_array(strtolower($type), ['$this'], true)) {
+ if (strtolower($type) === '$this') {
return strtolower($type);
}
}
}
```
< br >
2018-10-23 15:58:57 -03:00
### `SimplifyStrposLowerRector`
2018-10-22 00:26:45 +02:00
2018-10-23 15:58:57 -03:00
- class: `Rector\CodeQuality\Rector\FuncCall\SimplifyStrposLowerRector`
2018-10-22 00:26:45 +02:00
2018-10-23 15:58:57 -03:00
Simplify strpos(strtolower(), "...") calls
2018-10-22 00:26:45 +02:00
```diff
2018-10-23 15:58:57 -03:00
-strpos(strtolower($var), "...")"
+stripos($var, "...")"
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2019-03-31 14:25:39 +02:00
### `SimplifyRegexPatternRector`
- class: `Rector\CodeQuality\Rector\FuncCall\SimplifyRegexPatternRector`
Simplify regex pattern to known ranges
```diff
class SomeClass
{
public function run($value)
{
- preg_match('#[a-zA-Z0-9+]#', $value);
+ preg_match('#[\w\d+]#', $value);
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `SimplifyIfNotNullReturnRector`
- class: `Rector\CodeQuality\Rector\If_\SimplifyIfNotNullReturnRector`
Changes redundant null check to instant return
2018-10-22 00:26:45 +02:00
```diff
2018-11-11 13:22:35 +01:00
$newNode = 'something ;
-if ($newNode !== null) {
- return $newNode;
-}
-
-return null;
+return $newNode;
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2019-02-18 16:51:24 +01:00
### `ExplicitBoolCompareRector`
- class: `Rector\CodeQuality\Rector\If_\ExplicitBoolCompareRector`
Make if conditions more explicit
```diff
final class SomeController
{
public function run($items)
{
- if (!count($items)) {
2019-04-03 19:14:46 +02:00
+ if (count($items) === 0) {
2019-02-18 16:51:24 +01:00
return 'no items';
}
}
}
```
< br >
2018-12-31 12:50:32 +01:00
### `SimplifyIfElseToTernaryRector`
- class: `Rector\CodeQuality\Rector\If_\SimplifyIfElseToTernaryRector`
Changes if/else for same value as assign to ternary
```diff
class SomeClass
{
public function run()
{
- if (empty($value)) {
- $this->arrayBuilt[][$key] = true;
- } else {
- $this->arrayBuilt[][$key] = $value;
- }
+ $this->arrayBuilt[][$key] = empty($value) ? true : $value;
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `SimplifyIfReturnBoolRector`
2018-10-22 00:26:45 +02:00
2018-11-11 13:22:35 +01:00
- class: `Rector\CodeQuality\Rector\If_\SimplifyIfReturnBoolRector`
Shortens if return false/true to direct return
2018-10-22 00:26:45 +02:00
```diff
2018-11-11 13:22:35 +01:00
-if (strpos($docToken->getContent(), "\n") === false) {
- return true;
-}
-
-return false;
+return strpos($docToken->getContent(), "\n") === false;
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-23 15:58:57 -03:00
2019-02-18 16:51:24 +01:00
### `SimplifyIfIssetToNullCoalescingRector`
- class: `Rector\CodeQuality\Rector\If_\SimplifyIfIssetToNullCoalescingRector`
Simplify binary if to null coalesce
```diff
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);
}
}
```
< br >
2019-01-22 21:34:38 +01:00
### `ConsecutiveNullCompareReturnsToNullCoalesceQueueRector`
- class: `Rector\CodeQuality\Rector\If_\ConsecutiveNullCompareReturnsToNullCoalesceQueueRector`
Change multiple null compares to ?? queue
```diff
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;
}
}
```
< br >
2019-05-19 10:27:38 +02:00
### `AndAssignsToSeparateLinesRector`
- class: `Rector\CodeQuality\Rector\LogicalAnd\AndAssignsToSeparateLinesRector`
Split 2 assigns ands to separate line
```diff
class SomeClass
{
public function run()
{
$tokens = [];
- $token = 4 and $tokens[] = $token;
+ $token = 4;
+ $tokens[] = $token;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `SimplifyDeMorganBinaryRector`
- class: `Rector\CodeQuality\Rector\BinaryOp\SimplifyDeMorganBinaryRector`
Simplify negated conditions with de Morgan theorem
```diff
< ?php
$a = 5;
$b = 10;
-$result = !($a > 20 || $b < = 50);
+$result = $a < = 20 & & $b > 50;
```
< br >
### `SimplifyEmptyArrayCheckRector`
- class: `Rector\CodeQuality\Rector\BooleanAnd\SimplifyEmptyArrayCheckRector`
Simplify `is_array` and `empty` functions combination into a simple identical check for an empty array
```diff
-is_array($values) & & empty($values)
+$values === []
```
< br >
2018-12-22 12:22:28 +01:00
2019-03-09 13:24:30 +00:00
### `GetClassToInstanceOfRector`
2018-12-22 12:22:28 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\CodeQuality\Rector\Identical\GetClassToInstanceOfRector`
2018-12-22 12:22:28 +01:00
2019-03-09 13:24:30 +00:00
Changes comparison with get_class to instanceof
2018-12-22 12:22:28 +01:00
```diff
2019-03-09 13:24:30 +00:00
-if (EventsListener::class === get_class($event->job)) { }
+if ($event->job instanceof EventsListener) { }
2018-12-22 12:22:28 +01:00
```
< br >
2019-03-09 13:24:30 +00:00
### `SimplifyBoolIdenticalTrueRector`
2018-11-11 13:22:35 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\CodeQuality\Rector\Identical\SimplifyBoolIdenticalTrueRector`
2018-11-11 13:22:35 +01:00
2019-03-09 13:24:30 +00:00
Symplify bool value compare to true or false
2018-10-23 15:58:57 -03:00
```diff
2019-03-09 13:24:30 +00:00
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);
}
}
2018-10-23 15:58:57 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2019-05-02 01:56:58 +02:00
### `BooleanNotIdenticalToNotIdenticalRector`
- class: `Rector\CodeQuality\Rector\Identical\BooleanNotIdenticalToNotIdenticalRector`
Negated identical boolean compare to not identical compare (does not apply to non-bool values)
```diff
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
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `SimplifyConditionsRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodeQuality\Rector\Identical\SimplifyConditionsRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
Simplify conditions
2018-12-22 12:22:28 +01:00
```diff
2019-02-04 01:32:53 +01:00
-if (! ($foo !== 'bar')) {...
+if ($foo === 'bar') {...
2018-12-22 12:22:28 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `SimplifyArraySearchRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodeQuality\Rector\Identical\SimplifyArraySearchRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
Simplify array_search to in_array
2018-12-25 20:55:16 +01:00
```diff
2019-02-04 01:32:53 +01:00
-array_search("searching", $array) !== false;
+in_array("searching", $array, true);
2018-12-25 20:55:16 +01:00
```
2018-11-11 13:22:35 +01:00
```diff
2019-02-04 01:32:53 +01:00
-array_search("searching", $array) != false;
+in_array("searching", $array);
2018-11-11 13:22:35 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `JoinStringConcatRector`
2018-12-22 15:15:11 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodeQuality\Rector\Concat\JoinStringConcatRector`
2018-12-22 15:15:11 +01:00
2019-02-04 01:32:53 +01:00
Joins concat of 2 strings
2018-12-22 15:15:11 +01:00
```diff
2019-02-04 01:32:53 +01:00
class SomeClass
2018-12-22 15:15:11 +01:00
{
2019-02-04 01:32:53 +01:00
public function run()
{
- $name = 'Hi' . ' Tom';
+ $name = 'Hi Tom';
}
2018-12-22 15:15:11 +01:00
}
```
< br >
2019-05-26 21:26:33 +02:00
### `CompleteDynamicPropertiesRector`
- class: `Rector\CodeQuality\Rector\Class_\CompleteDynamicPropertiesRector`
Add missing dynamic properties
```diff
class SomeClass
{
+ /**
+ * @var int
+ */
+ public $value;
public function set()
{
$this->value = 5;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
## CodingStyle
2019-01-03 11:54:13 +01:00
2019-05-26 13:47:23 +02:00
### `SplitDoubleAssignRector`
- class: `Rector\CodingStyle\Rector\Assign\SplitDoubleAssignRector`
Split multiple inline assigns to each own lines default value, to prevent undefined array issues
```diff
class SomeClass
{
public function run()
{
- $one = $two = 1;
+ $one = 1;
+ $two = 1;
}
}
```
< br >
2019-01-03 11:54:13 +01:00
### `ReturnArrayClassMethodToYieldRector`
- class: `Rector\CodingStyle\Rector\ClassMethod\ReturnArrayClassMethodToYieldRector`
Turns yield return to array return in specific type and method
```yaml
services:
Rector\CodingStyle\Rector\ClassMethod\ReturnArrayClassMethodToYieldRector:
EventSubscriberInterface:
- getSubscribedEvents
```
↓
```diff
class SomeEventSubscriber implements EventSubscriberInterface
{
public static function getSubscribedEvents()
{
2019-01-03 13:47:44 +01:00
- yield 'event' => 'callback';
2019-01-03 11:54:13 +01:00
+ return ['event' => 'callback'];
}
}
```
< br >
### `YieldClassMethodToArrayClassMethodRector`
- class: `Rector\CodingStyle\Rector\ClassMethod\YieldClassMethodToArrayClassMethodRector`
Turns yield return to array return in specific type and method
```yaml
services:
Rector\CodingStyle\Rector\ClassMethod\YieldClassMethodToArrayClassMethodRector:
EventSubscriberInterface:
- getSubscribedEvents
```
↓
```diff
class SomeEventSubscriber implements EventSubscriberInterface
2019-01-03 13:47:44 +01:00
{
public static function getSubscribedEvents()
2019-01-03 11:54:13 +01:00
{
2019-01-03 13:47:44 +01:00
- yield 'event' => 'callback';
+ return ['event' => 'callback'];
}
}
2019-01-03 11:54:13 +01:00
```
< br >
2019-05-26 13:47:23 +02:00
### `CatchExceptionNameMatchingTypeRector`
- class: `Rector\CodingStyle\Rector\Catch_\CatchExceptionNameMatchingTypeRector`
Type and name of catch exception should match
```diff
class SomeClass
{
public function run()
{
try {
// ...
- } catch (SomeException $typoException) {
- $typoException->getMessage();
+ } catch (SomeException $someException) {
+ $someException->getMessage();
}
}
}
```
< br >
2019-03-09 13:24:30 +00:00
### `SymplifyQuoteEscapeRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\CodingStyle\Rector\String_\SymplifyQuoteEscapeRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:24:30 +00:00
Prefer quote that not inside the string
2019-02-04 01:32:53 +01:00
```diff
2019-03-09 13:24:30 +00:00
class SomeClass
2019-02-04 01:32:53 +01:00
{
2019-03-09 13:24:30 +00:00
public function run()
{
- $name = "\" Tom";
- $name = '\' Sara';
+ $name = '" Tom';
+ $name = "' Sara";
}
2019-02-04 01:32:53 +01:00
}
```
< br >
2018-11-11 13:22:35 +01:00
2019-05-02 01:56:58 +02:00
### `SplitStringClassConstantToClassConstFetchRector`
- class: `Rector\CodingStyle\Rector\String_\SplitStringClassConstantToClassConstFetchRector`
Separate class constant in a string to class constant fetch and string
```diff
class SomeClass
{
const HI = true;
}
class AnotherClass
{
public function get()
{
- return 'SomeClass::HI';
+ return SomeClass::class . '::HI';
}
}
```
< br >
### `ImportFullyQualifiedNamesRector`
- class: `Rector\CodingStyle\Rector\Namespace_\ImportFullyQualifiedNamesRector`
Import fully qualified names to use statements
```diff
+use SomeAnother\AnotherClass;
+
class SomeClass
{
public function create()
{
- return SomeAnother\AnotherClass;
+ return AnotherClass;
}
}
```
< br >
2019-05-26 13:47:23 +02:00
### `ArrayPropertyDefaultValueRector`
- class: `Rector\CodingStyle\Rector\Property\ArrayPropertyDefaultValueRector`
Array property should have default value, to prevent undefined array issues
```diff
class SomeClass
{
/**
* @var int[]
*/
- private $items;
+ private $items = [];
public function run()
{
foreach ($items as $item) {
}
}
}
```
< br >
2019-03-09 13:24:30 +00:00
### `RemoveUnusedAliasRector`
2019-01-03 11:54:13 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\CodingStyle\Rector\Use_\RemoveUnusedAliasRector`
2019-01-03 11:54:13 +01:00
2019-03-09 13:24:30 +00:00
Removes unused use aliases
2019-01-03 11:54:13 +01:00
```diff
2019-03-09 13:24:30 +00:00
-use Symfony\Kernel as BaseKernel;
+use Symfony\Kernel;
2019-01-03 11:54:13 +01:00
2019-03-09 13:24:30 +00:00
-class SomeClass extends BaseKernel
+class SomeClass extends Kernel
{
2019-01-03 11:54:13 +01:00
}
```
< br >
2019-05-26 13:47:23 +02:00
### `FollowRequireByDirRector`
- class: `Rector\CodingStyle\Rector\Include_\FollowRequireByDirRector`
include/require should be followed by absolute path
```diff
class SomeClass
{
public function run()
{
- require 'autoload.php';
+ require __DIR__ . '/autoload.php';
}
}
```
< br >
2019-01-03 11:54:13 +01:00
### `ConsistentImplodeRector`
- class: `Rector\CodingStyle\Rector\FuncCall\ConsistentImplodeRector`
Changes various implode forms to consistent one
```diff
class SomeClass
{
public function run(array $items)
{
- $itemsAsStrings = implode($items);
- $itemsAsStrings = implode($items, '|');
+ $itemsAsStrings = implode('', $items);
+ $itemsAsStrings = implode('|', $items);
$itemsAsStrings = implode('|', $items);
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `SimpleArrayCallableToStringRector`
- class: `Rector\CodingStyle\Rector\FuncCall\SimpleArrayCallableToStringRector`
Changes redundant anonymous bool functions to simple calls
```diff
-$paths = array_filter($paths, function ($path): bool {
- return is_dir($path);
-});
+array_filter($paths, "is_dir");
```
< br >
2019-05-26 13:47:23 +02:00
### `ConsistentPregDelimiterRector`
- class: `Rector\CodingStyle\Rector\FuncCall\ConsistentPregDelimiterRector`
Replace PREG delimiter with configured one
```diff
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);
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `NullableCompareToNullRector`
- class: `Rector\CodingStyle\Rector\If_\NullableCompareToNullRector`
Changes negate of empty comparison of nullable value to explicit === or !== compare
```diff
/** @var stdClass|null $value */
-if ($value) {
+if ($value !== null) {
}
-if (!$value) {
+if ($value === null) {
}
```
2018-10-22 00:26:45 +02:00
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2019-05-02 01:56:58 +02:00
### `SplitGroupedConstantsAndPropertiesRector`
- class: `Rector\CodingStyle\Rector\ClassConst\SplitGroupedConstantsAndPropertiesRector`
Separate constant and properties to own lines
```diff
class SomeClass
{
- const HI = true, AHOJ = 'true';
+ const HI = true;
+ const AHOJ = 'true';
/**
* @var string
*/
- public $isIt, $isIsThough;
+ public $isIt;
+
+ /**
+ * @var string
+ */
+ public $isIsThough;
}
```
< br >
2019-05-26 13:47:23 +02:00
### `VarConstantCommentRector`
- class: `Rector\CodingStyle\Rector\ClassConst\VarConstantCommentRector`
Constant should have a @var comment with type
```diff
class SomeClass
{
+ /**
+ * @var string
+ */
const HI = 'hi';
}
```
< br >
2019-02-04 01:32:53 +01:00
### `CompleteVarDocTypeConstantRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodingStyle\Rector\ClassConst\CompleteVarDocTypeConstantRector`
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
Complete constant `@var` annotations for missing one, yet known.
2018-10-22 00:26:45 +02:00
```diff
2019-02-04 01:32:53 +01:00
final class SomeClass
{
+ /**
+ * @var int
+ */
private const NUMBER = 5;
}
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
### `IdenticalFalseToBooleanNotRector`
2019-01-03 11:54:13 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\CodingStyle\Rector\Identical\IdenticalFalseToBooleanNotRector`
2019-01-03 11:54:13 +01:00
2019-02-04 01:32:53 +01:00
Changes === false to negate !
2019-01-03 11:54:13 +01:00
```diff
2019-02-04 01:32:53 +01:00
-if ($something === false) {}
+if (! $something) {}
```
2019-01-03 11:54:13 +01:00
2019-02-04 01:32:53 +01:00
< br >
### `BinarySwitchToIfElseRector`
- class: `Rector\CodingStyle\Rector\Switch_\BinarySwitchToIfElseRector`
Changes switch with 2 options to if-else
```diff
-switch ($foo) {
- case 'my string':
- $result = 'ok';
- break;
-
- default:
- $result = 'not ok';
+if ($foo == 'my string') {
+ $result = 'ok;
+} else {
+ $result = 'not ok';
2019-01-03 11:54:13 +01:00
}
```
< br >
2019-02-04 01:32:53 +01:00
## DeadCode
### `RemoveDoubleAssignRector`
- class: `Rector\DeadCode\Rector\Assign\RemoveDoubleAssignRector`
Simplify useless double assigns
```diff
-$value = 1;
$value = 1;
```
< br >
### `RemoveUnusedForeachKeyRector`
- class: `Rector\DeadCode\Rector\Foreach_\RemoveUnusedForeachKeyRector`
Remove unused key in foreach
```diff
$items = [];
-foreach ($items as $key => $value) {
+foreach ($items as $value) {
$result = $value;
}
```
< br >
2018-12-25 20:55:16 +01:00
### `RemoveDuplicatedArrayKeyRector`
- class: `Rector\DeadCode\Rector\Array_\RemoveDuplicatedArrayKeyRector`
Remove duplicated key in defined arrays.
```diff
$item = [
2019-03-31 14:25:39 +02:00
- 1 => 'A',
1 => 'B'
2018-12-25 20:55:16 +01:00
];
```
< br >
2019-02-04 01:32:53 +01:00
### `RemoveParentCallWithoutParentRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\DeadCode\Rector\StaticCall\RemoveParentCallWithoutParentRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
Remove unused parent call with no parent class
2018-12-25 20:55:16 +01:00
```diff
2019-02-04 01:32:53 +01:00
class OrphanClass
{
public function __construct()
{
- parent::__construct();
}
}
2018-12-25 20:55:16 +01:00
```
< br >
### `RemoveEmptyClassMethodRector`
- class: `Rector\DeadCode\Rector\ClassMethod\RemoveEmptyClassMethodRector`
Remove empty method calls not required by parents
```diff
class OrphanClass
{
- public function __construct()
- {
- }
}
```
< br >
2018-12-31 12:50:32 +01:00
### `RemoveUnusedParameterRector`
- class: `Rector\DeadCode\Rector\ClassMethod\RemoveUnusedParameterRector`
Remove unused parameter, if not required by interface or parent class
```diff
class SomeClass
{
- public function __construct($value, $value2)
+ public function __construct($value)
{
$this->value = $value;
}
}
```
< br >
2019-03-09 13:24:30 +00:00
### `RemoveDeadConstructorRector`
- class: `Rector\DeadCode\Rector\ClassMethod\RemoveDeadConstructorRector`
Remove empty constructor
```diff
class SomeClass
{
- public function __construct()
- {
- }
}
```
< br >
2019-02-21 15:36:16 +01:00
### `RemoveUnusedPrivateMethodRector`
- class: `Rector\DeadCode\Rector\ClassMethod\RemoveUnusedPrivateMethodRector`
Remove unused private method
```diff
final class SomeController
{
public function run()
{
return 5;
}
-
- private function skip()
- {
- return 10;
- }
}
```
< br >
2019-02-18 16:51:24 +01:00
### `RemoveOverriddenValuesRector`
- class: `Rector\DeadCode\Rector\ClassMethod\RemoveOverriddenValuesRector`
Remove initial assigns of overridden values
```diff
final class SomeController
{
public function run()
{
- $directories = [];
$possibleDirectories = [];
$directories = array_filter($possibleDirectories, 'file_exists');
}
}
```
< br >
2019-05-19 10:27:38 +02:00
### `RemoveDefaultArgumentValueRector`
- class: `Rector\DeadCode\Rector\MethodCall\RemoveDefaultArgumentValueRector`
Remove argument value, if it is the same as default value
```diff
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;
}
}
```
< br >
2019-03-31 14:25:39 +02:00
### `RemoveDeadIfForeachForRector`
- class: `Rector\DeadCode\Rector\For_\RemoveDeadIfForeachForRector`
Remove if, foreach and for that does not do anything
```diff
class SomeClass
{
public function run($someObject)
{
$value = 5;
- if ($value) {
- }
-
if ($someObject->run()) {
- }
-
- foreach ($values as $value) {
}
return $value;
}
}
```
< br >
2018-12-25 20:55:16 +01:00
### `RemoveUnusedPrivatePropertyRector`
- class: `Rector\DeadCode\Rector\Property\RemoveUnusedPrivatePropertyRector`
Remove unused private properties
```diff
class SomeClass
{
- private $property;
}
```
< br >
2019-02-04 01:32:53 +01:00
### `RemoveDeadStmtRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\DeadCode\Rector\Stmt\RemoveDeadStmtRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
Removes dead code statements
2018-12-25 20:55:16 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$value = 5;
-$value;
+$value = 5;
2018-12-25 20:55:16 +01:00
```
< br >
2019-05-19 10:27:38 +02:00
### `RemoveAndTrueRector`
- class: `Rector\DeadCode\Rector\BooleanAnd\RemoveAndTrueRector`
Remove and true that has no added value
```diff
class SomeClass
{
public function run()
{
- return true && 5 === 1;
+ return 5 === 1;
}
}
```
< br >
2019-02-21 15:36:16 +01:00
### `RemoveUnusedPrivateConstantRector`
- class: `Rector\DeadCode\Rector\ClassConst\RemoveUnusedPrivateConstantRector`
Remove unused private constant
```diff
final class SomeController
{
- private const SOME_CONSTANT = 5;
public function run()
{
return 5;
}
}
```
< br >
### `RemoveCodeAfterReturnRector`
- class: `Rector\DeadCode\Rector\FunctionLike\RemoveCodeAfterReturnRector`
Remove dead code after return statement
```diff
class SomeClass
{
public function run(int $a)
{
return $a;
- $a++;
}
}
```
< br >
2019-03-31 14:25:39 +02:00
### `RemoveDeadReturnRector`
- class: `Rector\DeadCode\Rector\FunctionLike\RemoveDeadReturnRector`
Remove last return in the functions, since does not do anything
```diff
class SomeClass
{
public function run()
{
$shallWeDoThis = true;
if ($shallWeDoThis) {
return;
}
-
- return;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `SimplifyMirrorAssignRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\DeadCode\Rector\Expression\SimplifyMirrorAssignRector`
2018-12-25 20:55:16 +01:00
2019-02-04 01:32:53 +01:00
Removes unneeded $a = $a assigns
2018-12-25 20:55:16 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$a = $a;
2018-12-25 20:55:16 +01:00
```
< br >
2018-08-01 22:09:34 +02:00
## Doctrine
### `AliasToClassRector`
- class: `Rector\Doctrine\Rector\AliasToClassRector`
Replaces doctrine alias with class.
```diff
2018-08-10 21:07:08 +02:00
$entityManager = new Doctrine\ORM\EntityManager();
-$entityManager->getRepository("AppBundle:Post");
+$entityManager->getRepository(\App\Entity\Post::class);
2018-08-01 22:09:34 +02:00
```
2018-07-31 23:47:59 +02:00
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
## DomainDrivenDesign
2018-07-31 21:43:39 +02:00
2018-12-14 20:35:35 +01:00
### `ObjectToScalarDocBlockRector`
2018-07-31 21:43:39 +02:00
2018-12-14 20:35:35 +01:00
- class: `Rector\DomainDrivenDesign\Rector\ObjectToScalar\ObjectToScalarDocBlockRector`
2018-05-05 13:48:33 +01:00
2018-10-22 00:26:45 +02:00
Turns defined value object to simple types in doc blocks
```yaml
services:
2018-12-14 20:35:35 +01:00
Rector\DomainDrivenDesign\Rector\ObjectToScalar\ObjectToScalarDocBlockRector:
2018-10-22 00:26:45 +02:00
$valueObjectsToSimpleTypes:
ValueObject: string
```
↓
2018-05-05 13:48:33 +01:00
```diff
2018-10-12 20:15:00 -03:00
/**
2018-10-22 00:26:45 +02:00
- * @var ValueObject|null
+ * @var string|null
2018-10-12 20:15:00 -03:00
*/
2018-10-22 00:26:45 +02:00
private $name;
2018-07-31 21:43:39 +02:00
2018-10-22 00:26:45 +02:00
-/** @var ValueObject|null */
+/** @var string|null */
$name;
```
2018-11-11 13:22:35 +01:00
< br >
2018-12-14 20:35:35 +01:00
### `ObjectToScalarRector`
2018-10-22 00:26:45 +02:00
2018-12-14 20:35:35 +01:00
- class: `Rector\DomainDrivenDesign\Rector\ObjectToScalar\ObjectToScalarRector`
2018-10-22 00:26:45 +02:00
Remove values objects and use directly the value.
```yaml
services:
2018-12-14 20:35:35 +01:00
Rector\DomainDrivenDesign\Rector\ObjectToScalar\ObjectToScalarRector:
2018-10-22 00:26:45 +02:00
$valueObjectsToSimpleTypes:
ValueObject: string
```
↓
2018-07-31 14:50:39 +02:00
```diff
2018-10-22 00:26:45 +02:00
-$name = new ValueObject("name");
+$name = "name";
2018-10-12 20:15:00 -03:00
2018-12-14 20:35:35 +01:00
-function someFunction(ValueObject $name): ?ValueObject {
+function someFunction(string $name): ?string {
}
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
## Guzzle
2018-11-11 13:22:35 +01:00
### `MessageAsArrayRector`
- class: `Rector\Guzzle\Rector\MethodCall\MessageAsArrayRector`
Changes getMessage(..., true) to getMessageAsArray()
```diff
/** @var GuzzleHttp\Message\MessageInterface */
-$value = $message->getMessage('key', true);
+$value = $message->getMessageAsArray('key');
```
< br >
2019-03-09 13:24:30 +00:00
## Laravel
### `MinutesToSecondsInCacheRector`
- class: `Rector\Laravel\Rector\StaticCall\MinutesToSecondsInCacheRector`
Change minutes argument to seconds in Illuminate\Contracts\Cache\Store and Illuminate\Support\Facades\Cache
```diff
class SomeClass
{
public function run()
{
- Illuminate\Support\Facades\Cache::put('key', 'value', 60);
+ Illuminate\Support\Facades\Cache::put('key', 'value', 60 * 60);
}
}
```
< br >
### `FacadeStaticCallToConstructorInjectionRector`
- class: `Rector\Laravel\Rector\StaticCall\FacadeStaticCallToConstructorInjectionRector`
Move Illuminate\Support\Facades\* static calls to constructor injection
```diff
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]);
}
}
```
< br >
### `Redirect301ToPermanentRedirectRector`
- class: `Rector\Laravel\Rector\StaticCall\Redirect301ToPermanentRedirectRector`
Change "redirect" call with 301 to "permanentRedirect"
```diff
class SomeClass
{
public function run()
{
- Illuminate\Routing\Route::redirect('/foo', '/bar', 301);
+ Illuminate\Routing\Route::permanentRedirect('/foo', '/bar');
}
}
```
< br >
### `RequestStaticValidateToInjectRector`
- class: `Rector\Laravel\Rector\StaticCall\RequestStaticValidateToInjectRector`
Change static validate() method to $request->validate()
```diff
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']);
}
}
```
< br >
2019-03-16 21:31:46 +01:00
### `HelperFunctionToConstructorInjectionRector`
- class: `Rector\Laravel\Rector\FuncCall\HelperFunctionToConstructorInjectionRector`
Move help facade-like function calls to constructor injection
```diff
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;
}
}
```
< br >
2019-05-26 18:38:09 +02:00
## Legacy
### `ChangeSingletonToServiceRector`
- class: `Rector\Legacy\Rector\ClassMethod\ChangeSingletonToServiceRector`
Change singleton class to normal class that can be registered as a service
```diff
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;
}
}
```
< br >
2019-02-21 15:36:16 +01:00
## MysqlToMysqli
### `MysqlAssignToMysqliRector`
- class: `Rector\MysqlToMysqli\Rector\Assign\MysqlAssignToMysqliRector`
Converts more complex mysql functions to mysqli
```diff
-$data = mysql_db_name($result, $row);
+mysqli_data_seek($result, $row);
+$fetch = mysql_fetch_row($result);
+$data = $fetch[0];
```
< br >
### `MysqlFuncCallToMysqliRector`
- class: `Rector\MysqlToMysqli\Rector\FuncCall\MysqlFuncCallToMysqliRector`
Converts more complex mysql functions to mysqli
```diff
-mysql_drop_db($database);
+mysqli_query('DROP DATABASE ' . $database);
```
< br >
### `MysqlPConnectToMysqliConnectRector`
- class: `Rector\MysqlToMysqli\Rector\FuncCall\MysqlPConnectToMysqliConnectRector`
Replace mysql_pconnect() with mysqli_connect() with host p: prefix
```diff
final class SomeClass
{
public function run($host, $username, $password)
{
- return mysql_pconnect($host, $username, $password);
+ return mysqli_connect('p:' . $host, $username, $password);
}
}
```
< br >
2019-04-02 15:35:35 +02:00
## Nette
### `SubstrStrlenFunctionToNetteUtilsStringsRector`
- class: `Rector\Nette\Rector\FuncCall\SubstrStrlenFunctionToNetteUtilsStringsRector`
Use Nette\Utils\Strings over bare string-functions
```diff
class SomeClass
{
public function run()
{
- return substr($value, 0, 3);
+ return \Nette\Utils\Strings::substring($value, 0, 3);
}
}
```
< br >
### `PregFunctionToNetteUtilsStringsRector`
- class: `Rector\Nette\Rector\FuncCall\PregFunctionToNetteUtilsStringsRector`
Use Nette\Utils\Strings over bare preg_* functions
```diff
class SomeClass
{
public function run()
{
$content = 'Hi my name is Tom';
- preg_match('#Hi #', $content);
+ \Nette\Utils\Strings::match($content, '#Hi #');
}
}
```
< br >
### `StrposToStringsContainsRector`
- class: `Rector\Nette\Rector\NotIdentical\StrposToStringsContainsRector`
Use Nette\Utils\Strings over bare string-functions
```diff
class SomeClass
{
public function run()
{
$name = 'Hi, my name is Tom';
- return strpos($name, 'Hi') !== false;
+ return \Nette\Utils\Strings::contains($name, 'Hi');
}
}
```
< br >
### `EndsWithFunctionToNetteUtilsStringsRector`
- class: `Rector\Nette\Rector\Identical\EndsWithFunctionToNetteUtilsStringsRector`
Use Nette\Utils\Strings over bare string-functions
```diff
class SomeClass
{
public function end($needle)
{
$content = 'Hi, my name is Tom';
- $yes = substr($content, -strlen($needle)) === $needle;
- $no = $needle !== substr($content, -strlen($needle));
+ $yes = \Nette\Utils\Strings::endsWith($content, $needle);
+ $no = !\Nette\Utils\Strings::endsWith($content, $needle);
}
}
```
< br >
### `StartsWithFunctionToNetteUtilsStringsRector`
- class: `Rector\Nette\Rector\Identical\StartsWithFunctionToNetteUtilsStringsRector`
Use Nette\Utils\Strings over bare string-functions
```diff
class SomeClass
{
public function start($needle)
{
$content = 'Hi, my name is Tom';
- $yes = substr($content, 0, strlen($needle)) === $needle;
- $no = $needle !== substr($content, 0, strlen($needle));
+ $yes = \Nette\Utils\Strings::startwith($content, $needle);
+ $no = !\Nette\Utils\Strings::startwith($content, $needle);
}
}
```
< br >
2019-03-16 21:31:46 +01:00
## NetteTesterToPHPUnit
2019-03-31 14:25:39 +02:00
### `NetteAssertToPHPUnitAssertRector`
- class: `Rector\NetteTesterToPHPUnit\Rector\StaticCall\NetteAssertToPHPUnitAssertRector`
Migrate Nette/Assert calls to PHPUnit
```diff
use Tester\Assert;
function someStaticFunctions()
{
- Assert::true(10 == 5);
+ \PHPUnit\Framework\Assert::assertTrue(10 == 5);
}
```
< br >
### `RenameTesterTestToPHPUnitToTestFileRector`
- class: `Rector\NetteTesterToPHPUnit\Rector\RenameTesterTestToPHPUnitToTestFileRector`
Rename "*.phpt" file to "*Test.php" file
< br >
2019-03-16 21:31:46 +01:00
### `NetteTesterClassToPHPUnitClassRector`
- class: `Rector\NetteTesterToPHPUnit\Rector\Class_\NetteTesterClassToPHPUnitClassRector`
Migrate Nette Tester test case to PHPUnit
```diff
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);
2019-03-31 14:25:39 +02:00
+ $this->assertInstanceOf(\Kdyby\Doctrine\EntityManager::cllass, $default);
+ $this->assertTrue(5);
+ $this->same($container->getService('kdyby.doctrine.default.entityManager'), $default);
2019-03-16 21:31:46 +01:00
}
-}
-
-(new \ExtensionTest())->run();
+}
```
< br >
2019-02-02 17:22:15 +01:00
## NetteToSymfony
### `RouterListToControllerAnnotationsRector`
2019-02-18 16:51:24 +01:00
- class: `Rector\NetteToSymfony\Rector\ClassMethod\RouterListToControllerAnnotationsRector`
2019-02-02 17:22:15 +01:00
Change new Route() from RouteFactory to @Route annotation above controller method
```diff
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;
}
}
final class SomePresenter
{
+ /**
+ * @Symfony \Component\Routing\Annotation\Route(path="some-path")
+ */
public function run()
{
}
}
```
< br >
2019-02-18 16:51:24 +01:00
### `RenameEventNamesInEventSubscriberRector`
- class: `Rector\NetteToSymfony\Rector\ClassMethod\RenameEventNamesInEventSubscriberRector`
Changes event names from Nette ones to Symfony ones
```diff
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
final class SomeClass implements EventSubscriberInterface
{
public static function getSubscribedEvents()
{
- return ['nette.application' => 'someMethod'];
+ return [\SymfonyEvents::KERNEL => 'someMethod'];
}
}
```
< br >
### `FromRequestGetParameterToAttributesGetRector`
- class: `Rector\NetteToSymfony\Rector\MethodCall\FromRequestGetParameterToAttributesGetRector`
Changes "getParameter()" to "attributes->get()" from Nette to Symfony
```diff
use Nette\Request;
final class SomeController
{
public static function someAction(Request $request)
{
- $value = $request->getParameter('abz');
+ $value = $request->attribute->get('abz');
}
}
```
< br >
### `FromHttpRequestGetHeaderToHeadersGetRector`
- class: `Rector\NetteToSymfony\Rector\MethodCall\FromHttpRequestGetHeaderToHeadersGetRector`
Changes getHeader() to $request->headers->get()
```diff
use Nette\Request;
final class SomeController
{
public static function someAction(Request $request)
{
- $header = $this->httpRequest->getHeader('x');
+ $header = $request->headers->get('x');
}
}
```
< br >
### `WrapTransParameterNameRector`
- class: `Rector\NetteToSymfony\Rector\MethodCall\WrapTransParameterNameRector`
Adds %% to placeholder name of trans() method if missing
```diff
use Symfony\Component\Translation\Translator;
final class SomeController
{
public function run()
{
$translator = new Translator('');
$translated = $translator->trans(
'Hello %name%',
- ['name' => $name]
+ ['%name%' => $name]
);
}
}
```
< br >
2019-03-31 14:25:39 +02:00
### `NetteControlToSymfonyControllerRector`
- class: `Rector\NetteToSymfony\Rector\Class_\NetteControlToSymfonyControllerRector`
Migrate Nette Component to Symfony Controller
```diff
use Nette\Application\UI\Control;
-class SomeControl extends Control
+class SomeController extends \Symfony\Bundle\FrameworkBundle\Controller\AbstractController
{
- public function render()
- {
- $this->template->param = 'some value';
- $this->template->render(__DIR__ . '/poll.latte');
- }
+ public function some()
+ {
+ $this->render(__DIR__ . '/poll.latte', ['param' => 'some value']);
+ }
}
```
< br >
### `NetteFormToSymfonyFormRector`
- class: `Rector\NetteToSymfony\Rector\Class_\NetteFormToSymfonyFormRector`
Migrate Nette\Forms in Presenter to Symfony
```diff
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'
+ ]);
}
}
```
< br >
2019-02-04 01:32:53 +01:00
## PHPStan
2018-12-06 01:13:34 +01:00
### `PHPStormVarAnnotationRector`
- class: `Rector\PHPStan\Rector\Assign\PHPStormVarAnnotationRector`
Change various @var annotation formats to one PHPStorm understands
```diff
-$config = 5;
-/** @var \Shopsys\FrameworkBundle\Model\Product\Filter\ProductFilterConfig $config */
+/** @var \Shopsys\FrameworkBundle\Model\Product\Filter\ProductFilterConfig $config */
+$config = 5;
```
< br >
2018-11-11 13:22:35 +01:00
### `RecastingRemovalRector`
- class: `Rector\PHPStan\Rector\Cast\RecastingRemovalRector`
Removes recasting of the same type
```diff
$string = '';
-$string = (string) $string;
+$string = $string;
$array = [];
-$array = (array) $array;
+$array = $array;
```
< br >
2018-10-22 00:26:45 +02:00
## PHPUnit
### `SimplifyForeachInstanceOfRector`
- class: `Rector\PHPUnit\Rector\Foreach_\SimplifyForeachInstanceOfRector`
Simplify unnecessary foreach check of instances
2018-10-12 20:15:00 -03:00
```diff
2018-10-22 00:26:45 +02:00
-foreach ($foos as $foo) {
- $this->assertInstanceOf(\SplFileInfo::class, $foo);
-}
+$this->assertContainsOnlyInstancesOf(\SplFileInfo::class, $foos);
2018-10-12 20:15:00 -03:00
```
2018-07-31 21:43:39 +02:00
2018-11-11 13:22:35 +01:00
< br >
2018-10-23 15:58:57 -03:00
2018-12-14 20:35:35 +01:00
### `AssertEqualsParameterToSpecificMethodsTypeRector`
- class: `Rector\PHPUnit\Rector\MethodCall\AssertEqualsParameterToSpecificMethodsTypeRector`
Change assertEquals()/assertNotEquals() method parameters to new specific alternatives
```diff
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');
}
}
```
< br >
### `SpecificAssertContainsRector`
- class: `Rector\PHPUnit\Rector\MethodCall\SpecificAssertContainsRector`
Change assertContains()/assertNotContains() method to new string and iterable alternatives
```diff
< ?php
final class SomeTest extends \PHPUnit\Framework\TestCase
{
public function test()
{
- $this->assertContains('foo', 'foo bar');
- $this->assertNotContains('foo', 'foo bar');
2019-03-16 17:01:40 +01:00
+ $this->assertStringContainsString('foo', 'foo bar');
+ $this->assertStringNotContainsString('foo', 'foo bar');
2019-03-16 21:31:46 +01:00
}
}
```
< br >
### `ReplaceAssertArraySubsetRector`
- class: `Rector\PHPUnit\Rector\MethodCall\ReplaceAssertArraySubsetRector`
Replace deprecated "assertArraySubset()" method with alternative methods
```diff
class SomeTest extends \PHPUnit\Framework\TestCase
{
public function test()
{
$checkedArray = [];
- $this->assertArraySubset([
- 'cache_directory' => 'new_value',
- ], $checkedArray);
+ $this->assertArrayHasKey('cache_directory', $checkedArray);
+ $this->assertSame('new_value', $checkedArray['cache_directory']);
2018-12-14 20:35:35 +01:00
}
}
```
< br >
2018-12-31 21:59:41 +01:00
### `UseSpecificWillMethodRector`
- class: `Rector\PHPUnit\Rector\MethodCall\UseSpecificWillMethodRector`
Changes ->will($this->xxx()) to one specific method
```diff
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 }}!');
}
}
```
< br >
2018-12-14 20:35:35 +01:00
### `SpecificAssertInternalTypeRector`
- class: `Rector\PHPUnit\Rector\MethodCall\SpecificAssertInternalTypeRector`
Change assertInternalType()/assertNotInternalType() method to new specific alternatives
```diff
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);
}
}
```
< br >
2019-05-02 01:56:58 +02:00
### `RemoveExpectAnyFromMockRector`
- class: `Rector\PHPUnit\Rector\MethodCall\RemoveExpectAnyFromMockRector`
Remove `expect($this->any())` from mocks as it has no added value
```diff
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 }}!');
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `ExceptionAnnotationRector`
- class: `Rector\PHPUnit\Rector\ExceptionAnnotationRector`
Takes `setExpectedException()` 2nd and next arguments to own methods in PHPUnit.
```diff
-/**
- * @expectedException Exception
- * @expectedExceptionMessage Message
- */
public function test()
{
+ $this->expectException('Exception');
+ $this->expectExceptionMessage('Message');
// tested code
}
```
< br >
2018-10-23 15:58:57 -03:00
2018-08-01 15:34:55 +02:00
### `AssertNotOperatorRector`
2018-07-31 14:50:39 +02:00
2018-08-01 15:34:55 +02:00
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertNotOperatorRector`
Turns not-operator comparisons to their method name alternatives in PHPUnit TestCase
2018-07-31 14:50:39 +02:00
```diff
2018-08-01 15:34:55 +02:00
-$this->assertTrue(!$foo, "message");
+$this->assertFalse($foo, "message");
2018-08-01 22:09:34 +02:00
```
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```diff
2018-08-01 15:34:55 +02:00
-$this->assertFalse(!$foo, "message");
+$this->assertTrue($foo, "message");
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-07-31 21:43:39 +02:00
2018-11-11 13:22:35 +01:00
### `AssertComparisonToSpecificMethodRector`
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertComparisonToSpecificMethodRector`
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
Turns comparison operations to their method name alternatives in PHPUnit TestCase
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
```diff
2018-11-11 13:22:35 +01:00
-$this->assertTrue($foo === $bar, "message");
+$this->assertSame($bar, $foo, "message");
2018-10-12 20:15:00 -03:00
```
```diff
2018-11-11 13:22:35 +01:00
-$this->assertFalse($foo >= $bar, "message");
+$this->assertLessThanOrEqual($bar, $foo, "message");
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-07-31 14:50:39 +02:00
2018-10-23 15:58:57 -03:00
### `AssertPropertyExistsRector`
2018-07-31 21:43:39 +02:00
2018-10-23 15:58:57 -03:00
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertPropertyExistsRector`
2018-07-31 14:50:39 +02:00
2018-10-23 15:58:57 -03:00
Turns `property_exists` comparisons to their method name alternatives in PHPUnit TestCase
2018-07-31 14:50:39 +02:00
```diff
2018-10-23 15:58:57 -03:00
-$this->assertTrue(property_exists(new Class, "property"), "message");
+$this->assertClassHasAttribute("property", "Class", "message");
2018-08-01 22:09:34 +02:00
```
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```diff
2018-10-23 15:58:57 -03:00
-$this->assertFalse(property_exists(new Class, "property"), "message");
+$this->assertClassNotHasAttribute("property", "Class", "message");
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
### `AssertTrueFalseInternalTypeToSpecificMethodRector`
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertTrueFalseInternalTypeToSpecificMethodRector`
Turns true/false with internal type comparisons to their method name alternatives in PHPUnit TestCase
```diff
-$this->assertTrue(is_{internal_type}($anything), "message");
+$this->assertInternalType({internal_type}, $anything, "message");
```
```diff
-$this->assertFalse(is_{internal_type}($anything), "message");
+$this->assertNotInternalType({internal_type}, $anything, "message");
```
< br >
2018-10-12 20:15:00 -03:00
### `AssertIssetToSpecificMethodRector`
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertIssetToSpecificMethodRector`
Turns isset comparisons to their method name alternatives in PHPUnit TestCase
```diff
-$this->assertTrue(isset($anything->foo));
+$this->assertFalse(isset($anything["foo"]), "message");
```
```diff
-$this->assertObjectHasAttribute("foo", $anything);
+$this->assertArrayNotHasKey("foo", $anything, "message");
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
### `AssertFalseStrposToContainsRector`
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertFalseStrposToContainsRector`
Turns `strpos` /`stripos` comparisons to their method name alternatives in PHPUnit TestCase
2018-10-12 20:15:00 -03:00
2018-10-22 00:26:45 +02:00
```diff
2018-11-11 13:22:35 +01:00
-$this->assertFalse(strpos($anything, "foo"), "message");
+$this->assertNotContains("foo", $anything, "message");
2018-10-22 00:26:45 +02:00
```
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
```diff
-$this->assertNotFalse(stripos($anything, "foo"), "message");
+$this->assertContains("foo", $anything, "message");
```
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2018-11-11 13:22:35 +01:00
### `AssertSameBoolNullToSpecificMethodRector`
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertSameBoolNullToSpecificMethodRector`
Turns same bool and null comparisons to their method name alternatives in PHPUnit TestCase
2018-10-12 20:15:00 -03:00
```diff
2018-11-11 13:22:35 +01:00
-$this->assertSame(null, $anything);
+$this->assertNull($anything);
2018-10-12 20:15:00 -03:00
```
```diff
2018-11-11 13:22:35 +01:00
-$this->assertNotSame(false, $anything);
+$this->assertNotFalse($anything);
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
### `AssertCompareToSpecificMethodRector`
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertCompareToSpecificMethodRector`
Turns vague php-only method in PHPUnit TestCase to more specific
```diff
-$this->assertSame(10, count($anything), "message");
+$this->assertCount(10, $anything, "message");
```
```diff
-$this->assertSame($value, {function}($anything), "message");
+$this->assert{function}($value, $anything, "message\");
```
```diff
-$this->assertEquals($value, {function}($anything), "message");
+$this->assert{function}($value, $anything, "message\");
```
```diff
-$this->assertNotSame($value, {function}($anything), "message");
+$this->assertNot{function}($value, $anything, "message")
```
```diff
-$this->assertNotEquals($value, {function}($anything), "message");
+$this->assertNot{function}($value, $anything, "message")
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
### `AssertRegExpRector`
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertRegExpRector`
Turns `preg_match` comparisons to their method name alternatives in PHPUnit TestCase
```diff
-$this->assertSame(1, preg_match("/^Message for ".*"\.$/", $string), $message);
+$this->assertRegExp("/^Message for ".*"\.$/", $string, $message);
```
```diff
-$this->assertEquals(false, preg_match("/^Message for ".*"\.$/", $string), $message);
+$this->assertNotRegExp("/^Message for ".*"\.$/", $string, $message);
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
### `AssertInstanceOfComparisonRector`
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertInstanceOfComparisonRector`
Turns instanceof comparisons to their method name alternatives in PHPUnit TestCase
2018-10-12 20:15:00 -03:00
```diff
2018-11-11 13:22:35 +01:00
-$this->assertTrue($foo instanceof Foo, "message");
2019-03-31 14:25:39 +02:00
+$this->assertInstanceOf("Foo", $foo, "message");
2018-10-12 20:15:00 -03:00
```
```diff
2019-03-31 14:25:39 +02:00
-$this->assertFalse($foo instanceof Foo, "message");
2018-11-11 13:22:35 +01:00
+$this->assertNotInstanceOf("Foo", $foo, "message");
```
< br >
### `AssertTrueFalseToSpecificMethodRector`
- class: `Rector\PHPUnit\Rector\SpecificMethod\AssertTrueFalseToSpecificMethodRector`
Turns true/false comparisons to their method name alternatives in PHPUnit TestCase when possible
```diff
-$this->assertTrue(is_readable($readmeFile), "message");
+$this->assertIsReadable($readmeFile, "message");
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `DelegateExceptionArgumentsRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PHPUnit\Rector\DelegateExceptionArgumentsRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Takes `setExpectedException()` 2nd and next arguments to own methods in PHPUnit.
2018-10-12 20:15:00 -03:00
```diff
2019-02-04 01:32:53 +01:00
-$this->setExpectedException(Exception::class, "Message", "CODE");
+$this->setExpectedException(Exception::class);
+$this->expectExceptionMessage("Message");
+$this->expectExceptionCode("CODE");
2018-11-11 13:22:35 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `TryCatchToExpectExceptionRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PHPUnit\Rector\TryCatchToExpectExceptionRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Turns try/catch to expectException() call
2018-10-12 20:15:00 -03:00
```diff
2019-02-04 01:32:53 +01:00
-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();
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `GetMockRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PHPUnit\Rector\GetMockRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Turns getMock*() methods to createMock()
2018-10-12 20:15:00 -03:00
```diff
2019-02-04 01:32:53 +01:00
-$this->getMock("Class");
+$this->createMock("Class");
2018-10-12 20:15:00 -03:00
```
```diff
2019-02-04 01:32:53 +01:00
-$this->getMockWithoutInvokingTheOriginalConstructor("Class");
+$this->createMock("Class");
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-23 15:58:57 -03:00
2019-03-31 14:25:39 +02:00
### `TestListenerToHooksRector`
- class: `Rector\PHPUnit\Rector\Class_\TestListenerToHooksRector`
Refactor "*TestListener.php" to particular "*Hook.php" files
```diff
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;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
## Php
2018-10-12 20:15:00 -03:00
### `AssignArrayToStringRector`
- class: `Rector\Php\Rector\Assign\AssignArrayToStringRector`
String cannot be turned into array by assignment anymore
```diff
-$string = '';
+$string = [];
$string[] = 1;
```
2018-11-11 13:22:35 +01:00
< br >
2019-01-22 21:34:38 +01:00
### `NullCoalescingOperatorRector`
- class: `Rector\Php\Rector\Assign\NullCoalescingOperatorRector`
Use null coalescing operator ??=
```diff
$array = [];
-$array['user_id'] = $array['user_id'] ?? 'value';
+$array['user_id'] ??= 'value';
```
< br >
2018-12-14 20:35:35 +01:00
### `MysqlAssignToMysqliRector`
- class: `Rector\Php\Rector\Assign\MysqlAssignToMysqliRector`
Converts more complex mysql functions to mysqli
```diff
-$data = mysql_db_name($result, $row);
+mysqli_data_seek($result, $row);
+$fetch = mysql_fetch_row($result);
+$data = $fetch[0];
```
< br >
2019-02-04 01:32:53 +01:00
### `TernaryToSpaceshipRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\Ternary\TernaryToSpaceshipRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Use < => spaceship instead of ternary with same effect
2018-10-12 20:15:00 -03:00
```diff
2019-02-04 01:32:53 +01:00
function order_func($a, $b) {
- return ($a < $b) ? -1 : (($a > $b) ? 1 : 0);
+ return $a < => $b;
}
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `TernaryToNullCoalescingRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\Ternary\TernaryToNullCoalescingRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Changes unneeded null check to ?? operator
2018-10-12 20:15:00 -03:00
```diff
2019-02-04 01:32:53 +01:00
-$value === null ? 10 : $value;
+$value ?? 10;
2018-10-12 20:15:00 -03:00
```
```diff
2019-02-04 01:32:53 +01:00
-isset($value) ? $value : 10;
+$value ?? 10;
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `StaticCallOnNonStaticToInstanceCallRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\StaticCall\StaticCallOnNonStaticToInstanceCallRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
Changes static call to instance call, where not useful
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
class Something
{
public function doWork()
{
}
}
class Another
{
public function run()
{
- return Something::doWork();
+ return (new Something)->doWork();
}
}
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ExportToReflectionFunctionRector`
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\StaticCall\ExportToReflectionFunctionRector`
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
Change export() to ReflectionFunction alternatives
2018-10-22 00:26:45 +02:00
```diff
2019-02-04 01:32:53 +01:00
-$reflectionFunction = ReflectionFunction::export('foo');
-$reflectionFunctionAsString = ReflectionFunction::export('foo', true);
+$reflectionFunction = new ReflectionFunction('foo');
+$reflectionFunctionAsString = (string) new ReflectionFunction('foo');
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ThisCallOnStaticMethodToStaticCallRector`
2019-01-22 21:34:38 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\MethodCall\ThisCallOnStaticMethodToStaticCallRector`
2019-01-22 21:34:38 +01:00
2019-02-04 01:32:53 +01:00
Changes $this->call() to static method to static call
2019-01-22 21:34:38 +01:00
```diff
class SomeClass
{
2019-02-04 01:32:53 +01:00
public static function run()
{
- $this->eat();
+ self::eat();
}
public static function eat()
2019-01-22 21:34:38 +01:00
{
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `PreferThisOrSelfMethodCallRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\MethodCall\PreferThisOrSelfMethodCallRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Changes $this->... to self:: or vise versa for specific types
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
```yaml
services:
Rector\Php\Rector\MethodCall\PreferThisOrSelfMethodCallRector:
PHPUnit\TestCase: self
2018-08-01 22:09:34 +02:00
```
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
↓
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
class SomeClass extends PHPUnit\TestCase
{
public function run()
{
- $this->assertThis();
+ self::assertThis();
}
2018-10-12 20:15:00 -03:00
}
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `UnsetCastRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\Unset_\UnsetCastRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
Removes (unset) cast
2018-07-31 14:50:39 +02:00
```diff
2019-02-04 01:32:53 +01:00
-$value = (unset) $value;
+$value = null;
2018-08-01 22:09:34 +02:00
```
2018-08-01 15:34:55 +02:00
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ReservedObjectRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\Name\ReservedObjectRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
Changes reserved "Object" name to "< Smart > Object" where < Smart > can be configured
2018-12-22 12:22:28 +01:00
```diff
2019-02-04 01:32:53 +01:00
-class Object
+class SmartObject
2018-12-22 12:22:28 +01:00
{
}
```
< br >
2019-05-24 22:30:15 +02:00
### `StringClassNameToClassConstantRector`
- class: `Rector\Php\Rector\String_\StringClassNameToClassConstantRector`
Replace string class names by < class > ::class constant
```diff
class AnotherClass
{
}
class SomeClass
{
public function run()
{
- return 'AnotherClass';
+ return \AnotherClass::class;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `SensitiveHereNowDocRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\String_\SensitiveHereNowDocRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
Changes heredoc/nowdoc that contains closing word to safe wrapper name
2018-11-11 13:22:35 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$value = < < < A
+$value = < < < A_WRAP
A
-A
+A_WRAP
2018-11-11 13:22:35 +01:00
```
< br >
2019-05-19 10:27:38 +02:00
### `ClosureToArrowFunctionRector`
- class: `Rector\Php\Rector\Closure\ClosureToArrowFunctionRector`
Change closure to arrow function
```diff
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));
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `SensitiveConstantNameRector`
2018-12-31 12:50:32 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\ConstFetch\SensitiveConstantNameRector`
2018-12-31 12:50:32 +01:00
2019-02-04 01:32:53 +01:00
Changes case insensitive constants to sensitive ones.
2018-12-31 12:50:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
define('FOO', 42, true);
var_dump(FOO);
-var_dump(foo);
+var_dump(FOO);
```
< br >
### `BarewordStringRector`
- class: `Rector\Php\Rector\ConstFetch\BarewordStringRector`
Changes unquoted non-existing constants to strings
```diff
-var_dump(VAR);
+var_dump("VAR");
```
< br >
2019-03-09 13:08:21 +00:00
### `RenameConstantRector`
2019-02-21 15:36:16 +01:00
2019-03-09 13:08:21 +00:00
- class: `Rector\Php\Rector\ConstFetch\RenameConstantRector`
2019-02-21 15:36:16 +01:00
Replace constant by new ones
```diff
final class SomeClass
{
public function run()
{
- return MYSQL_ASSOC;
+ return MYSQLI_ASSOC;
}
}
```
< br >
2019-05-02 01:56:58 +02:00
### `ReservedFnFunctionRector`
- class: `Rector\Php\Rector\Function_\ReservedFnFunctionRector`
Change fn() function name, since it will be reserved keyword
```diff
class SomeClass
{
public function run()
{
- function fn($value)
+ function f($value)
{
return $value;
}
- fn(5);
+ f(5);
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `MultiExceptionCatchRector`
- class: `Rector\Php\Rector\TryCatch\MultiExceptionCatchRector`
Changes multi catch of same exception to single one | separated.
```diff
try {
// Some code...
-} catch (ExceptionType1 $exception) {
- $sameCode;
-} catch (ExceptionType2 $exception) {
+} catch (ExceptionType1 | ExceptionType2 $exception) {
$sameCode;
}
```
< br >
### `CompleteVarDocTypePropertyRector`
- class: `Rector\Php\Rector\Property\CompleteVarDocTypePropertyRector`
Complete property `@var` annotations for missing one, yet known.
```diff
final class SomeClass
{
+ /**
+ * @var EventDispatcher
+ */
private $eventDispatcher;
public function __construct(EventDispatcher $eventDispatcher)
{
$this->eventDispatcher = $eventDispatcher;
}
}
```
< br >
### `TypedPropertyRector`
- class: `Rector\Php\Rector\Property\TypedPropertyRector`
Changes property `@var` annotations from annotation to type.
```diff
final class SomeClass
{
- /**
- * @var int
- */
- private count;
+ private int count;
}
```
< br >
2019-02-21 15:36:16 +01:00
### `VarToPublicPropertyRector`
- class: `Rector\Php\Rector\Property\VarToPublicPropertyRector`
Remove unused private method
```diff
final class SomeController
{
- var $name = 'Tom';
+ public $name = 'Tom';
}
```
< br >
2019-02-04 01:32:53 +01:00
### `ClassConstantToSelfClassRector`
- class: `Rector\Php\Rector\MagicConstClass\ClassConstantToSelfClassRector`
Change __CLASS__ to self::class
```diff
class SomeClass
{
public function callOnMe()
{
- var_dump(__CLASS__);
+ var_dump(self::class);
}
}
```
< br >
### `RealToFloatTypeCastRector`
- class: `Rector\Php\Rector\Double\RealToFloatTypeCastRector`
Change deprecated (real) to (float)
```diff
class SomeClass
{
public function run()
{
- $number = (real) 5;
+ $number = (float) 5;
$number = (float) 5;
$number = (double) 5;
}
}
```
< br >
### `CreateFunctionToAnonymousFunctionRector`
- class: `Rector\Php\Rector\FuncCall\CreateFunctionToAnonymousFunctionRector`
Use anonymous functions instead of deprecated create_function()
```diff
class ClassWithCreateFunction
{
public function run()
{
- $callable = create_function('$matches', "return '$delimiter' . strtolower(\$matches[1]);");
+ $callable = function($matches) use ($delimiter) {
+ return $delimiter . strtolower($matches[1]);
+ };
}
}
```
< br >
### `RandomFunctionRector`
- class: `Rector\Php\Rector\FuncCall\RandomFunctionRector`
Changes rand, srand and getrandmax by new md_* alternatives.
```diff
-rand();
+mt_rand();
```
< br >
2019-03-09 13:24:30 +00:00
### `PregReplaceEModifierRector`
- class: `Rector\Php\Rector\FuncCall\PregReplaceEModifierRector`
2019-05-02 21:14:01 +02:00
The /e modifier is no longer supported, use preg_replace_callback instead
2019-03-09 13:24:30 +00:00
```diff
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);
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `FilterVarToAddSlashesRector`
- class: `Rector\Php\Rector\FuncCall\FilterVarToAddSlashesRector`
Change filter_var() with slash escaping to addslashes()
```diff
$var= "Satya's here!";
-filter_var($var, FILTER_SANITIZE_MAGIC_QUOTES);
+addslashes($var);
```
2018-12-31 12:50:32 +01:00
< br >
2018-12-22 12:22:28 +01:00
### `GetClassOnNullRector`
- class: `Rector\Php\Rector\FuncCall\GetClassOnNullRector`
Null is no more allowed in get_class()
```diff
final class SomeClass
{
public function getItem()
{
$value = null;
- return get_class($value);
+ return $value !== null ? get_class($value) : self::class;
}
}
```
< br >
2018-12-31 12:50:32 +01:00
### `ArrayKeyExistsOnPropertyRector`
- class: `Rector\Php\Rector\FuncCall\ArrayKeyExistsOnPropertyRector`
Change array_key_exists() on property to property_exists()
```diff
class SomeClass {
public $value;
}
$someClass = new SomeClass;
-array_key_exists('value', $someClass);
+property_exists($someClass, 'value');
```
< br >
2019-05-24 22:30:15 +02:00
### `RemoveMissingCompactVariableRector`
- class: `Rector\Php\Rector\FuncCall\RemoveMissingCompactVariableRector`
Remove non-existing vars from compact()
```diff
class SomeClass
{
public function run()
{
$value = 'yes';
- compact('value', 'non_existing');
+ compact('value');
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `ArrayKeyFirstLastRector`
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Php\Rector\FuncCall\ArrayKeyFirstLastRector`
Make use of array_key_first() and array_key_last()
2018-07-31 14:50:39 +02:00
```diff
2018-11-11 13:22:35 +01:00
-reset($items);
-$firstKey = key($items);
+$firstKey = array_key_first($items);
2018-08-01 22:09:34 +02:00
```
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```diff
2018-11-11 13:22:35 +01:00
-end($items);
-$lastKey = key($items);
+$lastKey = array_key_last($items);
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-07-31 21:43:39 +02:00
2018-11-11 13:22:35 +01:00
### `CallUserMethodRector`
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Php\Rector\FuncCall\CallUserMethodRector`
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
Changes call_user_method()/call_user_method_array() to call_user_func()/call_user_func_array()
```diff
-call_user_method($method, $obj, "arg1", "arg2");
+call_user_func(array(& $obj, "method"), "arg1", "arg2");
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2018-11-11 13:22:35 +01:00
### `StringifyStrNeedlesRector`
2018-10-22 00:26:45 +02:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Php\Rector\FuncCall\StringifyStrNeedlesRector`
2018-10-22 00:26:45 +02:00
2018-11-11 13:22:35 +01:00
Makes needles explicit strings
2018-10-22 00:26:45 +02:00
2018-10-23 15:58:57 -03:00
```diff
2018-11-11 13:22:35 +01:00
$needle = 5;
-$fivePosition = strpos('725', $needle);
+$fivePosition = strpos('725', (string) $needle);
2018-10-23 15:58:57 -03:00
```
2018-10-22 00:26:45 +02:00
2018-11-11 13:22:35 +01:00
< br >
2018-10-22 00:26:45 +02:00
2018-12-22 12:22:28 +01:00
### `StringsAssertNakedRector`
- class: `Rector\Php\Rector\FuncCall\StringsAssertNakedRector`
String asserts must be passed directly to assert()
```diff
function nakedAssert()
{
- assert('true === true');
- assert("true === true");
+ assert(true === true);
+ assert(true === true);
}
```
< br >
### `ParseStrWithResultArgumentRector`
- class: `Rector\Php\Rector\FuncCall\ParseStrWithResultArgumentRector`
Use $result argument in parse_str() function
```diff
-parse_str($this->query);
-$data = get_defined_vars();
+parse_str($this->query, $result);
+$data = $result;
```
< br >
2019-02-21 15:36:16 +01:00
### `RemoveReferenceFromCallRector`
- class: `Rector\Php\Rector\FuncCall\RemoveReferenceFromCallRector`
Remove & from function and method calls
```diff
final class SomeClass
{
public function run($one)
{
- return strlen(&$one);
+ return strlen($one);
}
}
```
< br >
2018-12-22 12:22:28 +01:00
### `IsObjectOnIncompleteClassRector`
- class: `Rector\Php\Rector\FuncCall\IsObjectOnIncompleteClassRector`
Incomplete class returns inverted bool on is_object()
```diff
$incompleteObject = new __PHP_Incomplete_Class;
-$isObject = is_object($incompleteObject);
+$isObject = ! is_object($incompleteObject);
```
< br >
### `RemoveExtraParametersRector`
- class: `Rector\Php\Rector\FuncCall\RemoveExtraParametersRector`
Remove extra parameters
```diff
-strlen("asdf", 1);
+strlen("asdf");
```
< br >
2018-12-31 12:50:32 +01:00
### `GetCalledClassToStaticClassRector`
- class: `Rector\Php\Rector\FuncCall\GetCalledClassToStaticClassRector`
Change __CLASS__ to self::class
```diff
class SomeClass
-{
- public function callOnMe()
- {
- var_dump( get_called_class());
- }
-}
+ {
+ public function callOnMe()
+ {
+ var_dump( static::class);
+ }
+ }
```
< br >
2019-05-19 10:27:38 +02:00
### `ArraySpreadInsteadOfArrayMergeRector`
- class: `Rector\Php\Rector\FuncCall\ArraySpreadInsteadOfArrayMergeRector`
2019-05-24 22:30:15 +02:00
Change array_merge() to spread operator, except values with possible string key values
2019-05-19 10:27:38 +02:00
```diff
class SomeClass
{
public function run($iter1, $iter2)
{
- $values = array_merge(iterator_to_array($iter1), iterator_to_array($iter2));
2019-05-24 22:30:15 +02:00
+ $values = [...$iter1, ...$iter2];
2019-05-19 10:27:38 +02:00
// 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)
- );
2019-05-24 22:30:15 +02:00
+ $anotherValues = [...$iter1, ...$iter2];
2019-05-19 10:27:38 +02:00
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `CountOnNullRector`
2018-10-23 15:58:57 -03:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Php\Rector\FuncCall\CountOnNullRector`
Changes count() on null to safe ternary check
2018-10-22 00:26:45 +02:00
```diff
2018-11-11 13:22:35 +01:00
$values = null;
-$count = count($values);
+$count = is_array($values) || $values instanceof Countable ? count($values) : 0;
2018-08-01 22:09:34 +02:00
```
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
### `SensitiveDefineRector`
2018-07-31 21:43:39 +02:00
2018-10-12 20:15:00 -03:00
- class: `Rector\Php\Rector\FuncCall\SensitiveDefineRector`
2018-07-31 14:50:39 +02:00
2018-10-12 20:15:00 -03:00
Changes case insensitive constants to sensitive ones.
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
```diff
2018-10-12 20:15:00 -03:00
-define('FOO', 42, true);
+define('FOO', 42);
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-12-22 12:22:28 +01:00
### `RegexDashEscapeRector`
- class: `Rector\Php\Rector\FuncCall\RegexDashEscapeRector`
Escape - in some cases
```diff
-preg_match("#[\w-()]#", 'some text');
+preg_match("#[\w\-()]#", 'some text');
```
< br >
2018-10-12 20:15:00 -03:00
### `MultiDirnameRector`
2018-07-31 21:43:39 +02:00
2018-10-12 20:15:00 -03:00
- class: `Rector\Php\Rector\FuncCall\MultiDirnameRector`
2018-07-31 14:50:39 +02:00
2018-10-12 20:15:00 -03:00
Changes multiple dirname() calls to one with nesting level
2018-07-31 14:50:39 +02:00
```diff
2018-10-12 20:15:00 -03:00
-dirname(dirname($path));
+dirname($path, 2);
2018-08-01 22:09:34 +02:00
```
2018-08-01 15:34:55 +02:00
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
### `JsonThrowOnErrorRector`
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Php\Rector\FuncCall\JsonThrowOnErrorRector`
Adds JSON_THROW_ON_ERROR to json_encode() and json_decode() to throw JsonException on error
2018-10-12 20:15:00 -03:00
2018-08-01 22:09:34 +02:00
```diff
2018-11-11 13:22:35 +01:00
-json_encode($content);
-json_decode($json);
2019-03-31 14:25:39 +02:00
+json_encode($content, JSON_THROW_ON_ERROR);
+json_decode($json, null, null, JSON_THROW_ON_ERROR);
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-07-31 21:43:39 +02:00
2019-03-09 13:24:30 +00:00
### `StringifyDefineRector`
- class: `Rector\Php\Rector\FuncCall\StringifyDefineRector`
Make first argument of define() string
```diff
class SomeClass
{
public function run(int $a)
{
- define(CONSTANT_2, 'value');
+ define('CONSTANT_2', 'value');
define('CONSTANT', 'value');
}
}
```
< br >
2019-02-21 15:36:16 +01:00
### `SwapFuncCallArgumentsRector`
- class: `Rector\Php\Rector\FuncCall\SwapFuncCallArgumentsRector`
Swap arguments in function calls
```diff
final class SomeClass
{
public function run($one, $two)
{
- return some_function($one, $two);
+ return some_function($two, $one);
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `EregToPregMatchRector`
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Php\Rector\FuncCall\EregToPregMatchRector`
Changes ereg*() to preg*() calls
2018-07-31 14:50:39 +02:00
```diff
2018-11-11 13:22:35 +01:00
-ereg("hi")
+preg_match("#hi #");
2018-08-01 22:09:34 +02:00
```
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
2018-12-31 12:50:32 +01:00
### `MbStrrposEncodingArgumentPositionRector`
- class: `Rector\Php\Rector\FuncCall\MbStrrposEncodingArgumentPositionRector`
Change mb_strrpos() encoding argument position
```diff
-mb_strrpos($text, "abc", "UTF-8");
+mb_strrpos($text, "abc", 0, "UTF-8");
```
< br >
2018-11-11 13:22:35 +01:00
### `PowToExpRector`
2018-10-12 20:15:00 -03:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Php\Rector\FuncCall\PowToExpRector`
Changes pow(val, val2) to ** (exp) parameter
2018-10-12 20:15:00 -03:00
2018-08-01 22:09:34 +02:00
```diff
2018-11-11 13:22:35 +01:00
-pow(1, 2);
+1**2;
2018-05-05 13:48:33 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-02 01:56:58 +02:00
### `BreakNotInLoopOrSwitchToReturnRector`
- class: `Rector\Php\Rector\Break_\BreakNotInLoopOrSwitchToReturnRector`
Convert break outside for/foreach/switch context to return
```diff
class SomeClass
{
public function run()
{
$zhrs = abs($gmt)/3600;
$hrs = floor($zhrs);
if ($isphp5)
return sprintf('%s%02d%02d',($gmt< =0)?'+':'-',floor($zhrs),($zhrs-$hrs)*60);
else
return sprintf('%s%02d%02d',($gmt< 0 ) ? ' + ' : ' - ' , floor ( $ zhrs ) , ( $ zhrs- $ hrs ) * 60 ) ;
- break;
+ return;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `IfToSpaceshipRector`
- class: `Rector\Php\Rector\If_\IfToSpaceshipRector`
Changes if/else to spaceship < => where useful
```diff
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];
});
}
}
```
< br >
2019-05-24 22:30:15 +02:00
### `BinaryOpBetweenNumberAndStringRector`
- class: `Rector\Php\Rector\BinaryOp\BinaryOpBetweenNumberAndStringRector`
Change binary operation between some number + string to PHP 7.1 compatible version
```diff
class SomeClass
{
public function run()
{
- $value = 5 + '';
- $value = 5.0 + 'hi';
+ $value = 5 + 0;
+ $value = 5.0 + 0
$name = 'Tom';
- $value = 5 * $name;
+ $value = 5 * 0;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `IsCountableRector`
- class: `Rector\Php\Rector\BinaryOp\IsCountableRector`
Changes is_array + Countable check to is_countable
```diff
-is_array($foo) || $foo instanceof Countable;
+is_countable($foo);
```
< br >
### `IsIterableRector`
- class: `Rector\Php\Rector\BinaryOp\IsIterableRector`
Changes is_array + Traversable check to is_iterable
```diff
-is_array($foo) || $foo instanceof Traversable;
+is_iterable($foo);
```
< br >
### `PublicConstantVisibilityRector`
- class: `Rector\Php\Rector\ClassConst\PublicConstantVisibilityRector`
Add explicit public constant visibility.
```diff
class SomeClass
{
- const HEY = 'you';
+ public const HEY = 'you';
}
```
< br >
2018-07-31 21:43:39 +02:00
2018-10-12 20:15:00 -03:00
### `ExceptionHandlerTypehintRector`
2018-07-31 21:43:39 +02:00
2018-10-12 20:15:00 -03:00
- class: `Rector\Php\Rector\FunctionLike\ExceptionHandlerTypehintRector`
2018-11-11 13:22:35 +01:00
Changes property `@var` annotations from annotation to type.
2018-07-31 21:43:39 +02:00
```diff
2018-10-12 20:15:00 -03:00
-function handler(Exception $exception) { ... }
+function handler(Throwable $exception) { ... }
set_exception_handler('handler');
2018-07-31 21:43:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
### `Php4ConstructorRector`
2018-08-01 22:09:34 +02:00
2018-10-12 20:15:00 -03:00
- class: `Rector\Php\Rector\FunctionLike\Php4ConstructorRector`
2018-08-01 22:09:34 +02:00
2018-10-12 20:15:00 -03:00
Changes PHP 4 style constructor to __construct.
2018-08-01 22:09:34 +02:00
```diff
2018-10-12 20:15:00 -03:00
class SomeClass
2018-08-01 22:09:34 +02:00
{
2018-10-12 20:15:00 -03:00
- public function SomeClass()
+ public function __construct()
{
}
2018-08-01 22:09:34 +02:00
}
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-02 01:56:58 +02:00
### `AddDefaultValueForUndefinedVariableRector`
- class: `Rector\Php\Rector\FunctionLike\AddDefaultValueForUndefinedVariableRector`
Adds default value for undefined variable
```diff
class SomeClass
{
public function run()
{
+ $a = null;
if (rand(0, 1)) {
$a = 5;
}
echo $a;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `WhileEachToForeachRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\Each\WhileEachToForeachRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
each() function is deprecated, use foreach() instead.
2018-12-31 20:29:12 +01:00
```diff
2019-02-04 01:32:53 +01:00
-while (list($key, $callback) = each($callbacks)) {
+foreach ($callbacks as $key => $callback) {
// ...
}
```
```diff
-while (list($key) = each($callbacks)) {
+foreach (array_keys($callbacks) as $key) {
// ...
2018-12-31 20:29:12 +01:00
}
```
< br >
2019-02-04 01:32:53 +01:00
### `ListEachRector`
- class: `Rector\Php\Rector\Each\ListEachRector`
2019-04-02 15:35:35 +02:00
each() function is deprecated, use key() and current() instead
2019-02-04 01:32:53 +01:00
```diff
-list($key, $callback) = each($callbacks);
+$key = key($opt->option);
+$val = current($opt->option);
```
< br >
2018-08-01 22:09:34 +02:00
2018-10-12 20:15:00 -03:00
### `ListSplitStringRector`
2018-08-01 22:09:34 +02:00
2018-10-12 20:15:00 -03:00
- class: `Rector\Php\Rector\List_\ListSplitStringRector`
list() cannot split string directly anymore, use str_split()
2018-08-01 22:09:34 +02:00
```diff
2018-10-12 20:15:00 -03:00
-list($foo) = "string";
+list($foo) = str_split("string");
2018-08-01 22:09:34 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
### `EmptyListRector`
- class: `Rector\Php\Rector\List_\EmptyListRector`
list() cannot be empty
2018-08-01 22:09:34 +02:00
```diff
2018-10-12 20:15:00 -03:00
-list() = $values;
+list($generated) = $values;
2018-08-01 22:09:34 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-12 20:15:00 -03:00
### `ListSwapArrayOrderRector`
2018-08-01 22:09:34 +02:00
2018-10-12 20:15:00 -03:00
- class: `Rector\Php\Rector\List_\ListSwapArrayOrderRector`
2018-08-01 22:09:34 +02:00
2018-10-12 20:15:00 -03:00
list() assigns variables in reverse order - relevant in array assign
2018-08-01 22:09:34 +02:00
```diff
2018-10-12 20:15:00 -03:00
-list($a[], $a[]) = [1, 2];
2019-03-31 14:25:39 +02:00
+list($a[], $a[]) = array_reverse([1, 2]);
2018-08-01 22:09:34 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ReduceMultipleDefaultSwitchRector`
2018-12-31 12:50:32 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Php\Rector\Switch_\ReduceMultipleDefaultSwitchRector`
2018-12-31 12:50:32 +01:00
2019-02-04 01:32:53 +01:00
Remove first default switch, that is ignored
2018-12-31 12:50:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
switch ($expr) {
default:
- echo "Hello World";
-
- default:
echo "Goodbye Moon!";
break;
2018-12-31 12:50:32 +01:00
}
```
< br >
2019-02-21 15:36:16 +01:00
### `ContinueToBreakInSwitchRector`
- class: `Rector\Php\Rector\Switch_\ContinueToBreakInSwitchRector`
Use break instead of continue in switch statements
```diff
function some_run($value)
{
switch ($value) {
case 1:
echo 'Hi';
- continue;
+ break;
case 2:
echo 'Hello';
break;
}
}
```
< br >
2019-02-04 01:32:53 +01:00
## PhpParser
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
### `RemoveNodeRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PhpParser\Rector\RemoveNodeRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
Turns integer return to remove node to constant in NodeVisitor of PHP-Parser
2018-12-31 20:29:12 +01:00
```diff
2019-02-04 01:32:53 +01:00
public function leaveNode()
2018-12-31 20:29:12 +01:00
{
2019-02-04 01:32:53 +01:00
- return false;
+ return NodeTraverser::REMOVE_NODE;
2018-12-31 20:29:12 +01:00
}
```
< br >
2019-02-04 01:32:53 +01:00
### `ParamAndStaticVarNameRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PhpParser\Rector\ParamAndStaticVarNameRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
Turns old string `var` to `var->name` sub-variable in Node of PHP-Parser
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
```diff
-$paramNode->name;
+$paramNode->var->name;
2018-12-31 20:29:12 +01:00
```
```diff
2019-02-04 01:32:53 +01:00
-$staticVarNode->name;
+$staticVarNode->var->name;
2018-12-31 20:29:12 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `IdentifierRector`
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PhpParser\Rector\IdentifierRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Turns node string names to Identifier object in php-parser
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
$constNode = new PhpParser\Node\Const_;
-$name = $constNode->name;
+$name = $constNode->name->toString();'
2018-08-01 22:09:34 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `CatchAndClosureUseNameRector`
2018-12-22 15:15:11 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PhpParser\Rector\CatchAndClosureUseNameRector`
2018-12-22 15:15:11 +01:00
2019-02-04 01:32:53 +01:00
Turns `$catchNode->var` to its new `name` property in php-parser
2018-12-22 15:15:11 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$catchNode->var;
+$catchNode->var->name
2018-12-22 15:15:11 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `SetLineRector`
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PhpParser\Rector\SetLineRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Turns standalone line method to attribute in Node of PHP-Parser
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
-$node->setLine(5);
+$node->setAttribute("line", 5);
2018-08-01 22:09:34 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `UseWithAliasRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\PhpParser\Rector\UseWithAliasRector`
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
Turns use property to method and `$node->alias` to last name in UseAlias Node of PHP-Parser
2018-12-31 20:29:12 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$node->alias;
+$node->getAlias();
```
2018-12-31 20:29:12 +01:00
2019-02-04 01:32:53 +01:00
```diff
-$node->name->getLast();
+$node->alias
2018-12-31 20:29:12 +01:00
```
< br >
2019-03-16 21:31:46 +01:00
## PhpSpecToPHPUnit
2019-03-31 14:25:39 +02:00
### `PhpSpecMethodToPHPUnitMethodRector`
- class: `Rector\PhpSpecToPHPUnit\Rector\ClassMethod\PhpSpecMethodToPHPUnitMethodRector`
Migrate PhpSpec behavior to PHPUnit test
```diff
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);
}
}
```
< br >
### `MockVariableToPropertyFetchRector`
- class: `Rector\PhpSpecToPHPUnit\Rector\ClassMethod\MockVariableToPropertyFetchRector`
Migrate PhpSpec behavior to PHPUnit test
```diff
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);
}
}
```
< br >
### `PhpSpecMocksToPHPUnitMocksRector`
- class: `Rector\PhpSpecToPHPUnit\Rector\MethodCall\PhpSpecMocksToPHPUnitMocksRector`
Migrate PhpSpec behavior to PHPUnit test
```diff
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);
}
}
```
< br >
### `PhpSpecPromisesToPHPUnitAssertRector`
- class: `Rector\PhpSpecToPHPUnit\Rector\MethodCall\PhpSpecPromisesToPHPUnitAssertRector`
Migrate PhpSpec behavior to PHPUnit test
```diff
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);
}
}
```
< br >
2019-03-16 21:31:46 +01:00
### `RenameSpecFileToTestFileRector`
- class: `Rector\PhpSpecToPHPUnit\Rector\RenameSpecFileToTestFileRector`
Rename "*Spec.php" file to "*Test.php" file
< br >
2019-03-31 14:25:39 +02:00
### `AddMockPropertiesRector`
2019-03-16 21:31:46 +01:00
2019-03-31 14:25:39 +02:00
- class: `Rector\PhpSpecToPHPUnit\Rector\Class_\AddMockPropertiesRector`
2019-03-16 21:31:46 +01:00
2019-03-31 14:25:39 +02:00
Migrate PhpSpec behavior to PHPUnit test
2019-03-16 21:31:46 +01:00
```diff
2019-03-31 14:25:39 +02:00
namespace spec\SomeNamespaceForThisTest;
2019-03-16 21:31:46 +01:00
2019-03-31 14:25:39 +02:00
-use PhpSpec\ObjectBehavior;
-
class OrderSpec extends ObjectBehavior
2019-03-16 21:31:46 +01:00
{
2019-03-31 14:25:39 +02:00
- public function let(OrderFactory $factory, ShippingMethod $shippingMethod)
+ /**
+ * @var \SomeNamespaceForThisTest\Order
+ */
+ private $order;
2019-03-16 21:31:46 +01:00
+ protected function setUp()
{
2019-03-31 14:25:39 +02:00
- $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);
2019-03-16 21:31:46 +01:00
}
2019-03-31 14:25:39 +02:00
}
```
< br >
### `PhpSpecClassToPHPUnitClassRector`
- class: `Rector\PhpSpecToPHPUnit\Rector\Class_\PhpSpecClassToPHPUnitClassRector`
2019-03-16 21:31:46 +01:00
2019-03-31 14:25:39 +02:00
Migrate PhpSpec behavior to PHPUnit test
```diff
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()
2019-03-16 21:31:46 +01:00
{
2019-03-31 14:25:39 +02:00
- $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);
2019-03-16 21:31:46 +01:00
}
}
```
< br >
2019-05-02 01:56:58 +02:00
## RemovingStatic
### `PHPUnitStaticToKernelTestCaseGetRector`
- class: `Rector\RemovingStatic\Rector\Class_\PHPUnitStaticToKernelTestCaseGetRector`
Convert static calls in PHPUnit test cases, to get() from the container of KernelTestCase
```yaml
services:
Rector\RemovingStatic\Rector\Class_\PHPUnitStaticToKernelTestCaseGetRector:
staticClassTypes:
- EntityFactory
```
↓
```diff
-< ?php
+use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
-use PHPUnit\Framework\TestCase;
+final class SomeTestCase extends KernelTestCase
+{
+ /**
+ * @var EntityFactory
+ */
+ private $entityFactory;
+
+ protected function setUp(): void
+ {
+ parent::setUp();
+ $this->entityFactory = self::$container->get(EntityFactory::class);
+ }
-final class SomeTestCase extends TestCase
-{
public function test()
{
- $product = EntityFactory::create('product');
+ $product = $this->entityFactory->create('product');
}
}
```
< br >
### `StaticTypeToSetterInjectionRector`
- class: `Rector\RemovingStatic\Rector\Class_\StaticTypeToSetterInjectionRector`
Changes types to setter injection
```yaml
services:
Rector\RemovingStatic\Rector\Class_\StaticTypeToSetterInjectionRector:
$staticTypes:
- SomeStaticClass
```
↓
```diff
< ?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();
}
-}
+}
```
< br >
### `NewUniqueObjectToEntityFactoryRector`
- class: `Rector\RemovingStatic\Rector\Class_\NewUniqueObjectToEntityFactoryRector`
Convert new X to new factories
```diff
-< ?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();
}
}
```
< br >
### `PassFactoryToUniqueObjectRector`
- class: `Rector\RemovingStatic\Rector\Class_\PassFactoryToUniqueObjectRector`
Convert new X/Static::call() to factories in entities, pass them via constructor to each other
```yaml
services:
Rector\RemovingStatic\Rector\Class_\PassFactoryToUniqueObjectRector:
typesToServices:
- StaticClass
```
↓
```diff
-< ?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);
}
}
```
< br >
## SOLID
### `PrivatizeLocalClassConstantRector`
- class: `Rector\SOLID\Rector\ClassConst\PrivatizeLocalClassConstantRector`
Finalize every class constant that is used only locally
```diff
class ClassWithConstantUsedOnlyHere
{
- const LOCAL_ONLY = true;
+ private const LOCAL_ONLY = true;
public function isLocalOnly()
{
return self::LOCAL_ONLY;
}
}
```
< br >
2019-05-26 18:38:09 +02:00
### `MakeUnusedClassesWithChildrenAbstractRector`
2019-05-26 13:47:23 +02:00
2019-05-26 18:38:09 +02:00
- class: `Rector\SOLID\Rector\Class_\MakeUnusedClassesWithChildrenAbstractRector`
2019-05-26 13:47:23 +02:00
Classes that have no children nor are used, should have abstract
```diff
class SomeClass extends PossibleAbstractClass
{
}
-class PossibleAbstractClass
+abstract class PossibleAbstractClass
{
}
```
< br >
2019-05-02 01:56:58 +02:00
### `FinalizeClassesWithoutChildrenRector`
- class: `Rector\SOLID\Rector\Class_\FinalizeClassesWithoutChildrenRector`
Finalize every class that has no children
```diff
-class FirstClass
+final class FirstClass
{
}
class SecondClass
{
}
-class ThirdClass extends SecondClass
+final class ThirdClass extends SecondClass
{
}
```
< br >
2019-02-04 01:32:53 +01:00
## Sensio
2018-12-31 12:50:32 +01:00
2019-02-04 01:32:53 +01:00
### `TemplateAnnotationRector`
2018-12-31 12:50:32 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Sensio\Rector\FrameworkExtraBundle\TemplateAnnotationRector`
Turns `@Template` annotation to explicit method call in Controller of FrameworkExtraBundle in Symfony
2018-12-31 12:50:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
-/**
- * @Template ()
- */
public function indexAction()
{
+ return $this->render("index.html.twig");
}
2018-12-31 12:50:32 +01:00
```
< br >
2019-03-16 21:31:46 +01:00
## Shopware
### `ReplaceEnlightResponseWithSymfonyResponseRector`
- class: `Rector\Shopware\Rector\MethodCall\ReplaceEnlightResponseWithSymfonyResponseRector`
Replace Enlight Response methods with Symfony Response methods
```diff
class FrontendController extends \Enlight_Controller_Action
{
public function run()
{
- $this->Response()->setHeader('Foo', 'Yea');
+ $this->Response()->headers->set('Foo', 'Yea');
}
}
```
< br >
2019-02-04 01:32:53 +01:00
## Silverstripe
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
### `ConstantToStaticCallRector`
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Silverstripe\Rector\ConstantToStaticCallRector`
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
Turns defined constant to static method call.
2018-10-22 00:26:45 +02:00
```diff
2019-02-04 01:32:53 +01:00
-SS_DATABASE_NAME;
+Environment::getEnv("SS_DATABASE_NAME");
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `DefineConstantToStaticCallRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Silverstripe\Rector\DefineConstantToStaticCallRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
Turns defined function call to static method call.
2018-12-22 12:22:28 +01:00
```diff
2019-02-04 01:32:53 +01:00
-defined("SS_DATABASE_NAME");
+Environment::getEnv("SS_DATABASE_NAME");
2018-12-22 12:22:28 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
## Sylius
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
### `ReplaceCreateMethodWithoutReviewerRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Sylius\Rector\Review\ReplaceCreateMethodWithoutReviewerRector`
2018-12-22 12:22:28 +01:00
2019-02-04 01:32:53 +01:00
Turns `createForSubjectWithReviewer()` with null review to standalone method in Sylius
2018-12-22 12:22:28 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$this->createForSubjectWithReviewer($subject, null)
+$this->createForSubject($subject)
2018-12-22 12:22:28 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
## Symfony
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
### `GetRequestRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\HttpKernel\GetRequestRector`
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
Turns fetching of dependencies via `$this->get()` to constructor injection in Command and Controller in Symfony
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
+use Symfony\Component\HttpFoundation\Request;
+
class SomeController
{
2019-05-29 12:52:07 +02:00
- public function somethingAction()
+ public function something(Request $request)
2019-02-04 01:32:53 +01:00
{
- $this->getRequest()->...();
+ $request->...();
}
}
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `AddFlashRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Controller\AddFlashRector`
2018-10-12 20:15:00 -03:00
2019-02-04 01:32:53 +01:00
Turns long flash adding to short helper method in Controller in Symfony
2018-10-12 20:15:00 -03:00
```diff
2019-02-04 01:32:53 +01:00
class SomeController extends Controller
{
public function some(Request $request)
{
- $request->getSession()->getFlashBag()->add("success", "something");
+ $this->addFlash("success", "something");
}
2018-10-12 20:15:00 -03:00
}
2018-08-01 22:09:34 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `RedirectToRouteRector`
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Controller\RedirectToRouteRector`
2018-10-22 00:26:45 +02:00
2019-02-04 01:32:53 +01:00
Turns redirect to route to short helper method in Controller in Symfony
2018-10-22 00:26:45 +02:00
```diff
2019-02-04 01:32:53 +01:00
-$this->redirect($this->generateUrl("homepage"));
+$this->redirectToRoute("homepage");
2018-10-22 00:26:45 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ActionSuffixRemoverRector`
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Controller\ActionSuffixRemoverRector`
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
Removes Action suffixes from methods in Symfony Controllers
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
class SomeController
2018-08-01 22:09:34 +02:00
{
2019-02-04 01:32:53 +01:00
- public function indexAction()
+ public function index()
{
}
2018-08-01 22:09:34 +02:00
}
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `CascadeValidationFormBuilderRector`
2018-09-29 00:33:35 +08:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\MethodCall\CascadeValidationFormBuilderRector`
2018-09-29 00:33:35 +08:00
2019-02-04 01:32:53 +01:00
Change "cascade_validation" option to specific node attribute
2018-09-29 00:33:35 +08:00
```diff
2019-02-04 01:32:53 +01:00
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();
}
}
2018-09-29 00:33:35 +08:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-19 10:27:38 +02:00
### `MakeDispatchFirstArgumentEventRector`
- class: `Rector\Symfony\Rector\MethodCall\MakeDispatchFirstArgumentEventRector`
Make event object a first argument of dispatch() method, event name as second
```diff
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
class SomeClass
{
public function run(EventDispatcherInterface $eventDisptacher)
{
- $eventDisptacher->dispatch('event_name', new Event());
+ $eventDisptacher->dispatch(new Event(), 'event_name');
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `ReadOnlyOptionToAttributeRector`
2018-09-29 00:33:35 +08:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\MethodCall\ReadOnlyOptionToAttributeRector`
2018-09-29 00:33:35 +08:00
2019-02-04 01:32:53 +01:00
Change "read_only" option in form to attribute
2018-09-29 00:33:35 +08:00
```diff
2019-02-04 01:32:53 +01:00
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]]);
}
2018-09-29 00:33:35 +08:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `FormTypeInstanceToClassConstRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\MethodCall\FormTypeInstanceToClassConstRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
Changes createForm(new FormType), add(new FormType) to ones with "FormType::class"
2018-07-31 14:50:39 +02:00
```diff
2019-02-04 01:32:53 +01:00
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',
- ]);
+ ));
}
}
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-02 01:56:58 +02:00
### `SimplifyWebTestCaseAssertionsRector`
- class: `Rector\Symfony\Rector\MethodCall\SimplifyWebTestCaseAssertionsRector`
Simplify use of assertions in WebTestCase
```diff
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');
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `ConstraintUrlOptionRector`
- class: `Rector\Symfony\Rector\Validator\ConstraintUrlOptionRector`
Turns true value to `Url::CHECK_DNS_TYPE_ANY` in Validator in Symfony.
```diff
-$constraint = new Url(["checkDNS" => true]);
+$constraint = new Url(["checkDNS" => Url::CHECK_DNS_TYPE_ANY]);
```
< br >
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
### `ConsoleExceptionToErrorEventConstantRector`
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
- class: `Rector\Symfony\Rector\Console\ConsoleExceptionToErrorEventConstantRector`
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
Turns old event name with EXCEPTION to ERROR constant in Console in Symfony
2018-07-31 14:50:39 +02:00
```diff
2018-08-01 22:09:34 +02:00
-"console.exception"
+Symfony\Component\Console\ConsoleEvents::ERROR
```
```diff
-Symfony\Component\Console\ConsoleEvents::EXCEPTION
+Symfony\Component\Console\ConsoleEvents::ERROR
2018-08-01 15:34:55 +02:00
```
2018-07-31 14:50:39 +02:00
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `StringToArrayArgumentProcessRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\New_\StringToArrayArgumentProcessRector`
2018-08-01 15:34:55 +02:00
2019-02-04 01:32:53 +01:00
Changes Process string argument to an array
2018-08-01 15:34:55 +02:00
```diff
2019-02-04 01:32:53 +01:00
use Symfony\Component\Process\Process;
-$process = new Process('ls -l');
+$process = new Process(['ls', '-l']);
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `RootNodeTreeBuilderRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\New_\RootNodeTreeBuilderRector`
2018-07-31 14:50:39 +02:00
2019-02-04 01:32:53 +01:00
Changes Process string argument to an array
2018-07-31 14:50:39 +02:00
2018-08-01 15:34:55 +02:00
```diff
2019-02-04 01:32:53 +01:00
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();
2018-08-01 15:34:55 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ParseFileRector`
2018-08-01 15:34:55 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Yaml\ParseFileRector`
2018-08-01 15:34:55 +02:00
2019-02-04 01:32:53 +01:00
session > use_strict_mode is true by default and can be removed
2018-07-31 14:50:39 +02:00
```diff
2019-02-04 01:32:53 +01:00
-session > use_strict_mode: true
+session:
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 22:09:34 +02:00
### `ContainerBuilderCompileEnvArgumentRector`
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
- class: `Rector\Symfony\Rector\DependencyInjection\ContainerBuilderCompileEnvArgumentRector`
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
Turns old default value to parameter in ContinerBuilder->build() method in DI in Symfony
2018-07-31 14:50:39 +02:00
```diff
2018-08-01 22:09:34 +02:00
-$containerBuilder = new Symfony\Component\DependencyInjection\ContainerBuilder(); $containerBuilder->compile();
+$containerBuilder = new Symfony\Component\DependencyInjection\ContainerBuilder(); $containerBuilder->compile(true);
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `VarDumperTestTraitMethodArgsRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\VarDumper\VarDumperTestTraitMethodArgsRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
Adds new `$format` argument in `VarDumperTestTrait->assertDumpEquals()` in Validator in Symfony.
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
-$varDumperTestTrait->assertDumpEquals($dump, $data, $mesage = "");
+$varDumperTestTrait->assertDumpEquals($dump, $data, $context = null, $mesage = "");
2018-08-01 22:09:34 +02:00
```
2018-07-31 14:50:39 +02:00
```diff
2019-02-04 01:32:53 +01:00
-$varDumperTestTrait->assertDumpMatchesFormat($dump, $format, $mesage = "");
+$varDumperTestTrait->assertDumpMatchesFormat($dump, $format, $context = null, $mesage = "");
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-31 14:25:39 +02:00
### `ResponseStatusCodeRector`
- class: `Rector\Symfony\Rector\BinaryOp\ResponseStatusCodeRector`
Turns status code numbers to constants
```diff
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) {}
}
}
```
< br >
2018-11-11 13:22:35 +01:00
### `GetParameterToConstructorInjectionRector`
2018-05-04 23:30:32 +01:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Symfony\Rector\FrameworkBundle\GetParameterToConstructorInjectionRector`
2018-08-01 15:34:55 +02:00
2018-11-11 13:22:35 +01:00
Turns fetching of parameters via `getParameter()` in ContainerAware to constructor injection in Command and Controller in Symfony
2018-05-04 23:30:32 +01:00
```diff
2018-11-11 13:22:35 +01:00
-class MyCommand extends ContainerAwareCommand
+class MyCommand extends Command
2018-05-04 23:30:32 +01:00
{
2018-11-11 13:22:35 +01:00
+ private $someParameter;
+
+ public function __construct($someParameter)
2018-07-31 14:50:39 +02:00
+ {
2018-11-11 13:22:35 +01:00
+ $this->someParameter = $someParameter;
2018-08-01 15:34:55 +02:00
+ }
+
public function someMethod()
{
2018-11-11 13:22:35 +01:00
- $this->getParameter('someParameter');
+ $this->someParameter;
2018-05-04 23:30:32 +01:00
}
}
```
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 15:34:55 +02:00
### `GetToConstructorInjectionRector`
2018-07-31 21:43:39 +02:00
2018-08-01 15:34:55 +02:00
- class: `Rector\Symfony\Rector\FrameworkBundle\GetToConstructorInjectionRector`
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
Turns fetching of dependencies via `$this->get()` to constructor injection in Command and Controller in Symfony
2018-05-04 23:30:32 +01:00
```diff
2018-08-01 15:34:55 +02:00
-class MyCommand extends ContainerAwareCommand
+class MyCommand extends Command
2018-05-04 23:30:32 +01:00
{
2018-08-01 15:34:55 +02:00
+ public function __construct(SomeService $someService)
+ {
+ $this->someService = $someService;
+ }
+
2018-07-31 14:50:39 +02:00
public function someMethod()
{
2018-08-01 15:34:55 +02:00
- // ...
- $this->get('some_service');
+ $this->someService;
2018-07-31 14:50:39 +02:00
}
2018-05-04 23:30:32 +01:00
}
```
2018-11-11 13:22:35 +01:00
< br >
2018-07-31 21:43:39 +02:00
2018-11-11 13:22:35 +01:00
### `ContainerGetToConstructorInjectionRector`
2018-05-04 23:30:32 +01:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Symfony\Rector\FrameworkBundle\ContainerGetToConstructorInjectionRector`
Turns fetching of dependencies via `$container->get()` in ContainerAware to constructor injection in Command and Controller in Symfony
2018-05-04 23:30:32 +01:00
```diff
2018-11-11 13:22:35 +01:00
-final class SomeCommand extends ContainerAwareCommand
+final class SomeCommand extends Command
2018-05-04 23:30:32 +01:00
{
2018-11-11 13:22:35 +01:00
+ public function __construct(SomeService $someService)
2018-05-04 23:30:32 +01:00
+ {
2018-11-11 13:22:35 +01:00
+ $this->someService = $someService;
2018-05-04 23:30:32 +01:00
+ }
+
2018-08-01 15:34:55 +02:00
public function someMethod()
2018-05-04 23:30:32 +01:00
{
2018-11-11 13:22:35 +01:00
// ...
- $this->getContainer()->get('some_service');
- $this->container->get('some_service');
+ $this->someService;
+ $this->someService;
2018-05-04 23:30:32 +01:00
}
}
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `FormIsValidRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Form\FormIsValidRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
Adds `$form->isSubmitted()` validatoin to all `$form->isValid()` calls in Form in Symfony
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
-if ($form->isValid()) {
+if ($form->isSubmitted() & & $form->isValid()) {
2018-08-01 22:09:34 +02:00
}
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `OptionNameRector`
2018-10-23 15:58:57 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Form\OptionNameRector`
2018-10-23 15:58:57 -03:00
2019-02-04 01:32:53 +01:00
Turns old option names to new ones in FormTypes in Form in Symfony
2018-10-23 15:58:57 -03:00
```diff
2019-02-04 01:32:53 +01:00
$builder = new FormBuilder;
-$builder->add("...", ["precision" => "...", "virtual" => "..."];
+$builder->add("...", ["scale" => "...", "inherit_data" => "..."];
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-23 15:58:57 -03:00
2019-02-04 01:32:53 +01:00
### `StringFormTypeToClassRector`
2018-10-23 15:58:57 -03:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Form\StringFormTypeToClassRector`
2018-10-23 15:58:57 -03:00
2019-02-04 01:32:53 +01:00
Turns string Form Type references to their CONSTANT alternatives in FormTypes in Form in Symfony
2018-10-23 15:58:57 -03:00
```diff
2019-02-04 01:32:53 +01:00
$formBuilder = new Symfony\Component\Form\FormBuilder;
-$formBuilder->add('name', 'form.type.text');
+$form->add('name', \Symfony\Component\Form\Extension\Core\Type\TextType::class);
2018-11-11 13:22:35 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `FormTypeGetParentRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Symfony\Rector\Form\FormTypeGetParentRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
Turns string Form Type references to their CONSTANT alternatives in `getParent()` and `getExtendedType()` methods in Form in Symfony
2018-11-11 13:22:35 +01:00
```diff
2019-02-04 01:32:53 +01:00
-function getParent() { return "collection"; }
+function getParent() { return CollectionType::class; }
2018-11-11 13:22:35 +01:00
```
```diff
2019-02-04 01:32:53 +01:00
-function getExtendedType() { return "collection"; }
+function getExtendedType() { return CollectionType::class; }
2018-10-23 15:58:57 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 15:34:55 +02:00
### `ProcessBuilderGetProcessRector`
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
- class: `Rector\Symfony\Rector\Process\ProcessBuilderGetProcessRector`
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
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.
2018-07-31 14:50:39 +02:00
2018-08-01 15:34:55 +02:00
```diff
$processBuilder = new Symfony\Component\Process\ProcessBuilder;
-$process = $processBuilder->getProcess();
-$commamdLine = $processBuilder->getProcess()->getCommandLine();
+$process = $processBuilder;
+$commamdLine = $processBuilder->getCommandLine();
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
### `ProcessBuilderInstanceRector`
- class: `Rector\Symfony\Rector\Process\ProcessBuilderInstanceRector`
Turns `ProcessBuilder::instance()` to new ProcessBuilder in Process in Symfony. Part of multi-step Rector.
```diff
-$processBuilder = Symfony\Component\Process\ProcessBuilder::instance($args);
+$processBuilder = new Symfony\Component\Process\ProcessBuilder($args);
```
< br >
2018-09-29 00:33:35 +08:00
## Twig
### `SimpleFunctionAndFilterRector`
- class: `Rector\Twig\Rector\SimpleFunctionAndFilterRector`
Changes Twig_Function_Method to Twig_SimpleFunction calls in TwigExtension.
```diff
class SomeExtension extends Twig_Extension
{
public function getFunctions()
{
return [
- 'is_mobile' => new Twig_Function_Method($this, 'isMobile'),
+ new Twig_SimpleFunction('is_mobile', [$this, 'isMobile']),
];
}
2019-03-31 14:25:39 +02:00
public function getFilters()
2018-09-29 00:33:35 +08:00
{
return [
- 'is_mobile' => new Twig_Filter_Method($this, 'isMobile'),
+ new Twig_SimpleFilter('is_mobile', [$this, 'isMobile']),
];
}
}
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-19 10:27:38 +02:00
## TypeDeclaration
### `AddClosureReturnTypeRector`
- class: `Rector\TypeDeclaration\Rector\Closure\AddClosureReturnTypeRector`
Add known return type to functions
```diff
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);
});
}
}
```
< br >
### `ReturnTypeDeclarationRector`
- class: `Rector\TypeDeclaration\Rector\FunctionLike\ReturnTypeDeclarationRector`
Change @return types and type from static analysis to type declarations if not a BC-break
```diff
< ?php
class SomeClass
{
/**
* @return int
*/
- public function getCount()
+ public function getCount(): int
{
}
}
```
< br >
### `ParamTypeDeclarationRector`
- class: `Rector\TypeDeclaration\Rector\FunctionLike\ParamTypeDeclarationRector`
Change @param types to type declarations if not a BC-break
```diff
< ?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)
{
}
}
```
< br >
2018-08-01 22:09:34 +02:00
---
## General
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- [Core ](#core )
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
## Core
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
### `PropertyAssignToMethodCallRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\Assign\PropertyAssignToMethodCallRector`
Turns property assign of specific type and property name to method call
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Assign\PropertyAssignToMethodCallRector:
$oldPropertiesToNewMethodCallsByType:
SomeClass:
oldPropertyName: oldProperty
newMethodName: newMethodCall
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
-$someObject = new SomeClass;
-$someObject->oldProperty = false;
+$someObject = new SomeClass;
+$someObject->newMethodCall(false);
```
< br >
### `MultipleClassFileToPsr4ClassesRector`
- class: `Rector\Rector\Psr4\MultipleClassFileToPsr4ClassesRector`
Turns namespaced classes in one file to standalone PSR-4 classes.
```diff
+// new file: "app/Exceptions/FirstException.php"
namespace App\Exceptions;
use Exception;
final class FirstException extends Exception
2018-08-01 22:09:34 +02:00
{
2019-02-04 01:32:53 +01:00
}
+
+// new file: "app/Exceptions/SecondException.php"
+namespace App\Exceptions;
+
+use Exception;
final class SecondException extends Exception
{
2018-08-01 22:09:34 +02:00
}
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `StaticCallToFunctionRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\StaticCall\StaticCallToFunctionRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
Turns static call to function call.
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\StaticCall\StaticCallToFunctionRector:
$staticCallToFunction:
OldClass:
oldMethod: new_function
2018-05-04 23:30:32 +01:00
```
2018-08-01 22:09:34 +02:00
↓
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
-OldClass::oldMethod("args");
+new_function("args");
2018-08-01 22:09:34 +02:00
```
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
< br >
### `WrapReturnRector`
- class: `Rector\Rector\ClassMethod\WrapReturnRector`
Wrap return value of specific method
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\ClassMethod\WrapReturnRector:
SomeClass:
getItem: array
2018-08-01 22:09:34 +02:00
```
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
final class SomeClass
2018-07-31 14:50:39 +02:00
{
2019-02-04 01:32:53 +01:00
public function getItem()
2018-08-01 22:09:34 +02:00
{
2019-02-04 01:32:53 +01:00
- return 1;
+ return [1];
2018-08-01 22:09:34 +02:00
}
2018-07-31 14:50:39 +02:00
}
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 00:55:22 +01:00
### `AddReturnTypeDeclarationRector`
- class: `Rector\Rector\ClassMethod\AddReturnTypeDeclarationRector`
Changes defined return typehint of method and class.
```yaml
services:
Rector\Rector\ClassMethod\AddReturnTypeDeclarationRector:
SomeClass:
getData: array
```
↓
```diff
class SomeClass
{
- public getData();
+ public getData(): array;
}
```
< br >
2019-02-04 01:32:53 +01:00
### `MethodCallToAnotherMethodCallWithArgumentsRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\MethodCall\MethodCallToAnotherMethodCallWithArgumentsRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
Turns old method call with specfici type to new one with arguments
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\MethodCall\MethodCallToAnotherMethodCallWithArgumentsRector:
Nette\DI\ServiceDefinition:
setInject:
2018-10-22 00:26:45 +02:00
-
2019-02-04 01:32:53 +01:00
- addTag
-
- inject
2018-08-01 22:09:34 +02:00
```
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
↓
2018-07-31 21:43:39 +02:00
```diff
2019-02-04 01:32:53 +01:00
$serviceDefinition = new Nette\DI\ServiceDefinition;
-$serviceDefinition->setInject();
+$serviceDefinition->addTag('inject');
2018-07-31 21:43:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-09 12:55:17 +00:00
### `RenameMethodRector`
2018-07-31 21:43:39 +02:00
2019-03-09 12:55:17 +00:00
- class: `Rector\Rector\MethodCall\RenameMethodRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
Turns method names to new ones.
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-03-09 12:55:17 +00:00
Rector\Rector\MethodCall\RenameMethodRector:
2018-11-11 13:22:35 +01:00
SomeExampleClass:
2019-02-04 01:32:53 +01:00
oldMethod: newMethod
2018-05-04 23:30:32 +01:00
```
2018-08-01 22:09:34 +02:00
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
$someObject = new SomeExampleClass;
-$someObject->oldMethod();
+$someObject->newMethod();
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-09 13:24:30 +00:00
### `RenameStaticMethodRector`
2018-12-14 20:35:35 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\Rector\MethodCall\RenameStaticMethodRector`
2018-12-14 20:35:35 +01:00
2019-02-04 01:32:53 +01:00
Turns method names to new ones.
2018-12-14 20:38:47 +01:00
```yaml
services:
2019-03-09 13:24:30 +00:00
Rector\Rector\MethodCall\RenameStaticMethodRector:
2018-12-14 20:38:47 +01:00
SomeClass:
2019-02-04 01:32:53 +01:00
oldMethod:
- AnotherExampleClass
- newStaticMethod
2018-07-31 14:50:39 +02:00
```
2018-08-01 22:09:34 +02:00
↓
2018-07-31 14:50:39 +02:00
```diff
2019-02-04 01:32:53 +01:00
-SomeClass::oldStaticMethod();
+AnotherExampleClass::newStaticMethod();
2018-05-04 23:30:32 +01:00
```
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-03-09 13:24:30 +00:00
Rector\Rector\MethodCall\RenameStaticMethodRector:
2019-02-04 01:32:53 +01:00
$oldToNewMethodByClasses:
SomeClass:
oldMethod: newStaticMethod
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
-SomeClass::oldStaticMethod();
+SomeClass::newStaticMethod();
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-16 21:31:46 +01:00
### `RenameMethodCallRector`
- class: `Rector\Rector\MethodCall\RenameMethodCallRector`
Turns method call names to new ones.
```yaml
services:
Rector\Rector\MethodCall\RenameMethodCallRector:
SomeExampleClass:
oldMethod: newMethod
```
↓
```diff
$someObject = new SomeExampleClass;
-$someObject->oldMethod();
+$someObject->newMethod();
```
< br >
2019-02-04 01:32:53 +01:00
### `FluentReplaceRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\MethodBody\FluentReplaceRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
Turns fluent interface calls to classic ones.
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\MethodBody\FluentReplaceRector:
-
- SomeExampleClass
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
$someClass = new SomeClass();
-$someClass->someFunction()
- ->otherFunction();
+$someClass->someFunction();
+$someClass->otherFunction();
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `NormalToFluentRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\MethodBody\NormalToFluentRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
Turns fluent interface calls to classic ones.
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\MethodBody\NormalToFluentRector:
2018-11-11 13:22:35 +01:00
SomeClass:
2019-02-04 01:32:53 +01:00
- someFunction
- otherFunction
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
$someObject = new SomeClass();
-$someObject->someFunction();
-$someObject->otherFunction();
+$someObject->someFunction()
+ ->otherFunction();
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ReturnThisRemoveRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\MethodBody\ReturnThisRemoveRector`
2018-11-11 13:22:35 +01:00
2019-02-04 01:32:53 +01:00
Removes "return $this;" from *fluent interfaces* for specified classes.
2018-11-11 13:22:35 +01:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\MethodBody\ReturnThisRemoveRector:
-
- SomeExampleClass
2018-11-11 13:22:35 +01:00
```
↓
```diff
2019-02-04 01:32:53 +01:00
class SomeClass
2018-10-23 15:58:57 -03:00
{
2019-02-04 01:32:53 +01:00
public function someFunction()
2018-10-23 15:58:57 -03:00
{
2019-02-04 01:32:53 +01:00
- return $this;
2018-10-23 15:58:57 -03:00
}
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
public function otherFunction()
2018-10-22 00:26:45 +02:00
{
2019-02-04 01:32:53 +01:00
- return $this;
2018-10-22 00:26:45 +02:00
}
}
2018-10-12 20:15:00 -03:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 22:09:34 +02:00
### `ToStringToMethodCallRector`
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
- class: `Rector\Rector\MagicDisclosure\ToStringToMethodCallRector`
2018-07-31 14:50:39 +02:00
2018-08-10 21:07:08 +02:00
Turns defined code uses of "__toString()" method to specific method calls.
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\MagicDisclosure\ToStringToMethodCallRector:
2018-11-11 13:22:35 +01:00
SomeObject: getPath
2018-08-01 22:09:34 +02:00
```
↓
2018-07-31 14:50:39 +02:00
```diff
2018-08-10 21:07:08 +02:00
$someValue = new SomeObject;
2018-08-01 22:09:34 +02:00
-$result = (string) $someValue;
-$result = $someValue->__toString();
2018-11-11 13:22:35 +01:00
+$result = $someValue->getPath();
+$result = $someValue->getPath();
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 22:09:34 +02:00
### `GetAndSetToMethodCallRector`
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
- class: `Rector\Rector\MagicDisclosure\GetAndSetToMethodCallRector`
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
Turns defined `__get` /`__set` to specific method calls.
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\MagicDisclosure\GetAndSetToMethodCallRector:
2018-11-11 13:22:35 +01:00
SomeContainer:
set: addService
2018-08-01 22:09:34 +02:00
```
↓
2018-07-31 14:50:39 +02:00
```diff
2018-08-10 21:07:08 +02:00
$container = new SomeContainer;
2018-08-01 22:09:34 +02:00
-$container->someService = $someService;
+$container->setService("someService", $someService);
```
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\MagicDisclosure\GetAndSetToMethodCallRector:
$typeToMethodCalls:
SomeContainer:
get: getService
2018-07-31 14:50:39 +02:00
```
2018-08-01 22:09:34 +02:00
↓
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
```diff
2018-08-10 21:07:08 +02:00
$container = new SomeContainer;
2018-08-01 22:09:34 +02:00
-$someService = $container->someService;
+$someService = $container->getService("someService");
```
2018-05-04 23:30:32 +01:00
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 22:09:34 +02:00
### `UnsetAndIssetToMethodCallRector`
- class: `Rector\Rector\MagicDisclosure\UnsetAndIssetToMethodCallRector`
Turns defined `__isset` /`__unset` calls to specific method calls.
```yaml
services:
Rector\Rector\MagicDisclosure\UnsetAndIssetToMethodCallRector:
2018-11-11 13:22:35 +01:00
SomeContainer:
isset: hasService
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2018-11-11 13:22:35 +01:00
$container = new SomeContainer;
2018-08-01 22:09:34 +02:00
-isset($container["someKey"]);
+$container->hasService("someKey");
```
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\MagicDisclosure\UnsetAndIssetToMethodCallRector:
2018-11-11 13:22:35 +01:00
SomeContainer:
unset: removeService
2018-08-01 22:09:34 +02:00
```
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
↓
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```diff
2018-11-11 13:22:35 +01:00
$container = new SomeContainer;
-unset($container["someKey"]);
2018-08-01 22:09:34 +02:00
+$container->removeService("someKey");
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `ParentTypehintedArgumentRector`
2018-09-29 00:33:35 +08:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\Typehint\ParentTypehintedArgumentRector`
2018-09-29 00:33:35 +08:00
2019-02-04 01:32:53 +01:00
Changes defined parent class typehints.
2018-09-29 00:33:35 +08:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Typehint\ParentTypehintedArgumentRector:
SomeInterface:
read:
$content: string
2018-09-29 00:33:35 +08:00
```
↓
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
interface SomeInterface
2018-08-01 22:09:34 +02:00
{
2019-02-04 01:32:53 +01:00
public read(string $content);
}
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
class SomeClass implements SomeInterface
{
- public read($content);
+ public read(string $content);
2018-08-01 22:09:34 +02:00
}
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-23 15:58:57 -03:00
2019-02-18 16:51:24 +01:00
### `StringToClassConstantRector`
- class: `Rector\Rector\String_\StringToClassConstantRector`
Changes strings to specific constants
```yaml
services:
Rector\Rector\String_\StringToClassConstantRector:
compiler.post_dump:
- Yet\AnotherClass
- CONSTANT
```
↓
```diff
final class SomeSubscriber
{
public static function getSubscribedEvents()
{
- return ['compiler.post_dump' => 'compile'];
+ return [\Yet\AnotherClass::CONSTANT => 'compile'];
}
}
```
< br >
2019-02-04 01:32:53 +01:00
### `ArgumentAdderRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\Argument\ArgumentAdderRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
This Rector adds new default arguments in calls of defined methods and class types.
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Argument\ArgumentAdderRector:
SomeExampleClass:
someMethod:
2018-10-22 00:26:45 +02:00
-
2019-03-09 13:24:30 +00:00
name: someArgument
2019-02-04 01:32:53 +01:00
default_value: 'true'
2019-03-09 13:24:30 +00:00
type: SomeType
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
$someObject = new SomeExampleClass;
-$someObject->someMethod();
+$someObject->someMethod(true);
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Argument\ArgumentAdderRector:
2018-11-11 13:22:35 +01:00
SomeExampleClass:
2019-02-04 01:32:53 +01:00
someMethod:
-
2019-03-09 13:24:30 +00:00
name: someArgument
2019-02-04 01:32:53 +01:00
default_value: 'true'
2019-03-09 13:24:30 +00:00
type: SomeType
2018-11-11 13:22:35 +01:00
```
↓
```diff
2019-02-04 01:32:53 +01:00
class MyCustomClass extends SomeExampleClass
{
- public function someMethod()
+ public function someMethod($value = true)
{
}
}
2018-11-11 13:22:35 +01:00
```
< br >
2019-02-04 01:32:53 +01:00
### `ArgumentRemoverRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\Argument\ArgumentRemoverRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
Removes defined arguments in defined methods and their calls.
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Argument\ArgumentRemoverRector:
ExampleClass:
someMethod:
-
value: 'true'
2018-08-01 22:09:34 +02:00
```
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
↓
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
```diff
2019-02-04 01:32:53 +01:00
$someObject = new SomeClass;
-$someObject->someMethod(true);
+$someObject->someMethod();'
2018-05-04 23:30:32 +01:00
```
2019-02-04 01:32:53 +01:00
< br >
### `ArgumentDefaultValueReplacerRector`
- class: `Rector\Rector\Argument\ArgumentDefaultValueReplacerRector`
Replaces defined map of arguments in defined methods and their calls.
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Argument\ArgumentDefaultValueReplacerRector:
SomeExampleClass:
someMethod:
-
-
before: 'SomeClass::OLD_CONSTANT'
after: 'false'
2018-08-01 22:09:34 +02:00
```
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
$someObject = new SomeClass;
-$someObject->someMethod(SomeClass::OLD_CONSTANT);
+$someObject->someMethod(false);'
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-09 13:24:30 +00:00
### `NewToStaticCallRector`
2018-08-01 22:09:34 +02:00
2019-03-09 13:24:30 +00:00
- class: `Rector\Rector\New_\NewToStaticCallRector`
2018-07-31 21:43:39 +02:00
2019-03-09 13:24:30 +00:00
Change new Object to static call
2018-05-05 01:04:41 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-03-09 13:24:30 +00:00
Rector\Rector\New_\NewToStaticCallRector:
Cookie:
-
- Cookie
- create
2018-08-01 22:09:34 +02:00
```
2018-05-05 01:04:41 +01:00
2018-08-01 22:09:34 +02:00
↓
2018-08-01 15:34:55 +02:00
2018-08-01 22:09:34 +02:00
```diff
2019-03-09 13:24:30 +00:00
class SomeClass
{
public function run()
{
- new Cookie($name);
+ Cookie::create($name);
}
}
2018-05-05 01:04:41 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 22:09:34 +02:00
### `PseudoNamespaceToNamespaceRector`
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
- class: `Rector\Rector\Namespace_\PseudoNamespaceToNamespaceRector`
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
Replaces defined Pseudo_Namespaces by Namespace\Ones.
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\Namespace_\PseudoNamespaceToNamespaceRector:
2018-12-06 01:13:34 +01:00
-
Some_: { }
```
↓
```diff
-$someService = new Some_Object;
+$someService = new Some\Object;
```
```yaml
services:
Rector\Rector\Namespace_\PseudoNamespaceToNamespaceRector:
-
Some_:
- Some_Class_To_Keep
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-03-16 21:31:46 +01:00
-/** @var Some_Object $someService */
2018-12-06 01:13:34 +01:00
-$someService = new Some_Object;
2019-03-16 21:31:46 +01:00
+/** @var Some\Object $someService */
2018-12-06 01:13:34 +01:00
+$someService = new Some\Object;
$someClassToKeep = new Some_Class_To_Keep;
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-09 13:24:30 +00:00
### `RenameNamespaceRector`
- class: `Rector\Rector\Namespace_\RenameNamespaceRector`
Replaces old namespace by new one.
```yaml
services:
Rector\Rector\Namespace_\RenameNamespaceRector:
$oldToNewNamespaces:
SomeOldNamespace: SomeNewNamespace
```
↓
```diff
-$someObject = new SomeOldNamespace\SomeClass;
+$someObject = new SomeNewNamespace\SomeClass;
```
< br >
2019-02-04 01:32:53 +01:00
### `FunctionToMethodCallRector`
- class: `Rector\Rector\Function_\FunctionToMethodCallRector`
Turns defined function calls to local method calls.
```yaml
services:
Rector\Rector\Function_\FunctionToMethodCallRector:
view:
- this
- render
```
↓
```diff
-view("...", []);
+$this->render("...", []);
```
< br >
### `FunctionToStaticCallRector`
- class: `Rector\Rector\Function_\FunctionToStaticCallRector`
Turns defined function call to static method call.
```yaml
services:
Rector\Rector\Function_\FunctionToStaticCallRector:
view:
- SomeStaticClass
- render
```
↓
```diff
-view("...", []);
+SomeClass::render("...", []);
```
< br >
2019-03-09 13:24:30 +00:00
### `RenameFunctionRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\Rector\Function_\RenameFunctionRector`
2019-02-04 01:32:53 +01:00
Turns defined function call new one.
```yaml
services:
2019-03-09 13:24:30 +00:00
Rector\Rector\Function_\RenameFunctionRector:
2019-02-04 01:32:53 +01:00
view: Laravel\Templating\render
```
↓
```diff
-view("...", []);
+Laravel\Templating\render("...", []);
```
< br >
2018-07-31 21:43:39 +02:00
2018-08-01 22:09:34 +02:00
### `PropertyToMethodRector`
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
- class: `Rector\Rector\Property\PropertyToMethodRector`
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
Replaces properties assign calls be defined methods.
2018-07-31 14:50:39 +02:00
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\Property\PropertyToMethodRector:
$perClassPropertyToMethods:
SomeObject:
property:
2018-09-29 00:33:35 +08:00
get: getProperty
set: setProperty
2018-08-01 22:09:34 +02:00
```
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
↓
```diff
-$result = $object->property;
-$object->property = $value;
+$result = $object->getProperty();
+$object->setProperty($value);
2018-08-01 15:34:55 +02:00
```
2018-07-31 21:43:39 +02:00
2018-09-29 00:33:35 +08:00
```yaml
services:
Rector\Rector\Property\PropertyToMethodRector:
$perClassPropertyToMethods:
SomeObject:
property:
get:
method: getConfig
arguments:
- someArg
```
↓
```diff
-$result = $object->property;
+$result = $object->getProperty('someArg');
```
2018-11-11 13:22:35 +01:00
< br >
2019-05-19 10:27:38 +02:00
### `InjectAnnotationClassRector`
- class: `Rector\Rector\Property\InjectAnnotationClassRector`
Changes properties with specified annotations class to constructor injection
```yaml
services:
Rector\Rector\Property\InjectAnnotationClassRector:
$annotationClass: JMS\DiExtraBundle\Annotation\Inject
```
↓
```diff
use JMS\DiExtraBundle\Annotation as DI;
class SomeController
{
/**
- * @DI \Inject("entity.manager")
+ * @var EntityManager
*/
private $entityManager;
+
+ public function __construct(EntityManager $entityManager)
+ {
+ $this->entityManager = entityManager;
+ }
}
```
< br >
2019-03-09 13:04:33 +00:00
### `RenamePropertyRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:04:33 +00:00
- class: `Rector\Rector\Property\RenamePropertyRector`
2019-02-04 01:32:53 +01:00
Replaces defined old properties by new ones.
```yaml
services:
2019-03-09 13:04:33 +00:00
Rector\Rector\Property\RenamePropertyRector:
2019-02-04 01:32:53 +01:00
$oldToNewPropertyByTypes:
SomeClass:
someOldProperty: someNewProperty
```
↓
```diff
-$someObject->someOldProperty;
+$someObject->someNewProperty;
```
< br >
### `AnnotatedPropertyInjectToConstructorInjectionRector`
- class: `Rector\Rector\Architecture\DependencyInjection\AnnotatedPropertyInjectToConstructorInjectionRector`
Turns non-private properties with `@annotation` to private properties and constructor injection
```yaml
services:
Rector\Rector\Architecture\DependencyInjection\AnnotatedPropertyInjectToConstructorInjectionRector:
$annotation: inject
```
↓
```diff
/**
* @var SomeService
- * @inject
*/
-public $someService;
+private $someService;
+
+public function __construct(SomeService $someService)
+{
+ $this->someService = $someService;
+}
```
< br >
### `ReplaceVariableByPropertyFetchRector`
- class: `Rector\Rector\Architecture\DependencyInjection\ReplaceVariableByPropertyFetchRector`
Turns variable in controller action to property fetch, as follow up to action injection variable to property change.
```diff
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();
}
}
```
< br >
### `ActionInjectionToConstructorInjectionRector`
- class: `Rector\Rector\Architecture\DependencyInjection\ActionInjectionToConstructorInjectionRector`
Turns action injection in Controllers to constructor injection
```diff
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();
}
}
```
< br >
### `NewObjectToFactoryCreateRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\Architecture\Factory\NewObjectToFactoryCreateRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
Replaces creating object instances with "new" keyword with factory method.
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Architecture\Factory\NewObjectToFactoryCreateRector:
MyClass:
class: MyClassFactory
method: create
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
class SomeClass
2018-09-29 00:33:35 +08:00
{
2019-02-04 01:32:53 +01:00
+ /**
+ * @var \MyClassFactory
+ */
+ private $myClassFactory;
2018-09-29 00:33:35 +08:00
+
2019-02-04 01:32:53 +01:00
public function example() {
- new MyClass($argument);
+ $this->myClassFactory->create($argument);
}
2018-09-29 00:33:35 +08:00
}
```
2018-11-11 13:22:35 +01:00
< br >
### `ReplaceParentRepositoryCallsByRepositoryPropertyRector`
- class: `Rector\Rector\Architecture\RepositoryAsService\ReplaceParentRepositoryCallsByRepositoryPropertyRector`
Handles method calls in child of Doctrine EntityRepository and moves them to "$this->repository" property.
```diff
< ?php
use Doctrine\ORM\EntityRepository;
class SomeRepository extends EntityRepository
{
public function someMethod()
{
- return $this->findAll();
+ return $this->repository->findAll();
}
}
```
< br >
2018-08-01 15:34:55 +02:00
### `ServiceLocatorToDIRector`
2018-07-31 21:43:39 +02:00
2018-08-01 15:34:55 +02:00
- class: `Rector\Rector\Architecture\RepositoryAsService\ServiceLocatorToDIRector`
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
Turns "$this->getRepository()" in Symfony Controller to constructor injection and private property access.
2018-05-04 23:30:32 +01:00
```diff
2018-08-01 15:34:55 +02:00
class ProductController extends Controller
2018-07-31 14:50:39 +02:00
{
2018-08-01 15:34:55 +02:00
+ /**
+ * @var ProductRepository
+ */
+ private $productRepository;
+
+ public function __construct(ProductRepository $productRepository)
2018-07-31 14:50:39 +02:00
+ {
2018-08-01 15:34:55 +02:00
+ $this->productRepository = $productRepository;
2018-07-31 14:50:39 +02:00
+ }
+
2018-08-01 15:34:55 +02:00
public function someAction()
2018-07-31 14:50:39 +02:00
{
2018-08-01 15:34:55 +02:00
$entityManager = $this->getDoctrine()->getManager();
- $entityManager->getRepository('SomethingBundle:Product')->findSomething(...);
+ $this->productRepository->findSomething(...);
2018-07-31 14:50:39 +02:00
}
}
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-10-23 15:58:57 -03:00
2018-08-01 15:34:55 +02:00
### `MoveRepositoryFromParentToConstructorRector`
2018-07-31 21:43:39 +02:00
2018-08-01 15:34:55 +02:00
- class: `Rector\Rector\Architecture\RepositoryAsService\MoveRepositoryFromParentToConstructorRector`
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
Turns parent EntityRepository class to constructor dependency
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\Architecture\RepositoryAsService\MoveRepositoryFromParentToConstructorRector:
$entityRepositoryClass: Doctrine\ORM\EntityRepository
$entityManagerClass: Doctrine\ORM\EntityManager
```
↓
2018-05-04 23:30:32 +01:00
```diff
2018-08-01 15:34:55 +02:00
namespace App\Repository;
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
+use App\Entity\Post;
use Doctrine\ORM\EntityRepository;
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
-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);
+ }
}
2018-07-31 14:50:39 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-16 21:31:46 +01:00
### `FunctionToNewRector`
- class: `Rector\Rector\FuncCall\FunctionToNewRector`
Change configured function calls to new Instance
```diff
class SomeClass
{
public function run()
{
- $array = collection([]);
+ $array = new \Collection([]);
}
}
```
< br >
### `RemoveTraitRector`
- class: `Rector\Rector\ClassLike\RemoveTraitRector`
Remove specific traits from code
```diff
class SomeClass
{
- use SomeTrait;
}
```
< br >
2019-02-04 01:32:53 +01:00
### `RemoveInterfacesRector`
2018-07-31 21:43:39 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\Interface_\RemoveInterfacesRector`
2018-05-04 23:30:32 +01:00
2019-02-04 01:32:53 +01:00
Removes interfaces usage from class.
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Interface_\RemoveInterfacesRector:
- SomeInterface
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2019-02-04 01:32:53 +01:00
-class SomeClass implements SomeInterface
+class SomeClass
2018-08-01 22:09:34 +02:00
{
}
```
2018-11-11 13:22:35 +01:00
< br >
2019-02-04 01:32:53 +01:00
### `MergeInterfacesRector`
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
- class: `Rector\Rector\Interface_\MergeInterfacesRector`
2018-08-01 22:09:34 +02:00
2019-02-04 01:32:53 +01:00
Merges old interface to a new one, that already has its methods
2018-08-01 22:09:34 +02:00
```yaml
services:
2019-02-04 01:32:53 +01:00
Rector\Rector\Interface_\MergeInterfacesRector:
SomeOldInterface: SomeInterface
2018-08-01 22:09:34 +02:00
```
↓
```diff
2019-02-04 01:32:53 +01:00
-class SomeClass implements SomeInterface, SomeOldInterface
+class SomeClass implements SomeInterface
2018-08-01 15:34:55 +02:00
{
}
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
### `ChangeMethodVisibilityRector`
2018-08-01 22:09:34 +02:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Rector\Visibility\ChangeMethodVisibilityRector`
2018-08-01 22:09:34 +02:00
2018-11-11 13:22:35 +01:00
Change visibility of method from parent class.
2018-08-01 22:09:34 +02:00
```yaml
services:
2018-11-11 13:22:35 +01:00
Rector\Rector\Visibility\ChangeMethodVisibilityRector:
FrameworkClass:
someMethod: protected
2018-08-01 22:09:34 +02:00
```
↓
```diff
2018-10-22 00:26:45 +02:00
class FrameworkClass
{
2018-11-11 13:22:35 +01:00
protected someMethod()
{
}
2018-10-22 00:26:45 +02:00
}
2018-08-01 22:09:34 +02:00
2018-10-22 00:26:45 +02:00
class MyClass extends FrameworkClass
{
2018-11-11 13:22:35 +01:00
- public someMethod()
+ protected someMethod()
{
}
2018-10-22 00:26:45 +02:00
}
2018-08-01 22:09:34 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-07-31 21:43:39 +02:00
2018-11-11 13:22:35 +01:00
### `ChangePropertyVisibilityRector`
2018-05-04 23:30:32 +01:00
2018-11-11 13:22:35 +01:00
- class: `Rector\Rector\Visibility\ChangePropertyVisibilityRector`
Change visibility of property from parent class.
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
2018-11-11 13:22:35 +01:00
Rector\Rector\Visibility\ChangePropertyVisibilityRector:
FrameworkClass:
someProperty: protected
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2018-08-01 15:34:55 +02:00
class FrameworkClass
{
2018-11-11 13:22:35 +01:00
protected $someProperty;
2018-08-01 15:34:55 +02:00
}
2018-07-31 08:38:48 +02:00
2018-08-01 15:34:55 +02:00
class MyClass extends FrameworkClass
{
2018-11-11 13:22:35 +01:00
- public $someProperty;
+ protected $someProperty;
2018-08-01 15:34:55 +02:00
}
2018-07-31 08:38:48 +02:00
```
2018-11-11 13:22:35 +01:00
< br >
2018-08-01 15:34:55 +02:00
### `ChangeConstantVisibilityRector`
2018-07-31 21:43:39 +02:00
2018-08-01 15:34:55 +02:00
- class: `Rector\Rector\Visibility\ChangeConstantVisibilityRector`
2018-05-04 23:30:32 +01:00
2018-08-01 15:34:55 +02:00
Change visibility of constant from parent class.
2018-05-04 23:30:32 +01:00
2018-08-01 22:09:34 +02:00
```yaml
services:
Rector\Rector\Visibility\ChangeConstantVisibilityRector:
2018-11-11 13:22:35 +01:00
ParentObject:
SOME_CONSTANT: protected
2018-08-01 22:09:34 +02:00
```
↓
2018-05-04 23:30:32 +01:00
```diff
2018-08-01 15:34:55 +02:00
class FrameworkClass
{
protected const SOME_CONSTANT = 1;
}
2018-07-31 14:50:39 +02:00
2018-08-01 15:34:55 +02:00
class MyClass extends FrameworkClass
{
- public const SOME_CONSTANT = 1;
+ protected const SOME_CONSTANT = 1;
}
2018-05-04 23:30:32 +01:00
```
2018-11-11 13:22:35 +01:00
< br >
2019-03-09 13:15:14 +00:00
### `RenameAnnotationRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:15:14 +00:00
- class: `Rector\Rector\Annotation\RenameAnnotationRector`
2019-02-04 01:32:53 +01:00
Turns defined annotations above properties and methods to their new values.
```yaml
services:
2019-03-09 13:15:14 +00:00
Rector\Rector\Annotation\RenameAnnotationRector:
2019-02-04 01:32:53 +01:00
PHPUnit\Framework\TestCase:
test: scenario
```
↓
```diff
class SomeTest extends PHPUnit\Framework\TestCase
{
- /**
- * @test
+ /**
+ * @scenario
*/
public function someMethod()
{
}
}
```
< br >
2019-03-09 13:24:30 +00:00
### `RenameClassConstantRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\Rector\Constant\RenameClassConstantRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:24:30 +00:00
Replaces defined class constants in their calls.
2019-02-04 01:32:53 +01:00
```yaml
services:
2019-03-09 13:24:30 +00:00
Rector\Rector\Constant\RenameClassConstantRector:
SomeClass:
OLD_CONSTANT: NEW_CONSTANT
OTHER_OLD_CONSTANT: 'DifferentClass::NEW_CONSTANT'
2019-02-04 01:32:53 +01:00
```
↓
```diff
2019-03-09 13:24:30 +00:00
-$value = SomeClass::OLD_CONSTANT;
-$value = SomeClass::OTHER_OLD_CONSTANT;
+$value = SomeClass::NEW_CONSTANT;
+$value = DifferentClass::NEW_CONSTANT;
2019-02-04 01:32:53 +01:00
```
< br >
2019-03-09 13:24:30 +00:00
### `RenameClassConstantsUseToStringsRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:24:30 +00:00
- class: `Rector\Rector\Constant\RenameClassConstantsUseToStringsRector`
2019-02-04 01:32:53 +01:00
2019-03-09 13:24:30 +00:00
Replaces constant by value
2019-02-04 01:32:53 +01:00
```yaml
services:
2019-03-09 13:24:30 +00:00
Rector\Rector\Constant\RenameClassConstantsUseToStringsRector:
Nette\Configurator:
DEVELOPMENT: development
PRODUCTION: production
2019-02-04 01:32:53 +01:00
```
↓
```diff
2019-03-09 13:24:30 +00:00
-$value === Nette\Configurator::DEVELOPMENT
+$value === "development"
2019-02-04 01:32:53 +01:00
```
< br >
2019-03-09 12:51:57 +00:00
### `RenameClassRector`
2019-02-04 01:32:53 +01:00
2019-03-09 12:51:57 +00:00
- class: `Rector\Rector\Class_\RenameClassRector`
2019-02-04 01:32:53 +01:00
Replaces defined classes by new ones.
```yaml
services:
2019-03-09 12:51:57 +00:00
Rector\Rector\Class_\RenameClassRector:
2019-05-02 01:56:58 +02:00
App\SomeOldClass: App\SomeNewClass
2019-02-04 01:32:53 +01:00
```
↓
```diff
2019-05-02 01:56:58 +02:00
namespace App;
2019-02-04 01:32:53 +01:00
-use SomeOldClass;
+use SomeNewClass;
2019-03-16 21:31:46 +01:00
-function someFunction(SomeOldClass $someOldClass): SomeOldClass
+function someFunction(SomeNewClass $someOldClass): SomeNewClass
2019-02-04 01:32:53 +01:00
{
- if ($someOldClass instanceof SomeOldClass) {
- return new SomeOldClass;
+ if ($someOldClass instanceof SomeNewClass) {
+ return new SomeNewClass;
}
}
```
< br >
### `ParentClassToTraitsRector`
- class: `Rector\Rector\Class_\ParentClassToTraitsRector`
Replaces parent class to specific traits
```yaml
services:
Rector\Rector\Class_\ParentClassToTraitsRector:
Nette\Object:
- Nette\SmartObject
```
↓
```diff
-class SomeClass extends Nette\Object
+class SomeClass
{
+ use Nette\SmartObject;
}
```
< br >