Get selected values from database in checkboxes (Laravel)

<input type="checkbox" id="division_id" name="division_id[]" value="{{$division->id}}" {{ $division->id == $value_from_database ? 'checked' : '' }}> 

or from pivot

<input type="checkbox" id="division_id" name="division_id[]" value="{{$division->id}}" {{ $subject->divisions->contains($division->id) ? 'checked' : '' }}> 

@fransiskusbenny I get this error when I do the second solution suggested, because I don't seem to understand the first.

Undefined variable: subject

you can pass the related divisions to the modal by set it as data-* in modal button trigger, and then set the checked checkbox with javascript.

http://getbootstrap.com/javascript/#modals-related-target

// inside $subjects foreach
<button type="button" class="btn btn-primary" data-toggle="modal" data-target="#yourEditModal" data-divisions="{{ $subject->divisions->pluck('id')->toJson() }}">Edit</button>

// do your javascript to check all related division checkbox

@bunnypro I'm not that good with javascript and have not the slightly idea on how you want me to accomplish this. Can you please be a little bit practical? Also know that I want the list of all the divisions and the ones assigned to the subject to be edited should be checked.

@Tangaye I'm not sure if I followed your code correctly or not,or maybe that's because you didn't send us you model. What I can say is there is actually two problem is exists with your code:

1. you are not eager loading your model so for every @foreach($subject->divisions as $division) in your code there will be a query executed against the database. This means that if you have 10/100/1000 divisions you have this number of query. For more info please see here

2. and for selecting checked ones you can use in_array(...) or contains()

@foreach($divisions as $division)
    <label class="checkbox-inline">
    <input type="checkbox" id="division_id" name="division_id[]" value="{{$division->id}}" {{ in_array($division->id, $array_of_values_you_fetched_from_database_as_seleted_items) ? 'checked' : '' }}> {{$division->name}}
        </label>
    @endforeach                           

Hope it helps

@sobhanattar Thanks for being a bit explicit. This is what I have in my controller to display records in my table:

public function index(Subject $subject)
{
    //
    $subjects = Subject::all();
   
    return view('subjects.show', compact('subjects'));
}

If it is best practice and is advisible I can refactor to this:

 $subject = Subject::with('divisions')->get();

The reason I didn't do it that way is that I realize that the page loads a bit slower when I do that. Like it's loading all the subjects and divisions at once. But anyway if it's best practice as I sad I will go along.

concerning the checked values

You said I should do this:

{{ in_array($division->id, $array_of_values_you_fetched_from_database_as_seleted_items) ? 'checked' : '' }}

But I'm thinking if I should make an ajax call to get the divisions assigned to the subjects. If the divisions are many, how do I pass the array of divisions returned to the method you suggested. Will appreciate some practical examples.

@Tangaye Hi Sorry for being late answering the question, I thought that if some one mention me in a discussion, then I'll get an alert on my email which is not the case.

For the first part, eager loading is absolutely the better way to go as it costs you only 2 query instead of a query/loop in your index view. But maybe the slowness of the view, is related to something else.

So stay with eager loading for the sake of your application performance.:)

You can see the difference between number of queries by installing this

For the second part of your question, if I understand it correctly you have divisions and subjects and there is a relation between these two that is being stored in subject_division table.

In other words you should have an edit method with following signature in SubjectsController.php

public function edit(Request $request, Subject $subject)

As you already knew, here the $subject variable is the one you want to show info about and show which divisions it has. So your method will be something like this:

public function edit(Request $request, Subject $subject)
{
    //after validation

    $selectedDivisionsOfThisSubject = $subject->divisions()->get();
    $divisions = Division::all();
    
    return view('subjects.show', compact('subject',     'selectedDivisionsOfThisSubject', 'divisions'));
}

In this method you pass $subject so you have info like $subject->name. $divisions give you the ability to show all divisions for each subject by putting this piece of code

@foreach($divisions as $division)
    <label class="checkbox-inline">
    <input type="checkbox" id="division_id" name="division_id[]" value="{{$division->id}}"> {{$division->name}}
    </label>
@endforeach

So now you have $subject info and list of divisions from foreach on $divisions. Now the only thing you passed and still not used is $selectedDivisionsOfThisSubject which holds the ids of divisions that are selected for this subject and you can dd() it to see how it stores the data and then used it in your code. Generally it will be a collection and you can easily find your path around.

@sobhanattar thanks I got it working now. But I used ajax to to fetch the $selectedDivisionOfThisSubject and passed that to my view. Also I utilize the in_array() method you suggested in your first answer which lead me to returned the selected divisions like this $selectedDivisionOfThisSubject->toArray(). :heart_eyes: :kissing_closed_eyes: :wink:

@Tangaye I don't know why it's not working, as I have same code that is working. Maybe you can send me your tables with dummy data, and then I do the job on my side and send you the final code.

Because by having the subject, divisions and selectedDivisions you should not go the ajax way

This is how I usually organize my code:

@foreach ($divisions as $division)
    
    <label class="checkbox-inline">

        <input type="checkbox" name="division_id[]" id="division_id_{{ $division->id }}" value="{{ $division->id }}"

            @if ($ids = $subject->divisions->pluck('id')->toArray())

                @foreach ($ids as $division_id)
                        
                    {{ $division->id == $division_id ? 'checked' : '' }}
                    
                @endforeach

            @endif
        />
            
       {{ $division->name }}

    </label>
    
@endforeach

@sobhanattar I don't selectedDivision in order to get that I had to make an ajax request. Please know that I'm using a pop up bootstrap model for editing, which means I'm not loading out another view to edit the record. After I have send the id to the edit method and have gotten the values I want returned. I return a partial view with those values (subjects,selectedDivisios.divisions`) into my modal for editing. Hope this gives understanding. Please let me know if there's a better of cleaner way to do it. Thanks so much for the effort.

@Tangaye do you call public function edit(Request $request, Subject $subject) in your ajax call and get partial HTML to show in modal?

If that is the case you are passing $subject id as parameter to edit, so can have $subjectDivisions by calling $subject->divisions()->get() in that same edit method and pass it in partial view. I assume that your ajax call is waiting for html to receive.

In which divisions() is a belongsToMany relationship that shouldbe declared in Subject Model.

@sobhanattar that's right! Am I going about it the right way?

@Tangaye I think you do