Groundskeeper\Tokens\Elements\Optgroup

Complexity

Total Complexity

9

Size/Duplication

Total Lines	41
Duplicated Lines	31.71 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getAllowedAttributes()	0	12	1
B	removeInvalidChildren()	13	13	5
A	removeInvalidSelf()	0	11	3

<?php

namespace Groundskeeper\Tokens\Elements;

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

/**
 * "optgroup" element
 *
 * https://html.spec.whatwg.org/multipage/forms.html#the-optgroup-element
 */
class Optgroup extends OpenElement
{
    protected function getAllowedAttributes()
    {
        $optgroupAllowedAttributes = array(
            '/^disabled$/i' => Attribute::BOOL,
            '/^label$/i' => Attribute::CS_STRING
        );

        return array_merge(
            $optgroupAllowedAttributes,
            parent::getAllowedAttributes()
        );
    }

    protected function removeInvalidChildren(LoggerInterface $logger)

This method seems to be duplicated in your project.

    {
        foreach ($this->children as $child) {
            if ($child instanceof NonParticipating ||
                $child instanceof Option ||
                $child instanceof ScriptSupporting) {
                continue;
            }

            $logger->debug('Removing ' . $child . '. Only "option" and script supporting elements allowed as children of a "optgroup" element.');
            $this->removeChild($child);
        }
    }

    protected function removeInvalidSelf(LoggerInterface $logger)
    {
        $parent = $this->getParent();
        if ($parent !== null && !$parent instanceof Select) {
            $logger->debug('Removing ' . $this . '. Only "select" element allowed as parent of "optgroup" element.');

            return true;
        }

        return false;
    }
}