Xetaravel\Http\Controllers\User\EmailController::update()

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function update(UpdateEmailRequest $request): RedirectResponse
    {
        $user = Auth::user();
        $user->email = $request->input('email');

Accessing email on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        if ($user->save()) {
            return redirect()
                ->route('user.setting.index')

The method route() does not exist on Illuminate\Routing\Redirector.

                ->success('Your E-mail has been updated successfully !');
        }

        return redirect()
            ->route('user.setting.index')
            ->error('An error occurred while saving your E-mail!');
    }