Groundskeeper\Tokens\Elements\Hgroup

Complexity

Total Complexity

15

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	0	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
C	removeInvalidChildren()	0	31	11
A	removeInvalidSelf()	0	17	4

<?php

namespace Groundskeeper\Tokens\Elements;

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

/**
 * "hgroup" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-hgroup-element
 */
class Hgroup extends OpenElement implements FlowContent, HeadingContent
{
    protected function removeInvalidChildren(LoggerInterface $logger)
    {
        // One or more "h1", "h2", "h3", "h4", "h5", "h6", and

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

        // "template" elements required.
        $headingContentElementCount = 0;

$headingContentElementCount is not used, you could remove the assignment.

        foreach ($this->children as $child) {
            if ($child->getType() === Token::COMMENT) {
                continue;
            }

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

                continue;
            }

            if ($child->getName() !== 'h1' &&
                $child->getName() !== 'h2' &&
                $child->getName() !== 'h3' &&
                $child->getName() !== 'h4' &&
                $child->getName() !== 'h5' &&
                $child->getName() !== 'h6' &&
                $child->getName() !== 'tempate') {
                $logger->debug('Removing ' .  $child . '. Only "h1"-"h6" and "template" elements allowed as child of "hgroup" element.');
                $this->removeChild($child);

                continue;
            }
        }
    }

    protected function removeInvalidSelf(LoggerInterface $logger)
    {
        foreach ($this->children as $child) {
            if ($child instanceof HeadingContent) {
                return false;
            }
        }

        // Handle no child HeadingContent.
        if ($headingContentElementCount == 0) {

The variable $headingContentElementCount does not exist. Did you forget to declare it?

            $logger->debug($this . ' must contain at least one of the following elements: "h1", "h2", "h3", "h4", "h5", "h6", or "template".');

            return true;
        }

        return false;
    }
}