Magestead\Helper\Config

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	28.21 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	__get()	0	4	1
A	getConfigFile()	11	11	2

<?php namespace Magestead\Helper;

use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Yaml\Exception\ParseException;
use Symfony\Component\Yaml\Parser;

class Config
{
    protected $_projectPath;

    /**
     * Config constructor.
     * @param OutputInterface $output
     */
    public function __construct(OutputInterface $output)
    {
        $this->_projectPath = getcwd();
        $this->_config = $this->getConfigFile($output);

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

    }

    /**
     * @param $name
     * @return mixed
     */
    function __get($name)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return $this->_config['magestead']['apps']['mba_12345'][$name];
    }

    /**
     * @param OutputInterface $output
     * @return bool|mixed
     */
    protected function getConfigFile(OutputInterface $output)

This method seems to be duplicated in your project.

    {
        $config = new Parser();
        try {
            return $config->parse(file_get_contents($this->_projectPath . '/magestead.yaml'));
        } catch (ParseException $e) {
            $output->writeln('<error>Unable to parse the config file</error>');
        }

        return false;
    }
}