Madkom\RegistryApplication\Application\CarManagement\Command\Car\AddCarDocumentCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	execute()	0	15	1

<?php

namespace Madkom\RegistryApplication\Application\CarManagement\Command\Car;

use Madkom\RegistryApplication\Application\CarManagement\Command\CommandInterface;
use Madkom\RegistryApplication\Application\CarManagement\DocumentDTO;
use Madkom\RegistryApplication\Domain\CarManagement\DocumentFactory;
use Madkom\RegistryApplication\Infrastructure\CarManagement\CarInMemoryRepository;

class AddCarDocumentCommand implements CommandInterface
{
    private $preparedDocument;
    private $repository;
    private $carId;

    public function __construct(CarInMemoryRepository $repository, $carId, DocumentDTO $preparedDocument)
    {
        $this->preparedDocument = $preparedDocument;
        $this->repository = $repository;
        $this->carId = $carId;
    }

    public function execute()
    {
        $document = new DocumentFactory();
        $carDocument = $document->create(DocumentFactory::CAR_DOCUMENT,
                                            $this->preparedDocument->docId,
                                            $this->preparedDocument->title,
                                            $this->preparedDocument->description,
                                            $this->preparedDocument->source
        );

        $car = $this->repository->find($this->carId);
        $car->addCarDocument($carDocument);

It seems like $carDocument defined by $document->create(\Madkom\RegistryApplication\Domain\CarManagement\DocumentFactory::CAR_DOCUMENT, $this->preparedDocument->docId, $this->preparedDocument->title, $this->preparedDocument->description, $this->preparedDocument->source) on line 26 can also be of type null or object<Madkom\RegistryApplication\Domain\CarManagement\Insurances\InsuranceDocument>; however, Madkom\RegistryApplication\Domain\CarManagement\Car::addCarDocument() does only seem to accept object<Madkom\RegistryApplication\Domain\CarManagement\CarDocument>, maybe add an additional type check?

        $this->repository->add($car);
    }
}