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

Complexity

Conditions	6
Paths	10

Size

Total Lines	36
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

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

        try {
            $user->createAsStripeCustomer();
        } 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)
                                 ->trialDays(14);
            if ($couponId = $request->input('coupon_id')) {
                $subscription->withCoupon($couponId);
            }
            $subscription->create($paymentMethod);
            $user->notify(new SubscribeSuccessfully($plan_id));
        } elseif ($user->subscribed('default')) {
            $subscription = $user->subscription();
            if ($subscription->stripe_status == 'canceled') {
                $user->newSubscription('default', $plan_id)
                    ->trialDays(14)
                    ->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.

    }