Opened 6 years ago

Closed 6 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)

WebContent.zip (69.2 KB) - added by Chandler 6 years ago.
Complete Project
WebContent.2.zip (69.2 KB) - added by Chandler 6 years ago.
all custom classes

Download all attachments as: .zip

Change History (5)

comment:1 Changed 6 years ago by bill

Owner: set to Chandler
Status: newpending

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.

Changed 6 years ago by Chandler

Attachment: WebContent.zip added

Complete Project

comment:2 Changed 6 years ago by Chandler

Status: pendingnew

Attachment (WebContent?.zip) added by ticket reporter.

Changed 6 years ago by Chandler

Attachment: WebContent.2.zip added

all custom classes

comment:3 Changed 6 years ago by bill

Resolution: patchwelcome
Status: newclosed

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.