Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha User
    Join Date
    Sep 2012
    Posts
    12
    Vote Rating
    0
    TopoFurnier is on a distinguished road

      0  

    Default CellEditing throws exception on locked grid

    CellEditing throws exception on locked grid


    REQUIRED INFORMATION
    Ext version tested:
    • Ext 4.1.0-gpl
    Browser versions tested against:
    • Chrome 22
    Operating System:
    • Win7 64bits
    Description:
    • Uses an Editing plugin in a grid with locked columns throws exception on grid creation. If said plugin is removed OR locked set to false there is no issue.
    Steps to reproduce the problem:
    • Create a grid with at least one locked column and CellEditing(or RowEditing) plugin.
    The result that was expected:
    • An editable locked grid should be created.
    The result that occurs instead:
    • On page load the following exception is thrown:
      • Uncaught TypeError: Cannot read property 'isCellModel' of undefined
    Test Case:
    Code:
    Ext.define('MVCapp.view.user.List' ,{
        extend: 'Ext.grid.Panel',
        alias : 'widget.simpsonlist',    
        title:'Simpsons',
            store      : Ext.create('Ext.data.Store', {
            storeId : 'simpsonsStore',
            fields  : ['name', 'email', 'change'],
            data    : {'items' : [
                { 'name' : 'Lisa',  'email' : 'lisa@simpsons.com',  'change' : 100  },
                { 'name' : 'Bart',  'email' : 'bart@simpsons.com',  'change' : -20  },
                { 'name' : 'Homer', 'email' : 'home@simpsons.com',  'change' : 23   },
                { 'name' : 'Marge', 'email' : 'marge@simpsons.com', 'change' : -11  }
            ]},
            proxy   : {
                type   : 'memory',
                reader : {
                    type : 'json',
                    root : 'items'
                }
            }
        }),
        plugins: [Ext.create('Ext.grid.plugin.RowEditing')],
        height     : 200,
        width      : 400,
        columns    : [
            { header : 'Name', dataIndex : 'name', locked: true },
            { header : 'Email', dataIndex : 'email', flex : 1 },
            { header : 'Change', dataIndex : 'change' }
        ]
    }); 
    
    Ext.application({
            name : 'MVCapp',
            appFolder : 'app',
            requires : ['Ext.container.Viewport',
                'MVCapp.view.user.List'
            ],
            launch : function() {
                Ext.create('Ext.container.Viewport', {
                        layout : 'border',    
                        items : [{
                            xtype : 'simpsonlist',
                            region : 'center',
                            bodyStyle : 'padding:1px'
                        }]
                    });
                }
    });
    

  2. #2
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,145
    Vote Rating
    673
    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


    Thanks for the test case, this issue is already resolved in the latest release.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

Thread Participants: 1