CommerceLeague\ActiveCampaign\MessageQueue\Customer\CreateConsumer

Complexity

Total Complexity

5

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	consume()	0	24	4

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

namespace CommerceLeague\ActiveCampaign\MessageQueue\Customer;

use CommerceLeague\ActiveCampaign\Api\CustomerRepositoryInterface;
use CommerceLeague\ActiveCampaign\Helper\Client as ClientHelper;
use CommerceLeague\ActiveCampaign\Logger\Logger;
use CommerceLeague\ActiveCampaignApi\Exception\HttpException;
use Magento\Framework\Exception\CouldNotSaveException;

/**
 * Class CreateConsumer
 */
class CreateConsumer
{
    /**
     * @var CustomerRepositoryInterface
     */
    private $customerRepository;

    /**
     * @var Logger
     */
    private $logger;

    /**
     * @var ClientHelper
     */
    private $clientHelper;

    /**
     * @param CustomerRepositoryInterface $customerRepository
     * @param Logger $logger
     * @param ClientHelper $clientHelper
     */
    public function __construct(
        CustomerRepositoryInterface $customerRepository,
        Logger $logger,
        ClientHelper $clientHelper
    ) {
        $this->customerRepository = $customerRepository;
        $this->logger = $logger;
        $this->clientHelper = $clientHelper;
    }

    /**
     * @param CreateMessage $message
     */
    public function consume(CreateMessage $message): void
    {
        $customer = $this->customerRepository->getById($message->getEntityId());

        if (!$customer->getId()) {

The expression $customer->getId() of type integer|null is loosely compared to false; this is ambiguous if the integer can be 0. You might want to explicitly use === null instead.

            $this->logger->error(__('Unable to find customer with id "%1".', $message->getEntityId()));
            return;
        }

        $request = json_decode($message->getSerializedRequest(), true);

        try {
            $apiResponse = $this->clientHelper->getCustomerApi()->create(['ecomCustomer' => $request]);
        } catch (HttpException $e) {
            $this->logger->error($e->getMessage());
            return;
        }

        $customer->setActiveCampaignId($apiResponse['ecomCustomer']['id']);

        try {
            $this->customerRepository->save($customer);
        } catch (CouldNotSaveException $e) {
            $this->logger->error($e->getMessage());
        }
    }
}