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
    5,160
    Vote Rating
    190
    slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold

      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

    C
    heck out all of the 2013 SenchaCon presentations here:
    http://www.sencha.com/blog/senchacon...now-available/

Thread Participants: 1