Sorting feature is missing in MeiliSearch Engine

[bedus-creation]

• Scout Version: 9.3.4
• Scout Driver: MeiliSearch
• Laravel Version: 8.61
• PHP Version: 8.0.14
• Database Driver & Version: Mysql 8

Description:

MeiliSearch supports sorting feature out of the box after defining sortable parameters while indexing. The given orderBy method is supposed to work for sorting. Consider the following code.

return User::search('Admin')
        ->orderBy("id", "desc")
        ->get();

The above code should generate a query like in https://github.com/laravel/scout/blob/9.x/src/Engines/MeiliSearchEngine.php#L114

public function paginate(Builder $builder, $perPage, $page) 
 { 
     return $this->performSearch($builder, array_filter([ 
         'filters' => $this->filters($builder), 
         'limit' => (int) $perPage, 
         'sort'=> ['id:desc'],
         'offset' => ($page - 1) * $perPage, 
     ])); 
 } 

But the sort options is missing while performing the search.

scout/src/Engines/MeiliSearchEngine.php

Lines 114 to 121 in 7811f6d

	public function paginate(Builder $builder, $perPage, $page)
	{
	return $this->performSearch($builder, array_filter([
	'filters' => $this->filters($builder),
	'limit' => (int) $perPage,
	'offset' => ($page - 1) * $perPage,
	]));
	}

scout/src/Engines/MeiliSearchEngine.php

Lines 99 to 105 in 7811f6d

	public function search(Builder $builder)
	{
	return $this->performSearch($builder, array_filter([
	'filters' => $this->filters($builder),
	'limit' => $builder->limit,
	]));
	}

[driesvints]

This sounds like a feature request. We'd be happy to receive PR's.

/cc @mmachatschek

[mmachatschek]

Hi @bedus-creation,

this feature is plausable but was already tried to be implemented here #537 (See discussion)

Maybe this is useful to the DatabaseEngine too and all engines could benefit from a generic solution. @driesvints

[driesvints]

@taylorotwell can we maybe revisit the PR from above that @mmachatschek linked to?

[driesvints]

The related pr was merged

[bedus-creation]

I think sort option is still missing in paginate function #537

[driesvints]

@bedus-creation please send in a PR