Success! Looks like we've fixed this one. According to our records the fix was applied for a bug in our system in a recent build.
  1. #1
    Sencha Premium Member abeyer's Avatar
    Join Date
    Jan 2012
    Location
    Germany
    Posts
    47
    Vote Rating
    1
    abeyer is on a distinguished road

      0  

    Default Wrong column in beforeedit event of a row-editable grid with a locked action column

    Wrong column in beforeedit event of a row-editable grid with a locked action column


    REQUIRED INFORMATION

    Ext version tested:
    • Ext 4.2.1
    Browser versions tested against:
    • IE9
    • FF 21
    • Chrome 27
    Description:
    • Wrong context.column is sent to "beforeedit" event of a row-editable grid panel with a locked action columns and row editor form focuses on a wrong field.
    Steps to reproduce the problem:
    • Create a grid panel with row-editing plugin and a locked action column and a listener for beforeedit event. Then click on a row to start editing.
    The result that was expected:
    • the correct field be focused and correct context.column be sent to beforeedit event.
    The result that occurs instead:
    • the correct field is not focused and wrong context.column is sent to beforeedit event.
    Test Case:

    Code:
        var json = [{ name: 'Bicycle',
                color: 'red',
            }, {
                name: 'Table',
                color: 'blue',
            }, {
                name: 'Pen',
                color: 'yellow',
            }, {
                name: 'Shoe',
                color: 'green',
            }];
    
    
    Ext.define('MyApp.model.Article', {
        extend: 'Ext.data.Model',
        fields: ['name', 'color'],
    });
    
    
    var store = Ext.create('Ext.data.Store', {
        requires: ['MyApp.model.Article'],
        autoLoad: true,
        model: 'MyApp.model.Article',
        proxy: {
            type: 'memory',
            data: json,
            reader: { type: 'json' }
        }
    });
    
    
    var rowEditing = Ext.create('Ext.grid.plugin.RowEditing', {
        clicksToEdit: 1,
        autoCancel: false,
        clicksToMoveEditor: 1,
    });
    
    
    var title = 'Wrong context.column sent to beforeedit event of <BR />row-editable grid with a locked action columns! <BR />Click to edit a row to see the problem.<BR />'
    
    
    var grid = Ext.create('Ext.grid.Panel', {
        renderTo: Ext.getBody(),
        height: 300,
        width: 450,
        title: title,
        store: store,
        plugins: [rowEditing],
        columns: [{
            xtype: 'actioncolumn',
            width: 120,
            text: 'Action',
            locked: true,
            items: [{
                icon: 'extjs/examples/shared/icons/fam/cog_edit.png',
                text: 'Inline edit'
            }]
        }, {
            width: 200,
            dataIndex: 'name',
            text: 'Name',
            editor: { xtype: 'textfield' }
        }, {
            xtype: 'gridcolumn',
            width: 80,
            dataIndex: 'color',
            text: 'Color',
            editor: {
                xtype: 'textfield',
            }
        }]
    });
    
    
    grid.on('beforeedit', function (editor, context, eOpts) {
        grid.setTitle(title + 'You clicked on ' + context.column.text + ' column to edit');
        return true;
    });
    HELPFUL INFORMATION


    Screenshot or Video:See this URL for live test case: http://jsfiddle.net/zHr3D/


    Debugging already done:
    • none
    Possible fix:
    • not provided
    Additional CSS used:
    • only default ext-all.css
    Operating System:
    • Win 7
    Last edited by abeyer; 30 Jun 2013 at 11:40 PM. Reason: typo

  2. #2
    Sencha - Support Team slemmon's Avatar
    Join Date
    Mar 2009
    Location
    Boise, ID
    Posts
    6,062
    Vote Rating
    215
    slemmon has much to be proud of slemmon has much to be proud of slemmon has much to be proud of slemmon has much to be proud of slemmon has much to be proud of slemmon has much to be proud of slemmon has much to be proud of slemmon has much to be proud of slemmon has much to be proud of

      0  

    Default


    Thanks for the bug report!
    I tested in 4.2.1 and see the issue, but it looks like this has been fixed for the next version.


    Are you a Sencha products veteran who has wondered what it might be like to work at Sencha? If so, please reach out to our recruiting manager:
    sheryl@sencha.com

Thread Participants: 1