Skip to content
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] Ability to turn off PHP autocompletion by VSCode #9003

Closed
Ciantic opened this issue Jul 9, 2016 · 16 comments · Fixed by #19015
Closed

[php] Ability to turn off PHP autocompletion by VSCode #9003

Ciantic opened this issue Jul 9, 2016 · 16 comments · Fixed by #19015
Assignees
Labels
feature-request Request for new features or functionality php PHP support issues
Milestone

Comments

@Ciantic
Copy link

Ciantic commented Jul 9, 2016

There is now a better Intellisense plugins for Vscode PHP: vscode-php-intellisense and crane.

In it's known issues it says "There may be duplicate suggestions for functions or variables. This is because VS Code has it's own (very limited) code-completion support for PHP, and there is currently no way to turn this off."

This indeed is annoying, the vscode suggestions are too broad and are just noise compared to accurate entries by crane.

@jrieken
Copy link
Member

jrieken commented Jul 12, 2016

It should be possible to uninstall the VS Code php extension.

@jrieken jrieken added the feature-request Request for new features or functionality label Jul 12, 2016
@jrieken
Copy link
Member

jrieken commented Jul 12, 2016

@joaomoreno fyi

@Ciantic
Copy link
Author

Ciantic commented Jul 12, 2016

I think crane still uses vscode's syntax highlighter for PHP. So if one were to remove VSCode's internal PHP extension, would it also remove the syntax highlighter?

P.S. I think simple options would suffice, e.g. "php.vscode.autocomplete", so it could be turned off.

@jrieken
Copy link
Member

jrieken commented Jul 12, 2016

Yes - but we can also think about splitting our extension into one part that does highlighting and bracket config and another that does completions. That would allow for more freedom.

@jrieken jrieken assigned aeschli and unassigned jrieken Aug 12, 2016
@aeschli aeschli changed the title Ability to turn off PHP autocompletion by VSCode [php] Ability to turn off PHP autocompletion by VSCode Aug 29, 2016
@aeschli aeschli added the php PHP support issues label Aug 29, 2016
@aeschli aeschli added this to the Backlog milestone Aug 29, 2016
@Ciantic
Copy link
Author

Ciantic commented Dec 8, 2016

This is now annoying the testing of proper language server in here : felixfbecker/php-language-server#165

Quote:

VS Code has its own inferior completion. The only way to disable is going to the installation dir under resources/app/extensions/php/out/phpMain.js and commenting out the registration of the completionProvider.

P.S. I think the option would be a simplest thing to implement for now.

@felixfbecker
Copy link
Contributor

Related #16586

@felixfbecker
Copy link
Contributor

felixfbecker commented Dec 24, 2016

Any update on this?

@aeschli aeschli assigned mousetraps and unassigned aeschli Dec 31, 2016
@felixfbecker
Copy link
Contributor

I have been holding back the PHP IntelliSense release with intelligent completion for over a month now because it is useless if VS Code just spams the completion list with countless functions that do not make sense in the context. When will we get the option to disable or uninstall the built-in PHP completion?

@mousetraps
Copy link
Contributor

I agree it makes sense to provide a temporary option to disable this. Just submitted a PR here: #19015

mousetraps added a commit to mousetraps/vscode that referenced this issue Jan 23, 2017
…ompletion provider.

This option enables extension authors providing PHP support to prevent needless duplication in the completions list. We can remove it when we extract this functionality to a separate extension.
@mousetraps
Copy link
Contributor

mousetraps commented Jan 23, 2017

Also @felixfbecker, just checking - at this point in time, is there anything else about the built-in support that is interfering with the experience? (hover, signature help, etc.?)

@felixfbecker
Copy link
Contributor

@mousetraps thanks for looking into this. Signature help is being implemented right now, iirc there is no built-in hover. It would be great if all of them could be disabled (everything except syntax highlighting).

aeschli added a commit that referenced this issue Jan 23, 2017
#9003 Provide temporary option to disable the built-in PHP completion provider
@aeschli
Copy link
Contributor

aeschli commented Jan 23, 2017

I pushed the PR and changes the setting name for the code completion to "php.suggest.basic".
If disabled, hover and signature help are also disabled.

@ADmad
Copy link

ADmad commented Jan 26, 2017

I don't think the php.suggest.basic setting achieves the desired result. Setting php.suggest.basic to false I no longer get completion suggestions even from PHP Intellisense extension. If have to use Ctrl + space to trigger the completion suggestions list.

@felixfbecker
Copy link
Contributor

@ADmad I don't think this is related, this only removes the built-in completion provider.

@ADmad
Copy link

ADmad commented Jan 26, 2017

So something else which controls the display of suggestions list from extensions like php intellisense is broken?

@aeschli aeschli modified the milestones: January 2017, Backlog Feb 3, 2017
@aeschli
Copy link
Contributor

aeschli commented Feb 3, 2017

Part of the January build.
Add
"php.suggest.basic": false
to your settings to disable all completions from the built-in PHP extension.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality php PHP support issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants