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

Cannot convert undefined or null to object #57007

Closed
isidorn opened this issue Aug 22, 2018 · 5 comments · Fixed by #57491
Closed

Cannot convert undefined or null to object #57007

isidorn opened this issue Aug 22, 2018 · 5 comments · Fixed by #57491
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Aug 22, 2018

  1. Start debugging
  2. Have the loaded scripts view open
  3. Stop debugging, notice exception in the console
  4. I can not reproduce with Stable so it might be that my listener refactoring caused this

If you pause on exception you will see that this call caused it.

  ERR Cannot convert undefined or null to object: TypeError: Cannot convert undefined or null to object
    at Function.keys (<anonymous>)
    at file:///Users/isidor/Development/vscode/out/vs/base/parts/tree/browser/treeView.js:606:35
    at file:///Users/isidor/Development/vscode/out/vs/base/common/async.js:272:28
    at Object.notifySuccess [as _notify] (file:///Users/isidor/Development/vscode/out/vs/base/common/winjs.base.js:1191:59)
    at Object.enter (file:///Users/isidor/Development/vscode/out/vs/base/common/winjs.base.js:867:30)
    at Promise_ctor._run (file:///Users/isidor/Development/vscode/out/vs/base/common/winjs.base.js:1089:29)
    at Promise_ctor._completed (file:///Users/isidor/Development/vscode/out/vs/base/common/winjs.base.js:1057:18)
    at file:///Users/isidor/Development/vscode/out/vs/base/common/async.js:277:23
errors.js:42 WARNING: Promise with no error callback:24
errors.js:43 {exception: TypeError: Cannot convert undefined or null to object
    at Function.keys (<anonymous>)
    at file…, error: null, promise: _…e.C…s.d…e._creator, handler: undefined, id: 24, …}
errors.js:45 TypeError: Cannot convert undefined or null to object
    at Function.keys (<anonymous>)
    at treeView.js:606
    at async.js:272
    at Object.notifySuccess [as _notify] (winjs.base.js:1191)
    at Object.enter (winjs.base.js:867)
    at Promise_ctor._run (winjs.base.js:1089)
    at Promise_ctor._completed (winjs.base.js:1057)
    at async.js:277
@isidorn isidorn added bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues labels Aug 22, 2018
@isidorn
Copy link
Contributor Author

isidorn commented Aug 28, 2018

@weinand did you have a chance to look into this? Or would you like me to investigate?
I suspect this is breaking the loaded scripts view and is the root cause for #57401

@weinand weinand added this to the August 2018 milestone Aug 28, 2018
@weinand
Copy link
Contributor

weinand commented Aug 28, 2018

no, please investigate since this and #57401 was working before, and I haven't touched this code for two weeks.

@weinand
Copy link
Contributor

weinand commented Aug 28, 2018

@isidorn end session events are not arriving in https://github.com/Microsoft/vscode/blob/93e30cc319bb2184756819aefbd4f16a27e7a17c/src/vs/workbench/parts/debug/browser/loadedScriptsView.ts#L388

As a consequence the tree for the session is not properly removed.

@weinand weinand removed their assignment Aug 28, 2018
@isidorn
Copy link
Contributor Author

isidorn commented Aug 29, 2018

I see something differnetly, the end sessions are arriving however when the last session end arrives view starts refreshing the tree and while that is in progress the view gets disposed which throws an exception. Looking into a fix...

@isidorn
Copy link
Contributor Author

isidorn commented Aug 29, 2018

I have created a PR which address this issue #57491

@bpasero bpasero added the verified Verification succeeded label Aug 30, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants