Omnipay\Payline\AbstractGateway

Complexity

Total Complexity

23

Size/Duplication

Total Lines	194
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Test Coverage

Coverage

90.63%

Importance

Changes	3
Bugs	0	Features	0

19 Methods

Rating	Name	Duplication	Size	Complexity
	getEndPoint()	0	1	?
A	__construct()	0	6	1
A	createRequest()	0	9	3
A	getDefaultParameters()	0	13	1
A	getMerchantId()	0	4	1
A	setMerchantId()	0	4	1
A	getAccessKey()	0	4	1
A	setAccessKey()	0	4	1
A	getContractNumber()	0	4	1
A	setContractNumber()	0	4	1
A	getProxyHost()	0	4	1
A	setProxyHost()	0	4	1
A	getProxyPort()	0	4	1
A	setProxyPort()	0	4	1
A	getProxyLogin()	0	4	1
A	setProxyLogin()	0	4	1
A	getProxyPassword()	0	4	1
A	setProxyPassword()	0	4	1
A	getDefaultHttpClient()	0	20	4

<?php

/*
 * WebMoney driver for the Omnipay PHP payment processing library
 *
 * @link      https://github.com/ck-developer/omnipay-payline
 * @package   omnipay-payline
 * @license   MIT
 * @copyright Copyright (c) 2016 Claude Khedhiri <claude@khedhiri.com>
 */

namespace Omnipay\Payline;

use Symfony\Component\HttpFoundation\Request as HttpRequest;

abstract class AbstractGateway extends \Omnipay\Common\AbstractGateway
{
    protected $liveEndpoint = 'http://www.payline.com/wsdl/v4_0/production';
    protected $testEndpoint = 'http://www.payline.com/wsdl/v4_0/homologation';

    abstract public function getEndPoint();

    /**
     * AbstractGateway constructor.
     *
     * @param \SoapClient|null $httpClient
     * @param HttpRequest|null $httpRequest
     */
    public function __construct(\SoapClient $httpClient = null, HttpRequest $httpRequest = null)
    {
        $this->httpClient = $httpClient;

It seems like $httpClient can also be of type object<SoapClient>. However, the property $httpClient is declared as type object<Guzzle\Http\ClientInterface>. Maybe add an additional type check?

        $this->httpRequest = $httpRequest;
        $this->initialize();
    }

    protected function createRequest($class, array $parameters)
    {
        $this->httpClient = $this->httpClient ?: $this->getDefaultHttpClient();

It seems like $this->httpClient ?: $this->getDefaultHttpClient() can also be of type object<SoapClient>. However, the property $httpClient is declared as type object<Guzzle\Http\ClientInterface>. Maybe add an additional type check?

        $this->httpRequest = $this->httpRequest ?: $this->getDefaultHttpRequest();

        $obj = new $class($this->httpClient, $this->httpRequest);

        return $obj->initialize(array_replace($this->getParameters(), $parameters));
    }

    public function getDefaultParameters()
    {
        return array(
            'merchantId' => '',
            'accessKey' => '',
            'proxyHost' => '',
            'proxyPort' => '',
            'proxyLogin' => '',
            'proxyPassword' => '',
            'contractNumber' => '',
            'testMode' => true,
        );
    }

    /**
     * @return string
     */
    public function getMerchantId()
    {
        return $this->getParameter('merchantId');
    }

    /**
     * @param string $merchantId
     *
     * @return $this
     */
    public function setMerchantId($merchantId)
    {
        return $this->setParameter('merchantId', $merchantId);
    }

    /**
     * @return string
     */
    public function getAccessKey()
    {
        return $this->getParameter('accessKey');
    }

    /**
     * @param string $accessKey
     *
     * @return $this
     */
    public function setAccessKey($accessKey)
    {
        return $this->setParameter('accessKey', $accessKey);
    }

    /**
     * @return string
     */
    public function getContractNumber()
    {
        return $this->getParameter('contractNumber');
    }

    /**
     * @param string $contractNumber
     *
     * @return $this
     */
    public function setContractNumber($contractNumber)
    {
        return $this->setParameter('contractNumber', $contractNumber);
    }

    /**
     * @return string
     */
    public function getProxyHost()
    {
        return $this->getParameter('proxyHost');
    }

    /**
     * @param string $proxyHost
     *
     * @return $this
     */
    public function setProxyHost($proxyHost)
    {
        return $this->setParameter('proxyHost', $proxyHost);
    }

    /**
     * @return string
     */
    public function getProxyPort()
    {
        return $this->getParameter('proxyPort');
    }

    /**
     * @param string $proxyPort
     *
     * @return $this
     */
    public function setProxyPort($proxyPort)
    {
        return $this->setParameter('proxyPort', $proxyPort);
    }

    /**
     * @return string
     */
    public function getProxyLogin()
    {
        return $this->getParameter('proxyLogin');
    }

    /**
     * @param string $proxyLogin
     *
     * @return $this
     */
    public function setProxyLogin($proxyLogin)
    {
        return $this->setParameter('proxyLogin', $proxyLogin);
    }

    /**
     * @return string
     */
    public function getProxyPassword()
    {
        return $this->getParameter('proxyPassword');
    }

    /**
     * @param string $proxyPassword
     *
     * @return $this
     */
    public function setProxyPassword($proxyPassword)
    {
        return $this->setParameter('proxyPassword', $proxyPassword);
    }

    /**
     * @return \SoapClient
     */
    public function getDefaultHttpClient()
    {
        $header = array(
            'Content-Type' => 'text/xml; charset=utf-8',
            'login' => $this->getMerchantId(),
            'password' => $this->getAccessKey(),
            'style' => defined(SOAP_DOCUMENT) ? SOAP_DOCUMENT : 2,
            'use' => defined(SOAP_LITERAL) ? SOAP_LITERAL : 2,
            'connection_timeout' => 5,
        );

        if ($this->getProxyHost()) {
            $header['proxy_host'] = $this->getProxyHost();
            $header['proxy_port'] = $this->getProxyPort();
            $header['proxy_login'] = $this->getProxyLogin();
            $header['proxy_password'] = $this->getProxyPassword();
        }

        return new \SoapClient($this->getEndPoint(), $header);
    }
}