ConferenceTools\Tickets\Domain\ValueObject\DiscountType\FixedPerTicket

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	27.27 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

40%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	apply()	0	5	1
A	getDiscount()	0	4	1
A	fromArray()	12	12	2

<?php

namespace ConferenceTools\Tickets\Domain\ValueObject\DiscountType;

use ConferenceTools\Tickets\Domain\Service\Configuration;
use ConferenceTools\Tickets\Domain\ValueObject\Basket;
use ConferenceTools\Tickets\Domain\ValueObject\Money;
use ConferenceTools\Tickets\Domain\ValueObject\Price;
use JMS\Serializer\Annotation as Jms;

class FixedPerTicket implements DiscountTypeInterface
{
    /**
     * @JMS\Type("Price")
     * @var Price
     */
    private $discount;

    /**
     * Percentage constructor.
     * @param $discount
     */
    public function __construct(Price $discount)
    {
        $this->discount = $discount;
    }

    public function apply(Basket $to): Price
    {
        $tickets = count($to->getTickets());
        return $this->discount->multiply($tickets);
    }

    /**
     * @return Price
     */
    public function getDiscount(): Price
    {
        return $this->discount;
    }

    public static function fromArray(array $data, Configuration $configuration): DiscountTypeInterface

This method seems to be duplicated in your project.

    {
        if (isset($data['net'])) {
            $amount = new Money($data['net'], $configuration->getCurrency());
            $discount = Price::fromNetCost($amount, $configuration->getTaxRate());
        } else {
            $amount = new Money($data['gross'], $configuration->getCurrency());
            $discount = Price::fromGrossCost($amount, $configuration->getTaxRate());
        }

        return new static($discount);
    }
}