Hybrid View

    Looks like we cannot reproduce this. Please provide another test case to reproduce this issue.
  1. #1
    Ext JS Premium Member
    Join Date
    Apr 2007
    Posts
    295
    Vote Rating
    9
    jheid will become famous soon enough

      0  

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

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


    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

    PHP Code:
    this.getBenutzerView ().getStore ().insert (0, new User ({
       ....
       }));
    grid.getPlugin ('myRowEditingID').startEdit (00); 
    When I hit update, the request is send to the server but I get a null pointer exception in Ext.view.Table.updateColumns at

    PHP Code:
    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

    PHP Code:
    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

  2. #2
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,174
    Vote Rating
    676
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      0  

    Default


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

    Code:
    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);
        
    });
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

  3. #3
    Ext JS Premium Member
    Join Date
    Apr 2007
    Posts
    295
    Vote Rating
    9
    jheid will become famous soon enough

      0  

    Default


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

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

Thread Participants: 1