Input send NULL array on laravel 5.3

Question

i have this error when I try to send an array to my database table. i have this on my update method

$post = Post::findOrFail($id);
$post->PostTitle = $request->PostTitle;
$post->post = $request->post;

$tags = new Tag;

foreach ($request->tags as $tags) {

    $tag = Tag::create(['tag' => $tags]);

    if ($tag->save()) {
        $post->tags()->sync($tag->id);
    }
}

this is my html form

<div class="post-panel-3">
        <div class="form-group">
          <label for="postTags">Etiquetas: </label>
            <input type="text" name="tags[]" class="form-control" placeholder="Etiquetas" value="<?php if(isset($post->tags)){ foreach($post->tags as $tag); echo $tag->tag;} ?>" id="tagBox2">

I have ManyToMany relationship model, all run fine in create method but no on update. I think i miss something but not finded yet.

Your input name in your form is taggged, not tags. And the field is most likely malformed from the way you're creating it in your form. — aynber
– aynber, Commented Mar 13, 2017 at 17:34
I edit the input name, jut a typo. i dont understand what you said. i was try to changed variable name, but it doesn't work. — Johanng1993
– Johanng1993, Commented Mar 13, 2017 at 18:25
If you look at the source code of the form in the browser, you'll see your value is 12342236 if your post has the tags 1, 2, 3, 4, 22, and 36. You should probably put the tags field inside the foreach instead. — aynber
– aynber, Commented Mar 13, 2017 at 18:27
Also, you are using sync which means each time through the loop, you are going to erase all past tags you just saved for that post. You probably want to use attach instead. — user1669496
– user1669496, Commented Mar 13, 2017 at 19:02

EddyTheDove · Accepted Answer · 2017-03-13 22:43:39Z

1

The sync() method takes an array as parameter. You are passing an object.

Create an array of IDs first, then pass it to the sync function.

$ids = [];
foreach ($request->tags as $tags) {
    $tag = Tag::create(['tag'=>$tags]);
    $ids[] = $tag->id;
}

//sync 
$post->tags()->sync($ids); //array of ids

answered Mar 13, 2017 at 22:43

EddyTheDove

13.2k2 gold badges40 silver badges46 bronze badges

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

11 Comments

Johanng1993 Over a year ago

I thought that my mistake was to pass an object instead of an array, but i saw that my input don't send any value to my controller, i try with all method, post, put, but nothing i don't get it.

EddyTheDove Over a year ago

View the source code of your form. Do you see any value in the input?

Johanng1993 Over a year ago

Johanng1993 Over a year ago

I will order my idea again, i want to attach new tags on post if people write on it, if not just leave the tags same as the post was created.

EddyTheDove Over a year ago

so request->tags should return 'etqiueta 1'. In your controller do dd($request->tags); and see what it displays.

|

Collectives™ on Stack Overflow

Input send NULL array on laravel 5.3

1 Answer 1

11 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

11 Comments

Your Answer

Sign up or log in

Post as a guest

Related