Ticaje\BookingApi\Application\Service\Provider\BasePriceProvider

Complexity

Total Complexity

7

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getBasePrice()	0	8	2
A	execute()	0	8	3
A	calculateDays()	0	6	1

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

namespace Ticaje\BookingApi\Application\Service\Provider;

use DateInterval;
use DateTime;
use Exception;
use Ticaje\BookingApi\Application\Signatures\Provider\PriceProviderGatewaySignature;
use Ticaje\BookingApi\Application\Signatures\Provider\PriceProviderSignature;
use Ticaje\BookingApi\Application\Signatures\Provider\ServiceProviderSignature;
use Ticaje\Contract\Patterns\Interfaces\Dto\DtoInterface;
use Ticaje\Contract\Persistence\Entity\EntityInterface;
use Ticaje\Hexagonal\Application\Signatures\UseCase\UseCaseCommandInterface;

/**
 * Class BasePriceProvider
 * @package Ticaje\BookingApi\Application\Service\Provider
 */
abstract class BasePriceProvider implements ServiceProviderSignature, PriceProviderSignature
{
    /** @var EntityInterface instance */
    protected $instance;

    /** @var PriceProviderGatewaySignature $priceProviderGatewaySignature */
    private $priceProviderGatewaySignature;

    /**
     * BasePriceProvider constructor.
     *
     * @param PriceProviderGatewaySignature $priceProviderGatewaySignature
     */
    public function __construct(PriceProviderGatewaySignature $priceProviderGatewaySignature)
    {
        $this->priceProviderGatewaySignature = $priceProviderGatewaySignature;
    }

    /**
     * @inheritDoc
     */
    public function execute(UseCaseCommandInterface $command)
    {
        $price = (float)0;
        if ($this->instance = $this->priceProviderGatewaySignature->execute($command)) {
            $price = $this->getPrice($command);
        }

        return (float)0 === $price ? $this->getBasePrice($command) : $price;
    }

    /**
     * @param UseCaseCommandInterface $command
     *
     * @return float|int
     * @throws Exception
     */
    protected function getBasePrice(UseCaseCommandInterface $command): float
    {
        /** @var DtoInterface $product */
        $product = $command->getProduct();

The method getProduct() 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.

        $days = $this->calculateDays($command);
        $result = $product ? $product->getPrice() * (int)$days->days : 0;

$product is of type Ticaje\Contract\Patterns\Interfaces\Dto\DtoInterface, thus it always evaluated to true.

        return (float)$result;
    }

    /**
     * @param UseCaseCommandInterface $command
     *
     * @return float
     */
    abstract public function getPrice(UseCaseCommandInterface $command): float;

    /**
     * @param UseCaseCommandInterface $command
     *
     * @return DateInterval|false
     * @throws Exception
     */
    protected function calculateDays(UseCaseCommandInterface $command)
    {
        $from = $command->getFromDate();

The method getFromDate() 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.

        $to = $command->getToDate();

The method getToDate() 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.

        return date_diff(new DateTime($to), new DateTime($from));
    }
}