[4.2.0.265] Bug with RowEditing in Ext.view.Table.updateColumns

[jheid]

The bug is very strange so I do my best to describe it.

I have a grid with two combobox editors in the columns.

To add a new row, I use

this.getBenutzerView ().getStore ().insert (0, new User ({
   ....
   }));
grid.getPlugin ('myRowEditingID').startEdit (0, 0); 

When I hit update, the request is send to the server but I get a null pointer exception in Ext.view.Table.updateColumns at

Ext.fly(oldCell).syncContent(newCell); 

because oldCell is null. In fact when investigating the DOM tree behind the RowEditing bar, the columns which uses a combobox are missing (no tds) when the exception rises.

I added a check here

if (!oldCell || !newCell) continue; 

and it seems to work fine then. After the RowEditing bar closes the dom is intact again.

I haven't tested it with older versions.

Cheers,
Jörn

[evant]

Can't reproduce it, please post a proper test case:

Ext.define('Thing', {
    extend: 'Ext.data.Model',
    fields: ['name']
});

Ext.require('*');

Ext.onReady(function(){
    
    var rowEditing = new Ext.grid.plugin.RowEditing();
    
    var store = new Ext.data.Store({
        model: Thing,
        data: []
    });
    
    var grid = new Ext.grid.Panel({
        renderTo: document.body,
        width: 400,
        height: 200,
        store: store,
        columns: [{
            dataIndex: 'name',
            flex: 1,
            text: 'Name',
            field: {
                xtype: 'textfield'
            }
        }],
        plugins: [rowEditing]
    });
    
    setTimeout(function(){
        store.add({
            name: 'Thing 1'
        });
        rowEditing.startEdit (0, 0);  
    }, 500);
    
});

[jheid]

Seems to be related to http://www.sencha.com/forum/showthre...ataIndex-usage

Using one column with one dataIndex seems to "fix" this.