MedianetDev\LaravelAuthApi\Http\Controllers\ChangeUserProfileInfoController::update()

Complexity

Conditions	2
Paths	2

Size

Total Lines	18
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	5
Bugs	1	Features	1

<?php
    public function update(ChangeUserProfileInfoRequest $request)
    {
        //TODO: encapsulate all the responses with ApiRequest
        $user = Auth::guard('apiauth')->user();

        $user->name = $request->name;

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

        $user->email = $request->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->isDirty('email')) {
            // mark the email as not verified yet
            $user->email_verified_at = null;

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

            // send a verification email
            $user->sendEmailVerificationNotification();
        }

        $user->save();

        return ApiResponse::send(['status' => 'Account updated successfully'], 1, 200, 'Account updated successfully');
    }