Get value from one to many relationship. Laravel - php

Question

I have two models Vehicles and Image,
Vehicle hasMany images
public function images(){
return $this->hasMany('App\Image');
}
Image belongsTo a Vehicle:
public function vehicle(){
return $this->belongsTo('App\vehicle');
}
I've retrieved vehicles like:
In Controller:
$ads=ads::orderBy('views','desc')->get();
foreach ($ads as $ad) {
$popularvehicles[]=vehicles::where('id',$ad->Vehicleid)->get()->toArray();
}
View:
#foreach($popularvehicles as $popularvehicle)
#foreach($popularvehicle as $vehicle)
{{$vehicle['vname']}}
#endforeach
#endforeach
Now, I want to retrieve an image that belongs to $vehicle.
I've tried following syntax:
#foreach($popularvehicles as $popularvehicle)
#foreach($popularvehicle as $vehicle)
#foreach($vehicle->image as $image)
<a href="page-product-details.html">
<img src="{{ asset("images/$image->title") }}" alt="{{ $image->title }}" >
</a>
#endforeach
#endforeach
#endforeach
But it is throwing an exception of: Trying to get property of non-object.
What can I do to make it right?

Answer 1

Just a simple example:
You have a model, the model has details in another model.
class Vehicle extends Model
{
protected $fillable = [
'brand',
'etc',
];
public function details()
{
return $this->belongsToMany('App\VehicleDetails');
}
}
And the details model:
class VehicleDetails extends Model
{
protected $fillable = [
'color',
'engine',
];
public function vehicle() {
return $this->belongsTo('App\Vehicle');
}
}
I usually use transformers to return the needed data, but you don't have to. To make use of the hasMany you can call it like this:
return Vehicle::find($vehicleId); // Retrieve the vehicles from the server
// the object retrieved has the details.
$vehicle->details; // then you can use contains the details
// example:
#foreach($vehicle->details as $details)
<a href="page-product-details.html">
<img src="{{ asset("images/$details->imagetitle") }}"
alt="{{ $details->title }}" >
</a>
#endforeach
The returned object contains the details and you can loop trough it how ever you want.
Without more details about how you retrieve the data I can't be more specific.

Answer 2

I can see 2 problems;
Your popularvehicles is an array of arrays
$popularvehicles[]=vehicles::where('id',$ad->Vehicleid)->get()->toArray();
The toArray is turning everything into an array. You no longer have Vehicle objects. https://laravel.com/docs/5.2/collections#method-toarray
Use ->all() instead.
Second Problem
#foreach($vehicle->image as $image)
You want to use $vehicle->images