[FIXED][2.x] DataView tpl config option bug

[slobo]

the DataView's tpl config option
does not accept an array of strings (HTML fragments)
as stated in the docs
(can only accept a string or an XTemplate - this can also clearly be seen from the source code of DataView)

(I think this should be handled as a programming bug, not as a docs problem)

[aconran]

slobo -

Thanks for the bug report. We've added this into our issue tracker and will take a look at it.

Thanks,

[mystix]

this bug's still present in 2.2.

bugfix for this is trivial:

Ext.override(Ext.DataView, {
    // private
    initComponent : function() {
        Ext.DataView.superclass.initComponent.call(this);
        if (typeof this.tpl == "string" || Ext.type(this.tpl) == "array") {
            this.tpl = new Ext.XTemplate(this.tpl);
        }

        this.addEvents(
            "beforeclick",
            "click",
            "mouseenter",
            "mouseleave",
            "containerclick",
            "dblclick",
            "contextmenu",
            "selectionchange",
            "beforeselect"
        );

        this.all = new Ext.CompositeElementLite();
        this.selected = new Ext.CompositeElementLite();
    }
});

[Condor]

Maybe, instead, we should check if it is not a usable template class:

Ext.override(Ext.DataView, {
	initComponent : function(){
		Ext.DataView.superclass.initComponent.call(this);
		if(this.tpl && !this.tpl.overwrite){
			this.tpl = new Ext.XTemplate(this.tpl);
		}
		this.addEvents(
			"beforeclick",
			"click",
			"mouseenter",
			"mouseleave",
			"containerclick",
			"dblclick",
			"contextmenu",
			"selectionchange",
			"beforeselect"
		);
		this.all = new Ext.CompositeElementLite();
		this.selected = new Ext.CompositeElementLite();
	}
});

[evant]

Fixed in SVN.