Groundskeeper\Tokens\Elements\Dt

Complexity

Total Complexity

9

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

83.33%

Importance

Changes	4
Bugs	2	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
B	removeInvalidChildren()	0	13	6
A	removeInvalidSelf()	0	13	3

<?php

namespace Groundskeeper\Tokens\Elements;

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

/**
 * "dt" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-dt-element
 */
class Dt extends OpenElement
{
    protected function removeInvalidChildren(LoggerInterface $logger)
    {
        // No "header", "footer", sectioning content, or heading content descendants.
        foreach ($this->children as $child) {
            if ($child instanceof Header ||
                $child instanceof Footer ||
                $child instanceof SectioningContent ||
                $child instanceof HeadingContent) {
                $logger->debug('Removing ' . $child . '. No "header", "footer", and sectioning content, or heading content elements allowed as children of "dt" element.');
                $this->removeChild($child);

It seems like $child can also be of type object<Groundskeeper\Tokens\ElementTypes\HeadingContent> or object<Groundskeeper\Tokens\ElementTypes\SectioningContent>; however, Groundskeeper\Tokens\Element::removeChild() does only seem to accept object<Groundskeeper\Tokens\Token>, maybe add an additional type check?

            }
        }
    }

    protected function removeInvalidSelf(LoggerInterface $logger)
    {
        // Must be child of "dl" element.
        $parent = $this->getParent();
        if ($parent !== null &&
            !$parent instanceof Dl) {
            $logger->debug('Removing ' . $this . '. Must be a child of a "dl" element.');

            return true;
        }

        return false;
    }
}