Setono\SyliusLagersystemPlugin\ViewRepository\Product\ProductVariantViewRepository

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getAllPaginated()	0	15	2

<?php

declare(strict_types=1);

namespace Setono\SyliusLagersystemPlugin\ViewRepository\Product;

use Pagerfanta\Adapter\DoctrineORMAdapter;
use Pagerfanta\Pagerfanta;
use Setono\SyliusLagersystemPlugin\Factory\PageViewFactoryInterface;
use Setono\SyliusLagersystemPlugin\Factory\Product\ProductVariantViewFactoryInterface;
use Setono\SyliusLagersystemPlugin\Model\PaginatorDetails;
use Setono\SyliusLagersystemPlugin\Repository\ProductVariantRepositoryInterface;
use Setono\SyliusLagersystemPlugin\View\PageView;
use Webmozart\Assert\Assert;

class ProductVariantViewRepository implements ProductVariantViewRepositoryInterface
{
    /** @var ProductVariantRepositoryInterface */
    protected $productVariantRepository;

    /** @var PageViewFactoryInterface */
    protected $pageViewFactory;

    /** @var ProductVariantViewFactoryInterface */
    protected $productVariantViewFactory;

    public function __construct(
        ProductVariantRepositoryInterface $productVariantRepository,
        PageViewFactoryInterface $pageViewFactory,
        ProductVariantViewFactoryInterface $productVariantViewFactory
    ) {
        $this->productVariantRepository = $productVariantRepository;
        $this->pageViewFactory = $pageViewFactory;
        $this->productVariantViewFactory = $productVariantViewFactory;
    }

    public function getAllPaginated(PaginatorDetails $paginatorDetails, ?string $localeCode): PageView
    {
        Assert::notNull($localeCode);
        $queryBuilder = $this->productVariantRepository->createLagersystemListQueryBuilder($localeCode);

It seems like $localeCode can also be of type null; however, parameter $locale of Setono\SyliusLagersystemPlugin\Repository\ProductVariantRepositoryInterface::createLagersystemListQueryBuilder() does only seem to accept string, maybe add an additional type check?

        $pagerfanta = new Pagerfanta(new DoctrineORMAdapter($queryBuilder));
        $pagerfanta->setMaxPerPage($paginatorDetails->limit());
        $pagerfanta->setCurrentPage($paginatorDetails->page());

        $pageView = $this->pageViewFactory->create($pagerfanta, $paginatorDetails->route(), $paginatorDetails->parameters());
        foreach ($pagerfanta->getCurrentPageResults() as $currentPageResult) {
            $pageView->items[] = $this->productVariantViewFactory->create($currentPageResult, $localeCode);

It seems like $localeCode can also be of type null; however, parameter $localeCode of Setono\SyliusLagersystemPlugin\Factory\Product\ProductVariantViewFactoryInterface::create() does only seem to accept string, maybe add an additional type check?

        }

        return $pageView;
    }
}