fix null checks

aeschli · aeschli · commit e6d41395d929 · 2019-02-04T16:37:53.000+01:00
diff --git a/src/vs/workbench/services/themes/electron-browser/colorThemeStore.ts b/src/vs/workbench/services/themes/electron-browser/colorThemeStore.ts
@@ -155,7 +155,7 @@ export class ColorThemeStore {
 		});
 	}
 
-	public findThemeDataByParentLocation(parentLocation: URI): Promise<ColorThemeData[]> {
+	public findThemeDataByParentLocation(parentLocation: URI | undefined): Promise<ColorThemeData[]> {
 		if (parentLocation) {
 			return this.getColorThemes().then(allThemes => {
 				return allThemes.filter(t => t.location && resources.isEqualOrParent(t.location, parentLocation));
diff --git a/src/vs/workbench/services/themes/electron-browser/fileIconThemeStore.ts b/src/vs/workbench/services/themes/electron-browser/fileIconThemeStore.ts
@@ -154,7 +154,7 @@ export class FileIconThemeStore {
 		});
 	}
 
-	public findThemeDataByParentLocation(parentLocation: URI): any {
+	public findThemeDataByParentLocation(parentLocation: URI | undefined): any {
 		if (parentLocation) {
 			return this.getFileIconThemes().then(allThemes => {
 				return allThemes.filter(t => t.location && resources.isEqualOrParent(t.location, parentLocation));

Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,7 @@ export class ColorThemeStore {`
`155`	`155`	`});`
`156`	`156`	`}`
`157`	`157`
`158`		`- public findThemeDataByParentLocation(parentLocation: URI): Promise<ColorThemeData[]> {`
	`158`	`+ public findThemeDataByParentLocation(parentLocation: URI \| undefined): Promise<ColorThemeData[]> {`
`159`	`159`	`if (parentLocation) {`
`160`	`160`	`return this.getColorThemes().then(allThemes => {`
`161`	`161`	`return allThemes.filter(t => t.location && resources.isEqualOrParent(t.location, parentLocation));`
Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,7 @@ export class FileIconThemeStore {`
`154`	`154`	`});`
`155`	`155`	`}`
`156`	`156`
`157`		`- public findThemeDataByParentLocation(parentLocation: URI): any {`
	`157`	`+ public findThemeDataByParentLocation(parentLocation: URI \| undefined): any {`
`158`	`158`	`if (parentLocation) {`
`159`	`159`	`return this.getFileIconThemes().then(allThemes => {`
`160`	`160`	`return allThemes.filter(t => t.location && resources.isEqualOrParent(t.location, parentLocation));`