fixed bug that assumed minichartModel is always set

rueckstiess · rueckstiess · commit bb1447bfdadb · 2015-07-03T09:30:00.000+02:00
diff --git a/scout-ui/src/field-list/type-list-item.js b/scout-ui/src/field-list/type-list-item.js
@@ -50,8 +50,9 @@ module.exports = AmpersandView.extend({
   },
   template: require('./type-list-item.jade'),
   typeClicked: function() {
-    if (this.parent.parent.minichartModel.modelType !== this.model.modelType) {
-      this.parent.parent.switchView(this.model);
+    var fieldList = this.parent.parent;
+    if (!fieldList.minichartModel || (fieldList.minichartModel.modelType !== this.model.modelType)) {
+      fieldList.switchView(this.model);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -50,8 +50,9 @@ module.exports = AmpersandView.extend({`
`50`	`50`	`},`
`51`	`51`	`template: require('./type-list-item.jade'),`
`52`	`52`	`typeClicked: function() {`
`53`		`- if (this.parent.parent.minichartModel.modelType !== this.model.modelType) {`
`54`		`- this.parent.parent.switchView(this.model);`
	`53`	`+ var fieldList = this.parent.parent;`
	`54`	`+ if (!fieldList.minichartModel \|\| (fieldList.minichartModel.modelType !== this.model.modelType)) {`
	`55`	`+ fieldList.switchView(this.model);`
`55`	`56`	`}`
`56`	`57`	`}`
`57`	`58`