SlevomatCsobGateway\Call\ClosePaymentRequest::send()

Complexity

Conditions	3
Paths	2

Size

Total Lines	39
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	19
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

<?php declare(strict_types = 1);
	public function send(ApiClient $apiClient): PaymentResponse
	{
		$data = [
			'merchantId' => $this->merchantId,
			'payId' => $this->payId,
		];

		if ($this->totalAmount !== null) {
			$data['totalAmount'] = $this->totalAmount;
		}

		$response = $apiClient->put(
			'payment/close',
			$data,
			new SignatureDataFormatter([
				'merchantId' => null,
				'payId' => null,
				'dttm' => null,
				'totalAmount' => null,
			]),
			new SignatureDataFormatter([
				'payId' => null,
				'dttm' => null,
				'resultCode' => null,
				'resultMessage' => null,
				'paymentStatus' => null,
				'authCode' => null,
			])
		);

		$data = $response->getData();

		return new PaymentResponse(
			$data['payId'],
			DateTimeImmutable::createFromFormat('YmdHis', $data['dttm']),

It seems like DateTimeImmutable::createFromFormat('YmdHis', $data['dttm']) can also be of type false; however, parameter $responseDateTime of SlevomatCsobGateway\Call\PaymentResponse::__construct() does only seem to accept DateTimeImmutable, maybe add an additional type check?

			ResultCode::get($data['resultCode']),
			$data['resultMessage'],
			isset($data['paymentStatus']) ? PaymentStatus::get($data['paymentStatus']) : null,
			$data['authCode'] ?? null
		);