SlevomatCsobGateway\Call\Button\PaymentButtonRequest::send()

Complexity

Conditions	4
Paths	4

Size

Total Lines	68
Code Lines	54

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php declare(strict_types = 1);
	public function send(ApiClient $apiClient): PaymentButtonResponse
	{
		$requestData = [
			'merchantId' => $this->merchantId,
			'orderNo' => $this->orderId,
			'clientIp' => $this->clientIp,
			'totalAmount' => $this->totalPrice->getAmount(),
			'currency' => $this->totalPrice->getCurrency()->getValue(),
			'returnUrl' => $this->returnUrl,
			'returnMethod' => $this->returnMethod->getValue(),
			'brand' => $this->brand->getValue(),
			'language' => $this->language->getValue(),
		];

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

		$response = $apiClient->post(
			'button/init',
			$requestData,
			new SignatureDataFormatter([
				'merchantId' => null,
				'orderNo' => null,
				'dttm' => null,
				'clientIp' => null,
				'totalAmount' => null,
				'currency' => null,
				'returnUrl' => null,
				'returnMethod' => null,
				'brand' => null,
				'merchantData' => null,
				'language' => null,
			]),
			new SignatureDataFormatter([
				'payId' => null,
				'dttm' => null,
				'resultCode' => null,
				'resultMessage' => null,
				'paymentStatus' => null,
				'redirect' => [
					'method' => null,
					'url' => null,
					'params' => null,
				],
			])
		);

		$data = $response->getData();

		$redirectUrl = null;
		$redirectMethod = null;
		$redirectParams = [];
		if (isset($data['redirect'])) {
			$redirectUrl = $data['redirect']['url'];
			$redirectMethod = HttpMethod::get($data['redirect']['method']);
			$redirectParams = $data['redirect']['params'] ?? null;
		}

		return new PaymentButtonResponse(
			$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\Button\PaymentButtonResponse::__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,
			$redirectMethod,
			$redirectUrl,
			$redirectParams
		);