Xparse\Parser\Helper\ConvertUrlElementFinderModifier

Complexity

Total Complexity

11

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	execute()	0	22	6
A	attributeName()	0	10	4

<?php

declare(strict_types=1);

namespace Xparse\Parser\Helper;

use DOMNodeList;
use GuzzleHttp\Psr7\Uri;
use GuzzleHttp\Psr7\UriResolver;
use Xparse\ElementFinder\DomNodeListAction\DomNodeListActionInterface;

/**
 * @internal
 * @author Ivan Shcherbak <alotofall@gmail.com>
 */
class ConvertUrlElementFinderModifier implements DomNodeListActionInterface
{

    /**
     * @var string
     */
    private $affectedUrl;

    /**
     * @var string
     */
    private $baseUrl;


    public function __construct(string $affectedUrl, string $baseUrl)
    {
        $this->affectedUrl = $affectedUrl;
        $this->baseUrl = $baseUrl;
    }


    final public function execute(DOMNodeList $nodeList): void
    {
        $affected = new Uri($this->affectedUrl);
        foreach ($nodeList as $element) {
            /** @var \DOMElement $element */
            $attribute = $this->attributeName($element);
            $relative = $element->getAttribute($attribute);
            $isValid = parse_url($relative) !== false;
            if (
                $isValid
                &&
                !preg_match('!^\s*javascript\s*:\s*!', $relative)
            ) {
                if ($this->baseUrl !== '' && !preg_match('!^(/|http)!i', $relative)) {
                    $relative = UriResolver::resolve(new Uri($this->baseUrl), new Uri($relative));
                }
                $url = UriResolver::resolve($affected, new Uri($relative));

It seems like $relative defined by \GuzzleHttp\Psr7\UriResolver::resolve(new \GuzzleHttp\Psr7\Uri($this->baseUrl), new \GuzzleHttp\Psr7\Uri($relative)) on line 51 can also be of type object<Psr\Http\Message\UriInterface>; however, GuzzleHttp\Psr7\Uri::__construct() does only seem to accept string, maybe add an additional type check?

                /** @noinspection UnusedFunctionResultInspection */
                $element->setAttribute($attribute, (string)$url);
            }
        }
    }


    private function attributeName(\DOMElement $element): string
    {
        $name = 'href';
        if ($element->tagName === 'form' && $element->hasAttribute('action') === true) {
            $name = 'action';
        } else if ($element->hasAttribute('src') === true) {
            $name = 'src';
        }
        return $name;
    }

}