PDA

View Full Version : [FIXED] New bug in RowExpander plugin



pwuk
25 Jul 2011, 6:16 AM
REQUIRED INFORMATION Ext version tested:

Ext 4.02 rev a

Browser versions tested against:

____
Chrome 12
FF5 (firebug 1.3.0.10 installed)
IE6

Description:

In row expander - clicking + throws an error
"this.view is undefined
var rowNode = this.view.getNode(rowIdx)"
Was ok in previous version.

Steps to reproduce the problem:

Goto http://dev.sencha.com/deploy/ext-4.0.2a/examples/grid/grid-plugins.html
click a + icon

The result that was expected:

Shows text


The result that occurs instead:

error

Test Case:

http://dev.sencha.com/deploy/ext-4.0.2a/examples/grid/grid-plugins.html HELPFUL INFORMATION Screenshot or Video:

attached

See this URL for live test case: http://dev.sencha.com/deploy/ext-4.0.2a/examples/grid/grid-plugins.html Debugging already done:

none

Possible fix:

not provided

Additional CSS used:

only default ext-all.css
custom css (include details)

Operating System:

WinXP Pro

pwuk
5 Aug 2011, 6:49 AM
RowExpander.js : -

toggleRow: function(rowIdx) {
var rowNode = this.cmp.view.getNode(rowIdx),
row = Ext.get(rowNode),
nextBd = Ext.get(row).down(this.rowBodyTrSelector),
record = this.cmp.view.getRecord(rowNode);


if (row.hasCls(this.rowCollapsedCls)) {
row.removeCls(this.rowCollapsedCls);
nextBd.removeCls(this.rowBodyHiddenCls);
this.recordsExpanded[record.internalId] = true;
this.cmp.view.fireEvent('expandbody', rowNode, record, nextBd.dom);
} else {
row.addCls(this.rowCollapsedCls);
nextBd.addCls(this.rowBodyHiddenCls);
this.recordsExpanded[record.internalId] = false;
this.cmp.view.fireEvent('collapsebody', rowNode, record, nextBd.dom);
}
this.view.up('gridpanel').invalidateScroller();
},




In the bold lines I changed this.view to this.cmp.view

mike.estes
5 Aug 2011, 3:47 PM
this is already fixed in source, will be present in the next release