Laravel update controller with many inputs

Question

I have a resource and I'm trying to set up the update controller. In my case my edit form has many inputs and I need to update the database with them but there might be columns in the database not changed by the edit form. So I have my controller like this:

public function update($id)
{

    $hostess = Hostess::find($id);

    $inputs=Input::all();

    foreach ($inputs as $key => $value) {
        $hostess->$key= $value;
    }

    if ($hostess->save())
    {
        return Redirect::route('hostesses.show', $hostess->id);
    }

    return Redirect::back()->withInput()->withErrors($hostess->getErrors());
}

This gives me an error because I am using PUT in my view and I get

 Column not found: 1054 Unknown column '_method' in 'field list'

Because my Input::all() is getting the hidden inputs for the PUT method. I can use Input::except() for that, but is that the proper way of updating with laravel?

majidarif · Accepted Answer · 2014-04-23 10:59:17Z

4

You can actually do something like this:

$hostess   = Hostess::find($id)

$post_data = Input::all();
// or
$post_data = Input::except('_method');

// warning untested if block below
if ($hostess->update($post_data))
{
    return Redirect::route('hostesses.show', $hostess->id);
}

return Redirect::back()->withInput()->withErrors($hostess->getErrors());

As short as that would update all available key and value pairs.

Do note that you have to add the columns to the $fillable property in the model to avoid the mass assignment warning.

edited Apr 23, 2014 at 10:59

answered Apr 23, 2014 at 10:48

majidarif

20.3k18 gold badges97 silver badges137 bronze badges

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

2 Comments

Jarek Tkaczyk Over a year ago

Your answer is the way to go, but to make it even better add as an alternative fill(Input::all()) or new Hostess(Input::all()) as simpler, but when $fillable is set on the model.

Elaine Marley Over a year ago

Thanks, this looks like the proper way to go.

Darin Kolev · Accepted Answer · 2014-04-23 11:27:25Z

0

You could do something like this:

$inputs = Input::all();
$inputs = array_except($input, '_method');

edited Apr 23, 2014 at 11:27

Darin Kolev

3,40913 gold badges34 silver badges48 bronze badges

answered Apr 23, 2014 at 10:51

fr00t

6791 gold badge5 silver badges20 bronze badges

Collectives™ on Stack Overflow

Laravel update controller with many inputs

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related