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

Deprecated code completions crash #8310

Closed
amiramw opened this issue Aug 5, 2020 · 1 comment · Fixed by #8517
Closed

Deprecated code completions crash #8310

amiramw opened this issue Aug 5, 2020 · 1 comment · Fixed by #8517
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help vscode issues related to VSCode compatibility

Comments

@amiramw
Copy link
Member

amiramw commented Aug 5, 2020

Bug Description:

When providing code completion result with deprecated: true property there is an error on code completion and no suggestions are displayed.

Steps to Reproduce:

lsp-sample-deprecated.zip contains the vscode extension source code for reference.
vsix.zip contains two vsix files lsp-sample-deprecated-item-1.0.0.vsix that provides deprecated code completion. It works on vscode but not on theia. The other one provide code completions without deprecated items and it work on both.

  1. Deploy vscode extension lsp-sample-deprecated-item-1.0.0.vsix
  2. Create empty text file
  3. Try to do code completion.

Nothing works and the browser console shows this error:

Uncaught Error: Cannot read property 'Deprecated' of undefined

TypeError: Cannot read property 'Deprecated' of undefined
at asCompletionItem (c:\Users\i066470\AppData\Local\Temp\vscode-unpacked\lsp-sample-deprecated-item-1.0.0.vsix\extension\client\node_modules\vscode-languageclient\lib\protocolConverter.js:244:50)
at Array.map ()
at asCompletionResult (c:\Users\i066470\AppData\Local\Temp\vscode-unpacked\lsp-sample-deprecated-item-1.0.0.vsix\extension\client\node_modules\vscode-languageclient\lib\protocolConverter.js:171:26)
at http://localhost:3000/vs/editor/editor.main.js:1532:31

On VS code it works like this:

vsc

Additional Information

  • Operating System: windows, debian
  • Theia Version: master
@vince-fugnitto vince-fugnitto added help wanted issues meant to be picked up, require help vscode issues related to VSCode compatibility labels Aug 5, 2020
@akosyakov akosyakov added the bug bugs found in the application label Aug 5, 2020
@keka-star

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help vscode issues related to VSCode compatibility
Projects
None yet
4 participants