Error explode() expects parameter 2 to be string When using where function

Question

i want to remove some post that have specific restrict value the posts are 2 type (artists and songs) , songs belong artists

i get the songs of artist by below code but when i use where function to remove some songs its not working ang got error

Error :

explode() expects parameter 2 to be string, object given

My Code

$rel = collect([]);
foreach ($post->artists as $artist) {
   $artist_posts = $artist->getPosts->where(function ($query) {
     $query->whereNotIn('restrict', ['res1'])->orWhereNull('restrict');                  
   })->sortbydesc('id');
   $rel = $rel->concat($artist_posts->where('type', $type));
}
$col = $rel->unique('id');

Post.php (model)

...
public function getPosts()
    {
        return $this->belongsToMany(Post::class, 'artists_posts', 'artist_id', 'post_id')->with('artists');
    }
 ...

what is $type? also why are you passing a closure to where ... that is Collection@where not a Query Builder where — lagbox
– lagbox, Commented Sep 22, 2020 at 12:52
I am assuming that relationship method you are showing is on Artist not Post? — lagbox
– lagbox, Commented Sep 22, 2020 at 13:02

lagbox · Accepted Answer · 2020-09-22 13:05:04Z

1

You can't pass a Closure to the where method of Illuminate\Support\Collection. That is not a Query Builder you are calling where on, it is a Collection.

/**
 * Filter items by the given key value pair.
 *
 * @param  string  $key
 * @param  mixed  $operator
 * @param  mixed  $value
 * @return static
 */
public function where($key, $operator = null, $value = null)

If you wanted to call where on a Query Builder to do a database query you would have to use the actual relationship method not the dynamic property:

 $artist_posts = $artist->getPosts()->where(function ($query) {
                                  ^^

Then you would have to adjust sortbydesc to orderByDesc(...) and you would have to call get on the Builder to get the Collection of results.

answered Sep 22, 2020 at 13:05

lagbox

50.7k8 gold badges76 silver badges85 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Error explode() expects parameter 2 to be string When using where function

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related