Opened 7 years ago
Closed 7 years ago
#17514 closed defect (patchwelcome)
_onSet function only renderRow
Reported by: | Chandler | Owned by: | Chandler |
---|---|---|---|
Priority: | undecided | Milestone: | tbd |
Component: | DojoX Grid | Version: | 1.9.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Hello, i work with a DataGrid?, ObjectStore? and a Observable Store. After i call the put function on the Store no item in DataGrid? is updated. The newValue in _onSet function should be set because after the notify from Observable the _onSet function in DataGrid? is called.
Old:
_onSet: function(item, attribute, oldValue, newValue){ this._checkUpdateStatus(); var idx = this.getItemIndex(item); if(idx>-1){ this.updateRow(idx); } }
New:
_onSet: function(item, attribute, oldValue, newValue){ this._checkUpdateStatus(); var idx = this.getItemIndex(item); if(idx>-1){ var item = this.getItem(idx); item[attribute]=newValue; this.updateRow(idx); } }
Attachments (2)
Change History (5)
comment:1 Changed 7 years ago by
Owner: | set to Chandler |
---|---|
Status: | new → pending |
comment:2 Changed 7 years ago by
Status: | pending → new |
---|
Attachment (WebContent?.zip) added by ticket reporter.
comment:3 Changed 7 years ago by
Resolution: | → patchwelcome |
---|---|
Status: | new → closed |
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.
Note: See
TracTickets for help on using
tickets.
Can you attach a test case using the attach file button? I'm sure we've tested store updates in general and they are (or were) working.