App\Http\Controllers\Api\OAuthController::userByToken()

Complexity

Conditions	4
Paths	3

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function userByToken(Request $request)
    {
        $token = $request->input('wiki_token', null);

        if($token === null || $token === ''){
            return ['error' => 'invalid_token'];
        }
        $user = User::where('wiki_token', $token)->first();

        if($user === null){
            return ['error' => 'invalid_token'];
        }

        return [
            'id' => $user->uuid,

The property uuid does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

            'name' => ucfirst($user->firstname).' '.ucfirst($user->lastname),

The property lastname does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

It seems like $user->firstname can also be of type Illuminate\Database\Eloquent\Builder&Illuminate\Database\Eloquent\Relations\Relation and Illuminate\Database\Eloquent\Relations\Relation; however, parameter $str of ucfirst() does only seem to accept string, maybe add an additional type check?

The property firstname does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

            'realname' => ucfirst($user->firstname).' '.ucfirst($user->lastname),
            'email' => $user->email,

The property email does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

            'avatar' => $user->adminlte_image()
        ];