Apps\Model\Admin\Main\FormYandexCounter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	make()	0	6	1
A	getCounters()	0	3	1
A	__construct()	0	4	1
A	rules()	0	5	1

<?php

namespace Apps\Model\Admin\Main;


use Ffcms\Core\App;
use Ffcms\Core\Arch\Model;

/**
 * Class FormYandexCounter
 * @package Apps\Model\Admin\Main
 */
class FormYandexCounter extends Model
{
    public $counter;

    private $_counters;

    /**
     * FormYandexCounter constructor.
     * @param array $counters
     */
    public function __construct(array $counters)
    {
        $this->_counters = $counters;
        parent::__construct(false);
    }

    /**
     * Validation rules
     * @return array
     */
    public function rules(): array
    {
        return [
            ['counter', 'required'],
            ['counter', 'int']
        ];
    }

    /**
     * Save counter id to configuration file
     */
    public function make()
    {
        $cfg = App::$Properties->getAll('Yandex');
        $cfg['metrika']['id'] = (int)$this->counter;

        App::$Properties->writeConfig('Yandex', $cfg);

It seems like $cfg can also be of type null; however, parameter $data of Ffcms\Core\Properties::writeConfig() does only seem to accept array, maybe add an additional type check?

    }

    /**
     * Get counter id->name
     * @return array
     */
    public function getCounters(): array
    {
        return $this->_counters;
    }
}