Tucker-Eric / EloquentFilter

An Eloquent Way To Filter Laravel Models And Their Relationships
http://tucker-eric.github.io/EloquentFilter
MIT License
1.72k stars 120 forks source link

Combine filter by related and own fields #140

Closed git-itmailbox closed 4 years ago

git-itmailbox commented 4 years ago

Hi, can you help with such issue, assume that we have some entity i.e. User, it has relation hasMany Orders, order belongs to some AnotherEntity. So I need to filter AnotherEntity by user's name, email, phone and also by AnotherEntity's field (i.e. name).

public $relations = [
    'order.user' => [
        'user_search'  => 'search',
    ]
];

public function search($search)
{
     //so here I need somehow to combine  'user_search' and another additional filter by own field or even another relation
}
Tucker-Eric commented 4 years ago

If I'm understanding correctly you should be able to leave that as is and it should work. The $relations array will forward the user_search parameter to search in the UserFilter and then you can add methods locally on AnotherEntity model to filter on AnotherEntity.

git-itmailbox commented 4 years ago

I'd like to have filter like this one

public $relations = [
    'order.user' => [
        'user_search'  => 'search',
    ]
];

public function search($search)
{
     //so here I need somehow to combine  'user_search' and another additional filter by own field or even another relation
    return $this->userSearch($search)
                      ->name($search); //name of AnotherEntity
}

public function name($search)
{
    return $q->where('name', 'LIKE', "%$name%");
}

and ofcourse conditions must be connected with OR operator

Tucker-Eric commented 4 years ago

So this is kind of tricky in the filter because when using the $relations array or related method they both collect all calls to each related entity and nest all those in one root level whereHas query so combining either of those methods with an or could potentially lead to unexpected behavior when adding more parameters that would chain with and queries.

I would suggest to not use the $relations array or related() method for this use case and use a nested where query where you can join with an or.

So, given the query to AnotherEntity:

AnotherEntity::filter([
    'search'      => 'some_string',
    'user_search' => 'user_string'
])
->get()

And the AnotherEntityModelFilter to search both strings with an OR condition:

public function search($search)
{
    return $this->where(function($query) {
        $query->whereHas('order.users', function($q) {
            // $q is an instance of the User query builder
            // so it has access to the UserModelFilter
            // allowing us to call `filter` on it
            $q->filter([
                'search' => $this->input('user_search')
            ]);
        })
        ->whereLike('name', $search, 'or');
    });
}
git-itmailbox commented 4 years ago

thanks, will try your solution. @Tucker-Eric