Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-6724 in a recent build.
  1. #1
    Ext JS Premium Member skullbooks's Avatar
    Join Date
    Nov 2010
    Location
    Dillenburg, Hessen, Germany
    Posts
    35
    Vote Rating
    1
    skullbooks is on a distinguished road

      0  

    Exclamation Wrong getEditor() call in RowModel.

    Wrong getEditor() call in RowModel.


    REQUIRED INFORMATIONExt version tested:
    • Ext 4.1.0 RC3
    Browser versions tested against:
    • Chrome 18
    Description:
    The Ext.selection.RowModel#onEditorTab calles getEditor without parameter. As in the documentation in Ext.grid.colum.Column#getEditor there should be the record and the default field.

    Test Case:


    In our own column i need the record for my column editor and since ExtJS 4.1.0 RC2 i don't get the editor on TABing into the column editor.

    HELPFUL INFORMATION

    Debugging already done:
    • Wrong getEditor() call in RowModel.
    Possible fix:
    • think the solution should be a while like this
    • Code:
      while(position && !view.headerCt.getHeaderAtIndex(position.column).getEditor(record, header));
    Additional CSS used:
    • only default ext-all.css


  2. #2
    Sencha - Support Team scottmartin's Avatar
    Join Date
    Jul 2010
    Location
    Houston, Tx
    Posts
    8,650
    Vote Rating
    402
    scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future scottmartin has a brilliant future

      0  

    Default


    Have you tried 4.1 GA (final)?

    Regards,
    Scott.

  3. #3
    Ext JS Premium Member skullbooks's Avatar
    Join Date
    Nov 2010
    Location
    Dillenburg, Hessen, Germany
    Posts
    35
    Vote Rating
    1
    skullbooks is on a distinguished road

      0  

    Default


    now i did it, but it's still not working - well
    Ext.selection.RowModel#onEditorTab is caling getEditor without any parameter so the but for me still has to be there

  4. #4
    Ext JS Premium Member skullbooks's Avatar
    Join Date
    Nov 2010
    Location
    Dillenburg, Hessen, Germany
    Posts
    35
    Vote Rating
    1
    skullbooks is on a distinguished road

      0  

    Default


    will this be a bug in next time or can't i assume the parameter for getEditor in my own column as they are documented?

  5. #5
    Ext Premium Member yyogev's Avatar
    Join Date
    Jun 2009
    Location
    Shoham, Israel
    Posts
    196
    Vote Rating
    6
    yyogev is on a distinguished road

      0  

    Exclamation


    Quote Originally Posted by scottmartin View Post
    Have you tried 4.1 GA (final)?

    Regards,
    Scott.
    This code is still not fixed in ExtJS 4.1.0:
    (ext-all-debug.js, line 84254, or rowModel.js line 442)
    Last edited by yyogev; 3 Jul 2012 at 10:26 PM. Reason: give pointer to code
    Yaron Yogev

    IT Software Developer

  6. #6
    Ext JS Premium Member skullbooks's Avatar
    Join Date
    Nov 2010
    Location
    Dillenburg, Hessen, Germany
    Posts
    35
    Vote Rating
    1
    skullbooks is on a distinguished road

      0  

    Default


    also not fixed in EXT 4.1.1 RC2

    seems to me like sencha sees no need to fix that

    maybe it's helpful to write a line where this makes problems:

    if you write your own colum with a special editor that needs the record, i see the only way in implementing the getEditor in the column and give the selected record to the editor instance.
    but in the case you edit the grid by switching columns with TAB, you don't get the record. so you cant update the editor that is depending on some current values of the record.

  7. #7
    Ext Premium Member yyogev's Avatar
    Join Date
    Jun 2009
    Location
    Shoham, Israel
    Posts
    196
    Vote Rating
    6
    yyogev is on a distinguished road

      0  

    Default Workaround for this problem

    Workaround for this problem


    Code:
    if (record == undefined)
        record = ext.getCmp("my_grid").plugins[0].getActiveRecord();
    Yaron Yogev

    IT Software Developer

Thread Participants: 2

film izle

hd film izle

film sitesi

takipci kazanma sitesi

takipci kazanma sitesi

güzel olan herşey

takipci alma sitesi

komik eğlenceli videolar