AframeVR\Core\Helpers\AssetsAbstract

Complexity

Total Complexity

11

Size/Duplication

Total Lines	93
Duplicated Lines	7.53 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	id()	0	5	1
A	src()	0	5	1
A	domElement()	0	7	1
A	setDomElementTag()	0	4	1
A	appendAttributes()	0	6	2
A	setAttribute()	7	7	4

<?php
/** @formatter:off
 * ******************************************************************
 * Created by   Marko Kungla on Jun 28, 2016 - 3:47:35 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         AssetsAbstract.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\AssetsInterface;
use \DOMElement;

abstract class AssetsAbstract implements AssetsInterface
{

    /**
     * DOM tag name of asset item
     *
     * @var string
     */
    protected $element_tag = 'a-asset-item';
    
    protected $attrs = array();
    /**
     * Asset constructor set asset ID
     *
     * @param string $id            
     */
    public function __construct(string $id)
    {
        $this->id($id);
    }

    /**
     * Set ID attribute of the asset
     *
     * {@inheritdoc}
     *
     * @param string $id            
     */
    public function id(string $id = 'untitled'): AssetsInterface
    {
        $this->attrs['id'] = $id;
        return $this;
    }

    /**
     * Set Assets src attribute
     *
     * {@inheritdoc}
     *
     * @param null|string $src            
     * @return void

Should the return type not be AssetsInterface?

     */
    public function src(string $src = null): AssetsInterface
    {
        $this->attrs['src'] = $src;
        return $this;
    }

    /**
     * Create and add DOM element of the asset
     *
     * @param \DOMDocument $aframe_dom            
     * @return \DOMElement
     */
    public function domElement(&$aframe_dom): DOMElement
    {
        $a_asset = $aframe_dom->createElement($this->element_tag);
        /* Asset must have a id */
        $this->appendAttributes($a_asset);
        return $a_asset;
    }

    /**
     * Set Dom element name
     *
     * @param string $element_tag            
     * @return void
     */
    public function setDomElementTag(string $element_tag)
    {
        $this->element_tag = $element_tag;
    }
    
    /**
     * Append DOM attributes no set by components
     *
     * @param \DOMElement $a_entity
     */
    private function appendAttributes(\DOMElement &$a_entity)
    {
        foreach ($this->attrs as $attr => $val) {
            $this->setAttribute($a_entity, $attr, $val);
        }
    }
    
    private function setAttribute(&$a_entity, $attr, $val)

This method seems to be duplicated in your project.

    {
        if ($attr === 'id' && ($val === 'untitled' || is_numeric($val)))
            return;
    
            $a_entity->setAttribute($attr, $val);
    }
}