Bavix\AdvancedHtmlDom\AHTMLNode

Complexity

Total Complexity

35

Size/Duplication

Total Lines	246
Duplicated Lines	0 %

Test Coverage

Coverage

11.7%

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	attributes()	0	8	2
A	offsetExists()	0	3	1
A	__destruct()	0	5	1
A	offsetGet()	0	3	1
C	flatten()	0	28	7
A	replace()	0	6	2
A	before()	0	6	1
A	after()	0	8	2
A	get_fragment()	0	7	2
A	__construct()	0	6	1
A	decamelize()	0	15	2
A	title()	0	3	1
A	offsetSet()	0	14	3
A	offsetUnset()	0	3	1
C	__set()	0	28	8

<?php

namespace Bavix\AdvancedHtmlDom;

/**
 * Class AHTMLNode
 *
 * @package Bavix\AdvancedHtmlDom
 *
 * @property-read string $clean_text
 */
class AHTMLNode extends AdvancedHtmlBase implements \ArrayAccess
{

    /**
     * @var
     */
    private $_path;

    /**
     * AHTMLNode constructor.
     *
     * @param $node
     * @param $doc
     */
    public function __construct($node, $doc)
    {
        $this->node    = $node;
        $this->_path   = $node->getNodePath();
        $this->doc     = $doc;
        $this->is_text = $node->nodeName === '#text';
    }

    /**
     * @inheritdoc
     */
    public function __destruct()
    {
        $this->_path = null;
        unset($this->_path);
        parent::__destruct();
    }

    /**
     * @param $html
     *
     * @return mixed
     */
    private function get_fragment($html)
    {
        $dom = $this->doc->dom;
        $fragment = $dom->createDocumentFragment() or die('nope');

Using exit here is not recommended.

Using logical operators such as or instead of || is generally not recommended.

        $fragment->appendXML($html);

        return $fragment;
    }

    /**
     * @param $html
     *
     * @return AHTMLNode|null
     */
    public function replace($html)
    {
        $node = empty($html) ? null : $this->before($html);
        $this->remove();

        return $node;
    }

    /**
     * @param $html
     *
     * @return AHTMLNode
     */
    public function before($html)
    {
        $fragment = $this->get_fragment($html);
        $this->node->parentNode->insertBefore($fragment, $this->node);

        return new AHTMLNode($this->node->previousSibling, $this->doc);
    }

    /**
     * @param $html
     */
    public function after($html)
    {
        $fragment = $this->get_fragment($html);
        if ($ref_node = $this->node->nextSibling)
        {
            $this->node->parentNode->insertBefore($fragment, $ref_node);
        } else {
            $this->node->parentNode->appendChild($fragment);
        }
    }

    /**
     * @param $str
     *
     * @return mixed
     */
    public function decamelize($str)
    {
        $str = \preg_replace_callback(
            '/(^|[a-z])([A-Z])/',
            function($matches) {
                return
                    \strtolower(
                        \strlen($matches[1])
                            ? $matches[1] . '_' . $matches[2] : $matches[2]
                    );
            },
            $str
        );

        return \preg_replace('/ /', '_', \strtolower($str));
    }

    /**
     * @return array
     */
    public function attributes()
    {
        $ret = array();
        foreach ($this->node->attributes as $attr) {
            $ret[$attr->nodeName] = $attr->nodeValue;
        }

        return $ret;
    }

    /**
     * @param null $key

Are you sure the doc-type for parameter $key is correct as it would always require null to be passed?

     * @param int  $level
     *
     * @return array
     */
    public function flatten($key = null, $level = 1)
    {

        $children = $this->children;

The property children does not exist on Bavix\AdvancedHtmlDom\AHTMLNode. Since you implemented __get, consider adding a @property annotation.

        $ret      = array();
        $tag      = $this->tag;

The property tag does not exist on Bavix\AdvancedHtmlDom\AHTMLNode. Since you implemented __get, consider adding a @property annotation.

        if ($this->at('./preceding-sibling::' . $this->tag) || $this->at('./following-sibling::' . $this->tag) || ($key = $this->tag . 's'))

Are you sure $this->tag of type null|Bavix\AdvancedHtmlDom\AHTMLNode[]|Bavix\AdvancedHtmlDom\AHTMLNodeList can be used in concatenation?

The method at() does not exist on Bavix\AdvancedHtmlDom\AHTMLNode. Since you implemented __call, consider adding a @method annotation.

        {
            $count = $this->search('./preceding-sibling::' . $this->tag)->length + 1;

The property length does not exist on Bavix\AdvancedHtmlDom\AHTMLNodeList. Since you implemented __get, consider adding a @property annotation.

The method search() does not exist on Bavix\AdvancedHtmlDom\AHTMLNode. Since you implemented __call, consider adding a @method annotation.

            $tag .= '_' . $count;
        }

        if ($children->length == 0)
        {
            $ret[$this->decamelize(\implode(' ', \array_filter(array($key, $tag))))] = $this->text;

The property text does not exist on Bavix\AdvancedHtmlDom\AHTMLNode. Since you implemented __get, consider adding a @property annotation.

        } else {
            $flatten = [];
            foreach ($children as $child)
            {
                $flatten[] = $child->flatten(\implode(' ', \array_filter(array($key, $level <= 0 ? $tag : null))), $level - 1);
//                $ret = array_merge($ret, $child->flatten(implode(' ', array_filter(array($key, $level <= 0 ? $tag : null))), $level - 1));

57% of this comment could be valid code. Did you maybe forget this after debugging?

            }

            $ret = \array_merge($ret, ...$flatten);
        }

        return $ret;
    }

    /**
     * @param $name
     * @param $value
     */
    public function __set($name, $value)
    {
        switch ($name)
        {
            case 'text':
            case 'innertext':
            case 'innerText':
            case 'plaintext':
                $this->node->nodeValue = $value;

                return;
            case 'outertext':
                $this->replace($value);

                return;
            case 'tag':
                $el = $this->replace('<' . $value . '>' . $this->innerhtml . '</' . $value . '>');

The property innerhtml does not exist on Bavix\AdvancedHtmlDom\AHTMLNode. Since you implemented __get, consider adding a @property annotation.

Are you sure $this->innerhtml of type null|Bavix\AdvancedHtmlDom\AHTMLNode[]|Bavix\AdvancedHtmlDom\AHTMLNodeList can be used in concatenation?

                foreach ($this->node->attributes as $_name => $att)
                {
                    $el->$_name = $att->nodeValue;
                }
                $this->node = $el->node;

                return;

        }

        $this->offsetSet($name, $value);
    }

    /**
     * @param mixed $offset
     *
     * @return bool
     */
    public function offsetExists($offset)
    {
        return true;
    }

    /**
     * @param mixed $offset
     *
     * @return mixed
     */
    public function offsetGet($offset)
    {
        return $this->node->getAttribute($offset);
    }

    /**
     * @param mixed $key
     * @param mixed $value
     */
    public function offsetSet($key, $value)
    {
        if (\in_array($key, ['_path','dom','doc','node']))
        {
            return;
        }

        if ($value)
        {
            $this->node->setAttribute($key, $value);
            return;
        }

        $this->node->removeAttribute($key);
        //trigger_error('offsetSet not implemented', E_USER_WARNING);

63% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    /**
     * @param mixed $offset
     */
    public function offsetUnset($offset)
    {
        \trigger_error('offsetUnset not implemented', E_USER_WARNING);
    }

    /**
     * @return mixed
     */
    public function title()
    {
        return $this->node->getAttribute('title');
    }
}