Ghc\Rosetta\Transformers\MoneyValues

Complexity

Total Complexity

7

Size/Duplication

Total Lines	67
Duplicated Lines	14.93 %

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	17	1
A	transform()	9	9	3
A	parseMoney()	0	16	3

<?php

namespace Ghc\Rosetta\Transformers;

use Money\Currencies\ISOCurrencies;
use Money\Parser\AggregateMoneyParser;
use Money\Parser\DecimalMoneyParser;
use Money\Parser\IntlMoneyParser;

class MoneyValues extends Transformer
{
    /**
     * @var AggregateMoneyParser
     */
    private $moneyParser;

    /**
     * Boot Transformer.
     */
    protected function boot()
    {
        $defaultLocale = 'en_US';
        $numberFormatter = new \NumberFormatter($defaultLocale, \NumberFormatter::CURRENCY);

The type NumberFormatter was not found. Maybe you did not declare it correctly or list all dependencies?

        $this->addDefaultConfig([
            'locale'        => $defaultLocale,
            'currency_code' => $numberFormatter->getTextAttribute(\NumberFormatter::CURRENCY_CODE),
        ]);

        $currencies = new ISOCurrencies();
        $intlParser = new IntlMoneyParser($numberFormatter, $currencies);
        $decimalParser = new DecimalMoneyParser($currencies);

        $this->moneyParser = new AggregateMoneyParser([
            $intlParser,
            $decimalParser,
        ]);
    }

    /**
     * @param array $data
     *
     * @return array
     */
    public function transform($data)

This method seems to be duplicated in your project.

    {
        foreach ($this->config['properties'] as $property) {
            if (isset($data[$property])) {
                $data[$property] = $this->parseMoney($data[$property]);
            }
        }

        return $data;
    }

    /**
     * @param $value
     *
     * @return \Money\Money
     */
    private function parseMoney($value)
    {
        $isNegative = str_contains($value, '-');
        $valueToParse = str_replace([' ', '+', '-'], '', $value);

        try {
            /** @var \Money\Money $money */
            $money = $this->moneyParser->parse($valueToParse);
            $value = [
                'amount'   => (int) (!$isNegative ? $money->getAmount() : -$money->getAmount()),
                'currency' => $money->getCurrency()->getCode(),
            ];
        } catch (\Exception $e) {

Consider adding a comment why this CATCH block is empty.

        }

        return $value;

The expression return $value also could return the type array<string,integer|string> which is incompatible with the documented return type Money\Money.

    }
}