Firegento\DevDashboard\Model\ConfigRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	save()	0	9	2
A	__construct()	0	8	1
A	getByUserId()	0	7	1

<?php

namespace Firegento\DevDashboard\Model;

use Firegento\DevDashboard\Api\Data\ConfigInterface;
use Firegento\DevDashboard\Model\ResourceModel\Config\CollectionFactory;

The type Firegento\DevDashboard\Model\ResourceModel\Config\CollectionFactory was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Framework\Exception\CouldNotSaveException;
use Magento\Framework\Model\AbstractModel;

class ConfigRepository implements \Firegento\DevDashboard\Api\ConfigRepositoryInterface
{
    protected $objectFactory;
    protected $collectionFactory;
    /**
     * @var ResourceModel\Config
     */
    private $resource;

    public function __construct(
        ConfigFactory $objectFactory,

The type Firegento\DevDashboard\Model\ConfigFactory was not found. Maybe you did not declare it correctly or list all dependencies?

        CollectionFactory $collectionFactory,
        ResourceModel\Config $resource
    ) {
        $this->objectFactory = $objectFactory;
        $this->collectionFactory = $collectionFactory;
        $this->resource = $resource;
    }

    /**
     * @param ConfigInterface|AbstractModel $object
     * @return ConfigInterface
     * @throws CouldNotSaveException
     */
    public function save(ConfigInterface $object)
    {
        try {
            $this->resource->save($object);
        } catch (\Exception $e) {
            throw new CouldNotSaveException(__($e->getMessage()));
        }

        return $object;

The expression return $object returns the type Magento\Framework\Model\AbstractModel which is incompatible with the documented return type Firegento\DevDashboard\Api\Data\ConfigInterface.

    }

    /**
     * @param int $userId
     * @return Config
     */
    public function getByUserId($userId)
    {
        /** @var \Firegento\DevDashboard\Model\Config $object */
        $object = $this->objectFactory->create();
        $this->resource->load($object, $userId, 'user_id');
        $object->setData('user_id', $userId);
        return $object;
    }
}