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;

问题分析

  1. 原来使用的查询语句为

    $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 协议》,转载必须注明作者和本文链接
《L01 基础入门》
我们将带你从零开发一个项目并部署到线上,本课程教授 Web 开发中专业、实用的技能,如 Git 工作流、Laravel Mix 前端工作流等。
《G01 Go 实战入门》
从零开始带你一步步开发一个 Go 博客项目,让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。
讨论数量: 2

请问最上面的代码是写在控制器的吗

4年前 评论

@beaplat-61f 是的,直接在控制器里面加就好了

4年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!