adamfairholm / Elasticquent

Map Larvel Eloquent models to Elasticsearch types
MIT License
201 stars 38 forks source link

Sorting #17

Open stevepop opened 9 years ago

stevepop commented 9 years ago

Hi adamfairholm, thanks for providing this package. It gave me quite a quick entry into using Elasticsearch and Laravel compared to using Symfony 2 and Elastica which took me longer to wrap my head around.

In the documentation, under searching, you listed the parameters to the searchByQuery method and the last one was sort. I have however gone through your code and seen that the last parameter is offset and there is nowhere in the code where sorting is implemented. Is this an omission? How can I pass a sort query to elasticsearch via Elasticquent?

Thanks Steve

yswery commented 9 years ago

Here is where in the code it should happen:

https://github.com/adamfairholm/Elasticquent/blob/4d16e2df77449d75058231b92bdc3aab92bb87ec/src/ElasticquentTrait.php#L262

subdesign commented 9 years ago

what's the param for sort? string 'asc' or 'desc'? because none of them work.

yswery commented 9 years ago

You need to pass through thr ElasticSearch sort params.

Example: https://www.elastic.co/guide/en/elasticsearch/reference/current/search-request-sort.html

fusion52 commented 8 years ago

Appreciate the info. I tried '{"sort":[{"id" : {"order" : "asc"}},"_score"]}' only to get search_parse_exception: No mapping found. Would someone assist in the correct format? Thanks.