Jaxon\Utils\Config\Reader

Complexity

Total Complexity

7

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	read()	0	27	6
A	load()	0	6	1

<?php

/**
 * Reader.php - Jaxon config reader
 *
 * @package jaxon-core

Package name "jaxon-core" is not valid; consider "Jaxoncore" instead

 * @author Thierry Feuzeu <thierry.feuzeu@gmail.com>
 * @copyright 2016 Thierry Feuzeu <thierry.feuzeu@gmail.com>
 * @license https://opensource.org/licenses/BSD-3-Clause BSD 3-Clause License
 * @link https://github.com/jaxon-php/jaxon-core
 */

PHP version not specified

Missing @category tag in file comment

namespace Jaxon\Utils\Config;

class Reader

Missing doc comment for class Reader

{
    /**
     * Read options from a config file
     *
     * @param string        $sConfigFile        The full path to the config file

Expected 1 spaces after parameter type; 8 found

Expected 1 spaces after parameter name; 8 found

     *
     * @return array
     */
    public function read($sConfigFile)

Expected 2 blank lines before function; 0 found

    {
        if(!($sConfigFile = trim($sConfigFile)))

Variable assignment found within a condition. Did you mean to do a comparison ?

Consider using a different name than the parameter $sConfigFile. This often makes code more readable.

        {
            return [];
        }

        $sExt = pathinfo($sConfigFile, PATHINFO_EXTENSION);
        switch($sExt)
        {
        case 'php':
            $aConfigOptions = Php::read($sConfigFile);
            break;
        case 'yaml':
        case 'yml':
            $aConfigOptions = Yaml::read($sConfigFile);
            break;
        case 'json':
            $aConfigOptions = Json::read($sConfigFile);
            break;
        default:
            $sErrorMsg = jaxon_trans('errors.file.extension', ['path' => $sConfigFile]);
            throw new \Jaxon\Utils\Config\Exception\File($sErrorMsg);
        }

        return $aConfigOptions;
    }

Expected 2 blank lines after function; 1 found

    /**
     * Read options from a config file and setup the library
     *
     * @param string        $sConfigFile        The full path to the config file

Expected 1 spaces after parameter type; 8 found

Expected 4 spaces after parameter name; 8 found

     * @param string        $sConfigSection     The section of the config file to be loaded

Expected 1 spaces after parameter type; 8 found

Expected 1 spaces after parameter name; 5 found

     *
     * @return void
     */
    public function load($sConfigFile, $sConfigSection = '')
    {
        $aConfigOptions = $this->read($sConfigFile);
        // Setup the lib config options.
        jaxon()->di()->getConfig()->setOptions($aConfigOptions, $sConfigSection);
    }

Expected 2 blank lines after function; 0 found

}