Groundskeeper\Tokens\Elements\Address

Complexity

Total Complexity

9

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	2	Features	3

1 Method

Rating	Name	Duplication	Size	Complexity
D	doClean()	0	36	9

<?php

namespace Groundskeeper\Tokens\Elements;

use Groundskeeper\Configuration;
use Groundskeeper\Tokens\ElementTypes\FlowContent;
use Groundskeeper\Tokens\ElementTypes\HeadingContent;
use Groundskeeper\Tokens\ElementTypes\OpenElement;
use Groundskeeper\Tokens\ElementTypes\SectioningContent;
use Groundskeeper\Tokens\Token;
use Psr\Log\LoggerInterface;

/**
 * "address" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-address-element
 */
class Address extends OpenElement implements FlowContent
{
    protected function doClean(LoggerInterface $logger)
    {
        if ($this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
            // No HeadingContent descendants
            // No SectioningContent descendants
            // No "header", "footer", or "address" element descendants.

36% of this comment could be valid code. Did you maybe forget this after debugging?

            foreach ($this->children as $child) {
                if ($child instanceof HeadingContent) {
                    $logger->debug('Heading Content elements not allowed as "address" element child.');
                    $this->removeChild($child);

$child is of type object<Groundskeeper\Tokens\ElementTypes\HeadingContent>, but the function expects a object<Groundskeeper\Tokens\Token>.

                    continue;
                }

                if ($child instanceof SectioningContent) {
                    $logger->debug('Sectioning Content elements not allowed as "address" element child.');
                    $this->removeChild($child);

$child is of type object<Groundskeeper\Tokens\ElementTypes\SectioningContent>, but the function expects a object<Groundskeeper\Tokens\Token>.

                    continue;
                }

                if ($child->getType() == Token::ELEMENT) {
                    if ($child->getName() == 'header' ||
                        $child->getName() == 'footer' ||
                        $child->getName() == 'address') {
                        $logger->debug('Elements "header", "footer", and "address" not allowed as "address" element child.');
                        $this->removeChild($child);

                        continue;
                    }
                }
            }
        }

        return true;
    }
}