search

bgultekin / laravel4-datatables-package

Server-side handler of DataTables Jquery Plugin for Laravel 4
267 stars 108 forks source link

Fixed issue of showing all rows in paging function #197

Open sameer1750 opened 9 years ago

sameer1750 commented 9 years ago

By default only 10 rows were shown in the paging function.Fixed the issue to show all rows if all selected.

MarkVaughn commented 9 years ago

I thought the -1 check was what would allow returning everything? also there is no reason to nest an if statement within an if statement if it has no else. Are you sure this is the place where the error happened?

sameer1750 commented 9 years ago

Yes i agree there is no need for a nested if .. but yes this is where the problem is ..

sameer1750 commented 9 years ago

-1 allows returning everything but bcoz of ur previous code it return only 10 rows.