Ticaje\BookingApi\Application\Service\Provider\DisabledDaysProvider

Complexity

Total Complexity

5

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	5	1
A	transform()	0	13	3
A	__construct()	0	6	1

<?php
declare(strict_types=1);
/**
 * Application Service Class
 * @package Ticaje_BookingApi
 * @author  Hector Luis Barrientos <ticaje@filetea.me>
 */

namespace Ticaje\BookingApi\Application\Service\Provider;

use Exception;
use Ticaje\BookingApi\Application\Signatures\Provider\DisabledDaysGatewaySignature;
use Ticaje\BookingApi\Application\Signatures\Provider\DisabledDaysProviderSignature;
use Ticaje\BookingApi\Domain\Policies\Calendar\Disabling\CQRS\Query;
use Ticaje\BookingApi\Domain\Policies\Calendar\Disabling\CQRS\QuerySignature;
use Ticaje\Contract\Application\Service\ServiceLocatorInterface;
use Ticaje\Hexagonal\Application\Signatures\UseCase\UseCaseCommandInterface;

/**
 * Class DisabledDaysProvider
 * @package Ticaje\BookingApi\Application\Service\Provider
 */
class DisabledDaysProvider implements DisabledDaysProviderSignature
{
    /** @var ServiceLocatorInterface */
    private $serviceLocator;

    /** @var DisabledDaysGatewaySignature */
    private $disabledDaysGatewaySignature;

    /**
     * DisabledDaysProvider constructor.
     *
     * @param ServiceLocatorInterface      $serviceLocator
     * @param DisabledDaysGatewaySignature $disabledDaysGatewaySignature
     */
    public function __construct(
        ServiceLocatorInterface $serviceLocator,
        DisabledDaysGatewaySignature $disabledDaysGatewaySignature)
    {
        $this->serviceLocator = $serviceLocator;
        $this->disabledDaysGatewaySignature = $disabledDaysGatewaySignature;
    }

    /**
     * @inheritDoc
     */
    public function execute(UseCaseCommandInterface $command)
    {
        $items = $this->disabledDaysGatewaySignature->execute($command);

        return $this->transform($items, $command->getFormat());

The method getFormat() does not exist on Ticaje\Hexagonal\Application\Signatures\UseCase\UseCaseCommandInterface. It seems like you code against a sub-type of Ticaje\Hexagonal\Application\Signatures\UseCase\UseCaseCommandInterface such as Ticaje\BookingApi\Application\UseCase\Command\DisabledDaysCommand or Ticaje\BookingApi\Application\UseCase\Command\GetPickupLocationCommand or Ticaje\BookingApi\Application\UseCase\Command\GetPriceCommand or Ticaje\BookingApi\Application\UseCase\Command\GetPackageCommand or Ticaje\BookingApi\Application\UseCase\Command\GetAvailabilityCommand.

    }

    /**
     * @param array $items
     * @param       $format
     *
     * @return string
     */
    private function transform(array $items, $format): string
    {
        /** @var QuerySignature $domainFetcher */
        $domainFetcher = $this->serviceLocator->get(Query::class, ['format' => $format]);

The call to Ticaje\Contract\Application\Service\ServiceLocatorInterface::get() has too many arguments starting with array('format' => $format).

        foreach ($items as $item) {
            try {
                $domainFetcher->interpret($item->getRule(), $item->getType());
            } catch (Exception $exception) {
                // Log properly
            }
        }

        return json_encode($domainFetcher->fetchList());
    }
}