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

[ruudk]

Feature Request

We should create a rule for this RFC:

https://wiki.php.net/rfc/new_without_parentheses

Diff

-$request = (new Request())->withMethod('GET')->withUri('/hello-world');
+$request = new Request()->withMethod('GET')->withUri('/hello-world');

[samsonasik]

I think that need to be feature request for nikic/php-parser

[TomasVotruba]

I already asked in php-parser last week. This will most likely require php-paser 5, PHPStan 2 and then Rector 2 due to BC breaks.

nikic/PHP-Parser@7b0384c#commitcomment-143390979

[samsonasik]

I created downgrade PR rule first:

• [DowngradePhp84] Add DowngradeNewMethodCallWithoutParenthesesRector rector-downgrade-php#253

to allow code to be downgraded on the first place.

[gnutix]

This will most likely require php-paser 5, PHPStan 2 and then Rector 2 due to BC breaks.

@samsonasik it seems to me that these criterion are now met. Is there anything more needed to create this new rule ?

[samsonasik]

The no parenthesis syntax is not only on method call, but also static call, property fetch, etc, per rfc

https://wiki.php.net/rfc/new_without_parentheses

which downgrade need to be exists first so full feature downgrade need to be exists first to avoid flip-flop lookup.

I am not fan of this syntax sugar, but feel free to create PR if you're interested, always start with improving downgrade part, there is starting point you can look:

• [DowngradePhp84] Add DowngradeNewMethodCallWithoutParenthesesRector rector-downgrade-php#253

[github-actions]

This issue has been automatically locked because it has been closed for 150 days. Please open a new issue if you have a similar problem.