Odtnhj

I have two methods in my callsController. The first is called newCall and it displays all the records from a DB table, it looks like this :

public function newCall()

 $new_calls = new_calls::all(); //fetches everything in the new_calls table in the database
 return view('calls.newCall')->with('new_calls', $new_calls); //passes it to the view

This function is working well and all the records are successfully retrieved in my view.

The issue is with the second method viewCall whose aim is to show data for a single call when its name is clicked from the list of newCalls.

Currently, I have just this:

public function viewCall()
 return view('calls.viewCall');

I plan on formatting the data from the database (for a particular call) with some static text while displaying it in my viewCall.blade.php.

I wasn't really sure what to put in web.php, so my routes look like this :

Route::get('/calls/newCall', 'CallsController@newCall');
Route::get('/calls/viewCall/id', 'CallsController@viewCall')->name('viewCall');

Can someone help me make this work? I just want to see the associative array with the record from the database when I click on a call's name. What do I need to add in routes and in my view?

This is my newCall.blade.php

<div class="table-responsive">
 <table class="table no-margin table-striped">
 <thead>
 <tr>
 <th>Terminal ID</th>
 <th>Terminal Name</th>
 -- <th>Fault Description</th> --
 <th>Call Logged On</th>
 <th>ATM Variant</th>
 </tr>
 </thead>
 <tbody>
 @if(count($new_calls) > 0)
 @foreach($new_calls as $calls)
 <tr>
 <td><a href="/calls/viewCall/$calls->id" title="view more details">$calls->terminal_id</a></td>
 <td>$calls->terminal_name</td>
 -- <td style="width:350px">$calls->fault_description</td> --
 <td>$calls->created_at</td>
 <td>$calls->atm_variant</td>
 </tr>
 @endforeach
 @else
 <h1>No new Calls</h1>
 @endif
 </tbody>
 </table>
</div>

This is my viewCall.blade.php

<address>
 <strong>$new_call->client_name</strong><br>
 $new_call->address<br>
 <b>Phone: $new_call->phone</b> <br>
 <b>Email: $new_call->email</b>
</address>

nb: I have also tried using $calls->email but it's returning undefined variable: calls and new_calls in each case

What you need to do is get the new_calls model by the ID. Which you can do like so:

routes/web.php

Route::get('/calls/viewCall/id', 'CallsController@viewCall');

CallsController.php

public function viewCall($id) // the id in the url

 $new_call = new_calls::find($id);

 return view('calls.viewCall')->with('new_call', $new_call);

In your view (I assume you're looping through all the calls to display them) create a link.

// your existing foreach
@foreach($x as $y)
 <a href="/calls/viewCall/$y->id">View Call</a>
@endforeach

You can do something like this:

public function viewCall($id)

 return view('calls.viewCall', ['call' => new_calls::findOrFail($id)]);

This will get the id from the route and call a function to find the object with the given id.

If you have problems to deal with the variable in the view. Try to check the object using dd. Try this:

public function viewCall($id)

 $new_call = new_calls::find($id);
 dd($new_call);

Ref: https://laravel.com/docs/5.7/controllers#basic-controllers

I recommend too looking at naming convetions and route model binding and type hinting in the controller.

Then you'd have something like this:

 public function viewCall(NewCall $newcall)

 return view('calls.viewCall')->with($newcall);

and route would be like this:

Route::get('/calls/viewCall/newcall', 'CallsController@viewCall')->name('viewCall');