BitBag\SyliusInvoicingPlugin\Resolver\InvoiceFileResolver

Complexity

Total Complexity

5

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	resolveInvoicePath()	0	11	4

<?php

/*
 * This file has been created by developers from BitBag.
 * Feel free to contact us once you face any issues or want to start
 * another great project.
 * You can find more information about us on https://bitbag.shop and write us
 * an email on mikolaj.krol@bitbag.pl.
 */

declare(strict_types=1);

namespace BitBag\SyliusInvoicingPlugin\Resolver;

use BitBag\SyliusInvoicingPlugin\Entity\InvoiceInterface;
use BitBag\SyliusInvoicingPlugin\FileGenerator\FileGeneratorInterface;
use BitBag\SyliusInvoicingPlugin\Repository\InvoiceRepositoryInterface;
use Doctrine\ORM\EntityManagerInterface;

final class InvoiceFileResolver implements InvoiceFileResolverInterface
{
    /** @var InvoiceRepositoryInterface */
    private $invoiceFileGenerator;

    /** @var InvoiceRepositoryInterface */
    private $invoiceRepository;

    /** @var InvoiceRepositoryInterface */
    private $invoiceEntityManager;

    /** @var InvoiceRepositoryInterface */
    private $environment;

    public function __construct(
        FileGeneratorInterface $invoiceFileGenerator,
        InvoiceRepositoryInterface $invoiceRepository,
        EntityManagerInterface $invoiceEntityManager,
        string $environment
    ) {
        $this->invoiceFileGenerator = $invoiceFileGenerator;

$invoiceFileGenerator is of type object<BitBag\SyliusInvoicingPlugin\FileGenerator\FileGeneratorInterface>, but the property $invoiceFileGenerator was declared to be of type object<BitBag\SyliusInvoicingPlugin\Repository\InvoiceRepositoryInterface>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->invoiceRepository = $invoiceRepository;
        $this->invoiceEntityManager = $invoiceEntityManager;

$invoiceEntityManager is of type object<Doctrine\ORM\EntityManagerInterface>, but the property $invoiceEntityManager was declared to be of type object<BitBag\SyliusInvoicingPlugin\Repository\InvoiceRepositoryInterface>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->environment = $environment;

It seems like $environment of type string is incompatible with the declared type object<BitBag\SyliusInvoicingPlugin\Repository\InvoiceRepositoryInterface> of property $environment.

    }

    public function resolveInvoicePath(InvoiceInterface $invoice): string
    {
        if (null === $invoice->getPath() || (null !== $invoice->getPath() && 'prod' !== $this->environment)) {
            $invoiceFilename = $this->invoiceFileGenerator->generateFile($invoice);

            $invoice->setPath($invoiceFilename);
            $this->invoiceEntityManager->flush();
        }

        return $this->invoiceFileGenerator->getFilesDirectoryPath() . DIRECTORY_SEPARATOR . $invoice->getPath();
    }
}