Call queryCallback in DatabaseEngine

[Alanaktion]

The ->query() callback function on searches is helpful for things like eager loading relations, and is typically triggered via getScoutModelsByIds in engine implementations. Since the Database engine works directly with the models and does not call that method, I've added an explicit call to queryCallback when it's defined.

In my limited testing, this allows things like the following to work consistently on several engines including MeiliSearch and Database without requiring any engine-specific end-user code changes:

Post::search('example query')
    ->query(function ($builder) {
        $builder->with('comments');
    })
    ->paginate(15);

It doesn't seem like that callback is documented so it may not be widely used, but since it is implemented consistently in the other official engines this seemed like a good contribution.

[taylorotwell]

I think the intention is for you to use this: https://laravel.com/docs/9.x/scout#customizing-engine-searches ... for the database engine it receives the query as its first argument.

[Alanaktion]

That does require the user to write engine-specific search queries rather than have a generalized solution that works consistently with one implementation, but if that's what's preferred by the Laravel team that's fine I guess. I'll probably just not use Scout at all for the database engine if I have to write custom logic to perform simple searches across multiple engines either way.