PHPDaemon\Config\Section

Complexity

Total Complexity

19

Size/Duplication

Total Lines	139
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	3
A	count()	0	4	1
A	toArray()	0	11	3
A	offsetExists()	0	5	1
A	getRealPropertyName()	0	4	1
A	offsetGet()	0	5	2
A	offsetSet()	0	5	1
A	offsetUnset()	0	5	1
B	imposeDefault()	0	25	6

<?php
namespace PHPDaemon\Config;

use PHPDaemon\Config\Entry\Generic;

/**
 * Config section
 *
 * @property Entry\Boolean enable
 * @property Entry\Boolean allowoverridebinpath
 * @property Entry\Boolean allowoverridechroot
 * @property Entry\Boolean allowoverrideuser
 * @property Entry\Boolean allowoverridegroup
 * @property Entry\Boolean allowoverridecwd
 * @property Entry\Generic cwd
 * @property Entry\Generic indexfiles
 * @property Entry\Generic mastersocket
 * @package    Core
 * @subpackage Config
 *
 * @author     Vasily Zorin <maintainer@daemon.io>
 * @dynamic_fields
 */
class Section implements \ArrayAccess
{
    use \PHPDaemon\Traits\ClassWatchdog;

    /** @var */
    public $source;
    /** @var */
    public $revision;

    /**
     * Constructor
     * @param hash
     * @return object

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($arr = [])
    {
        foreach ($arr as $k => $v) {
            if (!is_object($v)) {
                $e = new Generic;
                $e->setHumanValue($v);
                $this->{$k} = $e;
            } else {
                $this->{$k} = $v;
            }
        }
    }

    /**
     * Count elements
     * @return integer
     */
    public function count()
    {
        return count($this) - 2;
    }

    /**
     * toArray handler
     * @return hash

Should the return type not be array?

     */
    public function toArray()
    {
        $arr = [];
        foreach ($this as $k => $entry) {

The expression $this of type this<PHPDaemon\Config\Section> is not traversable.

            if (!$entry instanceof Generic) {
                continue;
            }
            $arr[$k] = $entry->value;
        }
        return $arr;
    }

    /**
     * Checks if property exists
     * @param string Property name
     * @return boolean Exists?
     */

    public function offsetExists($prop)
    {
        $prop = $this->getRealPropertyName($prop);
        return property_exists($this, $prop);
    }

    /**
     * Get real property name
     * @param string Property name
     * @return string Real property name
     */
    public function getRealPropertyName($prop)
    {
        return str_replace('-', '', strtolower($prop));
    }

    /**
     * Get property by name
     * @param string Property name
     * @return mixed
     */
    public function offsetGet($prop)
    {
        $prop = $this->getRealPropertyName($prop);
        return isset($this->{$prop}) ? $this->{$prop}->value : null;
    }

    /**
     * Set property
     * @param string Property name
     * @param mixed  Value
     * @return void
     */
    public function offsetSet($prop, $value)
    {
        $prop = $this->getRealPropertyName($prop);
        $this->{$prop} = $value;
    }

    /**
     * Unset property
     * @param string Property name
     * @return void
     */
    public function offsetUnset($prop)
    {
        $prop = $this->getRealPropertyName($prop);
        unset($this->{$prop});
    }

    /**
     * Impose default config
     * @param array {"setting": "value"}
     * @return void
     */
    public function imposeDefault($settings = [])
    {
        foreach ($settings as $name => $value) {
            $name = strtolower(str_replace('-', '', $name));
            if (!isset($this->{$name})) {
                if (is_scalar($value)) {
                    $this->{$name} = new Generic($value);
                } else {
                    $this->{$name} = $value;
                }
            } elseif ($value instanceof Section) {
                $value->imposeDefault($value);

$value is of type object<PHPDaemon\Config\Section>, but the function expects a array.

            } else {
                $current = $this->{$name};
                if (!is_object($value)) {
                    $this->{$name} = new Generic($value);
                } else {
                    $this->{$name} = $value;
                }
                $this->{$name}->setHumanValue($current->value);
                $this->{$name}->source = $current->source;
                $this->{$name}->revision = $current->revision;
            }
        }
    }
}