Groundskeeper\Tokens\Elements\Head

Complexity

Total Complexity

21

Size/Duplication

Total Lines	70
Duplicated Lines	15.71 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

91.84%

Importance

Changes	5
Bugs	1	Features	3

1 Method

Rating	Name	Duplication	Size	Complexity
C	doClean()	11	67	21

<?php

namespace Groundskeeper\Tokens\Elements;

use Groundskeeper\Configuration;
use Groundskeeper\Tokens\ElementTypes\OpenElement;
use Groundskeeper\Tokens\ElementTypes\MetadataContent;
use Groundskeeper\Tokens\Token;
use Psr\Log\LoggerInterface;

/**
 * "head" element
 */
class Head extends OpenElement
{
    protected function doClean(LoggerInterface $logger = null)
    {
        // "head" element must be a child of "html" element.
        if ($this->getParent() !== null &&
            $this->getParent()->getType() === Token::ELEMENT &&
            $this->getParent()->getName() != 'html') {

It seems like you code against a concrete implementation and not the interface Groundskeeper\Tokens\Token as the method getName() does only exist in the following implementations of said interface: Groundskeeper\Tokens\Element, Groundskeeper\Tokens\ElementTypes\ClosedElement, Groundskeeper\Tokens\ElementTypes\OpenElement, Groundskeeper\Tokens\Elements\Article, Groundskeeper\Tokens\Elements\Aside, Groundskeeper\Tokens\Elements\Base, Groundskeeper\Tokens\Elements\Body, Groundskeeper\Tokens\Elements\Br, Groundskeeper\Tokens\Elements\Footer, Groundskeeper\Tokens\Elements\H1, Groundskeeper\Tokens\Elements\H2, Groundskeeper\Tokens\Elements\H3, Groundskeeper\Tokens\Elements\H4, Groundskeeper\Tokens\Elements\H5, Groundskeeper\Tokens\Elements\H6, Groundskeeper\Tokens\Elements\Head, Groundskeeper\Tokens\Elements\Header, Groundskeeper\Tokens\Elements\Hgroup, Groundskeeper\Tokens\Elements\Hr, Groundskeeper\Tokens\Elements\Html, Groundskeeper\Tokens\Elements\Link, Groundskeeper\Tokens\Elements\Main, Groundskeeper\Tokens\Elements\Meta, Groundskeeper\Tokens\Elements\Nav, Groundskeeper\Tokens\Elements\Noscript, Groundskeeper\Tokens\Elements\Script, Groundskeeper\Tokens\Elements\Section, Groundskeeper\Tokens\Elements\Style, Groundskeeper\Tokens\Elements\Template, Groundskeeper\Tokens\Elements\Title.

            if ($logger !== null) {
                $logger->debug('Element "head" must be a child of "html" element.');
            }

            return false;
        }

        // "head" element must contain only metadata content elements.
        // "head" element must contain exactly one "title" element.
        // "head" element must contain either 0 or 1 "base" element.
        $titleCount = 0;
        $baseCount = 0;
        foreach ($this->children as $child) {
            if (!$child instanceof MetadataContent &&
                $child->getType() !== 'comment' &&
                $this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
                $this->removeChild($child);
                if ($logger !== null) {
                    $logger->debug('Removing ' . $child . '. Only children of metadata content allowed.');
                }
            }

            if ($child->getType() != Token::ELEMENT) {
                continue;
            }

            if ($child->getName() == 'title') {
                $titleCount++;
                if ($titleCount > 1 &&
                    $this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
                    if ($logger !== null) {
                        $logger->debug('Removing ' . $child . '. Only one "title" element allowed.');
                    }

                    $this->removeChild($child);
                }
            } elseif ($child->getName() == 'base') {

This code seems to be duplicated across your project.

                $baseCount++;
                if ($baseCount > 1 &&
                    $this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
                    if ($logger !== null) {
                        $logger->debug('Removing ' . $child . '. Maximum one "base" element allowed.');
                    }

                    $this->removeChild($child);
                }
            }
        }

        // Missing title.
        if ($titleCount == 0) {
            if ($logger !== null) {
                $logger->debug('Adding "title" element. One "title" element required.');
            }

            $title = new Title($this->configuration, 'title');
            $this->prependChild($title);
        }

        return true;
    }
}