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
[NETBEANS-5599] PHP 8.1 Support: First class callable syntax #4240
[NETBEANS-5599] PHP 8.1 Support: First class callable syntax #4240
Conversation
- https://issues.apache.org/jira/browse/NETBEANS-5599 - https://wiki.php.net/rfc/first_class_callable_syntax - Fix the parser - Fix `PHP81UnhandledError` - Add unit tests
- https://issues.apache.org/jira/browse/NETBEANS-5599 - https://wiki.php.net/rfc/first_class_callable_syntax - Fix the GotoDeclaration and MarkOccurences - Add unit tests
- https://issues.apache.org/jira/browse/NETBEANS-5599 - https://wiki.php.net/rfc/first_class_callable_syntax - Add the HintError to check incorrect first-class callable syntax - Add unit tests
Probably, once this is merged, PHP 8.1 support is complete :) |
@tmysik Could you please have a look at this when you have time? If there is no problem, let's merge it. Thanks! |
php/php.editor/src/org/netbeans/modules/php/editor/completion/PHPCompletionItem.java
Outdated
Show resolved
Hide resolved
php/php.editor/src/org/netbeans/modules/php/editor/completion/PHPCompletionItem.java
Show resolved
Hide resolved
@junichi11 Junichi, please, let me know what you think about my nitpick comments. The change looks good to me, I am ready to merge. Thanks again for your great work! |
- https://issues.apache.org/jira/browse/NETBEANS-5599 - https://wiki.php.net/rfc/first_class_callable_syntax - Fix the code completion - Don't add callable(...) cc items by default - Add unit tests
7069e4d
to
34af956
Compare
@tmysik Fixed: https://github.com/apache/netbeans/compare/7069e4d2f2b90e385bc61681e1b8e4b37770bb24..34af956e74b9b92809cefdfd1126005d8ba35697 Thank you for your review! |
Part 1
PHP81UnhandledError
PHP 8.1
PHP 8.0
Part 2
Part 3
Part 4
Part 5