-
Notifications
You must be signed in to change notification settings - Fork 305
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 CodeIntel Updates #3770
PHP CodeIntel Updates #3770
Conversation
ssigwart
commented
Jul 6, 2019
- Use FQN for PHP types
- Skip outdated trg-from-pos requests.
- For large files, trg-from-pos requests can back up as you type. I can't think of any reason why we'd need to process a request if we got a newer request for a new position.
- PSR4 updates
- Allow filenames like MyClass.class.php
- When doing PSR4 lookup, only concider paths that match namespace
- Do not look at global scope with doing PSR4 lookups
- Use scandir for PHP import lookup.
It's probably best to look with whitespace ignored: |
I've been using this patch for the mast month and a half. It seems to work well. I also just adding another commit to fix a PSR4 bug: I think the bug was there before this change too, but I'm not positive. If a PSR4 lookup was attempted, but failed, it would throw an error causing the whole lookup to fail. I was seeing this when doing something like this: $arr = functionThatReturnsAnArrayContainingAClassInANamespace();
foreach ($arr as $row)
$row-> // Lookup fails here |
@th3coop, do you think this can be included before the next production release? I've been running it locally the past 4 months (by manually patching files in the nightly). While I can't guarantee that it has no bad side effects, it seems to have been working well. |
@ssigwart, looks like there's a conflict with |
@ssigwart Are there other PRs I've lagged on that you'd like to see in the next release? |
- For large files, trg-from-pos requests can back up as you type. I can't think of any reason why we'd need to process a request if we got a newer request for a new position.
- Allow filenames like MyClass.class.php - When doing PSR4 lookup, only concider paths that match namespace - Do not look at global scope with doing PSR4 lookups
bed4d63
to
330a1db
Compare
I rebased this branch with master.
I do have some others. Here they are in what I think are most to least important:
Very Low Priority
|
@ssigwart thanks for the list! |