SP\Crawler\Reader

Complexity

Total Complexity

29

Size/Duplication

Total Lines	295
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes	6
Bugs	1	Features	3

23 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getDocument()	0	4	1
A	getXPath()	0	4	1
A	getInputMap()	0	4	1
A	click()	0	15	3
A	select()	0	12	2
A	sendRequest()	0	6	1
A	open()	0	6	1
A	getUri()	0	4	1
A	query()	0	4	1
A	getElement()	0	12	2
A	getText()	0	6	1
A	getTagName()	0	4	1
A	getAttribute()	0	4	1
A	getHtml()	0	4	1
A	getFullHtml()	0	4	1
A	isVisible()	0	18	1
A	isSelected()	0	4	1
A	isChecked()	0	4	1
A	getInput()	0	4	1
A	getValue()	0	4	1
A	setValue()	0	8	2
A	queryIds()	0	12	2

<?php

namespace SP\Crawler;

use SP\Spiderling\CrawlerInterface;
use SP\Crawler\Element\ClickRequestInterface;
use SP\Crawler\Element\ClickableInterface;
use SP\Crawler\Element\SelectableInterface;
use Psr\Http\Message\RequestInterface;
use SP\Spiderling\Query\AbstractQuery;
use GuzzleHttp\Psr7\Uri;
use Psr\Http\Message\UriInterface;
use DOMDocument;
use DOMElement;
use DOMXPath;
use InvalidArgumentException;
use BadMethodCallException;

/**
 * @author    Ivan Kerin <ikerin@gmail.com>
 * @copyright 2015, Clippings Ltd.
 * @license   http://spdx.org/licenses/BSD-3-Clause
 */
class Reader implements CrawlerInterface
{
    /**
     * @var DOMDocument
     */
    private $document;

    /**
     * @var DOMXPath
     */
    private $xpath;

    /**
     * @var ElementMap
     */
    private $inputMap;

    /**
     * @param DOMDocument $document
     */
    public function __construct(DOMDocument $document)
    {
        $this->document = $document;

        $this->xpath = new DOMXPath($document);

        $this->inputMap = new InputMap($this);

It seems like new \SP\Crawler\InputMap($this) of type object<SP\Crawler\InputMap> is incompatible with the declared type object<SP\Crawler\ElementMap> of property $inputMap.

    }

    /**
     * @return DOMDocument
     */
    public function getDocument()
    {
        return $this->document;
    }

    /**
     * @return DOMXPath
     */
    public function getXPath()
    {
        return $this->xpath;
    }

    /**
     * @return ElementMap
     */
    public function getInputMap()
    {
        return $this->inputMap;
    }

    /**
     * @param  string $id
     * @throws BadMethodCallException
     */
    public function click($id)
    {
        $input = $this->getInput($this->getElement($id));

        if ($input instanceof ClickableInterface) {
            $input->click();
        } elseif ($input instanceof ClickRequestInterface) {
            $request = $input->clickRequest();
            $this->sendRequest($request);
        } else {
            throw new BadMethodCallException(
                sprintf('Cannot click on %s, %s', get_class($input), $id)
            );
        }
    }

    /**
     * @param  string $id
     * @throws BadMethodCallException
     */
    public function select($id)
    {
        $input = $this->getInput($this->getElement($id));

        if ($input instanceof SelectableInterface) {
            $input->select();
        } else {
            throw new BadMethodCallException(
                sprintf('Cannot select on %s, %s', get_class($input), $id)
            );
        }
    }

    /**
     * @param  RequestInterface $input

There is no parameter named $input. Was it maybe removed?

     * @throws BadMethodCallException
     */
    public function sendRequest(RequestInterface $request)
    {
        throw new BadMethodCallException(
            sprintf('Cannot send request to %s', $request->getUri())
        );
    }

    /**
     * @param  string $url
     * @throws BadMethodCallException
     */
    public function open(UriInterface $url)
    {
        throw new BadMethodCallException(
            sprintf('Method %s not supported by %s', __METHOD__, __CLASS__)
        );
    }

    /**
     * @return Psr\Http\Message\UriInterface
     */
    public function getUri()
    {
        return new Uri('');
    }

    /**
     * @param  string          $xpath
     * @param  DOMElement|null $scope
     * @return DOMNodeList
     */
    public function query($xpath, DOMElement $scope = null)
    {
        return $this->getXpath()->query($xpath, $scope);
    }

    /**
     * @param  string $xpath
     * @throws InvalidArgumentException when id not found
     * @return DOMElement
     */
    public function getElement($xpath)
    {
        $items = $this->query($xpath);

        if (0 === $items->length) {
            throw new InvalidArgumentException(
                sprintf('Node with id %s does not exist', $xpath)
            );
        }

        return $items->item(0);
    }

    /**
     * @param  string $id
     * @throws InvalidArgumentException when id not found
     * @return string
     */
    public function getText($id)
    {
        $element = $this->getElement($id);

        return trim(preg_replace('/[ \s\f\n\r\t\v ]+/u', ' ', $element->textContent));
    }

    /**
     * @param  string $id
     * @throws InvalidArgumentException when id not found
     * @return string
     */
    public function getTagName($id)
    {
        return $this->getElement($id)->tagName;
    }

    /**
     * @param  string $id
     * @param  string $name
     * @throws InvalidArgumentException when id not found
     * @return string
     */
    public function getAttribute($id, $name)
    {
        return $this->getElement($id)->getAttribute($name);
    }

    /**
     * @param  string $id
     * @throws InvalidArgumentException when id not found
     * @return string
     */
    public function getHtml($id)
    {
        return $this->document->saveXml($this->getElement($id));
    }

    /**
     * @return string
     */
    public function getFullHtml()
    {
        return $this->document->saveHtml();
    }

    /**
     * @param  string $id
     * @throws InvalidArgumentException when id not found
     * @return boolean
     */
    public function isVisible($id)
    {
        $element = $this->getElement($id);

        $conditions = [
            "contains(@style, 'display:none')",
            "contains(@style, 'display: none')",
            "self::script",
            "self::head",
        ];

        $hidden = $this->xpath->query(
            './ancestor-or-self::*['.join(' or ', $conditions).']',
            $element
        );

        return $hidden->length == 0;
    }

    /**
     * @param  string $id
     * @throws InvalidArgumentException when id not found
     * @return boolean
     */
    public function isSelected($id)
    {
        return $this->getElement($id)->hasAttribute('selected');
    }

    /**
     * @param  string $id
     * @throws InvalidArgumentException when id not found
     * @return boolean
     */
    public function isChecked($id)
    {
        return $this->getElement($id)->hasAttribute('checked');
    }

    /**
     * @param  DOMElement $element
     * @throws InvalidArgumentException when id not found
     * @return Element\AbstractElement
     */
    public function getInput(DOMElement $element)
    {
        return $this->inputMap->get($element);
    }

    /**
     * @param  string $id
     * @throws InvalidArgumentException when id not found
     * @return mixed
     */
    public function getValue($id)
    {
        return $this->getInput($this->getElement($id))->getValue();

It seems like you code against a specific sub-type and not the parent class SP\Crawler\Element\AbstractElement as the method getValue() does only exist in the following sub-classes of SP\Crawler\Element\AbstractElement: SP\Crawler\Element\Checkbox, SP\Crawler\Element\File, SP\Crawler\Element\Input, SP\Crawler\Element\Option, SP\Crawler\Element\Radio, SP\Crawler\Element\Select, SP\Crawler\Element\Textarea. Maybe you want to instanceof check for one of these explicitly?

    }

    /**
     * @param  string $id
     * @param  string $value
     * @throws InvalidArgumentException when id not found
     */
    public function setValue($id, $value)
    {
        $input = $this->getInput($this->getElement($id));

        if (false === $input->isDisabled()) {
            $input->setValue($value);
        }
    }

    /**
     * @param  AbstractQuery $query
     * @param  string        $parent
     * @return array
     */
    public function queryIds(AbstractQuery $query, $parent = null)
    {
        $xpath = $parent.$query->getXPath();

        $ids = [];

        foreach ($this->query($xpath) as $index => $element) {
            $ids []= "($xpath)[".($index+1)."]";
        }

        return $query->getFilters()->matchAll($this, $ids);
    }
}