ByTIC\Payments\Stripe\Message\CompletePurchaseResponse

Complexity

Total Complexity

9

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	31	4
A	canProcessModel()	0	3	1
A	getCode()	0	3	1
A	getSessionDebug()	0	6	1
A	getMessage()	0	3	1
A	isSuccessful()	0	3	1

<?php

namespace ByTIC\Payments\Stripe\Message;

use ByTIC\Omnipay\Common\Message\Traits\HtmlResponses\ConfirmHtmlTrait;
use Omnipay\Common\Message\AbstractResponse;
use ByTIC\Payments\Gateways\Providers\AbstractGateway\Message\Traits\CompletePurchaseResponseTrait;
use Stripe\PaymentIntent;

/**
 * Class CompletePurchaseResponse
 * @package ByTIC\Payments\Gateways\Providers\Paylike\Message
 */
class CompletePurchaseResponse extends AbstractResponse
{
    use ConfirmHtmlTrait;
    use CompletePurchaseResponseTrait;

    /**
     * @var bool
     */
    private $successful = false;

    /**
     * @inheritDoc
     */
    public function __construct(CompletePurchaseRequest $request, $data)
    {
        parent::__construct($request, $data);

        if (isset($data['notification']['paymentIntent'])) {
            /** @var \Stripe\PaymentIntent $paymentIntent */
            $paymentIntent = $data['notification']['paymentIntent'];

            $this->internalTransactionRef = $paymentIntent->id;

The property internalTransactionRef does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            // Amazingly there doesn't seem to be a simple code nor message when the payment succeeds (or fails).
            // For now, just use the status for the message, and leave the code blank.
            switch ($paymentIntent->status) {
                case PaymentIntent::STATUS_SUCCEEDED:
                    $this->successful = true;
                    $this->message = $paymentIntent->status;

The property message does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                    break;
                case PaymentIntent::STATUS_CANCELED:
                    $this->successful = false;
                    $this->message = 'Canceled by customer';
                    break;
                default:
                    // We don't know what happened, so act accordingly. Would be nice to make this better over time.
                    $this->successful = false;
                    $this->message = 'Unknown error';
                    break;

            }
        } else {
            $this->successful = false; // Just make sure.
            $this->message = 'Could not retrieve payment';
        }
    }

    public function isSuccessful()
    {
        return $this->successful;
    }

    public function getMessage()
    {
        return $this->message;
    }

    public function getCode()
    {
        return $this->code;

The property code does not exist on ByTIC\Payments\Stripe\Message\CompletePurchaseResponse. Did you maybe forget to declare it?

    }


    /** @noinspection PhpMissingParentCallCommonInspection
     * @return bool
     */
    protected function canProcessModel()
    {
        return true;
    }

    /**
     * @return []

The doc comment [] at position 0 could not be parsed: Unknown type name '[' at position 0 in [].

     */
    public function getSessionDebug(): array
    {
        $notification = $this->getDataProperty('notification');
        return [
            'session' => $notification['session']->toArray(),
            'paymentIntent' => $notification['paymentIntent']->toArray()
        ];
    }
}