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
    289
    Vote Rating
    0
    jheid is on a distinguished road

      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
    16,785
    Vote Rating
    597
    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
    289
    Vote Rating
    0
    jheid is on a distinguished road

      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

Turkiyenin en sevilen filmlerinin yer aldigi xnxx internet sitemiz olan ve porn sex tarzi bir site olan mobil porno izle sitemiz gercekten dillere destan bir durumda herkesin sevdigi bir site olarak tarihe gececege benziyor. Sitenin en belirgin ozelliklerinden birisi de Turkiyede gercekten kaliteli ve muntazam, duzenli porno izle siteleri olmamasidir. Bu yuzden iste. Ayrica en net goruntu kalitesine sahip adresinde yayinlanmaktadir. Mesela diğer sitelerimizden bahsedecek olursak, en iyi hd porno video arşivine sahip bir siteyiz. "The Best anal porn videos and slut anus, big asses movies set..."