Results 1 to 3 of 3

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

Hybrid View

Previous Post Previous Post   Next Post Next Post
    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
    299
    Vote Rating
    13
      0  

    Default [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,
    Jrn

  2. #2
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    18,590
    Vote Rating
    873
      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
    Twitter - @evantrimboli

  3. #3
    Ext JS Premium Member
    Join Date
    Apr 2007
    Posts
    299
    Vote Rating
    13
      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.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •