Groundskeeper\Tokens\Elements\Hgroup

Complexity

Total Complexity

13

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	4
Bugs	1	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
C	removeInvalidChildren()	0	24	10
A	removeInvalidSelf()	0	13	3

<?php

namespace Groundskeeper\Tokens\Elements;

use Groundskeeper\Tokens\ElementTypes\FlowContent;
use Groundskeeper\Tokens\ElementTypes\HeadingContent;
use Groundskeeper\Tokens\ElementTypes\OpenElement;
use Groundskeeper\Tokens\NonParticipating;
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 instanceof NonParticipating) {
                continue;
            }

            if (!$child instanceof H1 &&
                !$child instanceof H2 &&
                !$child instanceof H3 &&
                !$child instanceof H4 &&
                !$child instanceof H5 &&
                !$child instanceof H6 &&
                !$child instanceof Tempate) {

The class Groundskeeper\Tokens\Elements\Tempate does not exist. Did you forget a USE statement, or did you not list all dependencies?

                $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.
        $logger->debug('Removing ' . $this . '. Must contain at least one of the following elements: "h1", "h2", "h3", "h4", "h5", "h6", or "template".');

        return true;
    }
}