-
-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rectorphp or rector have a problem with fluent function \TYPO3\CMS\Core\Database\Query\Expression\CompositeExpression::add #4196
Comments
is related to rectorphp/rector#8563. I test the patch https://github.com/rectorphp/rector-src/pull/5743/files in vendor/rector/rector/src/PHPStan/NodeVisitor/ExprScopeFromStmtNodeVisitor.php fom @samsonasik , but it do not solved the problem. Can you help? |
Like mentioned on original issue, It is better to create a github repo with step by step to reproduce, otherwise, nothing we can do. |
You can find reproduceable repo here https://github.com/hdietrich-timespin/typo3-rector-testrepo/tree/bug_issues_4196. You can start rector process with:
|
Fixed at rectorphp/rector-src#5743 |
It works. Thanks for the Bugfix! But is it possible that the changed code more human readable? rector changes is very complicate, horrible and have to much redundancy code: See new issue #4201 |
The following Code throw an Error durinfg process
typo3-rector version 2.2.0
Code:
Error: (see trace in the end of that description!)
if i change the Code to:
It will be processed without Error:
Errortrace:
The text was updated successfully, but these errors were encountered: