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

OutputChannel.hide throws error #377

Closed
SamVerschueren opened this issue Nov 20, 2015 · 5 comments
Closed

OutputChannel.hide throws error #377

SamVerschueren opened this issue Nov 20, 2015 · 5 comments
Assignees
Milestone

Comments

@SamVerschueren
Copy link
Contributor

It seems that OutputChannel.hide() is throwing an error when called.

This is the error I'm receiving:

{ exception: null,
  error: 
   { stack: 'TypeError: Cannot read property \'getVisibleEditors\' of undefined\n    at e.close (out/vs/workbench/workbench.main.js:1661:15751)\n    at t.e.handle (out/vs/workbench/workbench.main.js:1613:7048)\n    at a (out/vs/workbench/workbench.main.js:1594:19297)\n    at Object.h.handle (out/vs/workbench/workbench.main.js:1594:19861)\n    at out/vs/workbench/workbench.main.js:1675:13\n    at ChildProcess.<anonymous> (out/vs/workbench/workbench.main.js:1675:2055)\n    at emitTwo (events.js:87:13)\n    at ChildProcess.emit (events.js:172:7)\n    at handleMessage (internal/child_process.js:685:10)\n    at Pipe.channel.onread (internal/child_process.js:440:11)',
     message: 'TypeError: Cannot read property \'getVisibleEditors\' of undefined',
     count: 1 },
  promise: 
   { _oncancel: null,
     _nextState: null,
     _state: 
      { name: 'error',
        enter: [Function],
        cancel: [Function: r],
        done: null,
        then: null,
        _completed: [Function: r],
        _error: [Function: r],
        _notify: [Function: f],
        _progress: [Function: r],
        _setCompleteValue: [Function: r],
        _setErrorValue: [Function: r] },
     _value: 
      { stack: 'TypeError: Cannot read property \'getVisibleEditors\' of undefined\n    at e.close (out/vs/workbench/workbench.main.js:1661:15751)\n    at t.e.handle (out/vs/workbench/workbench.main.js:1613:7048)\n    at a (out/vs/workbench/workbench.main.js:1594:19297)\n    at Object.h.handle (out/vs/workbench/workbench.main.js:1594:19861)\n    at out/vs/workbench/workbench.main.js:1675:13\n    at ChildProcess.<anonymous> (out/vs/workbench/workbench.main.js:1675:2055)\n    at emitTwo (events.js:87:13)\n    at ChildProcess.emit (events.js:172:7)\n    at handleMessage (internal/child_process.js:685:10)\n    at Pipe.channel.onread (internal/child_process.js:440:11)',
        message: 'TypeError: Cannot read property \'getVisibleEditors\' of undefined',
        count: 1 },
     _isException: false,
     _errorId: 2,
     done: [Function],
     then: [Function] },
  handler: undefined,
  id: 2,
  parent: undefined }

// @bpasero

@bpasero bpasero added this to the Nov 2015 - Recovery milestone Nov 20, 2015
@bpasero
Copy link
Member

bpasero commented Nov 20, 2015

@jrieken I can reproduce, somehow the MainThreadOutputService is created with an undefined editor service (the output service is there though).

@SamVerschueren
Copy link
Contributor Author

It's odd that it didn't popup in the closed beta phase. Never used .hide() though.

@SamVerschueren
Copy link
Contributor Author

Awesome! Great job.

@egamma egamma modified the milestones: Nov 2015 - end, Dec 2015 Nov 23, 2015
@natewallace
Copy link

Has this fix been tested successfully? I'm running version 0.10.3 of the app which appears would contain the fix applied in c3094f3 but I still get the undefined error described in this issue when calling the OutputChannel.hide function.

@bpasero
Copy link
Member

bpasero commented Dec 4, 2015

Fix should be in our upcoming December release.

@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
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants