AframeVR\Core\Assets\Mixin

Complexity

Total Complexity

2

Size/Duplication

Total Lines	88
Duplicated Lines	30.68 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	hp$0 ➔ __call()	0	5	1
A	component()	17	17	3
A	__call()	10	10	2
B	componentClosure()	0	26	1

<?php
/** @formatter:off
 * ******************************************************************
 * Created by   Marko Kungla on Jun 20, 2016 - 9:12:11 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         Mixin.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\Assets;

use \AframeVR\Core\Exceptions\BadComponentCallException;
use \AframeVR\Interfaces\Core\Assets\MixinInterface;
use \AframeVR\Core\Helpers\AssetsAbstract;
use \Closure;

final class Mixin extends AssetsAbstract implements MixinInterface
{

    /**
     * Array of used components
     *
     * @var array $components
     */
    protected $components = array();

    /**
     * Load component for this entity
     *
     * @param string $component_name            
     * @throws \AframeVR\Core\Exceptions\BadComponentCallException
     * @return object|null
     */
    public function component(string $component_name)

This method seems to be duplicated in your project.

    {
        $component_name = strtolower($component_name);
        
        /* Does this mixin already have this component loaded */
        if (! array_key_exists($component_name, $this->components)) {
            $component = sprintf('\AframeVR\Core\Components\%s\%sComponent', ucfirst($component_name), ucfirst($component_name));

This line exceeds maximum limit of 120 characters; contains 129 characters

            /* Does called component exist */
            if (class_exists($component)) {
                $this->components[$component_name] = $this->componentClosure();
            } else {
                throw new BadComponentCallException($component_name);
            }
        }
        
        return $this->components[$component_name] ?? null;
    }

    /**
     * Handle mixin components
     *
     * Since we might need to customize these to have
     * custom components loaded as $this->methosd aswell therefore
     * we have these placeholder magic methods here
     *
     * @param string $component_name            
     * @param array $args            
     */
    public function __call(string $component_name, array $args)

This method seems to be duplicated in your project.

    {
        if (! method_exists($this, $component_name)) {
            $this->{$component_name} = Closure::bind(function () use ($component_name) {
                return $this->component($component_name);
            }, $this, get_class());
        }
        
        return call_user_func($this->{$component_name}, $args);
    }

    /**
     * Create Closure to mock compnent to be aplied to entity using this mixin
     *
     * @return object

Consider making the return type a bit more specific; maybe use anonymous//src/Core/Assets/Mixin.php$0.

     */
    protected function componentClosure()
    {
        return new class() {

            /**
             * Record all menthod / args to be called when mixin is used on some entity
             *
             * @var array
             */
            protected $component_methods = array();

            /**
             * Call passes all calls to no existing methods to self::methodProvider
             *
             * @param string $method            
             * @param array $args            
             * @throws InvalidComponentMethodException
             * @return object

Consider making the return type a bit more specific; maybe use anonymous//src/Core/Assets/Mixin.php$0.

             */
            public function __call(string $method, $args)
            {
                $this->component_methods[$method] = $args;
                return $this;
            }
        };
    }
}