WebinoDomLib\Dom

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	2
A	locate()	0	5	1
A	__toString()	0	5	2

<?php

namespace WebinoDomLib;

/**
 * Class DomHtml
 *
 * @package WebinoDomLib
 */
class Dom implements Dom\NodeLocatorInterface
{
    use Dom\LocatorAwareTrait;

    /**
     * @var Dom\Document
     */
    private $doc;

    // TODO factory
    public function __construct($code)
    {
        // hack HTML5
        libxml_use_internal_errors(true);

        $this->doc = new Dom\Document;

        // TODO
        $this->doc->isXml = false;

The property isXml does not seem to exist in WebinoDomLib\Dom\Document.

        $this->doc->isXml ? $this->doc->loadXml($code)
               : $this->doc->loadHtml(mb_convert_encoding($code, 'HTML-ENTITIES', 'UTF-8'));
    }

    /**
     * Return an element collection
     *
     * @param string|array $locator
     * @return Dom\NodeList
     */
    public function locate($locator)
    {
        $nodeList = $this->getLocator()->query($this->doc->getDocumentElement(), $locator);

It seems like $this->doc->getDocumentElement() targeting WebinoDomLib\Dom\Document::getDocumentElement() can also be of type object<DOMElement>; however, WebinoDomLib\Dom\Locator::query() does only seem to accept object<WebinoDomLib\Dom\NodeInterface>, maybe add an additional type check?

        return new Dom\NodeList($nodeList);
    }

    /**
     * @return string
     */
    public function __toString()
    {
        // TODO resolve ->doc->isXml()

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

        return $this->doc->isXml ? $this->doc->saveXml() : $this->doc->saveHtml();

The property isXml does not seem to exist in WebinoDomLib\Dom\Document.

    }
}