Groundskeeper\Tokens\Elements\Ol

Complexity

Total Complexity

8

Size/Duplication

Total Lines	44
Duplicated Lines	61.36 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	3

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getAllowedAttributes()	0	13	1
C	doClean()	27	27	7

<?php

namespace Groundskeeper\Tokens\Elements;

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

/**
 * "ol" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-ol-element
 */
class Ol extends OpenElement implements FlowContent
{
    protected function getAllowedAttributes()
    {
        $olAllowedAttributes = array(
            '/^reversed$/i' => Element::ATTR_BOOL,
            '/^start$/i' => Element::ATTR_INT,
            '/^type$/i' => Element::ATTR_CS_STRING,
        );

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

    protected function doClean(LoggerInterface $logger)

This method seems to be duplicated in your project.

    {
        if ($this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
            // Only "li" and ScriptSupporting elements allowed.
            foreach ($this->children as $child) {
                if ($child->getType() == Token::COMMENT) {
                    continue;
                }

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

                    continue;
                }

                if ($child->getName() == 'li' || $child instanceof ScriptSupporting) {
                    continue;
                }

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

        return true;
    }
}