rector

rector copied to clipboard

# Bug Report | Subject | Details | | :------------- | :--------------------| | Rector version | last dev-main | | Installed as | composer dependency | ## Minimal PHP Code...

jorgsowa

[PHP 8.4] new MyClass()->method() without parentheses

3

# Feature Request We should create a rule for this RFC: https://wiki.php.net/rfc/new_without_parentheses ## Diff ```diff -$request = (new Request())->withMethod('GET')->withUri('/hello-world'); +$request = new Request()->withMethod('GET')->withUri('/hello-world'); ```

ruudk

# Feature Request I'd like to see combinations of `date` and `strtotime` like the one in the first diff converted to use Carbon. Always converting `strtotime($invoice->getDate())` to something like `Carbon::parse($invoice->getDate())->timestamp`...

OA

Explore way Rector or another tool can process blade templates

2

This is one of pain points when using Rector with Laravel. It works well on PHP code, but fails on Blade. Often Blade uses simle structures like PHP functions calls,...

TomasVotruba

We've done similar split in the past, to make type declaration level work in as small steps as possible. This would concert few `ReturnType*` rules that now handle all functions,...

TomasVotruba

New rule proposal: incorrect use of parenthesescan can cause unexpected behavior in code

1

Thanks for all this awesome tool, for me, is one of the best PHP tools ever done. Great job! # Feature Request I have seen the PHP rule on https://cloud-ci.sgs.com/sonar/coding_rules?open=php%3AS6600&rule_key=php%3AS6600...

palaueb

RemoveExtraParametersRector removes all parameters

1

# Bug Report | Subject | Details | | :------------- | :---------------------------------------------------------------| | Rector version | v1.2.4 | The `RemoveExtraParametersRector` removes the mandatory parameter. ## Minimal PHP Code Causing Issue...

anibalsanchez

earlyReturn confused by elseif

3

# Bug Report | Subject | Details | | :------------- | :---------------------------------------------------------------| | Rector version | v1.2.4 | The `earlyReturn` dropped a whole code branch. INPUT: ```php protected function checkACL($area)...

anibalsanchez

Create a rule to annotate a data provider (PHPUnit) with correct array based the test method signature that uses the data provider

3

# Feature Request I would like a Rector to add the correct return type docblock to a PHPUnit data provider method, based on the signature of the test method that...

DaveLiddament

# Feature Request It would be great if there is a rule (or an upgrade of `RemoveUnusedVariableAssignRector`?) to also remove unused variables assigns if they are inline. ## Diff ```diff

darthf1