CODEWITHSUNDEEP
phplaravelcheckboxlaravel-bladelaravelcollective
Norgul
Norgul

Reputation: 4783

Laravel checkboxes with many-to-many relationship

I am pulling diseases from DB and presenting them in front-end with checkboxes:

<div class="col-lg-12 col-md-12 col-sm-12 col-xs-12">
    @foreach($diseases as $disease)
        <div class="col-md-6">
            <div class="form-group">
                <label>{{ Form::checkbox('diseases','0','',
                                ['class' => 'grey','data-id'=> $disease->id]) }}
                    {{$disease->title}}
                </label>
                <div class="clearfix"></div>
            </div>
        </div>
    @endforeach
</div>

Once I submit the form, I get the result of the $request->diseases like this:

1:"1"
2:"1"
3:"0"
4:"1"
5:"0"
6:"0"
...

where first number represents disease_id, and other one presents whether the checkbox is checked or not. Relationships to the pivot table are made within models correctly, and what I try to do in controller is:

if ($request->has('diseases')) {
    $patient->diseases()->sync($request->diseases);
}

but this fails with error:

Integrity constraint violation: 1452 Cannot add or update a child row: a foreign key 
constraint fails (`db_name`.`disease_patient`, CONSTRAINT `disease_patient_patient_id_foreign` 
FOREIGN KEY (`patient_id`) REFERENCES `patients` (`id`) ON DELETE CASCADE) (SQL: insert into 
`disease_patient` (`disease_id`, `patient_id`) 
values (17, {"1":"0","2":"0","3":"1","4":"1","5":"1","6":"1","7":"1","8":"1","9":"0","10":"1","11":"0","12":"1","13":"0","14":"0","15":"0","16":"0","17":"0","18":"0","19":"0","20":"0","21":"0","22":"0","23":"0"}))

Relationship inside Patient model:

protected $guarded = ['id'];

public function diseases()
{
    return $this->belongsToMany('App\Disease', 'disease_patient', 'disease_id', 'patient_id');
}

EDIT:

When I do:

$diseases = array_keys(array_filter(json_decode($request->diseases, true)));
return $diseases;

I get the list of ID's like so:

0:1
1:2
2:8
3:14
4:15
5:21

And error:

(SQL: insert into `disease_patient` (`disease_id`, `patient_id`) values (33, 1))

With disease_id incrementing on each request

Upvotes: 1

Views: 1887

Answers (1)

Jeremy Harris
Jeremy Harris

Reputation: 24579

The issue here is that the sync method is NOT supposed to be an associative array. You should be passing it a single array containing the disease ID's to attach. To do this, you will need to pre-process your array:

if ($request->has('diseases')) {
    $diseases = array_keys(array_filter($request->diseases)));
    $patient->diseases()->sync($diseases);
}

If $request->diseases returns a Laravel Collection, you may need to call toArray() or cast it to an array first.

Reference: The "Syncing Associations" section of https://laravel.com/docs/5.4/eloquent-relationships#inserting-and-updating-related-models

Upvotes: 1

Related Questions