App\Shop\PaymentMethods\Stripe\StripeRepository::execute()

Complexity

Conditions	3
Paths	12

Size

Total Lines

35

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function execute(array $data, $total, $tax) : Charge
    {
        try {
            $courierRepo = new CourierRepository(new Courier);
            $courierId = $data['courier'];
            $courier = $courierRepo->findCourierById($courierId);

            $totalComputed = $total + $courier->cost;

            $customerRepo = new CustomerRepository($this->customer);
            $options['source'] = $data['stripeToken'];

$options was never initialized. Although not strictly required by PHP, it is generally a good practice to add $options = array(); before regardless.

            $options['currency'] = config('cart.currency');

            if ($charge = $customerRepo->charge($totalComputed, $options)) {

$totalComputed of type double is incompatible with the type integer expected by parameter $amount of App\Shop\Customers\Repositories\CustomerRepository::charge().

                $checkoutRepo = new CheckoutRepository;
                $checkoutRepo->buildCheckoutItems([
                    'reference' => Uuid::uuid4()->toString(),
                    'courier_id' => $courierId,
                    'customer_id' => $this->customer->id,
                    'address_id' => $data['billing_address'],
                    'order_status_id' => 1,
                    'payment' => strtolower(config('stripe.name')),
                    'discounts' => 0,
                    'total_products' => $total,
                    'total' => $totalComputed,
                    'total_paid' => $totalComputed,
                    'tax' => $tax
                ]);

                Cart::destroy();
            }

            return $charge;
        } catch (\Exception $e) {
            throw new StripeChargingErrorException($e);
        }