Groundskeeper\Tokens\Elements\Table

Complexity

Total Complexity

12

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
C	doClean()	0	31	12

<?php

namespace Groundskeeper\Tokens\Elements;

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

/**
 * "table" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-table-element
 */
class Table extends OpenElement implements FlowContent
{
    /**
     * @todo Deal with validation of the order of child elements.
     */
    protected function doClean(LoggerInterface $logger)
    {
        if ($this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
            foreach ($this->children as $child) {
                if ($child->getType() == Token::COMMENT) {
                    continue;
                }

                if ($child->getType() !== Token::ELEMENT) {
                    $logger->debug('Removing ' . $child . '. Only elements allowed as children of "table" element.');
                    $this->removeChild($child);

                    continue;
                }

                if ($child->getName() == 'caption' ||
                    $child->getName() == 'colgroup' ||
                    $child->getName() == 'thead' ||
                    $child->getName() == 'tbody' ||
                    $child->getName() == 'tr' ||
                    $child->getName() == 'tfoot' ||
                    $child instanceof ScriptSupporting)
                    continue;
            }

            $logger->debug('Removing ' . $child . '. Only "caption", "colgroup", "thead", "tbody", "tr", "tfoot", and script supporting elements allowed as children of "table" element.');

The variable $child seems to be defined by a foreach iteration on line 25. Are you sure the iterator is never empty, otherwise this variable is not defined?

            $this->removeChild($child);
        }

        return true;
    }
}