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 read property 'resource' of undefined #42626

Closed
chrmarti opened this issue Jan 31, 2018 · 5 comments
Closed

Cannot read property 'resource' of undefined #42626

chrmarti opened this issue Jan 31, 2018 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug file-explorer Explorer widget issues verified Verification succeeded
Milestone

Comments

@chrmarti
Copy link
Contributor

Issue Type

Bug

Description

Testing #42195

Found after doing many individual renames on the workspace folders:

ERR Cannot read property 'resource' of undefined: TypeError: Cannot read property 'resource' of undefined
at Function.e.mergeLocalWithDisk (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:2348:409)
at file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:4433:577
at Array.forEach (native)
at file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:4433:537
at Object.g [as _notify] (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:140:868)
at Object.enter (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:144:153)
at n.Class.derive._oncancel._run (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:145:970)
at n.Class.derive._oncancel._completed (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:145:411)
at l (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:148:888)
at j.then.o.(anonymous function) (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:149:78)

VS Code Info

VS Code version: Code - Insiders 1.20.0-insider (010de7d, 2018-01-31T09:45:32.303Z)
OS version: Linux x64 3.13.0-108-generic

System Info
Item Value
CPUs Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (4 x 2194)
Load (avg) 1, 1, 0
Memory (System) 3.86GB (0.29GB free)
Process Argv /usr/share/code-insiders/code-insiders --unity-launch
Screen Reader no
VM 100%
Extensions: none Reproduces only with extensions
@bpasero bpasero assigned isidorn and unassigned bpasero Jan 31, 2018
@bpasero bpasero added the file-explorer Explorer widget issues label Jan 31, 2018
@bpasero
Copy link
Member

bpasero commented Jan 31, 2018

Seems to originate from the explorer: mergeLocalWithDisk

@isidorn
Copy link
Contributor

isidorn commented Jan 31, 2018

All I can do is null guard for this. Which is what I just did

@isidorn isidorn added the bug Issue identified by VS Code Team member as probable bug label Jan 31, 2018
@isidorn isidorn added this to the January 2018 milestone Jan 31, 2018
@bpasero
Copy link
Member

bpasero commented Feb 1, 2018

@isidorn is that a smart fix? shouldn't you check where on the calling side you can get a NULL for the stat?

isidorn added a commit that referenced this issue Feb 1, 2018
@isidorn
Copy link
Contributor

isidorn commented Feb 1, 2018

@bpasero thanks. I have addressed your concern via 45e95c6

@bpasero
Copy link
Member

bpasero commented Feb 1, 2018

@isidorn great thanks

@chrmarti chrmarti added the verified Verification succeeded label Feb 2, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Mar 17, 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 file-explorer Explorer widget issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants