LauLamanApps\ApplePassbook\MetaData\Field\NumberField

Complexity

Total Complexity

11

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	setValue()	0	8	2
A	setCurrencyCode()	0	8	2
A	setNumberStyle()	0	8	2
A	getMetadata()	0	13	3

<?php

declare(strict_types=1);

namespace LauLamanApps\ApplePassbook\MetaData\Field;

use LauLamanApps\ApplePassbook\Exception\InvalidArgumentException;
use LauLamanApps\ApplePassbook\Style\NumberStyle;
use LogicException;

class NumberField extends Field
{
    /**
     * @var string|null
     */
    private $currencyCode;

    /**
     * @var NumberStyle|null
     */
    private $numberStyle;

    public function __construct(?string $key = null, $value = null, ?string $label = null)
    {
        parent::__construct($key, null, $label);

        if ($value !== null) {
            $this->setValue($value);
        }
    }

    public function setValue($value): void
    {
        if (!is_numeric($value)) {
            throw new InvalidArgumentException('Value should be numeric.');
        }

        parent::setValue($value);
    }

    public function setCurrencyCode(string $currencyCode): void
    {
        if ($this->numberStyle) {
            throw new LogicException('You can not set both a \'currencyCode\' and a \'numberStyle\'. Please set only one of the 2.');
        }

        $this->currencyCode = $currencyCode;
    }

    public function setNumberStyle(NumberStyle $numberStyle): void
    {
        if ($this->currencyCode) {

The expression $this->currencyCode of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            throw new LogicException('You can not set both a \'currencyCode\' and a \'numberStyle\'. Please set only one of the 2.');
        }

        $this->numberStyle = $numberStyle;
    }

    public function getMetadata(): array
    {
        $data = parent::getMetadata();
        if ($this->currencyCode) {

The expression $this->currencyCode of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            $data['currencyCode'] = $this->currencyCode;
        }

        if ($this->numberStyle) {
            $data['numberStyle'] = $this->numberStyle->getValue();
        }

        return $data;
    }
}