@tykus @martinbean Ok i have a table i want to show. But in one of the columns(bedtype), i want to get the name of the bedtype instead of the id. This is the code for table view template

                    <th scope="col">#</th>
                    <th scope="col">Bed</th>
                    <th scope="col">Bed Type</th>
                    <th scope="col">Charge</th>
                    {{-- <th scope="col">Available </th> --}}
                    <th scope="col">Edit</th>
                    <th scope="col">Delete</th>
                  </tr>
                </thead>
                <tbody>
                  @foreach ($beds as $bed)
                  <tr>
                    <td>{{ $bed->id}}</td>
                    <td>{{ $bed->name}}</td>
                    <td>{{ $bed->bed_type}}</td>
                    <td>{{ $bed->charge}}</td>
                    {{-- <td>{{ $bed->is_available}}</td> --}}
                    <td>
                        <a class="btn btn-primary" href="{{url('updateBed',$bed->id)}}">Update</a>                        
                    </td>  
                    <td>
                      <a onclick="return confirm('Are you sure you want to delete?')" class="btn btn-danger" href="{{url('deleteBed',$bed->id)}}">Delete</a>                     
                  </td>
            
                  </tr>
                  @endforeach
              </tbody>
            </table>
          </div>

and this is the code for the form to insert values into the above table

                        <form  action="{{url('upload_bed')}}" method="POST" id="nurse-form" enctype="multipart/form-data">
                          @csrf
                          <div class="row mb-3">
                              <label for="name" class="col-sm-2 col-form-label">Bed</label>
                              <div class="col-sm-10">
                                  <input type="text" class="form-control" id="name" name="name" placeholder="Bed" required>
                              </div>
                          </div>
                          <div class="row mb-3">
                              <label for="bedTypes" class="col-sm-2 col-form-label">Bed Type</label>
                              <div class="col-sm-10">
                                <select class="form-select form-select-sm" name="bed_type" aria-label=".form-select-sm example" >
                                  <option>--Select Bed Type--</option>
                                  @foreach($bedTypes as $bed_type)
                                  <option value="{{$bed_type->id}}" class="form-control">{{$bed_type->title}}</option>
                                  @endforeach
                                </select>
                              </div>
                          </div>
                          <div class="row mb-3">
                            <label for="number" class="col-sm-2 col-form-label">Charge</label>
                            <div class="col-sm-10">
                                <input type="number" class="form-control" id="charge" name="charge" placeholder="Ghc" required>
                            </div>
                        </div>
                          
                      </form> 

@saqar so a Bed belongs to a BedType? Do you have a relationship defined on the Bed model?

@tykus yeah this is the relationship in the Bed model

public function bed_types() { return $this->belongsTo(BedType::class); }

@saqar by convention this would be singular bed_type since it returns 0 or 1 instance, but it appears above that the foreign key is bed_type as well, so let's not change it!

Now you can eager-load the BedType along with the Beds Collection:

$beds = Bed::with('bed_types')
    // ->where(etc)
    ->get();

And in the view, you can use the bed_types property which is a BedType instance (the ?? operator will mitigate for orphaned data)

@foreach($beds as $bed)
    <!-- etc. -->
    {{ $bed->bed_types->name ?? 'no bed type'}}
@endforeach

@tykus So what you are saying is i should add this piece of code $beds = Bed::with('bed_types') // ->where(etc) ->get(); to the relationship in the Bed model?

@saqar no. Wherever you are getting the $beds variable for the view, you need to eager-load the bed_types relationship.

@tykus Alright thanks a lot. But now it seems to be getting only the 'no bed type' choice though i select a bedtype in the form

What does your code look like now? How are you assigning the bed_types foreign key on a Bed model whenever you are submitting the form?

@saqar A foreign key usually has the table name it references in singular form followd by an id. That is the naming convention that is assumed by Laravel, unless you tell it something else.

So you have a beds table and a bed_types table, and since a bed can have a bed_type the foreign key should reside in the beds table. Thus the name of the foreign key should be bed_types in it's sindular form bed_type followed by the name if the primary key in the bed_types table, which should be id. Putting all that together makes the foreign ke column name bed_type_id, and that is what Laravel expects you to have in your beds table.

@tykus So i guess you want to see thecodes for the form ?

@saqar form, controller, views please

@tykus This is the table view.

                    <th scope="col">#</th>
                    <th scope="col">Bed</th>
                    <th scope="col">Bed Type</th>
                    <th scope="col">Charge</th>
                    {{-- <th scope="col">Available </th> --}}
                    <th scope="col">Edit</th>
                    <th scope="col">Delete</th>
                  </tr>
                </thead>
                <tbody>
                  @foreach ($beds as $bed)
                  <tr>
                    <td>{{ $bed->id}}</td>
                    <td>{{ $bed->name}}</td>
                    <td>{{  $bed->bed_types->name ?? 'no bed type'}}</td>
                    <td>{{ $bed->charge}}</td>
                    {{-- <td>{{ $bed->is_available}}</td> --}}
                    <td>
                        <a class="btn btn-primary" href="{{url('updateBed',$bed->id)}}">Update</a>                        
                    </td>  
                    <td>
                      <a onclick="return confirm('Are you sure you want to delete?')" class="btn btn-danger" href="{{url('deleteBed',$bed->id)}}">Delete</a>                     
                  </td>
            
                  </tr>
                  @endforeach
              </tbody>

This is the form

       <form  action="{{url('upload_bed')}}" method="POST"  enctype="multipart/form-data">
                @csrf
                <div class="mb-3">
                  <label for="name" class="form-label">Bed</label>
                  <input type="text" class="form-control" style="color:black"  name="name" required>
                </div>
                <div class="col-12 col-sm-6 py-2 wow fadeInRight" data-wow-delay="300ms">
                  <label for="title" class="form-label">Bed Type</label>
                    <select name="title" id="title"  class="custom-select" required>
                      <option>--Select Bed Type--</option>
                      @foreach($bed_types as $bed_type)
                      <option value="{{$bed_type->id}}" class="form-control">{{$bed_type->title}}</option>
                      @endforeach
                    </select>
                  <div class="mb-3">
                    <label for="charge" class="form-label">Charge</label>
                    <input type="number" class="form-control" style="color:black"  name="charge" required>
                  </div>

this is the controller

class BedController extends Controller{ public function showBed(){

    // $beds = Bed::all();
    $beds = Bed::with('bed_types')
 ->where('bed_type', '!=', null)
->get();
  $bedTypes = BedType::all();
    
    return view("nurse.showBed",compact("beds"))->with('bedTypes', $bedTypes);
}



   public function upload(Request $request){

       $request->validate([
    'name'=> 'required|unique:beds,name',
    'charge'=> 'required',
    // 'bed_type' => 'required|exists:bed_types,title',
    
    ]);

Bed::create([
    'name' => $request->name,
    'charge' => $request->charge,
    'bed_type' => $request->bed_type,
]);



return redirect()->back()->with('message','Bed Added Successfully');

}

public function deleteBed($id){

$beds = Bed::find($id);
$beds->delete();

return redirect()->back();

}

public function updateBed($id){

$beds = Bed::find($id);

return view('nurse.update_bed',compact("beds"));

}

public function editBed(Request $request , $id){

$beds=Bed::find($id);
$beds = new Bed();
$beds->name = $request->name;
$beds->charge = $request->charge;
$beds->bedtype_id = $request->bed_types;

$beds->save();

return redirect()->back->with('message','Bed Details updated Successfully');

}

@saqar the select input is named incorrectly as title, and not bed_type

<select name="title" id="title"  class="custom-select" required>
// change to 
<select name="bed_type" id="bed_type"  class="custom-select" required>

Good validation rules would have caught this...

Also, is the database column bed_type or bedtype_id -- the inconsistency is maddening!

Last, WTF is this $beds assignment?

public function editBed(Request $request , $id)
{
    $beds=Bed::find($id);
    $beds = new Bed();
    // ...

@tykus the database column is bed_type.

$bed assignment - is it wrong?? Please bear with me i'm only a beginner

@saqar in the form, the input should be named correctly, and represented in the validation rules and getting data out of the Request:

<select name="bed_type" id="bed_type"  class="custom-select" required>

Then in the controller actions, you get that data from the Request using the same name:

public function upload(Request $request)
{
    $request->validate([
        'name'=> 'required|unique:beds,name',
        'charge'=> 'required',
        'bed_type' => 'required|exists:bed_types,title',
    ]);

    Bed::create([
        'name' => $request->name,
        'charge' => $request->charge,
        'bed_type' => $request->bed_type,
    ]);

    return redirect()->back()->with('message','Bed Added Successfully');
}

public function editBed(Request $request , $id)
{
    $beds=Bed::find($id);
    $beds->name = $request->name;
    $beds->charge = $request->charge;
    $beds->bed_type = $request->bed_type;
    $beds->save();

    return redirect()->back->with('message','Bed Details updated Successfully');
}

As I mentioned yesterday, and @tray2 today, there are conventions that we use for foreign keys (such as the _id suffix) which will make other aspects of your development easier. But, I don't think it is worth laboring on it for now.

@saqar When you have time I strongly recommend reading these three posts.

• https://tray2.se/posts/database-design This will cover the basics of database design and naming conventions.
• https://tray2.se/posts/database-design-part-2 This one goes further into database design and the options you have.
• https://tray2.se/posts/sqlerrm This one covers almost all the common error the database will throw at you when unsing Laravel..

@tykus I've done all those changes you asked but it still shows the 'no selected bed' tho i chose a bed type

@saqar does BedType model have a name property? I see you are using title in the select options!

<option value="{{$bed_type->id}}" class="form-control">{{$bed_type->title}}</option>

but name in the table:

<td>{{  $bed->bed_types->name ?? 'no bed type'}}</td>

If there is no name, then you should be using title here!