App\Http\Controllers\Ajax\AccountController::updateAvatar()

Complexity

Conditions	5
Paths	4

Size

Total Lines	41
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function updateAvatar(Request $request)
    {
        $isValid = $request->file('avatar')->isValid();
        if($isValid){
            $extension = $request->file('avatar')->extension();
        }else{
            $output=[
                'ret' => '400',
                'desc' => 'Invalid file',
                'data' => null
            ];
            return response()->json($output);

The expression return response()->json($output) returns the type Illuminate\Http\JsonResponse which is incompatible with the documented return type Illuminate\Http\Response.

        }
        $allow_extension = ['jpg','png','jpeg'];
        if($isValid && in_array($extension,$allow_extension)){
            $path = $request->file('avatar')->store('/static/img/avatar','NOJPublic');

            $user = Auth::user();
            $old_path = $user->avatar;

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

            if($old_path != '/static/img/avatar/default.png'){
                Storage::disk('NOJPublic')->delete($old_path);
            }

            $user->avatar = '/'.$path;

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

Are you sure $path of type false|string can be used in concatenation?

            $user->save();

            $output=[
                'ret' => '200',
                'desc' => 'success',
                'data' => [
                    'url' => '/'.$path
                ]
            ];
            return response()->json($output);

The expression return response()->json($output) returns the type Illuminate\Http\JsonResponse which is incompatible with the documented return type Illuminate\Http\Response.

        }else{
            $output=[
                'ret' => '400',
                'desc' => 'Invalid file',
                'data' => null
            ];
            return response()->json($output);

The expression return response()->json($output) returns the type Illuminate\Http\JsonResponse which is incompatible with the documented return type Illuminate\Http\Response.

        }