Groundskeeper\Tokens\Elements\Link

Complexity

Total Complexity

20

Size/Duplication

Total Lines	93
Duplicated Lines	7.53 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes	5
Bugs	1	Features	2

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getAllowedAttributes()	0	17	1
C	doClean()	7	49	13
B	isAllowedInBody()	0	15	6

<?php

namespace Groundskeeper\Tokens\Elements;

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()
    {
        $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 = null)
    {
        // Must have "href" attribute.
        if (!$this->hasAttribute('href')) {

This code seems to be duplicated across your project.

            if ($logger !== null) {
                $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.
                if ($logger !== null) {
                    $logger->debug('Element "link" requires either "rel" or "itemprop" attribute, but not both.');
                }

                return false;
            }
        }

        if ($attrCount == 0) {
            if ($logger !== null) {
                $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()) {
            if ($logger !== null) {
                $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;
    }
}