Thruster\Component\XMLIterator\Element

Complexity

Total Complexity

7

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

52.38%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getXMLElementAround()	0	4	1
A	getAttributes()	0	4	1
A	getAttribute()	0	4	1
A	__toString()	0	4	1
A	initializeFrom()	0	14	2

<?php

namespace Thruster\Component\XMLIterator;

use RuntimeException;

/**
 * Class Element
 *
 * @package Thruster\Component\XMLIterator
 * @author  Aurimas Niekis <aurimas@niekis.lt>
 */
class Element extends Node
{
    private $name_;
    private $attributes_;

    public function __construct(XMLReader $reader)
    {
        parent::__construct($reader);

        $this->initializeFrom($reader);
    }

    public function getXMLElementAround($innerXML = '')
    {
        return XMLBuild::wrapTag($this->name_, $this->attributes_, $innerXML);
    }

    public function getAttributes()
    {
        return $this->attributes_;
    }

    public function getAttribute(string $name, $default = null)
    {
        return $this->attributes_[$name] ?? $default;
    }

    public function __toString()
    {
        return $this->name_;
    }

    private function initializeFrom(XMLReader $reader)
    {
        if ($reader->nodeType !== XMLReader::ELEMENT) {
            $node = new Node($reader);

            throw new RuntimeException(sprintf(
                'Reader must be at an XMLReader::ELEMENT, is XMLReader::%s given.',
                $node->getNodeTypeName()
            ));
        }

        $this->name_       = $reader->name;
        $this->attributes_ = parent::getAttributes()->getArrayCopy();

It seems like you call parent on a different method (getAttributes() instead of initializeFrom()). Are you sure this is correct? If so, you might want to change this to $this->getAttributes().

    }
}