ByTIC\Migrations\Config\Traits\HasParamsTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

83.33%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	setParams()	0	3	1
A	mergeParams()	0	3	1
A	toArray()	0	3	1
A	generateBaseParams()	0	8	1
A	getParams()	0	3	1

<?php

namespace ByTIC\Migrations\Config\Traits;

/**
 * Trait HasParamsTrait
 * @package ByTIC\Migrations\Config\Traits
 */
trait HasParamsTrait
{
    protected $params = [];

    /**
     * @return null
     */
    public function getParams()
    {
        return $this->params;

The expression return $this->params returns the type array which is incompatible with the documented return type null.

    }

    /**
     * @param null $params

Are you sure the doc-type for parameter $params is correct as it would always require null to be passed?

     */
    public function setParams($params)
    {
        $this->params = $params;
    }

    /**
     * @param $params
     */
    public function mergeParams($params)
    {
        $this->params = array_merge($this->params, $params);
    }

    /**
     * @return array
     */
    public function toArray()
    {
        return $this->params;
    }

    protected function generateBaseParams()
    {
        $this->params = [
            'paths' => [
                'migrations' => [],
                'seeds' => [],
            ],
            'environments' => [],
        ];
    }
}