-
Notifications
You must be signed in to change notification settings - Fork 843
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-377] Add PSR-4 hints #2630
Conversation
We should leave a comment for unsupported cases somewhere. Add a test for the following case? // file: Vendor/App/ClassName.php
namespace Vendor\App;
class classname {
} |
php/php.editor/src/org/netbeans/modules/php/editor/verification/PSR4Hint.java
Outdated
Show resolved
Hide resolved
e6bef1a
to
4fe6751
Compare
I added tests where casing is different. Also added note about case-sensitivity to hint descriptions:
|
php/php.editor/src/org/netbeans/modules/php/editor/verification/Bundle.properties
Show resolved
Hide resolved
php/php.editor/src/org/netbeans/modules/php/editor/verification/PSR4Hint.java
Show resolved
Hide resolved
https://issues.apache.org/jira/browse/NETBEANS-377 - Added hints for [PSR-4 Autoloader](https://www.php-fig.org/psr/psr-4/) standard. - Checks that class/interface/trait declaration matches file name. - Checks that namespace matches directory structure. - Highlights only non-matching part of namespace.
4fe6751
to
7efb4f8
Compare
Will re-run jobs. |
@junichi11 @tmysik Thanks for review. |
Does this feature support psr-4 declaration from composer.json? |
@ahajzik-crido No, mapping from composer.json is separate task - https://issues.apache.org/jira/browse/NETBEANS-5341 |
https://issues.apache.org/jira/browse/NETBEANS-377