SlevomatCsobGateway\Call\Extension\MaskedCardNumberExtension

Complexity

Total Complexity

3

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

77.78%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	createResponse()	0	6	1
A	getSignatureDataFormatter()	0	3	1
A	parseExpiration()	0	3	1

<?php declare(strict_types = 1);

namespace SlevomatCsobGateway\Call\Extension;

use DateTimeImmutable;
use SlevomatCsobGateway\Call\ResponseExtensionHandler;
use SlevomatCsobGateway\Crypto\SignatureDataFormatter;
use function array_flip;

class MaskedCardNumberExtension implements ResponseExtensionHandler
{

	public const NAME = 'maskClnRP';

	/**
	 * @param mixed[] $data
	 * @return MaskedCardNumberResponse
	 */
	public function createResponse(array $data): MaskedCardNumberResponse
	{
		return new MaskedCardNumberResponse(
			$data['longMaskedCln'],
			$data['maskedCln'],
			$this->parseExpiration($data['expiration'])
		);
	}

	public function getSignatureDataFormatter(): SignatureDataFormatter
	{
		return new SignatureDataFormatter(array_flip(['extension', 'dttm', 'maskedCln', 'expiration', 'longMaskedCln']));
	}

	private function parseExpiration(string $expiration): DateTimeImmutable
	{
		return DateTimeImmutable::createFromFormat('d/m/y', '01/' . $expiration)->modify('last day of this month today');

The expression return DateTimeImmutable::createFromFormat('d/m/y', '01/' . $expiration)->modify('last day of this month today') could return the type false which is incompatible with the type-hinted return DateTimeImmutable. Consider adding an additional type-check to rule them out.

	}

}