FuelPHP Forums

Ran into one issue with the package. The sort vars seem to be global. So if you have multiple views using this and sort on one column in one view and switch to a different view that one will try to sort by the same sort_by. This causes a SQL error if the column doesn't exist in the table, obvious. For now, I'm just doing a check to see if a sort_by is set and if it is in an acceptable field list.

Thanks, glad you find it handy

WanWizard wrote on Tuesday 11th of October 2011:

It suffers from the same issue as the pagination class though, in that the location of the control variables in the URI are hardcoded. So you run into issues when your URI has to be http://darklow.fuel.first.lv/this/that/tablesort/1/name-desc.

Well it is not exactly hardcoded. I left config parameter: base_url outside the class, so you can control it with the config. You can always auto-detect needed url using URI class. I could put autodetect in the class, but it will never satisfy 100% of all the users, that is why also in Pagination class and here you can specify location url by yourself. For example if your url is /items/2/ class could never know if 2 is pagination and should be striped out, or it is some kind of a section and should be there and final url should be /items/2/$page/$sort. Changed hardcoded string to URI function, so that people don't confused
'base_url' => '/'.URI::segment(1).'/',

Peter Wiggers wrote on Thursday 13th of October 2011:

I'd suggest to use http://tablesorter.com/docs/#Demo for "small" tables (because you're loading all the results in one time)

You mean IF you are loading all results in one time, then yes, you can use local javascript sorting, otherwise if you are using database queries (as in my example) you will need to use Pagination and also TableSort class...