Xparse\Parser\Helper\ConvertUrlElementFinderModifier

Complexity

Total Complexity

11

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	modify()	0	20	6
A	attributeName()	0	9	4

<?php

  declare(strict_types=1);

  namespace Xparse\Parser\Helper;

  use GuzzleHttp\Psr7\Uri;
  use GuzzleHttp\Psr7\UriResolver;
  use Xparse\ElementFinder\ElementFinder\ElementFinderModifierInterface;

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


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

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


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


    /**
     * @param \DOMNodeList $nodeList
     * @return void
     */
    public function modify(\DOMNodeList $nodeList) {
      $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 (
          !preg_match('!^\s*javascript\s*:\s*!', $relative)
          and
          $isValid
        ) {
          if ($this->baseUrl !== '' and !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 52 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?

          $element->setAttribute($attribute, (string) $url);
        }
      }
    }


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

  }