ByTIC\Payments\Actions\Transactions\ChargeWithToken::process()

Complexity

Conditions	2
Paths	2

Size

Total Lines	20
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public static function process($transaction)
    {
        $purchase = $transaction->getPurchase();
        $method = $transaction->getPaymentMethod();
        $gateway = $method->getGateway();

        if (!method_exists($gateway, 'purchaseWithToken')) {
            throw RequestNotSupportedException::create('purchaseWithToken', $gateway);

$gateway of type ByTIC\Payments\Gateways\Providers\AbstractGateway\Traits\GatewayTrait is incompatible with the type Omnipay\Common\AbstractGateway|null expected by parameter $gateway of ByTIC\Payments\Exception\RequestNotSupportedException::create().

        }

        $parameters = $purchase->getPurchaseParameters();
        $parameters['token'] = $transaction->getToken()->getTokenId();

        /** @var AbstractResponse $response */
        $response = $gateway->purchaseWithToken($parameters);

The method purchaseWithToken() does not exist on ByTIC\Payments\Gateways\Providers\AbstractGateway\Traits\GatewayTrait. Since you implemented __call, consider adding a @method annotation.

        $response->processModel();

The method processModel() does not exist on Omnipay\Common\Message\AbstractResponse. It seems like you code against a sub-type of Omnipay\Common\Message\AbstractResponse such as ByTIC\Payments\Stripe\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Twispay\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Twispay\Message\ServerCompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Payu\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Payu\Message\ServerCompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Paylike\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Euplatesc\Message\ServerCompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Euplatesc\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Romcard\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\PlatiOnline\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\PlatiOnline\Message\ServerCompletePurchaseResponse or ByTIC\Common\Payments\Gateways\Providers\AbstractGateway\Message\CompletePurchaseResponse or ByTIC\Common\Payments\Gateways\Providers\AbstractGateway\Message\ServerCompletePurchaseResponse or ByTIC\Payments\Mobilpay\Message\ServerCompletePurchaseResponse or ByTIC\Payments\Mobilpay\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Librapay\Message\CompletePurchaseResponse or ByTIC\Payments\Gateways\Providers\Librapay\Message\ServerCompletePurchaseResponse.

        UpdatePaymentModelsFromResponse::handle($response, $model, 'IPN');

The variable $model seems to be never defined.

        return $response;
    }
}