Laravel update method not working correctly

Question

Here's my controller code:

public function update(Request $request, $id)
{
    $currency = Currency::find($id);

    $this->validate($request, [
        'cur_name' => 'required',
        'cur_price' => 'required|numeric',
        'cur_icon' => 'required|image|max:100',
        'cur_reserve' => 'required|numeric',
    ]);        

    $currency->cur_name = strip_tags($request->input('cur_name'));
    $currency->cur_price = $request->input('cur_price');

    // image
    $file = $request->file('cur_icon');

    if (isset($file)) {
        $filename = Str::lower(
            pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME) . '-' . uniqid() . '.' . $file->getClientOriginalExtension()
        );
        $destination = 'uploads/';
        $file->move($destination, $filename);
        $currency->cur_icon = $filename;
    }

    $currency->cur_reserve = $request->input('cur_reserve');
    $currency->slug = \Slug::make(strip_tags($request->input('cur_name')));
    dd($currency);
    $currency->save();

    return redirect('/admin/currency');
}

edit.blade.php (short):

<form class="form-horizontal" role="form" method="POST" enctype="multipart/form-data" action="/admin/currency/{{ $currency->id }}">
                {{ csrf_field() }}
                <input name="_method" type="hidden" value="PUT">
...
</form>

routes:

Route::group(['prefix' => 'admin', 'middleware' => 'auth'], function() {
    Route::resource('currency', 'CurrencyController');
});

When i click on submit - i am not being redirected and nothing happens, just page reloads - that's all. Even if i modify data no changes are saved to database.

share your routes too

Sulthan Allaudeen
– Sulthan Allaudeen

2016-11-15 17:27:51 +00:00
Commented Nov 15, 2016 at 17:27 — Sulthan Allaudeen
– Sulthan Allaudeen, Commented Nov 15, 2016 at 17:27

Eric Tucker · Accepted Answer · 2016-11-15 17:38:40Z

3

If you aren't getting to dd($currency); before $currency->save()? then it may be the validator redirecting. Try debugging with this in your view to check:

{{ count($errors) > 0 ? dd($errors->all()) : ''}}

answered Nov 15, 2016 at 17:38

Eric Tucker

6,3851 gold badge28 silver badges39 bronze badges

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

2 Comments

Alexander Kim Over a year ago

array:1 [▼ 0 => "The cur icon field is required." ] You were right, that's validator. Could you tell, how can i separate validation rule if there's file input and when there's no file input in my case.

Eric Tucker Over a year ago

That's it. In your validator change 'cur_icon' => 'required|image|max:100', to 'cur_icon' => 'image', passing the required parameter looks for a field, not a file. image is enough to require a file be present and also be an image.

Anandhan · Accepted Answer · 2016-11-15 17:29:26Z

1

Try this, you need to use url() method to get full url.

action="{{ url('/admin/currency/' . $currency->id) }}">

answered Nov 15, 2016 at 17:29

Anandhan

6874 silver badges9 bronze badges

1 Comment

Alexander Kim Over a year ago

Thank you for the tip, but same behavior.

David · Accepted Answer · 2017-03-15 09:12:38Z

0

Include something like:

protected $fillable = [
        'name', 'email', 'password',
        'gender', 'phone' ,'isActive', 
        'photo','setup', 'address'
 ];

In your Model

edited Mar 15, 2017 at 9:12

David

1,1434 gold badges18 silver badges30 bronze badges

answered Mar 15, 2017 at 8:01

Charles

1

Collectives™ on Stack Overflow

Laravel update method not working correctly

3 Answers 3

2 Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related