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

Uncaught TypeError: Cannot read property 'uri' of null #41080

Closed
vscodeerrors opened this issue Jan 3, 2018 · 3 comments
Closed

Uncaught TypeError: Cannot read property 'uri' of null #41080

vscodeerrors opened this issue Jan 3, 2018 · 3 comments
Assignees
Labels
error-telemetry VS Code - Issues generated by telemetry
Milestone

Comments

@joaomoreno joaomoreno added this to the December 2017/January 2018 milestone Jan 8, 2018
@joaomoreno joaomoreno added bug Issue identified by VS Code Team member as probable bug error-telemetry VS Code - Issues generated by telemetry labels Jan 8, 2018
@KamilSzot
Copy link
Contributor

KamilSzot commented Jan 22, 2018

also present on 1.19.2, prevented me from installing or even viewing cjsx extension

Caused by assuming here: e6b421a#diff-dcdf54eabe92fa9ae7150f090a9f809eR128
that: this.gallery.assets.repository is always an object and never null or undefined

And it is null because this if 232d83f#diff-5a704040f30e3eb4463336a47e6500cbR205 has no else fallback for case when version.properties is not defined. And properties is optional for version because it's of type IRawGalleryExtensionVersion bd1ada3#diff-5a704040f30e3eb4463336a47e6500cbR41

@KamilSzot
Copy link
Contributor

Fix is in the same style as special case for default icon if icon asset is missing.

@joaomoreno joaomoreno assigned sandy081 and unassigned joaomoreno Jan 26, 2018
@joaomoreno
Copy link
Member

@sandy081 I'm forwarding this and the respective PR to you, I'm sure you'll know immediately how to review it.

@sandy081 sandy081 removed the bug Issue identified by VS Code Team member as probable bug label Jan 29, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Mar 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
error-telemetry VS Code - Issues generated by telemetry
Projects
None yet
Development

No branches or pull requests

4 participants