Repository navigation
EloquentQueryProvider #5
Description
We should have a replacement ready for the old QueryEngine, so a user can pass a query builder or similar to the datatable.
What i found was really 'annoying' was the small little details i had to put into the old QueryEngine to make the search work across different databases.
So i would like to include search functionality behaviour as method on this provider so the user can implement it as needed.
I also see a problem if the user pre selected any data and then passes it to the EloquentQueryProvider.
E.g. if a user passes all users sorted by name to the datatable but skips the first 10 we have a problem, because the initial dataset can change.
So when a user orders by id (and the user skips the first 10 and passes it to the provider) then there is a changed dataset where different 10 users are missing.
Also should we override the skip and take parameter of the user?
Activity
timgws commentedon Mar 2, 2016
We should consider allowing a maximum value for
skipandlimit.For example, I don't want users to be able to export all of my data by just having them provide me with a
$largeNumberfor thetakeparameter.But I don't think this needs to be handled in the
QueryProvider, I think this should be done somewhere else, that way the providers don't need to worry about handling minimum and maximum parameters, it happens somewhere in a level above.timgws commentedon Feb 6, 2017
Are you specifically talking about Chumper/Datatable#267?
I don't see that as a bug - I see that as a feature :) By definition, if you are doing filtering & ordering, the datatable should represent what is in the database. If that means unexpectedly showing and hiding different rows from users, then so be it :)
I am sure if it's a problem that we will hear about it.
Chumper commentedon Feb 6, 2017
I was not talking about any issue specifically, that was just my impression when working on the old 'QueryEngine`.
Sounds good, I think with the current eloquent there should be quite some possibilities.