Propaganistas\LaravelIntl\Number

Complexity

Total Complexity

7

Size/Duplication

Total Lines	101
Duplicated Lines	9.9 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

76%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	format()	0	7	1
A	percent()	0	7	1
A	parse()	10	10	1
A	get()	0	4	1
A	all()	0	5	1
A	getDigitsOfFormat()	0	10	1

<?php namespace Propaganistas\LaravelIntl;

use CommerceGuys\Intl\Formatter\NumberFormatter;
use CommerceGuys\Intl\NumberFormat\NumberFormat;
use CommerceGuys\Intl\NumberFormat\NumberFormatRepository;
use Propaganistas\LaravelIntl\Base\Intl;
use ReflectionClass;

class Number extends Intl
{
    /**
     * @var \CommerceGuys\Intl\NumberFormat\NumberFormatRepository
     */
    protected $data;

    /**
     * Number constructor.
     *
     * @param \CommerceGuys\Intl\NumberFormat\NumberFormatRepository $data
     */
    public function __construct(NumberFormatRepository $data)
    {
        $this->data = $data;
    }

    /**
     * Format an value in the given locale (or app locale if not specified).
     *
     * @param int|float $value
     * @return string
     */
    public function format($value)
    {
        $format = $this->get();
        $formatter = new NumberFormatter($format);

        return $formatter->format($value);
    }

    /**
     * Format an value as percents in the given locale (or app locale if not specified).
     *
     * @param int|float $value
     * @return string
     */
    public function percent($value)
    {
        $format = $this->get();
        $formatter = new NumberFormatter($format, NumberFormatter::PERCENT);

        return $formatter->format($value);
    }

    /**
     * Parse a localized number into native PHP format.
     *
     * @param string|int|float $value
     * @return int|float
     */
    public function parse($value)

This method seems to be duplicated in your project.

    {
        $format = $this->get(null);
        $formatter = new NumberFormatter($format);

        // At time of writing, commerceguys/intl has number parsing still coupled to a currency. Parsing does
        // succeed however,even though a value is provided without any currency. So let's just pass in
        // a very rare currency to avoid unwanted formatting behavior. Sorry Cape Verdean Escudo!
        return $formatter->parseCurrency($value, $currency = currency()->get('CVE'));

The method get() does not seem to exist on object<CommerceGuys\Intl\Currency\Currency>.

    }

    /**
     * Get a localized entry.
     *
     * @param string|null $code
     * @return mixed
     */
    public function get($code = null)
    {
        return $this->data->get(null);
    }

    /**
     * Get a localized list of entries, keyed by their code.
     *
     * @return array
     */
    public function all()
    {
        // Unsupported.
        return [];
    }

    /**
     * Get the 'digits' property of a given NumberFormat.
     *
     * @param \CommerceGuys\Intl\NumberFormat\NumberFormat $format
     * @return array
     */
    private function getDigitsOfFormat(NumberFormat $format)

This method is not used, and could be removed.

    {
        $formatter = new NumberFormatter($format);

        $reflection = new ReflectionClass($formatter);
        $property = $reflection->getProperty('digits');
        $property->setAccessible(true);

        return $property->getValue($formatter);
    }
}