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 8.3 Support: Dynamic class constant fetch #6701 #6752
PHP 8.3 Support: Dynamic class constant fetch #6701 #6752
Conversation
b14721e
to
e53b927
Compare
@tmysik Could you please have a look at each commit when you have time? |
php/php.editor/src/org/netbeans/modules/php/editor/parser/astnodes/StaticConstantAccess.java
Outdated
Show resolved
Hide resolved
php/php.editor/src/org/netbeans/modules/php/editor/parser/astnodes/StaticConstantAccess.java
Outdated
Show resolved
Hide resolved
php/php.editor/test/unit/src/org/netbeans/modules/php/editor/PHPBracesMatcherTest.java
Show resolved
Hide resolved
php/php.editor/src/org/netbeans/modules/php/editor/verification/PHP55UnhandledError.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, looks OK to me. As always, thanks a lot for all the tests! 👍
- apache#6701 - https://wiki.php.net/rfc/dynamic_class_constant_fetch - Fix the grammar file (parser) - Add the `PHP83UnhandledError` - Add/Fix unit tests for the parser Example: ```php class Example { const CONSTANT = 'constant'; } $constant = 'CONSTANT'; echo Example::{$constant}; ```
- apache#6701 - https://wiki.php.net/rfc/dynamic_class_constant_fetch - Fix the `SemanticAnalysis` - Fix the mark occurrences and the go to declaration features - Add unit tests
- apache#6701 - https://wiki.php.net/rfc/dynamic_class_constant_fetch - Fix the formatter - Add the option of the around spaces for the scope resolution operator (`::`) - Don't add spaces within braces (`{}`) e.g. {$example} - Add unit tests
- apache#6701 - https://wiki.php.net/rfc/dynamic_class_constant_fetch - Fix the code completion feature - Add unit tests for the code completion and brace matching features
- apache#6701 - https://wiki.php.net/rfc/dynamic_class_constant_fetch - Fix the `IntroduceSuggestion` and `PHP55UnhandledError` - Add unit tests
e53b927
to
7e70563
Compare
@tmysik Fixed : https://github.com/apache/netbeans/compare/e53b9278c63011edf23dc592d66827e77d98edbe..7e705639b19ce57a0fe0984cdc8e91ac2f4272bc Thank you for your review and your time :) |
@junichi11 Nothing to thank for, really. NetBeans users have to thank you, Junichi. Without you, there would not be any up-to-date PHP support in NetBeans at all! 👍 |
PHP 8.3 Support: Add PHP 8.3 to the PhpVersion
PHP 8.3 Support: Dynamic class constant fetch (Part 1)
PHP83UnhandledError
Example:
PHP 8.3:
PHP 8.2:
PHP 8.3 Support: Dynamic class constant fetch (Part 2)
SemanticAnalysis
PHP 8.3 Support: Dynamic class constant fetch (Part 3)
::
){}
) e.g. {$example}PHP 8.3 Support: Dynamic class constant fetch (Part 4)
PHP 8.3 Support: Dynamic class constant fetch (Part 5)
IntroduceSuggestion
andPHP55UnhandledError