Bavix\AdvancedHtmlDom\AdvancedHtmlDom

Complexity

Total Complexity

11

Size/Duplication

Total Lines	116
Duplicated Lines	0 %

Test Coverage

Coverage

60%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	loadFile()	0	5	1
A	cache()	0	8	2
A	text()	0	3	1
A	title()	0	3	1
A	setCache()	0	3	1
A	load()	0	14	2
A	load_file()	0	3	1
A	__construct()	0	6	2

<?php

namespace Bavix\AdvancedHtmlDom;

use Bavix\AdvancedHtmlDom\CacheSystem\InterfaceCache;

class AdvancedHtmlDom extends AdvancedHtmlBase
{

    /**
     * @var
     */
    public $xpath;

    /**
     * @var
     */
    public $root;

    /**
     * @var InterfaceCache
     */
    protected $cache;

    /**
     * AdvancedHtmlDom constructor.
     *
     * @param null $html

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

     * @param bool $is_xml
     */
    public function __construct($html = null, $is_xml = false)
    {
        $this->doc = $this;
        if ($html)
        {
            $this->load($html, $is_xml);
        }
    }

    /**
     * @param InterfaceCache $cache
     */
    public function setCache(InterfaceCache $cache)
    {
        $this->cache = $cache;
    }

    /**
     * @param string $url
     *
     * @return mixed
     */
    public function cache($url)
    {
        if (!$this->cache)
        {
            return \file_get_contents($url);
        }

        return $this->cache->get($url);
    }

    /**
     * @param      $html
     * @param bool $is_xml
     */
    public function load($html, $is_xml = false)
    {
        $this->dom = new \DOMDocument();

The call to DOMDocument::__construct() has too few arguments starting with version.

        if ($is_xml)
        {
            $this->dom->loadXML(\preg_replace('/xmlns=".*?"/ ', '', $html));
        }
        else
        {
            $this->dom->loadHTML($html);
        }
        $this->xpath = new \DOMXPath($this->dom);
        //$this->root = new AHTMLNode($this->dom->documentElement, $this->doc);

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

        $this->root = $this->at('body');

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

    }

    /**
     * @param      $file
     * @param bool $is_xml
     *
     * @deprecated loadFile
     */
    public function load_file($file, $is_xml = false)
    {
        $this->loadFile($file, $is_xml);
    }

    /**
     * @param string $file
     * @param bool   $is_xml
     *
     * @return $this
     */
    public function loadFile($file, $is_xml = false)
    {
        $this->load($this->cache($file), $is_xml);

        return $this;
    }

    // special cases

    /**
     * @return mixed
     */
    public function text()
    {
        return $this->root->text;
    }

    /**
     * @return mixed
     */
    public function title()
    {
        return $this->at('title')->text();

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

    }

}