CommerceLeague\ActiveCampaign\Service\ExportOrderService

Complexity

Total Complexity

2

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	export()	0	8	1
A	__construct()	0	6	1

<?php
declare(strict_types=1);
/**
 */

namespace CommerceLeague\ActiveCampaign\Service;

use CommerceLeague\ActiveCampaign\Gateway\Request\OrderBuilder as OrderRequestBuilder;
use CommerceLeague\ActiveCampaign\MessageQueue\Topics;
use Magento\Framework\MessageQueue\PublisherInterface;
use Magento\Sales\Model\Order as MagentoOrder;

class ExportOrderService
{
    /**
     * @var OrderRequestBuilder
     */
    private $orderRequestBuilder;

    /**
     * @var PublisherInterface
     */
    private $publisher;

    /**
     * @param OrderRequestBuilder $orderRequestBuilder
     * @param PublisherInterface $publisher
     */
    public function __construct(
        OrderRequestBuilder $orderRequestBuilder,
        PublisherInterface $publisher
    ) {
        $this->orderRequestBuilder = $orderRequestBuilder;
        $this->publisher = $publisher;
    }

    public function export(MagentoOrder $magentoOrder): void
    {
        $data = [
            'magento_order_id' => $magentoOrder->getId(),
            'request' => $this->orderRequestBuilder->build($magentoOrder)
        ];

        $this->publisher->publish(Topics::ORDER_EXPORT, json_encode($data));

json_encode($data) of type string is incompatible with the type array|object expected by parameter $data of Magento\Framework\MessageQueue\PublisherInterface::publish().

    }
}