SprykerEco\Zed\Afterpay\Business\Payment\Transaction\Logger\TransactionLogger

Complexity

Total Complexity

3

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	logTransaction()	0	14	1
A	__construct()	0	3	1
A	getRequestTransferEncoded()	0	3	1

<?php

/**
 * MIT License
 * For full license information, please view the LICENSE file that was distributed with this source code.
 */

namespace SprykerEco\Zed\Afterpay\Business\Payment\Transaction\Logger;

use Generated\Shared\Transfer\AfterpayApiResponseTransfer;

The type Generated\Shared\Transfer\AfterpayApiResponseTransfer was not found. Maybe you did not declare it correctly or list all dependencies?

use Orm\Zed\Afterpay\Persistence\SpyPaymentAfterpayTransactionLog;

The type Orm\Zed\Afterpay\Persistence\SpyPaymentAfterpayTransactionLog was not found. Maybe you did not declare it correctly or list all dependencies?

use Spryker\Shared\Kernel\Transfer\AbstractTransfer;
use SprykerEco\Zed\Afterpay\Dependency\Service\AfterpayToUtilEncodingInterface;

class TransactionLogger implements TransactionLoggerInterface
{
    /**
     * @var \SprykerEco\Zed\Afterpay\Dependency\Service\AfterpayToUtilEncodingInterface
     */
    protected $utilEncoding;

    /**
     * @param \SprykerEco\Zed\Afterpay\Dependency\Service\AfterpayToUtilEncodingInterface $utilEncoding
     */
    public function __construct(AfterpayToUtilEncodingInterface $utilEncoding)
    {
        $this->utilEncoding = $utilEncoding;
    }

    /**
     * @param string $transactionType
     * @param string $orderReference
     * @param \Spryker\Shared\Kernel\Transfer\AbstractTransfer $transactionRequest
     * @param \Generated\Shared\Transfer\AfterpayApiResponseTransfer $transactionResponse
     *
     * @return void
     */
    public function logTransaction(
        string $transactionType,
        string $orderReference,
        AbstractTransfer $transactionRequest,
        AfterpayApiResponseTransfer $transactionResponse
    ): void {
        $transactionLog = new SpyPaymentAfterpayTransactionLog();
        $transactionLog
            ->setOrderReference($orderReference)
            ->setTransactionType($transactionType)
            ->setOutcome($transactionResponse->getOutcome())
            ->setRequestPayload($this->getRequestTransferEncoded($transactionRequest))
            ->setResponsePayload($transactionResponse->getResponsePayload())
            ->save();
    }

    /**
     * @param \Spryker\Shared\Kernel\Transfer\AbstractTransfer $requestTransfer
     *
     * @return string
     */
    protected function getRequestTransferEncoded(AbstractTransfer $requestTransfer): string
    {
        return $this->utilEncoding->encodeJson($requestTransfer->toArray());

The expression return $this->utilEncoding->encodeJson($requestTransfer->toArray()) could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}