CurrencyConverter\View\Helper\CurrencyConverterHelper

Complexity

Total Complexity

17

Size/Duplication

Total Lines	141
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	10	10	1
A	convert()	7	7	1
A	rate()	4	4	1
B	getRateToUse()	32	32	10
A	_formatConvert()	4	4	1
A	_getRateFromAPI()	19	19	3

<?php

namespace CurrencyConverter\View\Helper;

use Cake\View\Helper;
use Cake\ORM\TableRegistry;
use Cake\I18n\Time;

/**
 * @property \Cake\View\Helper\HtmlHelper $Html
 * @property \Cake\View\Helper\FormHelper $Form
 * @property \Cake\View\Helper\NumberHelper $Number
 */
class CurrencyConverterHelper extends Helper

This class seems to be duplicated in your project.

{
    /**
     * CurrencyratesTable Object
     * @var \Cake\ORM\Table
     */
    private $_currencyratesTable;

    /**
     * Default settings
     *
     * @var array
     */
    protected $_defaultConfig = [
        'database' => 2, // Use database to store rate
        'refresh' => 24, // Time interval for refreshing rate in database
        'decimal' => 2, // Number of decimal to use for convert number
    ];

    /**
     * @param array $config
     * @return void
     */
    public function initialize(array $config = []) {

        $config = $this->getConfig();

        $this->database = $config['database'];

The property database does not exist. Did you maybe forget to declare it?

        $this->refresh = $config['refresh'];

The property refresh does not exist. Did you maybe forget to declare it?

        $this->decimal = $config['decimal'];

The property decimal does not exist. Did you maybe forget to declare it?

        $this->_currencyratesTable = TableRegistry::get('CurrencyConverter.Currencyrates');

The method Cake\ORM\TableRegistry::get() has been deprecated with message: 3.6.0 Use \Cake\ORM\Locator\TableLocator::get() instead.

    }

    /**
     * Convert method take an amount as first parameter and convert it using $from currency and $to currency.
     *
     * @param float|string $amount the amount to convert.
     * @param string $from currency to convert from
     * @param string $to currency to convert to
     * @return float $amount converted
     */
    public function convert($amount, $from, $to)
    {
        $amount = floatval($amount);
        $rate = $this->getRateToUse($from, $to);

        return $convert = $this->_formatConvert($rate * $amount);

$convert is not used, you could remove the assignment.

    }

    /**
     * Rate method return the rate of two currencies
     *
     * @param string $from currency to get the rate from
     * @param string $to currency to get the rate to
     * @return float|null $rate
     */
    public function rate($from, $to)
    {
        return $this->getRateToUse($from, $to);
    }

    /**
     * getRateToUse return rate to use
     * Using $from and $to parameters representing currency to deal with and the configuration settings
     * This method save or update currencyrates Table if necesseray too.
     *
     * @param string $from currency to get the rate from
     * @param string $to currency to get the rate to
     * @return float|null $rate
     */
    public function getRateToUse($from, $to)
    {
        if ($from == $to) return 1;
        if ($this->database) {
            // Get a currency rate from table
            $result = $this->_currencyratesTable->find('all')->where(['from_currency' => $from, 'to_currency' => $to])->first();
            // If currency rate is in table and it doesn't have to be updated
            if ($result && $result->modified->wasWithinLast($this->refresh . ' hours')) return $rate = $result->rate;

$rate is not used, you could remove the assignment.

            // If currency rate is in table and it have to be updated
            if ($result && !$result->modified->wasWithinLast($this->refresh . ' hours')) {
                if ($rate = $this->_getRateFromAPI($from, $to)) {
                    $result->rate = $rate;
                    $this->_currencyratesTable->save($result);

It seems like $result defined by $this->_currencyratesTable->find('all')->where(array('from_currency' => $from, 'to_currency' => $to))->first() on line 90 can also be of type array; however, Cake\ORM\Table::save() does only seem to accept object<Cake\Datasource\EntityInterface>, maybe add an additional type check?

                }
                return $rate;
            }
            // If currency rate isn't in table
            if (!$result) {
                if ($rate = $this->_getRateFromAPI($from, $to)) {
                    $entity = $this->_currencyratesTable->newEntity([
                        'from_currency' => $from,
                        'to_currency' => $to,
                        'rate' => $rate
                    ]);
                    $this->_currencyratesTable->save($entity);
                }
                return $rate;
            }
        }

        return $this->_getRateFromAPI($from, $to);
    }

    /**
     * Format float number using configuration
     *
     * @return floatval
     */
    private function _formatConvert($number)
    {
        return number_format($number, $this->decimal);
    }

    /**
     * Call free.currencyconverterapi.com API to get a rate for one currency to an other one currency
     *
     * @param string $from the currency
     * @param string $to the currency
     * @return int|null $rate
     */
    private function _getRateFromAPI($from, $to)
    {
        $rate = null;

        $url = 'https://free.currencyconverterapi.com/api/v5/convert?q=' . $from . '_' . $to . '&compact=ultra';
        $request = @fopen($url, 'r');

        if ($request) {
            $response = fgets($request, 4096);
            fclose($request);

            $response = json_decode($response, true);
            if (isset($response[$from . '_' . $to])) {
                $rate = $response[$from . '_' . $to];
            }
        }
        
        return $rate;
    }
}