Opened 10 years ago

Closed 7 years ago

#10845 closed enhancement (wontfix)

Filtering grid data

Reported by: Goran Miskovic Owned by: Evan
Priority: low Milestone: 1.9
Component: DojoX Grid Version: 1.4.0
Keywords: DataGrid, filter, server side, PHP, json_decode, undefined Cc:
Blocked By: Blocking:

Description

According to example given in documentation of DataGrid? method filter is intended for client side data filtering. However, I don't see why it can't be used for server side filtering as well.

The only problem that I run into was that isRender parameter passed in query is undefined which is causing PHP function json_decode to fails parsing JSON string.

Very simple patch is needed to overcome the problem. I prefer the DataGrid2.patch of two given patches.

PHP is fine with setting isRender to either false or null. As far as I can see it makes no difference to DataGrid? as well since both methods relaying on isRender parameter, _onFetchBegin and _onFetchComplete, are checking only if there is property isRender in request object.

if(req.isRender)

Attachments (3)

DataGrid.patch (712 bytes) - added by Goran Miskovic 10 years ago.
setting isRender withing _fetch method
DataGrid2.patch (609 bytes) - added by Goran Miskovic 10 years ago.
setting isRender within filter method
dojox.grid.DataGridFilter patch.zip (11.6 KB) - added by Chris Mitchell 9 years ago.

Download all attachments as: .zip

Change History (9)

Changed 10 years ago by Goran Miskovic

Attachment: DataGrid.patch added

setting isRender withing _fetch method

Changed 10 years ago by Goran Miskovic

Attachment: DataGrid2.patch added

setting isRender within filter method

comment:1 Changed 10 years ago by Douglas Hays

Owner: changed from bryanforbes to Bryan Forbes

comment:2 Changed 9 years ago by bill

Owner: changed from Bryan Forbes to Eavn

looks like these are ones that evan is working on

comment:3 Changed 9 years ago by bill

Owner: changed from Eavn to Evan

Changed 9 years ago by Chris Mitchell

comment:4 Changed 9 years ago by bill

Owner: changed from Evan to evan

comment:5 Changed 8 years ago by Evan

Owner: changed from evan to Evan

comment:6 Changed 7 years ago by Evan

Milestone: tbd1.9
Resolution: wontfix
Status: newclosed

Closing as wontfix since this is already supported by the new dgrid or gridx

Note: See TracTickets for help on using tickets.