-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 autocomplete doesn't work with latest Intelephense #17493
Comments
Taking a look at the issue, I see you are using Intelephense version 1.5.3 and 1.4.0. Right now we only have 1.3.11 in the plugin registry. I can't say for sure, but it looks like there is some theia API missing. Thanks for the bug report, we'll look into updating to Intelephense 1.5.3, including all the needed API to go with it. |
Thanks for taking a look @ericwill |
@l0rd Are you sure it is Intelephense only as it is also not working with CSS? |
@danidoedel no I am not but things may be related and the original title wasn't explicit enough. |
@danidoedel I tried reproducing the issue with a Symfony project and intelphense 1.5.3, but I was unable to reproduce the error. Can you share your devfile or a link to the project which reproduces the crash? |
@ericwill as the devfile and project are work related I can't share them, but I'll try to set something simple up to reproduce it. |
@ericwill I did some tests and found out the following:
However if I create a workspace with this devfile: And try to edit some SCSS file and type something like
I hope you can reproduce it with these information and a fix might also solve the Intelephense issue.. hopfully ;) |
Sadly I still can't reproduce it. How have you deployed Che? Judging by the error, it might be worth it to try a more recent che-theia release, like 7.17 or 7.16.2. |
Currently we are running 7.17.1 on kubernetes, not sure how it was deployed though. I also tired it with che-theia next. |
@ericwill seems to be not just me: eclipse-theia/theia#8475 |
Looks like there is a PR open for that issue, maybe it will fix the PHP case as well. I'm also planning on updating Che's PHP plugin this sprint. |
The theia PR has been merged, I would try testing this on Che 7.20 (when it's released) and see if the bug is still there. |
@ericwill I can confirm it is fixed with Che Theia next 👍 |
Perfect! |
Describe the bug
Auto-completion is not working anymore/only working partially.
Che version
Steps to reproduce
Response
as return type, there will be suggestions forR
, but if I typeRe
* or more the suggestions disappear. If I want to writeRediretResponse
there will be suggestions forR
, but not forRe
, but again forRed
and every following letter.Similar situation for CSS properties in a SCSS file. If I want to write
display
**, there are no suggestions at all, but afterdisplay: i
, there will be suggestions forinline-block
for example.I tried it in Gitpod and VS Code and there were no issues.
See screenshots.
*This error message appears in the console after typing
Re
(see above):**This error message appears in the console when trying to type
display
in a SCSS file (see above):Expected behavior
Suggestions and auto-completion should always work
Runtime
kubectl version
)oc version
)minikube version
andkubectl version
)minishift version
andoc version
)docker version
andkubectl version
)Screenshots
Installation method
chectl version
commandEnvironment
The text was updated successfully, but these errors were encountered: