vaened / laravel-searcher

src/WithIndexes.php

<?php
/**
 * Created by enea dhack - 13/06/2020 13:01.
 */

namespace Vaened\Searcher;

trait WithIndexes
{
    abstract protected function getIndexProvider(): Indexable;

    public function search(string $index, ?string $q): self
    {
        if ($q != null) {

It seems like you are loosely comparing $q of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            $this->apply($this->getIndexProvider()->getConstraint($index, $q));

It seems like apply() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $this;
    }
}