App\Http\Controllers\Stripe\Subscribe::__invoke()

Complexity

Conditions	6
Paths	10

Size

Total Lines	40
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function __invoke(Request $request)
    {
        $user = auth()->user();
        \Stripe\Stripe::setApiKey(config('cashier.secret'));

        try {
            $user->createAsStripeCustomer();

The method createAsStripeCustomer() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

        } catch(\Exception) {

Consider adding a comment why this CATCH block is empty.

        }

        $plan_id = $request->input('plan_id');

        if ($request->has('payment_method')) {
            $paymentMethod = $request->payment_method;

            $subscription = $user->newSubscription('default', $plan_id)

The method newSubscription() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

                                 ->trialDays(14);

            if ($couponId = $request->input('coupon_id')) {
                $subscription->withCoupon($couponId);
            }

            $subscription->create($paymentMethod);

            $user->notify(new SubscribeSuccessfully($plan_id));

The method notify() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

        } else {
            if ($user->subscribed('default')) {

The method subscribed() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

                $subscription = $user->subscription();

The method subscription() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

                if ($subscription->stripe_status == 'canceled') {
                    $user->newSubscription('default', $plan_id)->create();
                    $user->notify(new SubscribeSuccessfully($plan_id));
                } else {
                    $user->subscription('default')->swap($plan_id);
                }
            } else {
                $user->subscription('default')->swap($plan_id);
            }
        }

        return ['success' => true];

The expression return array('success' => true) returns the type array<string,true> which is incompatible with the documented return type Illuminate\Http\Response.

    }