Groundskeeper\Tokens\Elements\Tr

Complexity

Total Complexity

12

Size/Duplication

Total Lines	35
Duplicated Lines	42.86 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	removeInvalidSelf()	0	16	6
B	removeInvalidChildren()	15	15	6

<?php

namespace Groundskeeper\Tokens\Elements;

use Groundskeeper\Tokens\ElementTypes\OpenElement;
use Groundskeeper\Tokens\ElementTypes\ScriptSupporting;
use Groundskeeper\Tokens\NonParticipating;
use Psr\Log\LoggerInterface;

/**
 * "tr" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-tr-element
 */
class Tr extends OpenElement
{
    protected function removeInvalidChildren(LoggerInterface $logger)

This method seems to be duplicated in your project.

    {
        // Children can be "td", "th", and script supporting elements.
        foreach ($this->children as $child) {
            if ($child instanceof NonParticipating ||
                $child instanceof Td ||
                $child instanceof Th ||
                $child instanceof ScriptSupporting) {
                continue;
            }

            $logger->debug('Removing ' . $child . '. Only "td", "th", and script supporting elements allowed as children of "tr" element.');
            $this->removeChild($child);
        }
    }

    protected function removeInvalidSelf(LoggerInterface $logger) : bool
    {
        // "table" must be parent.
        $parent = $this->getParent();
        if ($parent !== null &&
            !$parent instanceof Thead &&
            !$parent instanceof Tbody &&
            !$parent instanceof Tfoot &&
            !$parent instanceof Table) {
            $logger->debug('Removing ' . $this . '. Must be a child of the "thead", "tbody", "tfoot", or "table" elements.');

            return true;
        }

        return false;
    }
}