Opened 7 years ago

Closed 6 years ago

#15048 closed defect (patchwelcome)

DataGrid can not submit alwaysEditing rows into store using grid.edit.apply()

Reported by: zhuxw Owned by: Evan
Priority: undecided Milestone: tbd
Component: DojoX Grid Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

In alwaysEditing mode, the grid.edit.apply does not work. The store is not updated at all. I debugged and found the function doApplyEdit in DataGrid?:

doApplyEdit: function(inRowIndex, inDataAttr){

var cache = this._cache[inRowIndex]; /*if(cache){

var data = this.getItem(inRowIndex); if(this.store.getValue(data, inDataAttr) != cache){

this.update(cache, data, inRowIndex);

} delete this._cache[inRowIndex];

}*/ this.onApplyEdit(inRowIndex);

},

The update part is commented out, and there's no store.setValue() used.

This issue comes from the following post: http://dojo-toolkit.33424.n3.nabble.com/How-to-get-correct-value-when-alwaysEditing-true-in-EnhancedGrid-tp3838940p3838940.html

Test link: http://jsfiddle.net/AsYWR/33/

Change History (2)

comment:1 Changed 6 years ago by 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.

comment:2 Changed 6 years ago by bill

Resolution: patchwelcome
Status: newclosed
Note: See TracTickets for help on using tickets.