DataGrid when after cell edit enter pressed fires onClickEvent with undefined rowIndex

[Marcin Gołębski]

Scenario:

1. DataGrid?, one text cell in edition state.
2. press Enter
3. _Events.onKeyDown fires and the following part of this function doesn't work properly:

if (!this.edit.isEditing()){
  var curView = this.focus.focusView || this.views.views[0];  //if no focusView than only one view
  curView.content.decorateEvent(e);
  this.onRowClick(e);

eg. curView.content.decorateEvent(e); doesn't set e.rowIndex.

I we change this block to the following:

if (!this.edit.isEditing()){
  var curView = this.focus.focusView || this.views.views[0];  //if no focusView than only one view
  curView.content.decorateEvent(e);
  if(!e.rowIndex) {
    e.rowIndex = this.focus.rowIndex;
  }
  this.onRowClick(e);

it works fine. Obviously this is not a solution.

I don't know if it is helpful but I found thad the method _Builder.findTarget:

findTarget: function(inSource, inTag){
  var n = inSource;
  while(n && (n!=this.domNode) && (!(inTag in n) || (gridViewTag in n && n[gridViewTag] != this.view.id))){
    n = n.parentNode;
  }
  return (n != this.domNode) ? n : null;
},

has this.domNode == undefined during curView.content.decorateEvent(e); call.

[bill]

DojoX Grid and EnhancedGrid are deprecated in favor of ​dgrid and ​gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.