leikind / wice_grid

A Rails grid plugin to create grids with sorting, pagination, and (automatically generated) filters
MIT License
537 stars 215 forks source link

filterDeclaration can sometimes be null, esp for Time datatypes #222

Closed asgeo1 closed 9 years ago

asgeo1 commented 9 years ago

In my grid, I have a column which has a Time data type. In this scenario, the javascript breaks, because the filterDeclaration variable is null - presumably because wice_grid supports DateTime and Dates, but not Times for columns.

This fixes the javascript error, which is important, because this javascript error prevents everything else from working (any of the filters, the 'show all link', the export to CSV - they all break).

I guess there is still the issue of no time filter being present, but that can be fixed later. For now, this at least lets the grid work as before.

leikind commented 9 years ago

thanks