Laravel 5.8+scout7.0 使用 orderBy 排序失效解决方案
最近在使用elasticSearch6.2.4做搜索时发现,排序字段失效,所以在这记录下
先看下解决方案,话不多说,直接上代码
$list = Article::search($words)->orderBy('created_at','desc')->paginateRaw(10)->toArray();
$results = $list['data'];
if ($results['hits']['total'] === 0) {
return $this->model->newCollection();
}
$builder =new Builder(new static(),$this->model->newModelQuery());
$keys = collect($results['hits']['hits'])->pluck('_id')->values()->all();
$query = $this->newQuery();
if ($builder->queryCallback) {
call_user_func($builder->queryCallback, $query);
}
//查询数据
$scoutModelsLists = $query->whereIn(
$this->model->qualifyColumn($this->model->getKeyName()), $keys
)->orderBy('created_at','desc')->get();
//过滤数据
$scoutModelsLists->filter(function () use ($keys) {
return in_array($this->model->getKey(), $keys);
});
//这里为最终排序好的数据
$data = $scoutModelsLists;
问题分析
-
原来使用的查询语句为
$list = Article::search($words)->orderBy('created_at','desc')->paginate(10)->toArray();
上面查询语句虽然设置了排序字段,但是最终输出的时候却没有排序,经分析,在ES搜索结果里面的确是排了序,但最终输出时,ES数据结构转化为集合时,并未加上排序字段
代码分析
文件1: /vendor/laravel/scout/src/builder.php 约261行-305行
仔细观察这个文件会有两个方法 paginate、paginateRaw ,前一个返回laravel 集合,后一个返回es的原生查询结构,
这两者代码的不同点在于这块$results = $this->model->newCollection($engine->map( $this, $rawResults = $engine->paginate($this, $perPage, $page), $this->model )->all());
文件2:vendor/tamayo/laravel-scout-elastic/src/ElasticsearchEngine.php 211行,map方法,因为这里我们使用的是ES引擎,如果使用别的,可能有所不同,代码:
public function map(Builder $builder, $results, $model) { //无数据返回空集合 if ($results['hits']['total'] === 0) { return $model->newCollection(); } //获取所有键为_id的ES数据 //$keys = collect($results['hits']['hits'])->pluck('_id')->values()->all(); //转化ES数据并过滤 return $model->getScoutModelsByIds( $builder, $keys )->filter(function ($model) use ($keys) { return in_array($model->getScoutKey(), $keys); }); }
从代码看来,es搜索出来有数据,则转化并过滤一下返回符合条件的集合,不满足直接返回空
文件3: /vendor/laravel/scout/src/Searchable.php 约171行getScoutModelsByIds方法,代码public function getScoutModelsByIds(Builder $builder, array $ids) { //加入软删除 $query = static::usesSoftDelete() ? $this->withTrashed() : $this->newQuery(); if ($builder->queryCallback) { call_user_func($builder->queryCallback, $query); } // 重点这里,自改代码 // return $query->whereIn( // $this->getScoutKeyName(), $ids // )->orderBy('orderBy','desc')->get(); //官方代码 return $query->whereIn( $this->getScoutKeyName(), $ids )->get(); }
这个文件是重点,主要是这里最后返回的时候并没有加上
orderBy
排序字段,所以最后输出时虽然es排序了,这里又重置了,为防止改组件后其他地方无法更新,所以在最后返回数据时加了排序处理,方案参考文章开头。
暂时告一段落,空了再完善。
本作品采用《CC 协议》,转载必须注明作者和本文链接
请问最上面的代码是写在控制器的吗
@beaplat-61f 是的,直接在控制器里面加就好了