Laravel Call to undefined method Query\Builder

Question

I have product Model which belongs to Shop Model. Shop Can be Active or in Active, So I need to get products from active shops only. So I added this method

public function isActiveShop(){
    return $this->where($this->shop->status,'active');
}

and from my controller

$prodcuts = Product::where('id',$rawArray)->isActiveShop()->get();

And I get this error

"Call to undefined method Illuminate\Database\Query\Builder::isActiveShop()"

Jonas Staudenmeir · Accepted Answer · 2018-07-13 14:24:21Z

1

You have to use a scope and whereHas():

public function scopeIsActiveShop($query) {
    return $query->whereHas('shop', function($query) {
        $query->where('status', 'active');
    });
}

edited Jul 13, 2018 at 14:24

answered Jul 13, 2018 at 13:56

Jonas Staudenmeir

26k7 gold badges70 silver badges114 bronze badges

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

5 Comments

Michael George Over a year ago

That's work as I suppose but I got a new error Trying to get property of non-object

Jonas Staudenmeir Over a year ago

$this->shop->status looks a bit strange. Do you just mean ->where('status', 'active')?

Michael George Over a year ago

shop is a relation method public function shop() { return $this->belongsTo(Shop::class); }

Michael George Over a year ago

Thanks, that's works perfectly. any reference to study that part!!

Jonas Staudenmeir Over a year ago

laravel.com/docs/5.6/…

J. Doe · Accepted Answer · 2018-07-13 13:49:24Z

0

Try this:

$products = (new Product())->isActiveShop()->where('id',$rawArray)->get();

answered Jul 13, 2018 at 13:49

J. Doe

1,7521 gold badge14 silver badges23 bronze badges

1 Comment

Michael George Over a year ago

That's work ,scope is better for solution as I think

Collectives™ on Stack Overflow

Laravel Call to undefined method Query\Builder

2 Answers 2

5 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related