AframeVR\Core\Helpers\ComponentAbstract

Complexity

Total Complexity

13

Size/Duplication

Total Lines	152
Duplicated Lines	0 %

Coupling/Cohesion

Components	4
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	__call()	0	15	4
A	setMethodProvider()	0	5	1
A	hasDOMAttributes()	0	4	1
A	getDOMAttributesArray()	0	4	1
A	getComponentScripts()	0	4	1
A	addComponentScripts()	0	4	1
A	getDOMAttr()	0	4	1
A	getDomAttributeName()	0	4	1
A	setDomAttributeName()	0	4	1

<?php
/** @formatter:off
 * ******************************************************************
 * Created by   Marko Kungla on Jun 25, 2016 - 7:55:29 PM
 * Contact      marko@okramlabs.com
 * @copyright   2016 Marko Kungla - https://github.com/mkungla
 * @license     The MIT License (MIT)
 * 
 * @category       AframeVR
 * @package        aframe-php
 * 
 * Lang         PHP (php version >= 7)
 * Encoding     UTF-8
 * File         ComponentAbstract.php
 * Code format  PSR-2 and 12
 * @link        https://github.com/mkungla/aframe-php
 ^ @issues      https://github.com/mkungla/aframe-php/issues
 * ********************************************************************
 * Contributors:
 * @author Marko Kungla <marko@okramlabs.com>
 * ********************************************************************
 * Comments:
 * @formatter:on */
namespace AframeVR\Core\Helpers;

use \AframeVR\Interfaces\ComponentInterface;
use \AframeVR\Core\Exceptions\{
    InvalidComponentMethodException,
    InvalidComponentArgumentException
};
use \DOMAttr;

abstract class ComponentAbstract implements ComponentInterface
{

    /**
     * Array of dom attributes
     *
     * @var array
     */
    protected $dom_attributes = array();

    /**
     * Array of component scripts
     *
     * @var array
     */
    protected $component_scripts = array();

    /**
     * Name of DOM attribute
     *
     * @var unknown
     */
    protected $dom_attribute_name;

    /**
     * Loaded Component Method object
     *
     * @var object
     */
    protected $methodProvider;

    /**
     * Component Constructor
     */
    public function __construct()
    {
        $this->initializeComponent();
        $this->setMethodProvider();
    }

    /**
     * Call passes all calls to no existing methods to self::methodProvider
     *
     * @param string $method            
     * @param array $args            
     * @throws InvalidComponentMethodException
     */
    public function __call(string $method, $args)
    {
        if (is_object($this->methodProvider) && method_exists($this->methodProvider, $method)) {
            array_unshift($args, 0);
            $args[0] = &$this->dom_attributes;
            
            return call_user_func_array(array(
                $this->methodProvider,
                (string) $method
            ), $args);
        } else {
            $class = is_object($this->methodProvider) ? get_class($this->methodProvider) : get_called_class();
            throw new InvalidComponentMethodException($method, $class);
        }
    }

    /**
     * Set class providing component methods
     *
     * @param string $method_provider            
     * @return void
     */
    public function setMethodProvider(string $method_provider = 'DefaultMethods')
    {
        $method_provider_class = substr(get_called_class(), 0, strrpos(get_called_class(), '\\')) . '\Methods\\' . $method_provider;
        $this->methodProvider = new $method_provider_class();
    }

    /**
     * Does component have DOM Atributes
     *
     * {@inheritdoc}
     *
     * @return bool
     */
    public function hasDOMAttributes(): bool
    {
        return ! empty($this->dom_attributes);
    }

    /**
     * Get component DOM Atributes array
     *
     * {@inheritdoc}
     *
     * @return bool

Should the return type not be array? Also, consider making the array more specific, something like array<String>, or String[].

     */
    public function getDOMAttributesArray(): array
    {
        return $this->dom_attributes;
    }

    /**
     * Get Component scripts
     *
     * {@inheritdoc}
     *
     * @return array
     */
    public function getComponentScripts(): array
    {
        return (array) $this->component_scripts;
    }

    /**
     * Add component scripts
     *
     * {@inheritdoc}
     *
     */
    public function addComponentScripts(string $vendor_component, string $script_name)
    {
        $this->component_scripts[$vendor_component] = $script_name;
    }

    /**
     * Get DOMAttr for the entity
     *
     * @return DOMAttr
     */
    public function getDOMAttr(): DOMAttr
    {
        return new DOMAttr($this->getDomAttributeName(), $this->getDomAttributeString());
    }

    /**
     * Get Dom attribute name
     *
     * @return string
     */
    public function getDomAttributeName(): string
    {
        return $this->dom_attribute_name;
    }

    /**
     * Set Dom Attribute name
     *
     * @return void
     */
    public function setDomAttributeName(string $dom_attribute_name)
    {
        $this->dom_attribute_name = $dom_attribute_name;

It seems like $dom_attribute_name of type string is incompatible with the declared type object<AframeVR\Core\Helpers\unknown> of property $dom_attribute_name.

    }
}