-
Notifications
You must be signed in to change notification settings - Fork 118
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
PHP named attributes are not taken into account #1925
Comments
The cause seems to be in this line of code where the arguments length is being checked as greater or equal to 4 I guess if possible the arguments should be checked for the index flags and take that argument |
@ea-inspections-team: +1 request to address this at least in the mentioned inspections. |
Thank you for reporting @guerinetn. Question for everyone: are |
Hello @ea-inspections-team at the moment it's the only method with this issues. |
@ea-inspections-team Is there a reason this open PR #1815 has not been picked up yet to resolve this issue? |
+1 |
@kalessil I wanted to kindly check with you if this is something @ea-inspections-team is working on? Perhaps some pointers could be given in order help me (or other people watching this issue) out in trying to create a PR addressing this issue? |
Thank you for your patience, everyone. We are working on fixes for the functions and inspections mentioned in this issue and will release Extended and Ultimate plugins after the fixes. |
We are still working on fixes for the issue and delaying the next releases for around week - v2024.2 for EA ultimate and v 5.1.0 for EA Extended. If no solution found today, we will proceed with the releases tomorrow. |
Current behaviour
If I use a named attributes for a function it's not understood.
If I write :
EA raise the error : Please provide 'more_entropy' parameter in order to increase likelihood of uniqueness.
It would be nice, il named attributes are understood by EA because this feature helps a lot when you have a lots of optional parameter to a function.
The text was updated successfully, but these errors were encountered: