Groundskeeper\Tokens\Elements\Link

Complexity

Total Complexity

17

Size/Duplication

Total Lines	87
Duplicated Lines	19.54 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes	7
Bugs	2	Features	4

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getAllowedAttributes()	17	17	1
D	doClean()	0	43	10
B	isAllowedInBody()	0	15	6

<?php

namespace Groundskeeper\Tokens\Elements;

use Groundskeeper\Configuration;
use Groundskeeper\Tokens\Element;
use Groundskeeper\Tokens\ElementTypes\ClosedElement;
use Groundskeeper\Tokens\ElementTypes\MetadataContent;
use Psr\Log\LoggerInterface;

class Link extends ClosedElement implements MetadataContent
{
    protected function getAllowedAttributes()

This method seems to be duplicated in your project.

    {
        $linkAllowedAttributes = array(
            '/^href$/i' => Element::ATTR_URI,
            '/^crossorigin$/i' => Element::ATTR_CS_STRING,
            '/^rel$/i' => Element::ATTR_CI_SSENUM . '("alternate","author","help","icon","license","next","pingback","prefetch","prev","search","stylesheet")',
            '/^media$/i' => Element::ATTR_CI_STRING,
            '/^hreflang$/i' => Element::ATTR_CS_STRING,
            '/^type$/i' => Element::ATTR_CI_STRING,
            '/^sizes$/i' => Element::ATTR_CI_STRING
        );

        return array_merge(
            $linkAllowedAttributes,
            parent::getAllowedAttributes()
        );
    }

    protected function doClean(LoggerInterface $logger)
    {
        if ($this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
            // Must have "href" attribute.
            if (!$this->hasAttribute('href')) {
                $logger->debug('Element "link" requires "href" attribute.');

                return false;
            }

            // Must have either "rel" or "itemprop" attribute, but not both.
            $attrCount = 0;
            foreach ($this->attributes as $key => $value) {
                if ($key == 'rel' || $key == 'itemprop') {
                    $attrCount++;
                }

                if ($attrCount > 1) {
                    // If both, then we don't know which one should be kept,
                    // so we recommend to delete the entire element.
                    $logger->debug('Element "link" requires either "rel" or "itemprop" attribute, but not both.');

                    return false;
                }
            }

            if ($attrCount == 0) {
                $logger->debug('Element "link" requires either "rel" or "itemprop" attribute.');

                return false;
            }

            // If inside "body" element, then we check if allowed.
            $body = new Body($this->configuration, 'body');
            if ($this->hasAncestor($body) && !$this->isAllowedInBody()) {
                $logger->debug('Element "link" does not have the correct attributes to be allowed inside the "body" element.');

                return false;
            }
        }

        return true;
    }

    /**
     * Is this element allowed inside a body element?
     *
     * https://html.spec.whatwg.org/multipage/semantics.html#allowed-in-the-body
     *
     * @return boolean True if allowed.
     */
    public function isAllowedInBody()
    {
        if ($this->hasAttribute('itemprop')) {
            return true;
        }

        if ($this->hasAttribute('rel') &&
            ($this->attributes['rel'] == 'pingback' ||
             $this->attributes['rel'] == 'prefetch' ||
             $this->attributes['rel'] == 'stylesheet')) {
            return true;
        }

        return false;
    }
}