Fix potential npe

mjbvz · mjbvz · commit 75d370ca5d1a · 2018-07-12T16:45:18.000-07:00
Related to microsoft#53830
diff --git a/src/vs/editor/contrib/codeAction/codeActionCommands.ts b/src/vs/editor/contrib/codeAction/codeActionCommands.ts
@@ -122,7 +122,9 @@ export class QuickFixController implements IEditorContribution {
 	}
 
 	private _handleLightBulbSelect(coords: { x: number, y: number }): void {
-		this._codeActionContextMenu.show(this._lightBulbWidget.model.actions, coords);
+		if (this._lightBulbWidget.model.actions) {
+			this._codeActionContextMenu.show(this._lightBulbWidget.model.actions, coords);
+		}
 	}
 
 	public triggerFromEditorSelection(filter?: CodeActionFilter, autoApply?: CodeActionAutoApply): Thenable<CodeAction[] | undefined> {

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,9 @@ export class QuickFixController implements IEditorContribution {`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`private _handleLightBulbSelect(coords: { x: number, y: number }): void {`
`125`		`- this._codeActionContextMenu.show(this._lightBulbWidget.model.actions, coords);`
	`125`	`+ if (this._lightBulbWidget.model.actions) {`
	`126`	`+ this._codeActionContextMenu.show(this._lightBulbWidget.model.actions, coords);`
	`127`	`+ }`
`126`	`128`	`}`
`127`	`129`
`128`	`130`	`public triggerFromEditorSelection(filter?: CodeActionFilter, autoApply?: CodeActionAutoApply): Thenable<CodeAction[] \| undefined> {`