Ogone\FlexCheckout\FlexCheckoutPaymentRequest

Complexity

Total Complexity

14

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getCheckoutUrl()	0	4	1
A	getRequiredFields()	0	11	1
A	getValidOgoneUris()	0	4	1
A	setPspId()	0	4	1
A	setOrderId()	0	4	1
A	setAliasId()	0	4	1
A	setPm()	0	7	2
A	setAccepturl()	0	5	1
A	setExceptionurl()	0	5	1
A	setLanguage()	0	4	1
A	setShaSign()	0	4	1
A	getValidOperations()	0	4	1

<?php namespace Ogone\FlexCheckout;

use Ogone\AbstractPaymentRequest;
use Ogone\ShaComposer\ShaComposer;

class FlexCheckoutPaymentRequest extends AbstractPaymentRequest
{
	const TEST = "https://ogone.test.v-psp.com/Tokenization/HostedPage";
	const PRODUCTION = "https://secure.ogone.com/Tokenization/HostedPage";

	protected $payment_methods = [
		"CreditCard",
		"DirectDebit",
	];

	public function __construct(ShaComposer $shaComposer)
	{
		$this->shaComposer = $shaComposer;
		$this->ogoneUri    = self::TEST;
	}

	public function getCheckoutUrl()
	{
		return $this->getOgoneUri()."?". http_build_query($this->toArray());
	}

	public function getRequiredFields()
	{
		return array(
			'account.pspid',
			'alias.aliasid',
			'alias.orderid',
			'card.paymentmethod',
			'parameters.accepturl',
			'parameters.exceptionurl',
		);
	}

	public function getValidOgoneUris()
	{
		return array(self::TEST, self::PRODUCTION);
	}

	public function setPspId($pspid)
	{
		$this->parameters['account.pspid'] = $pspid;
	}

	public function setOrderId($orderid)
	{
		$this->parameters['alias.orderid'] = $orderid;
	}

	public function setAliasId(Alias $alias)
	{
		$this->parameters['alias.aliasid'] = $alias->getAlias();
	}

	public function setPm($payment_method)
	{
		if (!in_array($payment_method, $this->payment_methods)) {
			throw new \InvalidArgumentException("Unknown Payment method [$payment_method].");
		}
		$this->parameters['card.paymentmethod'] = $payment_method;
	}

	public function setAccepturl($accepturl)
	{
		$this->validateUri($accepturl);
		$this->parameters['parameters.accepturl'] = $accepturl;
	}

	public function setExceptionurl($exceptionurl)
	{
		$this->validateUri($exceptionurl);
		$this->parameters['parameters.exceptionurl'] = $exceptionurl;
	}

	public function setLanguage($language)
	{
		$this->parameters['layout.language'] = $language;
	}

	public function setShaSign()
	{
		$this->parameters['shasignature.shasign'] = parent::getShaSign();

It seems like you call parent on a different method (getShaSign() instead of setShaSign()). Are you sure this is correct? If so, you might want to change this to $this->getShaSign().

	}

	protected function getValidOperations()
	{
		return [];
	}
}