Undefined variable: products in @foreach loop

Question

I am trying to override a view of voyager-admin and edit it but I get an "Undefined variable: products" error and I can't figure out why. Here is my OrdersController:

$order = Orders::find($id);
$products = $order->products;

return Voyager::view($view, compact('dataType', 'dataTypeContent', 'isModelTranslatable','order', 'products'))

Here is my view:

<ul>
    @foreach ($products as $product)
    <li style="margin-bottom: 10px">
    <div>ID Produs: {{$product->id}}</div>
    <div>Nume Produs: {{$product->name}}</div>
    <div>Pret Produs: {{$product->presentPrice()}}</div>
    <div>Cantitate: {{$product->pivot->quantity}}</div>
    </li>
    @endforeach
 </ul>

If anything more needed please let me know, I just can't understand why is throwing me this error.

My Orders Model

 public function products(){
  return $this->belongsToMany('App\Product')->withPivot('quantity');
}

My Products Model

 public function orders(){

    return $this->belongsToMany('App\Orders');
}

`return` sounds like that previous code is in a function, is that correct? — user3783243, Jun 12 '19 at 15:46
It looks like it should work. You could always skip it and iterate through `$orders->products` instead, if `$orders` works in your view. — aynber, Jun 12 '19 at 15:53
What happens with `$orders->products`? Is it complaining that `$orders` is not defined, or that it can't iterate through it? — aynber, Jun 12 '19 at 16:28
Try with dd($orders) to make sure what relationships it brings — Eduardo Hernandez, Jun 12 '19 at 17:31
where exactly do I have to do the dd($orders), I am new to laravel. I tried some different options but it doesn't show me the array so I am doing something wrong. — Negru Andrei, Jun 12 '19 at 18:15
I did the $order->products in the foreach loop and now it says that $order is not defined — Negru Andrei, Jun 12 '19 at 19:41
double check your product and order relation, did you write something on model to map them(eloquent) — afikri, Jun 13 '19 at 04:35

score 0 · Answer 1 · answered Jun 13 '19 at 04:02

0

Add validation to check your collectio

$order = Orders::find($id);
if($order != null){
$products = $order->products;

return Voyager::view($view, compact('dataType', 'dataTypeContent', 'isModelTranslatable','order', 'products'))
} else {
return null;
}

answered Jun 13 '19 at 04:02

Carl Angelo Orale

119
1
7

same result, products is undefined – Negru Andrei Jun 13 '19 at 06:58

score 0 · Answer 2 · edited Mar 02 '20 at 12:27

0

I forgot to override the controller in the Voyager-Admin.

edited Mar 02 '20 at 12:27

double-beep

5,031
17
33
41

answered Jun 13 '19 at 08:12

Negru Andrei

3
4

Undefined variable: products in @foreach loop

2 Answers2