Opened 10 years ago

Closed 7 years ago

#10983 closed defect (patchwelcome)

a bug in dojox.grid.DataGrid.filter

Reported by: huangyi Owned by: Bryan Forbes
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.4.0
Keywords: Cc:
Blocked By: Blocking:

Description

I create a datagrid and use the function filter to filter datas. And I find when the data is larger than displays, after load the new data, the filter result is incorrect. If I click the column to sort the data, it turn to correct.

Attachments (3)

filterProblem.PNG (116.9 KB) - added by fphan 10 years ago.
Screen shot of filter failure
filterProblem.html (10.4 KB) - added by fphan 10 years ago.
HTML code
filterProblemData.json (87.1 KB) - added by fphan 10 years ago.
JSON data for the HTML

Download all attachments as: .zip

Change History (9)

comment:1 Changed 10 years ago by Adam Peller

Component: DojoxDojoX Grid
Owner: changed from Adam Peller to bryanforbes

might need a concrete example here

Changed 10 years ago by fphan

Attachment: filterProblem.PNG added

Screen shot of filter failure

Changed 10 years ago by fphan

Attachment: filterProblem.html added

HTML code

Changed 10 years ago by fphan

Attachment: filterProblemData.json added

JSON data for the HTML

comment:2 Changed 10 years ago by fphan

I'm seeing the same problem (though in an EnhancedGrid?). After calling filter(), the results are wrong, although the result size is correct (and smaller than the original full data set).

comment:3 Changed 9 years ago by bill

Owner: changed from bryanforbes to Bryan Forbes

comment:4 Changed 9 years ago by snapedia

Just wanted to add that I am seeing the same problem. Seems that the filter() method works on smaller data sets, but once it gets past 100 or so items in the datastore, it starts to display the correct number of rows in the grid but the contents of those rows are not 100% correct. The suggested "workaround" of sorting a column seems to work, but at some point I need to reload the grid/datastore completely in order to even have that work.

comment:5 Changed 7 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:6 Changed 7 years ago by bill

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