Merge pull request microsoft#4614 from Microsoft/ben/4576

bpasero · bpasero · commit 4d4831f35f1b · 2016-03-23T16:41:28.000+01:00
Cannot read property 'dispose' of undefined (fixes microsoft#4573)
diff --git a/src/vs/base/parts/quickopen/browser/quickOpenModel.ts b/src/vs/base/parts/quickopen/browser/quickOpenModel.ts
@@ -650,21 +650,21 @@ class Renderer implements IRenderer<QuickOpenEntry> {
 	public disposeTemplate(templateId: string, templateData: any): void {
 		if (templateId === templateEntryItem) {
 			this.entryItemRenderer.disposeTemplate(null, templateId, templateData);
+		} else {
+			const data = templateData as IQuickOpenEntryGroupTemplateData;
+			data.actionBar.dispose();
+			data.actionBar = null;
+			data.container = null;
+			data.description.dispose();
+			data.description = null;
+			data.detail.dispose();
+			data.detail = null;
+			data.group = null;
+			data.icon = null;
+			data.label.dispose();
+			data.label = null;
+			data.prefix = null;
 		}
-
-		const data = templateData as IQuickOpenEntryGroupTemplateData;
-		data.actionBar.dispose();
-		data.actionBar = null;
-		data.container = null;
-		data.description.dispose();
-		data.description = null;
-		data.detail.dispose();
-		data.detail = null;
-		data.group = null;
-		data.icon = null;
-		data.label.dispose();
-		data.label = null;
-		data.prefix = null;
 	}
 }
 
