App\Http\Controllers\Stripe\Webhook

Complexity

Total Complexity

14

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Importance

Changes	6
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
C	__invoke()	0	56	14

<?php

namespace App\Http\Controllers\Stripe;

use App\Http\Controllers\Controller;
use App\Models\User;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Log;
use LaravelEnso\Roles\Models\Role;
use Stripe;

class Webhook extends Controller
{
    /**
     * Handle the incoming request.
     *
     * @return \Illuminate\Http\Response
     */
    public function __invoke(Request $request)
    {
        Stripe\Stripe::setApiKey(\Config::get('services.stripe.secret'));
        Stripe\Plan::all();
        $data = request()->all();

        if ($data) {

The expression $data of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            switch ($data['type']) {
                case 'customer.subscription.deleted':
                    $user = User::where('stripe_id', $data['data']['object']['customer'])->first();
                    if ($user) {
                        $user->role_id = 4;
                        $user->save();
                    }
                    break;

                case 'customer.subscription.created':
                case 'customer.subscription.updated':
                    $user = User::where('stripe_id', $data['data']['object']['customer'])->first();
                    if ($user) {
                        $plan_nickname = $data['data']['object']['plan']['nickname'];
                        $roles = Role::where('name', strtolower((string) $plan_nickname))->first();
                        if ($roles) {
                            $user->role_id = $roles->id;

The property id does not seem to exist on Illuminate\Database\Eloquent\Relations\HasOneThrough.

The property id does not seem to exist on Illuminate\Database\Eloquent\Relations\HasManyThrough.

                            $user->save();
                        }
                    }
                    break;

                case 'invoice.payment_succeeded':
                    $user = User::where('stripe_id', $data['data']['object']['customer'])->first();
                    if ($user) {
                        $plan_nickname = $data['data']['object']['lines']['data'][0]['plan']['nickname'];
                        $roles = Role::where('name', strtolower((string) $plan_nickname))->first();
                        if ($roles) {
                            $user->role_id = $roles->id;
                            $user->save();
                        }
                    }
                    break;
                case 'invoice.payment_failed':
                    $user = User::where('stripe_id', $data['data']['object']['customer'])->first();
                    if ($user) {
                        $roles = Role::where('name', strtolower('free'))->first();
                        if ($roles) {
                            $user->role_id = $roles->id;
                            $user->save();
                        }
                    }
                    break;
            }
        } else {
            echo 'User not found!';
        }

        return true;

The expression return true returns the type true which is incompatible with the documented return type Illuminate\Http\Response.

    }
}