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

[vscode] TreeItem fails to render if it has a command tied to it #5666

Closed
tetchel opened this issue Jul 8, 2019 · 3 comments
Closed

[vscode] TreeItem fails to render if it has a command tied to it #5666

tetchel opened this issue Jul 8, 2019 · 3 comments
Labels
bug bugs found in the application vscode issues related to VSCode compatibility

Comments

@tetchel
Copy link
Contributor

tetchel commented Jul 8, 2019

#4975 seems to have regressed since #5556, on TreeItems which have their command field set.

root ERROR [hosted-plugin: 73] [ERRO: 03:36:57.738 extension.js.on():43]: Unhandled promise rejection: TypeError: Converting circular structure to JSON
    at JSON.stringify (<anonymous>)
    at Function.MessageFactory.replyOK (/home/theia/node_modules/@theia/plugin-ext/lib/api/rpc-protocol.js:280:103)
    at /home/theia/node_modules/@theia/plugin-ext/lib/api/rpc-protocol.js:158:51
    at process._tickCallback (internal/process/next_tick.js:68:7)

You can see that I had to not set command on my TreeItems to work around this bug. eclipse-archived/codewind-vscode@92b5683

It also appears to be intermittent with some treeitems, but happens every time with others - I could not figure out what caused the difference.

@akosyakov since you fixed this originally

@tetchel tetchel changed the title TreeItem fails to render if it has a command tied to it [vscode] TreeItem fails to render if it has a command tied to it Jul 8, 2019
@akosyakov akosyakov added bug bugs found in the application vscode issues related to VSCode compatibility labels Jul 9, 2019
@akosyakov
Copy link
Member

Please have a look at it after #5665 gets merged, plugin trees were refactored to allow state restoration.

@tolusha
Copy link
Contributor

tolusha commented Aug 8, 2019

@tetchel Is this issue still reproducible since #5665 has been merged?

@tetchel
Copy link
Contributor Author

tetchel commented Oct 9, 2019

Sorry, I verified this like a month ago! It's fixed.

@tetchel tetchel closed this as completed Oct 9, 2019
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 vscode issues related to VSCode compatibility
Projects
None yet
Development

No branches or pull requests

3 participants