mirror of
https://github.com/rectorphp/rector.git
synced 2025-03-22 08:19:50 +01:00
use PHP highlight for herenow doc
This commit is contained in:
parent
93d3a46a13
commit
ca7d0cba15
7
ecs.yaml
7
ecs.yaml
@ -185,3 +185,10 @@ parameters:
|
||||
# intentional "assertEquals()"
|
||||
- 'tests/PhpParser/Node/NodeFactoryTest.php'
|
||||
- '*TypeResolverTest.php'
|
||||
|
||||
Symplify\CodingStandard\Fixer\LineLength\LineLengthFixer:
|
||||
# buggy with PHP heredoc
|
||||
- 'packages/SOLID/src/Rector/ClassConst/PrivatizeLocalClassConstantRector.php'
|
||||
- 'packages/Php/src/Rector/FuncCall/SensitiveDefineRector.php'
|
||||
- 'packages/Php/src/Rector/Name/ReservedObjectRector.php'
|
||||
- 'packages/Php/src/Rector/Assign/AssignArrayToStringRector.php'
|
||||
|
@ -52,7 +52,7 @@ final class ConstructorInjectionToActionInjectionRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
/**
|
||||
@ -70,9 +70,9 @@ final class SomeController
|
||||
$products = $this->productRepository->fetchAll();
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function default(ProductRepository $productRepository)
|
||||
@ -80,7 +80,7 @@ final class SomeController
|
||||
$products = $productRepository->fetchAll();
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -30,7 +30,7 @@ final class RemoveRepositoryFromEntityAnnotationRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Removes repository class from @Entity annotation', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Doctrine\ORM\Mapping as ORM;
|
||||
|
||||
/**
|
||||
@ -39,9 +39,9 @@ use Doctrine\ORM\Mapping as ORM;
|
||||
class Product
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Doctrine\ORM\Mapping as ORM;
|
||||
|
||||
/**
|
||||
@ -50,7 +50,7 @@ use Doctrine\ORM\Mapping as ORM;
|
||||
class Product
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -41,7 +41,7 @@ final class MoveEntitiesToEntityDirectoryRector extends AbstractFileSystemRector
|
||||
public function getDefinition(): RectorDefinition
|
||||
{
|
||||
return new RectorDefinition('Move entities to Entity namespace', [new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
// file: app/Controller/Product.php
|
||||
|
||||
namespace App\Controller;
|
||||
@ -54,9 +54,9 @@ use Doctrine\ORM\Mapping as ORM;
|
||||
class Product
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
// file: app/Entity/Product.php
|
||||
|
||||
namespace App\Entity;
|
||||
@ -69,7 +69,7 @@ use Doctrine\ORM\Mapping as ORM;
|
||||
class Product
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
)]);
|
||||
}
|
||||
|
||||
|
@ -30,7 +30,7 @@ final class MoveInterfacesToContractNamespaceDirectoryRector extends AbstractFil
|
||||
public function getDefinition(): RectorDefinition
|
||||
{
|
||||
return new RectorDefinition('Move interface to "Contract" namespace', [new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
// file: app/Exception/Rule.php
|
||||
|
||||
namespace App\Exception;
|
||||
@ -38,9 +38,9 @@ namespace App\Exception;
|
||||
interface Rule
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
// file: app/Contract/Rule.php
|
||||
|
||||
namespace App\Contract;
|
||||
@ -48,7 +48,7 @@ namespace App\Contract;
|
||||
interface Rule
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
)]);
|
||||
}
|
||||
|
||||
|
@ -46,7 +46,7 @@ final class MoveServicesBySuffixToDirectoryRector extends AbstractFileSystemRect
|
||||
public function getDefinition(): RectorDefinition
|
||||
{
|
||||
return new RectorDefinition('Move classes by their suffix to their own group/directory', [new ConfiguredCodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
// file: app/Entity/ProductRepository.php
|
||||
|
||||
namespace App/Entity;
|
||||
@ -54,9 +54,9 @@ namespace App/Entity;
|
||||
class ProductRepository
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
// file: app/Repository/ProductRepository.php
|
||||
|
||||
namespace App/Repository;
|
||||
@ -64,7 +64,7 @@ namespace App/Repository;
|
||||
class ProductRepository
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
[
|
||||
'$groupNamesBySuffix' => ['Repository'],
|
||||
|
@ -36,7 +36,7 @@ final class ModalToGetSetRector extends AbstractRector
|
||||
'Changes combined set/get `value()` to specific `getValue()` or `setValue(x)`.',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$object = new InstanceConfigTrait;
|
||||
|
||||
$config = $object->config();
|
||||
@ -44,9 +44,9 @@ $config = $object->config('key');
|
||||
|
||||
$object->config('key', 'value');
|
||||
$object->config(['key' => 'value']);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$object = new InstanceConfigTrait;
|
||||
|
||||
$config = $object->getConfig();
|
||||
@ -54,7 +54,7 @@ $config = $object->getConfig('key');
|
||||
|
||||
$object->setConfig('key', 'value');
|
||||
$object->setConfig(['key' => 'value']);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
@ -19,7 +19,7 @@ final class ChangeSnakedFixtureNameToCamelRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes $fixtues style from snake_case to CamelCase.', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeTest
|
||||
{
|
||||
protected $fixtures = [
|
||||
@ -27,9 +27,9 @@ class SomeTest
|
||||
'app.users',
|
||||
'some_plugin.posts/special_posts',
|
||||
];
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeTest
|
||||
{
|
||||
protected $fixtures = [
|
||||
@ -37,7 +37,7 @@ class SomeTest
|
||||
'app.Users',
|
||||
'some_plugin.Posts/SpeectialPosts',
|
||||
];
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -22,17 +22,17 @@ final class LogicalToBooleanRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Change OR, AND to ||, && with more common understanding', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
if ($f = false or true) {
|
||||
return $f;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
if (($f = false) || true) {
|
||||
return $f;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -45,7 +45,7 @@ final class SetTypeToCastRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes settype() to (type) where possible', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($foo)
|
||||
@ -55,9 +55,9 @@ class SomeClass
|
||||
return settype($foo, 'integer');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(array $items)
|
||||
@ -67,7 +67,7 @@ class SomeClass
|
||||
return (int) $foo;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -19,7 +19,7 @@ final class CommonNotEqualRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Use common != instead of less known <> with same meaning', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run($one, $two)
|
||||
@ -27,9 +27,9 @@ final class SomeClass
|
||||
return $one <> $two;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run($one, $two)
|
||||
@ -37,7 +37,7 @@ final class SomeClass
|
||||
return $one != $two;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -45,7 +45,7 @@ final class CallableThisArrayToAnonymousFunctionRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Convert [$this, "method"] to proper anonymous function', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -61,9 +61,9 @@ class SomeClass
|
||||
return $first <=> $second;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -81,7 +81,7 @@ class SomeClass
|
||||
return $first <=> $second;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -34,21 +34,21 @@ final class SimplifyDeMorganBinaryRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Simplify negated conditions with de Morgan theorem', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
<?php
|
||||
|
||||
$a = 5;
|
||||
$b = 10;
|
||||
$result = !($a > 20 || $b <= 50);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
<?php
|
||||
|
||||
$a = 5;
|
||||
$b = 10;
|
||||
$result = $a <= 20 && $b > 50;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -26,7 +26,7 @@ final class ThrowWithPreviousExceptionRector extends AbstractRector
|
||||
'When throwing into a catch block, checks that the previous exception is passed to the new throw clause',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -38,9 +38,9 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -52,7 +52,7 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
@ -53,7 +53,7 @@ final class CompleteDynamicPropertiesRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Add missing dynamic properties', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function set()
|
||||
@ -61,9 +61,9 @@ class SomeClass
|
||||
$this->value = 5;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
/**
|
||||
@ -75,7 +75,7 @@ class SomeClass
|
||||
$this->value = 5;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -18,7 +18,7 @@ final class JoinStringConcatRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Joins concat of 2 strings', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -26,9 +26,9 @@ class SomeClass
|
||||
$name = 'Hi' . ' Tom';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -36,7 +36,7 @@ class SomeClass
|
||||
$name = 'Hi Tom';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -22,7 +22,7 @@ final class UseIdenticalOverEqualWithSameTypeRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Use ===/!== over ==/!=, it values have the same type', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(int $firstValue, int $secondValue)
|
||||
@ -31,9 +31,9 @@ class SomeClass
|
||||
$isDiffernt = $firstValue != $secondValue;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(int $firstValue, int $secondValue)
|
||||
@ -42,7 +42,7 @@ class SomeClass
|
||||
$isDiffernt = $firstValue !== $secondValue;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -47,7 +47,7 @@ final class ForToForeachRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Change for() to foreach() where useful', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($tokens)
|
||||
@ -63,9 +63,9 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($tokens)
|
||||
@ -81,7 +81,7 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -47,7 +47,7 @@ final class ForeachToInArrayRector extends AbstractRector
|
||||
'Simplify `foreach` loops into `in_array` when possible',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
foreach ($items as $item) {
|
||||
if ($item === "something") {
|
||||
return true;
|
||||
@ -55,7 +55,7 @@ foreach ($items as $item) {
|
||||
}
|
||||
|
||||
return false;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
'in_array("something", $items, true);'
|
||||
),
|
||||
|
@ -25,7 +25,7 @@ final class SimplifyForeachToArrayFilterRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Simplify foreach with function filtering to array filter', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$directories = [];
|
||||
$possibleDirectories = [];
|
||||
foreach ($possibleDirectories as $possibleDirectory) {
|
||||
@ -33,12 +33,12 @@ foreach ($possibleDirectories as $possibleDirectory) {
|
||||
$directories[] = $possibleDirectory;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$possibleDirectories = [];
|
||||
$directories = array_filter($possibleDirectories, 'file_exists');
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -42,7 +42,7 @@ final class SimplifyForeachToCoalescingRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes foreach that returns set value to ??', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
foreach ($this->oldToNewFunctions as $oldFunction => $newFunction) {
|
||||
if ($currentFunction === $oldFunction) {
|
||||
return $newFunction;
|
||||
@ -50,11 +50,11 @@ foreach ($this->oldToNewFunctions as $oldFunction => $newFunction) {
|
||||
}
|
||||
|
||||
return null;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
return $this->oldToNewFunctions[$currentFunction] ?? null;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -23,7 +23,7 @@ final class CompactToVariablesRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Change compact() call to own array', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -34,9 +34,9 @@ class SomeClass
|
||||
return compact('checkout', 'form');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -47,7 +47,7 @@ class SomeClass
|
||||
return ['checkout' => $checkout, 'form' => $form];
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -19,7 +19,7 @@ final class IsAWithStringWithThirdArgumentRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function __construct(string $value)
|
||||
@ -27,9 +27,9 @@ class SomeClass
|
||||
return is_a($value, 'stdClass');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function __construct(string $value)
|
||||
@ -37,7 +37,7 @@ class SomeClass
|
||||
return is_a($value, 'stdClass', true);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -44,7 +44,7 @@ final class SimplifyRegexPatternRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Simplify regex pattern to known ranges', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -52,9 +52,9 @@ class SomeClass
|
||||
preg_match('#[a-zA-Z0-9+]#', $value);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -62,7 +62,7 @@ class SomeClass
|
||||
preg_match('#[\w\d+]#', $value);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -20,7 +20,7 @@ final class SingleInArrayToCompareRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes in_array() with single element to ===', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -30,9 +30,9 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -42,7 +42,7 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -20,7 +20,7 @@ final class StrlenZeroToIdenticalEmptyStringRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -28,9 +28,9 @@ class SomeClass
|
||||
$empty = strlen($value) === 0;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -38,7 +38,7 @@ class SomeClass
|
||||
$empty = $value === '';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -23,7 +23,7 @@ final class BooleanNotIdenticalToNotIdenticalRector extends AbstractRector
|
||||
'Negated identical boolean compare to not identical compare (does not apply to non-bool values)',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -36,9 +36,9 @@ class SomeClass
|
||||
var_dump($a !== $b); // true
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -51,7 +51,7 @@ class SomeClass
|
||||
var_dump($a !== $b); // true
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
@ -21,7 +21,7 @@ final class SimplifyBoolIdenticalTrueRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Symplify bool value compare to true or false', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(bool $value, string $items)
|
||||
@ -30,9 +30,9 @@ class SomeClass
|
||||
$match = in_array($value, $items, TRUE) !== FALSE;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(bool $value, string $items)
|
||||
@ -41,7 +41,7 @@ class SomeClass
|
||||
$match = in_array($value, $items, TRUE);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -43,7 +43,7 @@ final class ConsecutiveNullCompareReturnsToNullCoalesceQueueRector extends Abstr
|
||||
{
|
||||
return new RectorDefinition('Change multiple null compares to ?? queue', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -59,9 +59,9 @@ class SomeClass
|
||||
return null;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -69,7 +69,7 @@ class SomeClass
|
||||
return $this->orderItem ?? $this->orderItemUnit;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -35,7 +35,7 @@ final class ExplicitBoolCompareRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Make if conditions more explicit', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run($items)
|
||||
@ -45,9 +45,9 @@ final class SomeController
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run($items)
|
||||
@ -57,7 +57,7 @@ final class SomeController
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -44,7 +44,7 @@ final class RemoveAlwaysTrueConditionSetInConstructorRector extends AbstractRect
|
||||
{
|
||||
return new RectorDefinition('If conditions is always true, perform the content right away', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
private $value;
|
||||
@ -61,9 +61,9 @@ final class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
private $value;
|
||||
@ -78,7 +78,7 @@ final class SomeClass
|
||||
return 'yes';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -28,7 +28,7 @@ final class SimplifyIfElseToTernaryRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes if/else for same value as assign to ternary', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -40,9 +40,9 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -50,7 +50,7 @@ class SomeClass
|
||||
$this->arrayBuilt[][$key] = empty($value) ? true : $value;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -26,7 +26,7 @@ final class SimplifyIfIssetToNullCoalescingRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Simplify binary if to null coalesce', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run($possibleStatieYamlFile)
|
||||
@ -38,9 +38,9 @@ final class SomeController
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run($possibleStatieYamlFile)
|
||||
@ -48,7 +48,7 @@ final class SomeController
|
||||
$possibleStatieYamlFile['import'] = array_merge($possibleStatieYamlFile['import'] ?? [], $filesToImport);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -30,19 +30,19 @@ final class SimplifyIfNotNullReturnRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes redundant null check to instant return', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$newNode = 'something ;
|
||||
if ($newNode !== null) {
|
||||
return $newNode;
|
||||
}
|
||||
|
||||
return null;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$newNode = 'something ;
|
||||
return $newNode;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -26,13 +26,13 @@ final class SimplifyIfReturnBoolRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Shortens if return false/true to direct return', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
if (strpos($docToken->getContent(), "\n") === false) {
|
||||
return true;
|
||||
}
|
||||
|
||||
return false;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
'return strpos($docToken->getContent(), "\n") === false;'
|
||||
),
|
||||
|
@ -21,7 +21,7 @@ final class AndAssignsToSeparateLinesRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Split 2 assigns ands to separate line', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -30,9 +30,9 @@ class SomeClass
|
||||
$token = 4 and $tokens[] = $token;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -42,7 +42,7 @@ class SomeClass
|
||||
$tokens[] = $token;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -34,18 +34,18 @@ final class SimplifyUselessVariableRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Removes useless variable assigns', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
function () {
|
||||
$a = true;
|
||||
return $a;
|
||||
};
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
function () {
|
||||
return true;
|
||||
};
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -18,7 +18,7 @@ final class SimplifyDuplicatedTernaryRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove ternary that duplicated return value of true : false', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(bool $value, string $name)
|
||||
@ -27,9 +27,9 @@ class SomeClass
|
||||
$isName = $name ? true : false;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(bool $value, string $name)
|
||||
@ -38,7 +38,7 @@ class SomeClass
|
||||
$isName = $name ? true : false;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -19,19 +19,19 @@ final class TernaryToElvisRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Use ?: instead of ?, where useful', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
function elvis()
|
||||
{
|
||||
$value = $a ? $a : false;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
function elvis()
|
||||
{
|
||||
$value = $a ?: false;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -65,17 +65,17 @@ final class PseudoNamespaceToNamespaceRector
|
||||
]
|
||||
),
|
||||
new ConfiguredCodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
/** @var Some_Object $someService */
|
||||
$someService = new Some_Object;
|
||||
$someClassToKeep = new Some_Class_To_Keep;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
/** @var Some\Object $someService */
|
||||
$someService = new Some\Object;
|
||||
$someClassToKeep = new Some_Class_To_Keep;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
[
|
||||
['Some_' => ['Some_Class_To_Keep']],
|
||||
|
@ -19,7 +19,7 @@ final class SplitDoubleAssignRector extends AbstractRector
|
||||
'Split multiple inline assigns to each own lines default value, to prevent undefined array issues',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -27,9 +27,9 @@ class SomeClass
|
||||
$one = $two = 1;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -38,7 +38,7 @@ class SomeClass
|
||||
$two = 1;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
@ -29,7 +29,7 @@ final class CatchExceptionNameMatchingTypeRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Type and name of catch exception should match', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -41,9 +41,9 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -55,7 +55,7 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -20,7 +20,7 @@ final class SplitGroupedConstantsAndPropertiesRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Separate constant and properties to own lines', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
const HI = true, AHOJ = 'true';
|
||||
@ -30,9 +30,9 @@ class SomeClass
|
||||
*/
|
||||
public $isIt, $isIsThough;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
const HI = true;
|
||||
@ -48,7 +48,7 @@ class SomeClass
|
||||
*/
|
||||
public $isIsThough;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -29,14 +29,14 @@ final class VarConstantCommentRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Constant should have a @var comment with type', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
const HI = 'hi';
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
/**
|
||||
@ -44,7 +44,7 @@ class SomeClass
|
||||
*/
|
||||
const HI = 'hi';
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -34,7 +34,7 @@ final class NewlineBeforeNewAssignSetRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Add extra space before new assign set', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -45,9 +45,9 @@ final class SomeClass
|
||||
$value2->setValue(1);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -59,7 +59,7 @@ final class SomeClass
|
||||
$value2->setValue(1);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -55,7 +55,7 @@ final class ReturnArrayClassMethodToYieldRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Turns yield return to array return in specific type and method', [
|
||||
new ConfiguredCodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeEventSubscriber implements EventSubscriberInterface
|
||||
{
|
||||
public static function getSubscribedEvents()
|
||||
@ -63,9 +63,9 @@ class SomeEventSubscriber implements EventSubscriberInterface
|
||||
yield 'event' => 'callback';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeEventSubscriber implements EventSubscriberInterface
|
||||
{
|
||||
public static function getSubscribedEvents()
|
||||
@ -73,7 +73,7 @@ class SomeEventSubscriber implements EventSubscriberInterface
|
||||
return ['event' => 'callback'];
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
[
|
||||
'EventSubscriberInterface' => ['getSubscribedEvents'],
|
||||
|
@ -43,7 +43,7 @@ final class YieldClassMethodToArrayClassMethodRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Turns yield return to array return in specific type and method', [
|
||||
new ConfiguredCodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeEventSubscriber implements EventSubscriberInterface
|
||||
{
|
||||
public static function getSubscribedEvents()
|
||||
@ -51,9 +51,9 @@ class SomeEventSubscriber implements EventSubscriberInterface
|
||||
yield 'event' => 'callback';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeEventSubscriber implements EventSubscriberInterface
|
||||
{
|
||||
public static function getSubscribedEvents()
|
||||
@ -61,7 +61,7 @@ class SomeEventSubscriber implements EventSubscriberInterface
|
||||
return ['event' => 'callback'];
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
[
|
||||
'EventSubscriberInterface' => ['getSubscribedEvents'],
|
||||
|
@ -52,7 +52,7 @@ final class AddArrayDefaultToArrayPropertyRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Adds array default value to property to prevent foreach over null error', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
/**
|
||||
@ -65,9 +65,9 @@ class SomeClass
|
||||
return $this->values === null;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
/**
|
||||
@ -80,7 +80,7 @@ class SomeClass
|
||||
return $this->values === [];
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -23,7 +23,7 @@ final class EncapsedStringsToSprintfRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Convert enscaped {$string} to more readable sprintf', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run(string $format)
|
||||
@ -31,9 +31,9 @@ final class SomeClass
|
||||
return "Unsupported format {$format}";
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run(string $format)
|
||||
@ -41,7 +41,7 @@ final class SomeClass
|
||||
return sprintf('Unsupported format %s', $format);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -21,7 +21,7 @@ final class ConsistentImplodeRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes various implode forms to consistent one', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(array $items)
|
||||
@ -32,9 +32,9 @@ class SomeClass
|
||||
$itemsAsStrings = implode('|', $items);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(array $items)
|
||||
@ -45,7 +45,7 @@ class SomeClass
|
||||
$itemsAsStrings = implode('|', $items);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -65,7 +65,7 @@ final class ConsistentPregDelimiterRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Replace PREG delimiter with configured one', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -74,9 +74,9 @@ class SomeClass
|
||||
preg_match_all('~value~im', $value);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -85,7 +85,7 @@ class SomeClass
|
||||
preg_match_all('#value#im', $value);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -35,11 +35,11 @@ final class SimpleArrayCallableToStringRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes redundant anonymous bool functions to simple calls', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$paths = array_filter($paths, function ($path): bool {
|
||||
return is_dir($path);
|
||||
});
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
'array_filter($paths, "is_dir");'
|
||||
),
|
||||
|
@ -29,23 +29,23 @@ final class NullableCompareToNullRector extends AbstractRector
|
||||
'Changes negate of empty comparison of nullable value to explicit === or !== compare',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
/** @var stdClass|null $value */
|
||||
if ($value) {
|
||||
}
|
||||
|
||||
if (!$value) {
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
/** @var stdClass|null $value */
|
||||
if ($value !== null) {
|
||||
}
|
||||
|
||||
if ($value === null) {
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
@ -21,7 +21,7 @@ final class FollowRequireByDirRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('include/require should be followed by absolute path', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -29,9 +29,9 @@ class SomeClass
|
||||
require 'autoload.php';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -39,7 +39,7 @@ class SomeClass
|
||||
require __DIR__ . '/autoload.php';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -70,7 +70,7 @@ final class ImportFullyQualifiedNamesRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Import fully qualified names to use statements', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function create()
|
||||
@ -78,9 +78,9 @@ class SomeClass
|
||||
return SomeAnother\AnotherClass;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use SomeAnother\AnotherClass;
|
||||
|
||||
class SomeClass
|
||||
@ -90,7 +90,7 @@ class SomeClass
|
||||
return AnotherClass;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -49,7 +49,7 @@ final class ManualJsonStringToJsonEncodeArrayRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Add extra space before new assign set', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -57,9 +57,9 @@ final class SomeClass
|
||||
$someJsonAsString = '{"role_name":"admin","numberz":{"id":"10"}}';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -72,7 +72,7 @@ final class SomeClass
|
||||
$someJsonAsString = Nette\Utils\Json::encode($data);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -21,7 +21,7 @@ final class SplitStringClassConstantToClassConstFetchRector extends AbstractRect
|
||||
{
|
||||
return new RectorDefinition('Separate class constant in a string to class constant fetch and string', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
const HI = true;
|
||||
@ -34,9 +34,9 @@ class AnotherClass
|
||||
return 'SomeClass::HI';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
const HI = true;
|
||||
@ -49,7 +49,7 @@ class AnotherClass
|
||||
return SomeClass::class . '::HI';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -18,7 +18,7 @@ final class SymplifyQuoteEscapeRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Prefer quote that not inside the string', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -27,9 +27,9 @@ class SomeClass
|
||||
$name = '\' Sara';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -38,7 +38,7 @@ class SomeClass
|
||||
$name = "' Sara";
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -25,7 +25,7 @@ final class BinarySwitchToIfElseRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes switch with 2 options to if-else', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
switch ($foo) {
|
||||
case 'my string':
|
||||
$result = 'ok';
|
||||
@ -34,15 +34,15 @@ switch ($foo) {
|
||||
default:
|
||||
$result = 'not ok';
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
if ($foo == 'my string') {
|
||||
$result = 'ok;
|
||||
} else {
|
||||
$result = 'not ok';
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -59,21 +59,21 @@ final class RemoveUnusedAliasRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Removes unused use aliases', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Symfony\Kernel as BaseKernel;
|
||||
|
||||
class SomeClass extends BaseKernel
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Symfony\Kernel;
|
||||
|
||||
class SomeClass extends Kernel
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -19,18 +19,18 @@ final class RemoveDuplicatedArrayKeyRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove duplicated key in defined arrays.', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$item = [
|
||||
1 => 'A',
|
||||
1 => 'B'
|
||||
];
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$item = [
|
||||
1 => 'B'
|
||||
];
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -46,10 +46,10 @@ final class RemoveDoubleAssignRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Simplify useless double assigns', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$value = 1;
|
||||
$value = 1;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
'$value = 1;'
|
||||
),
|
||||
|
@ -17,7 +17,7 @@ final class RemoveAndTrueRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove and true that has no added value', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -25,9 +25,9 @@ class SomeClass
|
||||
return true && 5 === 1;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -35,7 +35,7 @@ class SomeClass
|
||||
return 5 === 1;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -28,7 +28,7 @@ final class RemoveUnusedPrivateConstantRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove unused private constant', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
private const SOME_CONSTANT = 5;
|
||||
@ -37,9 +37,9 @@ final class SomeController
|
||||
return 5;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run()
|
||||
@ -47,7 +47,7 @@ final class SomeController
|
||||
return 5;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -17,20 +17,20 @@ final class RemoveDeadConstructorRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove empty constructor', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function __construct()
|
||||
{
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -37,7 +37,7 @@ final class RemoveDelegatingParentCallRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function prettyPrint(array $stmts): string
|
||||
@ -45,13 +45,13 @@ class SomeClass
|
||||
return parent::prettyPrint($stmts);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -30,20 +30,20 @@ final class RemoveEmptyClassMethodRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove empty method calls not required by parents', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class OrphanClass
|
||||
{
|
||||
public function __construct()
|
||||
{
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class OrphanClass
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -39,7 +39,7 @@ final class RemoveOverriddenValuesRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove initial assigns of overridden values', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run()
|
||||
@ -49,9 +49,9 @@ final class SomeController
|
||||
$directories = array_filter($possibleDirectories, 'file_exists');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run()
|
||||
@ -60,7 +60,7 @@ final class SomeController
|
||||
$directories = array_filter($possibleDirectories, 'file_exists');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -77,7 +77,7 @@ final class RemoveUnusedParameterRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove unused parameter, if not required by interface or parent class', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function __construct($value, $value2)
|
||||
@ -85,9 +85,9 @@ class SomeClass
|
||||
$this->value = $value;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function __construct($value)
|
||||
@ -95,7 +95,7 @@ class SomeClass
|
||||
$this->value = $value;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -32,7 +32,7 @@ final class RemoveUnusedPrivateMethodRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove unused private method', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run()
|
||||
@ -45,9 +45,9 @@ final class SomeController
|
||||
return 10;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeController
|
||||
{
|
||||
public function run()
|
||||
@ -55,7 +55,7 @@ final class SomeController
|
||||
return 5;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -44,7 +44,7 @@ final class RemoveSetterOnlyPropertyAndMethodCallRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Removes method that set values that are never used', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
private $name;
|
||||
@ -63,9 +63,9 @@ class ActiveOnlySetter
|
||||
$someClass->setName('Tom');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
}
|
||||
@ -77,7 +77,7 @@ class ActiveOnlySetter
|
||||
$someClass = new SomeClass();
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -73,7 +73,7 @@ final class RemoveUnusedDoctrineEntityMethodAndPropertyRector extends AbstractRe
|
||||
{
|
||||
return new RectorDefinition('Removes unused methods and properties from Doctrine entity classes', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Doctrine\ORM\Mapping as ORM;
|
||||
|
||||
/**
|
||||
@ -96,9 +96,9 @@ class UserEntity
|
||||
$this->name = $name;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Doctrine\ORM\Mapping as ORM;
|
||||
|
||||
/**
|
||||
@ -107,7 +107,7 @@ use Doctrine\ORM\Mapping as ORM;
|
||||
class UserEntity
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -19,7 +19,7 @@ final class RemoveConcatAutocastRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove (string) casting when it comes to concat, that does this by default', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeConcatingClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -27,9 +27,9 @@ class SomeConcatingClass
|
||||
return 'hi ' . (string) $value;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeConcatingClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -37,7 +37,7 @@ class SomeConcatingClass
|
||||
return 'hi ' . $value;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -22,7 +22,7 @@ final class RemoveDeadIfForeachForRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove if, foreach and for that does not do anything', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($someObject)
|
||||
@ -40,9 +40,9 @@ class SomeClass
|
||||
return $value;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($someObject)
|
||||
@ -54,7 +54,7 @@ class SomeClass
|
||||
return $value;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -17,19 +17,19 @@ final class RemoveUnusedForeachKeyRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove unused key in foreach', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$items = [];
|
||||
foreach ($items as $key => $value) {
|
||||
$result = $value;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$items = [];
|
||||
foreach ($items as $value) {
|
||||
$result = $value;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -22,7 +22,7 @@ final class RemoveCodeAfterReturnRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove dead code after return statement', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(int $a)
|
||||
@ -31,9 +31,9 @@ class SomeClass
|
||||
$a++;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run(int $a)
|
||||
@ -41,7 +41,7 @@ class SomeClass
|
||||
return $a;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -21,7 +21,7 @@ final class RemoveDeadReturnRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove last return in the functions, since does not do anything', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -35,9 +35,9 @@ class SomeClass
|
||||
return;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -49,7 +49,7 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -18,7 +18,7 @@ final class RemoveAlwaysTrueIfConditionRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove if condition that is always true', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function go()
|
||||
@ -30,9 +30,9 @@ final class SomeClass
|
||||
return 'no';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
final class SomeClass
|
||||
{
|
||||
public function go()
|
||||
@ -42,7 +42,7 @@ final class SomeClass
|
||||
return 'no';
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -25,7 +25,7 @@ final class RemoveDuplicatedInstanceOfRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -34,9 +34,9 @@ class SomeClass
|
||||
$isIt = $value instanceof A && $value instanceof A;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run($value)
|
||||
@ -45,7 +45,7 @@ class SomeClass
|
||||
$isIt = $value instanceof A;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -44,7 +44,7 @@ final class RemoveDefaultArgumentValueRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove argument value, if it is the same as default value', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -63,9 +63,9 @@ class SomeClass
|
||||
return $cards;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -84,7 +84,7 @@ class SomeClass
|
||||
return $cards;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -24,7 +24,7 @@ final class RemoveDeadZeroAndOneOperationRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -33,9 +33,9 @@ class SomeClass
|
||||
$value = 5 + 0;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -44,7 +44,7 @@ class SomeClass
|
||||
$value = 5;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -24,7 +24,7 @@ final class RemoveZeroAndOneBinaryRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -33,9 +33,9 @@ class SomeClass
|
||||
$value = 5 + 0;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -44,7 +44,7 @@ class SomeClass
|
||||
$value = 5;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -19,19 +19,19 @@ final class RemoveNullPropertyInitializationRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove initialization with null value from property declarations', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SunshineCommand extends ParentClassWithNewConstructor
|
||||
{
|
||||
private $myVar = null;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SunshineCommand extends ParentClassWithNewConstructor
|
||||
{
|
||||
private $myVar;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -35,18 +35,18 @@ final class RemoveUnusedPrivatePropertyRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove unused private properties', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
private $property;
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -30,7 +30,7 @@ final class RemoveParentCallWithoutParentRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Remove unused parent call with no parent class', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class OrphanClass
|
||||
{
|
||||
public function __construct()
|
||||
@ -38,16 +38,16 @@ class OrphanClass
|
||||
parent::__construct();
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class OrphanClass
|
||||
{
|
||||
public function __construct()
|
||||
{
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -22,14 +22,14 @@ final class RemoveDeadStmtRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Removes dead code statements', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$value = 5;
|
||||
$value;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$value = 5;
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -18,7 +18,7 @@ final class RemoveDuplicatedCaseInSwitchRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('2 following switch keys with identical will be reduced to one result', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -35,9 +35,9 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -53,7 +53,7 @@ class SomeClass
|
||||
}
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -48,7 +48,7 @@ final class ManagerRegistryGetManagerToEntityManagerRector extends AbstractRecto
|
||||
{
|
||||
return new RectorDefinition('', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Doctrine\Common\Persistence\ManagerRegistry;
|
||||
|
||||
class CustomRepository
|
||||
@ -69,9 +69,9 @@ class CustomRepository
|
||||
$someRepository = $entityManager->getRepository('Some');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Doctrine\ORM\EntityManagerInterface;
|
||||
|
||||
class CustomRepository
|
||||
@ -91,7 +91,7 @@ class CustomRepository
|
||||
$someRepository = $this->entityManager->getRepository('Some');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -39,15 +39,15 @@ final class EntityAliasToClassConstantReferenceRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Replaces doctrine alias with class.', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$entityManager = new Doctrine\ORM\EntityManager();
|
||||
$entityManager->getRepository("AppBundle:Post");
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$entityManager = new Doctrine\ORM\EntityManager();
|
||||
$entityManager->getRepository(\App\Entity\Post::class);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -22,7 +22,7 @@ final class MigrateFilterToQueryRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Migrates addFilter to addQuery', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -34,9 +34,9 @@ class SomeClass
|
||||
);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -49,7 +49,7 @@ class SomeClass
|
||||
);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -30,15 +30,15 @@ final class MessageAsArrayRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Changes getMessage(..., true) to getMessageAsArray()', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
/** @var GuzzleHttp\Message\MessageInterface */
|
||||
$value = $message->getMessage('key', true);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
/** @var GuzzleHttp\Message\MessageInterface */
|
||||
$value = $message->getMessageAsArray('key');
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -32,7 +32,7 @@ final class InlineValidationRulesToArrayDefinitionRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Transforms inline validation rules to array definition', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Illuminate\Foundation\Http\FormRequest;
|
||||
|
||||
class SomeClass extends FormRequest
|
||||
@ -44,9 +44,9 @@ class SomeClass extends FormRequest
|
||||
];
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Illuminate\Foundation\Http\FormRequest;
|
||||
|
||||
class SomeClass extends FormRequest
|
||||
@ -58,7 +58,7 @@ class SomeClass extends FormRequest
|
||||
];
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -173,7 +173,7 @@ final class HelperFunctionToConstructorInjectionRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Move help facade-like function calls to constructor injection', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeController
|
||||
{
|
||||
public function action()
|
||||
@ -182,9 +182,9 @@ class SomeController
|
||||
$viewFactory = view();
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeController
|
||||
{
|
||||
/**
|
||||
@ -203,7 +203,7 @@ class SomeController
|
||||
$viewFactory = $this->viewFactory;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -75,7 +75,7 @@ final class FacadeStaticCallToConstructorInjectionRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Move Illuminate\Support\Facades\* static calls to constructor injection', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Illuminate\Support\Facades\Response;
|
||||
|
||||
class ExampleController extends Controller
|
||||
@ -85,9 +85,9 @@ class ExampleController extends Controller
|
||||
return Response::view('example', ['new_example' => 123]);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Illuminate\Support\Facades\Response;
|
||||
|
||||
class ExampleController extends Controller
|
||||
@ -107,7 +107,7 @@ class ExampleController extends Controller
|
||||
return $this->responseFactory->view('example', ['new_example' => 123]);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -36,7 +36,7 @@ final class MinutesToSecondsInCacheRector extends AbstractRector
|
||||
'Change minutes argument to seconds in Illuminate\Contracts\Cache\Store and Illuminate\Support\Facades\Cache',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -44,9 +44,9 @@ class SomeClass
|
||||
Illuminate\Support\Facades\Cache::put('key', 'value', 60);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -54,7 +54,7 @@ class SomeClass
|
||||
Illuminate\Support\Facades\Cache::put('key', 'value', 60 * 60);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
@ -24,7 +24,7 @@ final class Redirect301ToPermanentRedirectRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Change "redirect" call with 301 to "permanentRedirect"', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -32,9 +32,9 @@ class SomeClass
|
||||
Illuminate\Routing\Route::redirect('/foo', '/bar', 301);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function run()
|
||||
@ -42,7 +42,7 @@ class SomeClass
|
||||
Illuminate\Routing\Route::permanentRedirect('/foo', '/bar');
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -40,7 +40,7 @@ final class RequestStaticValidateToInjectRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Change static validate() method to $request->validate()', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Illuminate\Http\Request;
|
||||
|
||||
class SomeClass
|
||||
@ -50,9 +50,9 @@ class SomeClass
|
||||
$validatedData = Request::validate(['some_attribute' => 'required']);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
use Illuminate\Http\Request;
|
||||
|
||||
class SomeClass
|
||||
@ -62,7 +62,7 @@ class SomeClass
|
||||
$validatedData = $request->validate(['some_attribute' => 'required']);
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -31,7 +31,7 @@ final class ChangeSingletonToServiceRector extends AbstractRector
|
||||
{
|
||||
return new RectorDefinition('Change singleton class to normal class that can be registered as a service', [
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
private static $instance;
|
||||
@ -49,16 +49,16 @@ class SomeClass
|
||||
return static::$instance;
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
class SomeClass
|
||||
{
|
||||
public function __construct()
|
||||
{
|
||||
}
|
||||
}
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]);
|
||||
}
|
||||
|
@ -42,15 +42,15 @@ final class MysqlAssignToMysqliRector extends AbstractRector
|
||||
'Converts more complex mysql functions to mysqli',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
$data = mysql_db_name($result, $row);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
mysqli_data_seek($result, $row);
|
||||
$fetch = mysql_fetch_row($result);
|
||||
$data = $fetch[0];
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
@ -25,13 +25,13 @@ final class MysqlFuncCallToMysqliRector extends AbstractRector
|
||||
'Converts more complex mysql functions to mysqli',
|
||||
[
|
||||
new CodeSample(
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
mysql_drop_db($database);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
,
|
||||
<<<'CODE_SAMPLE'
|
||||
<<<'PHP'
|
||||
mysqli_query('DROP DATABASE ' . $database);
|
||||
CODE_SAMPLE
|
||||
PHP
|
||||
),
|
||||
]
|
||||
);
|
||||
|
Some files were not shown because too many files have changed in this diff Show More
Loading…
x
Reference in New Issue
Block a user