laravel display variable from second table

Question

Welcome ! I made app where user has it's own notes ( based on foreign key). Now in table I want to display his name but i can't pass it :

Pilot model:

class Pilot extends Model
{
  protected $table = 'pilots';
    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'phone', 'email',
    ];

    public function note() {
      return $this->hasMany(Note::class);
    }

Note model:

class Note extends Model

{
  protected $table = 'notes';
    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'data', 'zadanie', 'uwagi', 'pilot_id',
    ];

    public function pilot() {
      return $this->belongsTo(Pilot::class);
    }

Notes Controller ( i want to display pilot name in index page)

public function index(Request $request)

{
    $notes = Note::all();

    $pilots = Pilot::find($id);
    return view('uwagi.index',['notes'=>$notes,'pilots'=>$pilots]);
}

When i pass it to view via

{{$pilots->name}}

I have an error undefined $id. Each note belongs to user based on foreign key pilot_id which is in note table. This pilot_id key refers to table pilots -> id and i want to display in index page name of this id but i don't know how to pass it.

Regards and thank you for help

What exactly do you want to show? just one pilot with its associated notes? — Onix
– Onix, Commented Mar 16, 2017 at 9:03

Nickstery · Accepted Answer · 2017-03-16 09:43:22Z

0

There is no defined variable $id in your Controller's index method. Also if you are defined model relations, you can grab needed data using Pilot::with('note')->all(); Take a look at this: https://laravel.com/docs/5.4/eloquent-relationships#eager-loading

But the best way to pass pilot_id to route like

Route::get('pilot/{id}', 'NotesController@index')

and index method will be like public function index(Request $request, $pilot_id)

or

Route::resource('pilot', 'NotesController')

and inside controller define show($pilot_id) method than use

$pilot_data = Pilot::with('note')->find($pilot_id);

$pilot_data->note will contain all notes connected to pilot

And according to the code, there is no doubt for binding Request $request inside controller's action, as you do not use it inside the code

edited Mar 16, 2017 at 9:43

answered Mar 16, 2017 at 9:02

Nickstery

1164 bronze badges

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

2 Comments

tomczas Over a year ago

Missing argument 2 for App\Http\Controllers\NotesController::index()

Nickstery Over a year ago

remove Request parameter from index method, leave only $pilot_id

tomczas · Accepted Answer · 2017-03-16 10:57:18Z

-1

Ok answer is easiest as i thought :

Controller:

public function index(Request $request)

{
    $notes = Note::all();

    return view('uwagi.index',['notes'=>$notes,]);
}

Blade:

@foreach ($notes as $note)
{{ $note->pilot->name }}
@endforeach

answered Mar 16, 2017 at 10:57

tomczas

1791 gold badge4 silver badges20 bronze badges

1 Comment

jakub wrona Over a year ago

Except it doesn't match the question at all (not to mention it's your own question), because the code you've pasted suggests you wanted only notes from a given Pilot, not all of them.

Collectives™ on Stack Overflow

laravel display variable from second table

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related