CommerceLeague\ActiveCampaign\Model\ContactRepository

Complexity

Total Complexity

11

Size/Duplication

Total Lines	109
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	deleteById()	0	11	2
A	__construct()	0	6	1
A	save()	0	9	2
A	getById()	0	7	1
A	getOrCreateByCustomer()	0	10	2
A	getByCustomerId()	0	7	1
A	delete()	0	9	2

<?php
declare(strict_types=1);
/**
 */
namespace CommerceLeague\ActiveCampaign\Model;

use CommerceLeague\ActiveCampaign\Api\ContactRepositoryInterface;
use CommerceLeague\ActiveCampaign\Api\Data;
use CommerceLeague\ActiveCampaign\Model\ResourceModel\Contact as ContactResource;
use Magento\Customer\Model\Customer;
use Magento\Framework\Exception\CouldNotDeleteException;
use Magento\Framework\Exception\CouldNotSaveException;
use Magento\Framework\Exception\NoSuchEntityException;
use Magento\Framework\Model\AbstractModel;

/**
 * Class ContactRepository
 */
class ContactRepository implements ContactRepositoryInterface
{
    /**
     * @var ContactResource
     */
    private $contactResource;

    /**
     * @var ContactFactory

The type CommerceLeague\ActiveCampaign\Model\ContactFactory was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    private $contactFactory;

    /**
     * @param ContactResource $contactResource
     * @param ContactFactory $contactFactory
     */
    public function __construct(
        ContactResource $contactResource,
        ContactFactory $contactFactory
    ) {
        $this->contactResource = $contactResource;
        $this->contactFactory = $contactFactory;
    }

    /**
     * @param Data\ContactInterface|AbstractModel $contact
     * @return Data\ContactInterface
     * @throws CouldNotSaveException
     */
    public function save(Data\ContactInterface $contact): Data\ContactInterface
    {
        try {
            $this->contactResource->save($contact);
        } catch (\Exception $e) {
            throw new CouldNotSaveException(__($e->getMessage()));
        }

        return $contact;

The expression return $contact returns the type Magento\Framework\Model\AbstractModel which is incompatible with the type-hinted return CommerceLeague\ActiveCampaign\Api\Data\ContactInterface.

    }

    /**
     * @inheritDoc
     */
    public function getById($contactId): Data\ContactInterface
    {
        /** @var Contact $contact */
        $contact = $this->contactFactory->create();
        $this->contactResource->load($contact, $contactId);

        return $contact;
    }

    /**
     * @inheritDoc
     */
    public function getByCustomerId($customerId): Data\ContactInterface
    {
        /** @var Contact $contact */
        $contact = $this->contactFactory->create();
        $this->contactResource->load($contact, $customerId, Data\ContactInterface::CUSTOMER_ID);

        return $contact;
    }

    /**
     * @inheritDoc
     */
    public function getOrCreateByCustomer(Customer $customer): Data\ContactInterface
    {
        $contact = $this->getByCustomerId($customer->getId());

        if (!$contact->getId()) {
            $contact->setCustomerId($customer->getId());
            $this->save($contact);
        }

        return $contact;
    }

    /**
     * @param Data\ContactInterface|AbstractModel $contact
     * @return bool
     * @throws CouldNotDeleteException
     */
    public function delete(Data\ContactInterface $contact): bool
    {
        try {
            $this->contactResource->delete($contact);
        } catch (\Exception $e) {
            throw new CouldNotDeleteException(__($e->getMessage()));
        }

        return true;
    }

    /**
     * @inheritDoc
     */
    public function deleteById($contactId): bool
    {
        $contact = $this->getById($contactId);

        if (!$contact->getId()) {
            throw new NoSuchEntityException(
                __('The Contact with the "%1" ID doesn\'t exist', $contactId)
            );
        }

        return $this->delete($contact);
    }
}